Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2708059145 Hey @kevinjqliu Did you get a chance to look at the PR again? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2677114643 Hey @kevinjqliu I have resolved all comments. Will resolve the merge conflicts once you have gone through these changes (to avoid rework). Let me know once you have revie

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1966883670 ## tests/table/test_init.py: ## @@ -982,28 +982,43 @@ def test_assert_table_uuid(table_v2: Table) -> None: def test_assert_ref_snapshot_id(table_v2: Table) -> No

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1966882693 ## pyiceberg/table/__init__.py: ## @@ -1003,22 +1015,27 @@ def overwrite( overwrite_filter: ALWAYS_TRUE when you overwrite all the data,

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1966881022 ## tests/table/test_init.py: ## @@ -982,28 +982,43 @@ def test_assert_table_uuid(table_v2: Table) -> None: def test_assert_ref_snapshot_id(table_v2: Table) -> No

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1966876035 ## tests/integration/test_writes/test_writes.py: ## @@ -1570,3 +1572,158 @@ def test_abort_table_transaction_on_exception( # Validate the transaction is abor

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1966875880 ## pyiceberg/table/update/__init__.py: ## @@ -609,11 +609,14 @@ class AssertRefSnapshotId(ValidatableTableRequirement): type: Literal["assert-ref-snapshot-id

Re: [PR] Feature: Write to branches [iceberg-python]

kevinjqliu commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2629019931 thank you! @vinjai feel free to tag me again for review :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2628949861 Hey @kevinjqliu I was traveling for the past two months and couldn’t complete the review comments. I’ll be back in a week, will address the comments then, and get this moving.

Re: [PR] Feature: Write to branches [iceberg-python]

kevinjqliu commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2628614211 hey @vinjai are you interested to pick this back up? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] Feature: Write to branches [iceberg-python]

kevinjqliu commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1866354514 ## pyiceberg/table/update/__init__.py: ## @@ -609,11 +609,14 @@ class AssertRefSnapshotId(ValidatableTableRequirement): type: Literal["assert-ref-snapsho

Re: [PR] Feature: Write to branches [iceberg-python]

kevinjqliu commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1866358508 ## tests/table/test_init.py: ## @@ -982,28 +982,43 @@ def test_assert_table_uuid(table_v2: Table) -> None: def test_assert_ref_snapshot_id(table_v2: Table) -

Re: [PR] Feature: Write to branches [iceberg-python]

kevinjqliu commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2512331585 @vinjai coming back to this after the 0.8.1 release :) Feel free to tag me again once the comments above are addressed. Thanks again for the contribution! -- This is an aut

Re: [PR] Feature: Write to branches [iceberg-python]

kevinjqliu commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1866359013 ## tests/table/test_init.py: ## @@ -982,28 +982,43 @@ def test_assert_table_uuid(table_v2: Table) -> None: def test_assert_ref_snapshot_id(table_v2: Table) -

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1865177262 ## tests/table/test_init.py: ## @@ -982,28 +982,43 @@ def test_assert_table_uuid(table_v2: Table) -> None: def test_assert_ref_snapshot_id(table_v2: Table) -> No

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1865176166 ## tests/table/test_init.py: ## @@ -982,28 +982,43 @@ def test_assert_table_uuid(table_v2: Table) -> None: def test_assert_ref_snapshot_id(table_v2: Table) -> No

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1855301400 ## tests/table/test_init.py: ## @@ -982,28 +982,43 @@ def test_assert_table_uuid(table_v2: Table) -> None: def test_assert_ref_snapshot_id(table_v2: Table) -> No

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1855299774 ## pyiceberg/table/update/__init__.py: ## @@ -609,11 +609,14 @@ class AssertRefSnapshotId(ValidatableTableRequirement): type: Literal["assert-ref-snapshot-id

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1855299357 ## pyiceberg/table/__init__.py: ## @@ -1003,22 +1015,27 @@ def overwrite( overwrite_filter: ALWAYS_TRUE when you overwrite all the data,

Re: [PR] Feature: Write to branches [iceberg-python]

kevinjqliu commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2491889472 cc @HonahX / @Fokko / @sungwy -- 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

Re: [PR] Feature: Write to branches [iceberg-python]

kevinjqliu commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1852553109 ## tests/table/test_init.py: ## @@ -982,28 +982,43 @@ def test_assert_table_uuid(table_v2: Table) -> None: def test_assert_ref_snapshot_id(table_v2: Table) -

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2487692994 @kevinjqliu What are the next steps to get this merged? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] Feature: Write to branches [iceberg-python]

MChamberlin commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2483824116 @kevinjqliu Is there anyway this can be included in the 0.8.0 release? This is an important feature add with only relatively simple refactoring. -- This is an automated message

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1841381304 ## tests/integration/test_writes/test_writes.py: ## @@ -1448,3 +1450,100 @@ def test_rewrite_manifest_after_partition_evolution(session_catalog: Catalog) ->

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2475142299 Thank you for the review @kevinjqliu -- 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] Feature: Write to branches [iceberg-python]

vinjai commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1841381304 ## tests/integration/test_writes/test_writes.py: ## @@ -1448,3 +1450,100 @@ def test_rewrite_manifest_after_partition_evolution(session_catalog: Catalog) ->

Re: [PR] Feature: Write to branches [iceberg-python]

kevinjqliu commented on code in PR #941: URL: https://github.com/apache/iceberg-python/pull/941#discussion_r1835751569 ## tests/integration/test_writes/test_writes.py: ## @@ -1448,3 +1450,100 @@ def test_rewrite_manifest_after_partition_evolution(session_catalog: Catalog) ->

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2463348171 Hey @kevinjqliu Did you get a chance to look at this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2424158737 I have mostly tried to cover all edge cases. The idea is that the branch is just another iceberg table where the snapshots append independently of the main branch. I also agr

Re: [PR] Feature: Write to branches [iceberg-python]

kevinjqliu commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2424130438 Thanks for the contribution! I'll take a look. I remember adding support for branch is complicated since we need to consider different edge cases. -- This is an automated m

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2417891570 Fixed another bug. Writes with same name but different ref types Please review whenever you get some time. -- This is an automated message from the Apache Git Service. To respo

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2416056139 @sungwy @kevinjqliu This PR is ready for review -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

Re: [PR] Feature: Write to branches [iceberg-python]

vinjai commented on PR #941: URL: https://github.com/apache/iceberg-python/pull/941#issuecomment-2416055849 @sungwy @kevinjqliu This PR is ready for review -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab