[GitHub] [iceberg] CircArgs commented on pull request #4816: remove operations from base expressions; add in expression

[GitBox <gi...@apache.org>]

CircArgs commented on PR #4816:
URL: https://github.com/apache/iceberg/pull/4816#issuecomment-1159209476

   Hey @rdblue sorry for the delay I was on vacation. I see I didn't change the abstract definition attribute names so I've changed that now. Do you also mean the `And` and `Or` as well? I've left those two alone for now because it's not clear to me why they would have the names `term` and `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: issues-unsubscribe@iceberg.apache.org

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


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