samredai commented on code in PR #4816:
URL: https://github.com/apache/iceberg/pull/4816#discussion_r877409487
##########
python/src/iceberg/expressions/base.py:
##########
@@ -267,6 +204,34 @@ def __str__(self) -> str:
return "false"
+class In(BooleanExpression):
Review Comment:
Now that we're moving away from having the single `Predicate` class, I still
think we can do some things to allow grouping downstream (for like methods that
must take a "predicate" or more specifically an "unbound predicate"). What do
you think of adding this? (includes renaming this `In` class to `UnboundIn`,
and we'd add a `BoundIn` class)
```py
class Predicate:
@abstractmethod
def __init__(self, left, right):
"""A concrete predicate must have a `left` and `right`"""
class Bound:
pass
class Unbound:
pass
class UnboundIn(BooleanExpression, Predicate, Unbound):
def __init__(self, left: 'UnboundReference', right: List[Literal]):
...
```
--
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]
