Fokko commented on PR #419:
URL: https://github.com/apache/iceberg-python/pull/419#issuecomment-2007184688
Thanks @Gowthami03B for adding this 🙌
--
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
Fokko merged PR #419:
URL: https://github.com/apache/iceberg-python/pull/419
--
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...@iceberg.
Fokko commented on code in PR #419:
URL: https://github.com/apache/iceberg-python/pull/419#discussion_r1529938224
##
pyiceberg/table/__init__.py:
##
@@ -2487,6 +2491,7 @@ def __init__(
operation: Operation,
transaction: Transaction,
io: FileIO,
+
Gowthami03B commented on code in PR #419:
URL: https://github.com/apache/iceberg-python/pull/419#discussion_r1525754450
##
pyiceberg/table/__init__.py:
##
@@ -1019,21 +1020,27 @@ def append(self, df: pa.Table) -> None:
if len(self.spec().fields) > 0:
raise
Gowthami03B commented on code in PR #419:
URL: https://github.com/apache/iceberg-python/pull/419#discussion_r1524869065
##
pyiceberg/table/__init__.py:
##
@@ -1019,21 +1020,27 @@ def append(self, df: pa.Table) -> None:
if len(self.spec().fields) > 0:
raise
Fokko commented on code in PR #419:
URL: https://github.com/apache/iceberg-python/pull/419#discussion_r1524454409
##
pyiceberg/table/__init__.py:
##
@@ -1019,21 +1020,27 @@ def append(self, df: pa.Table) -> None:
if len(self.spec().fields) > 0:
raise ValueE
Fokko commented on code in PR #419:
URL: https://github.com/apache/iceberg-python/pull/419#discussion_r1496570473
##
pyiceberg/table/__init__.py:
##
@@ -1019,21 +1020,27 @@ def append(self, df: pa.Table) -> None:
if len(self.spec().fields) > 0:
raise ValueE
Gowthami03B commented on PR #419:
URL: https://github.com/apache/iceberg-python/pull/419#issuecomment-1955150881
> @Gowthami03B Now the release is out, we can merge this. Could you fix the
merge conflicts? Thanks!
@Fokko Done!
--
This is an automated message from the Apache Git Ser
Fokko commented on PR #419:
URL: https://github.com/apache/iceberg-python/pull/419#issuecomment-1955098638
@Gowthami03B Now the release is out, we can merge this. Could you fix the
merge conflicts? Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the mess
Fokko commented on code in PR #419:
URL: https://github.com/apache/iceberg-python/pull/419#discussion_r1489812651
##
mkdocs/docs/api.md:
##
@@ -446,6 +446,20 @@ table =
table.transaction().remove_properties("abc").commit_transaction()
assert table.properties == {}
```
+## S
10 matches
Mail list logo
