rdblue commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797229861


##########
api/src/main/java/org/apache/iceberg/expressions/ResidualEvaluator.java:
##########
@@ -89,6 +89,12 @@ public static ResidualEvaluator of(PartitionSpec spec, 
Expression expr, boolean
     }
   }
 
+  public static ResidualEvaluator of(Expression expr) {
+    // TODO this constructor is used for REST scan planning

Review Comment:
   If there isn't enough information, then why create the evaluator at that 
point? Seems like we should wait until that information is available.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to