Fokko opened a new pull request, #6128:
URL: https://github.com/apache/iceberg/pull/6128

   - Adds projection for each of the transforms
   - Removes the dataclasses from the expressions. I noticed that the IDE 
didn't like the way we were using the dataclasses in a hierarchical structure: 
   
![image](https://user-images.githubusercontent.com/1134248/200150958-3634a40c-a530-424c-956d-94b843bb5cb9.png)
   It complaints about unexpected arguments. Also, noticed that mypy was 
struggling with it. I've removed the dataclasses and added the abstract 
classes. I've implemented the `__eq__` and `__repr__` where required. Also 
removed some inconsistencies along the way (`In` was expecting a tuple, and 
`BoundIn` a set).
   
   More tests still to be added


-- 
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]

Reply via email to