Change to warn instead of error for invalid ORRS (dydxprotocol#3036)

Author: jusbar23

protocol/x/revshare/keeper/revshare.go

@@ -332,7 +332,11 @@ func (k Keeper) getOrderRouterRevShares(
 	orderRouterRevShares := []types.RevShare{}
 	takerOrderRouterRevSharePpm, err := k.GetOrderRouterRevShare(ctx, fill.TakerOrderRouterAddr)
 	if err != nil {
-		k.Logger(ctx).Error("order router rev share invalid for taker: " + fill.TakerOrderRouterAddr)
+		k.Logger(ctx).Warn("order router rev share invalid for taker, ignoring ",
+			"taker_order_router_addr: ", fill.TakerOrderRouterAddr,
+			"taker_addr: ", fill.TakerAddr,
+			"error: ", err,
+		)
 	} else {
 		if fill.TakerOrderRouterAddr != "" {
 			// Orders can have 2 rev share ids, we need to calculate each side separately
@@ -351,7 +355,11 @@ func (k Keeper) getOrderRouterRevShares(
 
 	makerOrderRouterRevSharePpm, err := k.GetOrderRouterRevShare(ctx, fill.MakerOrderRouterAddr)
 	if err != nil {
-		k.Logger(ctx).Error("order router rev share invalid for maker: " + fill.MakerOrderRouterAddr)
+		k.Logger(ctx).Warn("order router rev share invalid for maker, ignoring ",
+			"maker_order_router_addr: ", fill.MakerOrderRouterAddr,
+			"maker_addr: ", fill.MakerAddr,
+			"error: ", err,
+		)
 	} else {
 		if fill.MakerOrderRouterAddr != "" {
 			// maker ppm * max(0, maker)