ccmao1130 commented on PR #2169:
URL: https://github.com/apache/iceberg-python/pull/2169#issuecomment-3053649259
Thanks Kevin!
--
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
kevinjqliu merged PR #2169:
URL: https://github.com/apache/iceberg-python/pull/2169
--
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-unsubscr...@i
kevinjqliu commented on PR #2169:
URL: https://github.com/apache/iceberg-python/pull/2169#issuecomment-3050095128
Looks like something changed in the newer version
This test failed
https://github.com/apache/iceberg-python/blob/e33cf5ac1adf47131d4992bdb686f0e58f4e4669/tests/integrat
ccmao1130 commented on PR #2169:
URL: https://github.com/apache/iceberg-python/pull/2169#issuecomment-3050036221
@kevinjqliu Okay I think it should be okay now? (sorry not a developer
hahaha)
--
This is an automated message from the Apache Git Service.
To respond to the message, pleas
kevinjqliu commented on PR #2169:
URL: https://github.com/apache/iceberg-python/pull/2169#issuecomment-3047138488
@ccmao1130 could you run `make install` and then `poetry lock` locally? That
should resolve the CI issue
--
This is an automated message from the Apache Git Service.
To respon
ccmao1130 commented on code in PR #2169:
URL: https://github.com/apache/iceberg-python/pull/2169#discussion_r2190844881
##
pyproject.toml:
##
@@ -78,7 +78,7 @@ gcsfs = { version = ">=2023.1.0", optional = true }
huggingface-hub = { version = ">=0.24.0", optional = true }
psyco
Fokko commented on code in PR #2169:
URL: https://github.com/apache/iceberg-python/pull/2169#discussion_r2184700624
##
pyproject.toml:
##
@@ -78,7 +78,7 @@ gcsfs = { version = ">=2023.1.0", optional = true }
huggingface-hub = { version = ">=0.24.0", optional = true }
psycopg2-
ccmao1130 opened a new pull request, #2169:
URL: https://github.com/apache/iceberg-python/pull/2169
# Rationale for this change
We recently changed our site domain so want to update all Daft documentation
links. And noticed that our package should be updated from `
