Re: [PR] Fix uuid partition internal representation from str to bytes [iceberg-python]

2024-03-04 Thread via GitHub
Fokko commented on PR #489: URL: https://github.com/apache/iceberg-python/pull/489#issuecomment-1978153309 @jqin61 sorry for the confusion there. We do this for the Literal itself: https://github.com/apache/iceberg-python/blob/9d01b02f14e761b36fe0b88a82274037f7ced5a7/pyiceberg/expressions/li

Re: [PR] Fix uuid partition internal representation from str to bytes [iceberg-python]

2024-03-04 Thread via GitHub
jqin61 closed pull request #489: Fix uuid partition internal representation from str to bytes URL: https://github.com/apache/iceberg-python/pull/489 -- 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 t

Re: [PR] Fix uuid partition internal representation from str to bytes [iceberg-python]

2024-03-04 Thread via GitHub
jqin61 commented on PR #489: URL: https://github.com/apache/iceberg-python/pull/489#issuecomment-1976833227 Got it, was under previous discussion impression that we need to use bytes as internal representation of uuid. -- This is an automated message from the Apache Git Service. To respon