iceberg-python icon indicating copy to clipboard operation
iceberg-python copied to clipboard

The `between` keyword seems to be missing from pyiceberg expression

Open maytasm opened this issue 1 year ago • 1 comments

Feature Request / Improvement

It used to be there in the old python iceberg: https://github.com/apache/iceberg/commit/a49fb924cd6ce0da688f873b9107e91f597529d4#diff-36a7624a2703d8198b7c73c2424890c094fa4734ac76babdf0519a8cbd6fad39R92

maytasm avatar Jun 03 '24 19:06 maytasm

@maytasm I'm open to adding this. I saw that in legacy python it is inclusive, and so it is for SQL. We can update parser.py to rewrite this a between 19 and 25 to a <= 19 and a >= 25.

Fokko avatar Jun 03 '24 20:06 Fokko

This issue has been automatically marked as stale because it has been open for 180 days with no activity. It will be closed in next 14 days if no further activity occurs. To permanently prevent this issue from being considered stale, add the label 'not-stale', but commenting on the issue is preferred when possible.

github-actions[bot] avatar Dec 01 '24 00:12 github-actions[bot]

This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale'

github-actions[bot] avatar Dec 16 '24 00:12 github-actions[bot]