Fokko commented on code in PR #14791:
URL: https://github.com/apache/iceberg/pull/14791#discussion_r2597680776
##########
open-api/rest-catalog-open-api.py:
##########
@@ -946,6 +994,27 @@ class PlanTask(BaseModel):
)
+class ResidualFilter1(BaseModel):
+ """
+ An optional filter to be applied to rows in this file scan task.
+ If the residual is not present, the client must produce the residual or
use the original filter.
+ """
+
+
+class ResidualFilter2(TrueExpression, ResidualFilter1):
+ """
+ An optional filter to be applied to rows in this file scan task.
+ If the residual is not present, the client must produce the residual or
use the original filter.
+ """
+
+
+class ResidualFilter3(FalseExpression, ResidualFilter1):
Review Comment:
That's definitely possible, but that would require an update to the spec. I
think it would be better to do that in a PR separate from this version bump.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
