Re: [I] PyIceberg Production Use case survey [iceberg-python]

2024-10-27 Thread via GitHub
djouallah commented on issue #1202: URL: https://github.com/apache/iceberg-python/issues/1202#issuecomment-2440232128 I use it mainly for testing xtable conversion from iceberg to delta, it is by the far the easiest way to generate Iceberg tables :) -- This is an automated message from

Re: [I] Iceberg Glue - Timeouts (maybe others client side error cases) can result in missing metadata_location [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on issue #9618: URL: https://github.com/apache/iceberg/issues/9618#issuecomment-2440238550 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache Gi

Re: [I] add manifest path to all_entries meta table [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on issue #9608: URL: https://github.com/apache/iceberg/issues/9608#issuecomment-2440238506 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache Gi

Re: [I] Iceberg Glue - Timeouts (maybe others client side error cases) can result in missing metadata_location [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed issue #9618: Iceberg Glue - Timeouts (maybe others client side error cases) can result in missing metadata_location URL: https://github.com/apache/iceberg/issues/9618 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [I] Migrate RESTCatalogServlet to use jakarta.* package for Spring boot 3 [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed issue #9626: Migrate RESTCatalogServlet to use jakarta.* package for Spring boot 3 URL: https://github.com/apache/iceberg/issues/9626 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

Re: [I] Migrate RESTCatalogServlet to use jakarta.* package for Spring boot 3 [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on issue #9626: URL: https://github.com/apache/iceberg/issues/9626#issuecomment-2440238570 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache Gi

Re: [I] HMS lock timeout [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed issue #9654: HMS lock timeout URL: https://github.com/apache/iceberg/issues/9654 -- 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-ma

Re: [I] HMS lock timeout [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on issue #9654: URL: https://github.com/apache/iceberg/issues/9654#issuecomment-2440238585 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache Gi

Re: [I] Docs: Go over docs to check rendering of pages/sections [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed issue #9657: Docs: Go over docs to check rendering of pages/sections URL: https://github.com/apache/iceberg/issues/9657 -- 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: [I] Docs: Go over docs to check rendering of pages/sections [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on issue #9657: URL: https://github.com/apache/iceberg/issues/9657#issuecomment-2440238606 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache Gi

Re: [I] Docs: Add Mandarin translation of the docs site [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed issue #9665: Docs: Add Mandarin translation of the docs site URL: https://github.com/apache/iceberg/issues/9665 -- 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 specifi

Re: [I] Docs: Add Mandarin translation of the docs site [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on issue #9665: URL: https://github.com/apache/iceberg/issues/9665#issuecomment-2440238626 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache Gi

Re: [I] Core: complete task JSON serialization for other types (like data task, manifest task) [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed issue #9597: Core: complete task JSON serialization for other types (like data task, manifest task) URL: https://github.com/apache/iceberg/issues/9597 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [I] Performance optimization [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on issue #9588: URL: https://github.com/apache/iceberg/issues/9588#issuecomment-2440238459 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache Gi

Re: [I] Core: complete task JSON serialization for other types (like data task, manifest task) [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on issue #9597: URL: https://github.com/apache/iceberg/issues/9597#issuecomment-2440238486 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache Gi

Re: [I] `latestOffset` call in SparkMicroBatchStream limits files and records based on the startTs and not last processed snapshot [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on issue #9593: URL: https://github.com/apache/iceberg/issues/9593#issuecomment-2440238471 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache Gi

Re: [I] add manifest path to all_entries meta table [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed issue #9608: add manifest path to all_entries meta table URL: https://github.com/apache/iceberg/issues/9608 -- 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 co

Re: [I] Fix header links with underscores in title. [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on issue #9617: URL: https://github.com/apache/iceberg/issues/9617#issuecomment-2440238533 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache Gi

Re: [I] Fix header links with underscores in title. [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed issue #9617: Fix header links with underscores in title. URL: https://github.com/apache/iceberg/issues/9617 -- 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 co

Re: [I] Performance optimization [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed issue #9588: Performance optimization URL: https://github.com/apache/iceberg/issues/9588 -- 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 unsubscri

Re: [PR] Spark: Support rewrite file with z-order for nested Struct type [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on PR #9818: URL: https://github.com/apache/iceberg/pull/9818#issuecomment-2440238751 This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If y

Re: [PR] Views, Spark: Add support for Materialized Views; Integrate with Spark SQL [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed pull request #9830: Views, Spark: Add support for Materialized Views; Integrate with Spark SQL URL: https://github.com/apache/iceberg/pull/9830 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

Re: [PR] Spark: Support rewrite file with z-order for nested Struct type [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed pull request #9818: Spark: Support rewrite file with z-order for nested Struct type URL: https://github.com/apache/iceberg/pull/9818 -- 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

Re: [PR] Views, Spark: Add support for Materialized Views; Integrate with Spark SQL [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on PR #9830: URL: https://github.com/apache/iceberg/pull/9830#issuecomment-2440238796 This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If y

Re: [I] `latestOffset` call in SparkMicroBatchStream limits files and records based on the startTs and not last processed snapshot [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed issue #9593: `latestOffset` call in SparkMicroBatchStream limits files and records based on the startTs and not last processed snapshot URL: https://github.com/apache/iceberg/issues/9593 -- This is an automated message from the Apache Git Service. To respond to the

Re: [I] Iceberg Rewrite DataFiles unmanageable behavior [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] closed issue #9674: Iceberg Rewrite DataFiles unmanageable behavior URL: https://github.com/apache/iceberg/issues/9674 -- 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 specifi

Re: [I] Iceberg Rewrite DataFiles unmanageable behavior [iceberg]

2024-10-27 Thread via GitHub
github-actions[bot] commented on issue #9674: URL: https://github.com/apache/iceberg/issues/9674#issuecomment-2440238641 This issue has been closed because it has not received any activity in the last 14 days since being marked as 'stale' -- This is an automated message from the Apache Gi

[I] add an option to automatically use the table name in the path [iceberg-python]

2024-10-27 Thread via GitHub
djouallah opened a new issue, #1254: URL: https://github.com/apache/iceberg-python/issues/1254 ### Feature Request / Improvement currently by default, pyiceberg write all table inside the same path, which is arguable a very bad idea as later, there is no way to provide a fine level s

Re: [PR] Iceberg/Comet integration POC [iceberg]

2024-10-27 Thread via GitHub
bmorck commented on PR #9841: URL: https://github.com/apache/iceberg/pull/9841#issuecomment-2440275506 @huaxingao Very interested in this work and thanks a bunch for taking this on! Wanted to see if this PR addresses all changes needed on the iceberg side needed to integrate comet with iceb

Re: [PR] Flink 1.20: Update Flink to use planned Avro reads [iceberg]

2024-10-27 Thread via GitHub
jbonofre commented on PR #11386: URL: https://github.com/apache/iceberg/pull/11386#issuecomment-2440687833 I fixed all types (primitves and arrays) reads. Tests should be happy now πŸ˜„ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [I] failed to access class org.apache.iceberg.flink.FlinkEnvironmentContext from class org.apache.iceberg.flink.TableLoader$HadoopTableLoader [iceberg]

2024-10-27 Thread via GitHub
pvary commented on issue #11408: URL: https://github.com/apache/iceberg/issues/11408#issuecomment-2440681436 > Normally, this error is caught by the compiler; this error can only occur at run time if the definition of a class has incompatibly changed. Could you please check the class

Re: [PR] Flink 1.20: Update Flink to use planned Avro reads [iceberg]

2024-10-27 Thread via GitHub
jbonofre commented on PR #11386: URL: https://github.com/apache/iceberg/pull/11386#issuecomment-2439933515 I'm fixing a new `ClassCastException`. -- 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] abort the whole table transaction if any updates in the transaction has failed [iceberg-python]

2024-10-27 Thread via GitHub
stevie9868 commented on code in PR #1246: URL: https://github.com/apache/iceberg-python/pull/1246#discussion_r1818144237 ## pyiceberg/table/__init__.py: ## @@ -231,9 +233,13 @@ def __enter__(self) -> Transaction: """Start a transaction to update the table.""" r

Re: [PR] Build: Upgrade to Gradle 7.x [iceberg]

2024-10-27 Thread via GitHub
shivjha30 commented on code in PR #2826: URL: https://github.com/apache/iceberg/pull/2826#discussion_r1818346539 ## build.gradle: ## @@ -247,13 +250,30 @@ project(':iceberg-data') { exclude group: 'org.slf4j', module: 'slf4j-log4j12' } -testCompile("org.apache.

Re: [PR] Build: Upgrade to Gradle 7.x [iceberg]

2024-10-27 Thread via GitHub
shivjha30 commented on code in PR #2826: URL: https://github.com/apache/iceberg/pull/2826#discussion_r1818346539 ## build.gradle: ## @@ -247,13 +250,30 @@ project(':iceberg-data') { exclude group: 'org.slf4j', module: 'slf4j-log4j12' } -testCompile("org.apache.

Re: [PR] Data: Add partition stats writer and reader [iceberg]

2024-10-27 Thread via GitHub
ajantha-bhat commented on code in PR #11216: URL: https://github.com/apache/iceberg/pull/11216#discussion_r1818391768 ## core/src/main/java/org/apache/iceberg/data/PartitionStatsRecord.java: ## @@ -0,0 +1,170 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] Add `all_manifests` metadata table with tests [iceberg-python]

2024-10-27 Thread via GitHub
kevinjqliu commented on code in PR #1241: URL: https://github.com/apache/iceberg-python/pull/1241#discussion_r1818300495 ## pyiceberg/table/inspect.py: ## @@ -32,6 +33,41 @@ from pyiceberg.table import Table +def get_manifests_schema() -> "pa.Schema": Review Comment:

Re: [I] Flaky test `TestFlinkIcebergSinkRangeDistributionBucketing > testBucketNumberHigherThanWriterParallelismNotDivisible()` [iceberg]

2024-10-27 Thread via GitHub
stevenzwu commented on issue #11397: URL: https://github.com/apache/iceberg/issues/11397#issuecomment-2440488043 probably the PR with failed test was created before the disabling is merged. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] chore(deps): Bump crate-ci/typos from 1.26.0 to 1.26.8 [iceberg-rust]

2024-10-27 Thread via GitHub
liurenjie1024 merged PR #683: URL: https://github.com/apache/iceberg-rust/pull/683 -- 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...@ic

Re: [PR] Data: Add partition stats writer and reader [iceberg]

2024-10-27 Thread via GitHub
ajantha-bhat commented on code in PR #11216: URL: https://github.com/apache/iceberg/pull/11216#discussion_r1818418685 ## core/src/main/java/org/apache/iceberg/PartitionStats.java: ## @@ -249,4 +250,45 @@ public void set(int pos, T value) { throw new UnsupportedOperatio

Re: [PR] Data: Add partition stats writer and reader [iceberg]

2024-10-27 Thread via GitHub
ajantha-bhat commented on code in PR #11216: URL: https://github.com/apache/iceberg/pull/11216#discussion_r1818418685 ## core/src/main/java/org/apache/iceberg/PartitionStats.java: ## @@ -249,4 +250,45 @@ public void set(int pos, T value) { throw new UnsupportedOperatio

Re: [PR] abort the whole table transaction if any updates in the transaction has failed [iceberg-python]

2024-10-27 Thread via GitHub
kevinjqliu commented on code in PR #1246: URL: https://github.com/apache/iceberg-python/pull/1246#discussion_r1818186347 ## tests/catalog/test_base.py: ## @@ -766,3 +766,26 @@ def test_table_properties_raise_for_none_value(catalog: InMemoryCatalog) -> None with pytest.rais

Re: [I] Add view support to the Rest Catalog [iceberg-python]

2024-10-27 Thread via GitHub
kevinjqliu commented on issue #818: URL: https://github.com/apache/iceberg-python/issues/818#issuecomment-2440219672 > look at how load_table works today: we return a Table model with all the metadata about the table, and this model exposes functionality for data scans, etc. So load_view wo

[PR] build(deps): bump github.com/aws/aws-sdk-go-v2/service/s3 from 1.66.0 to 1.66.1 [iceberg-go]

2024-10-27 Thread via GitHub
dependabot[bot] opened a new pull request, #184: URL: https://github.com/apache/iceberg-go/pull/184 Bumps [github.com/aws/aws-sdk-go-v2/service/s3](https://github.com/aws/aws-sdk-go-v2) from 1.66.0 to 1.66.1. Commits https://github.com/aws/aws-sdk-go-v2/commit/896793a6822d2cec

[I] failed to access class org.apache.iceberg.flink.FlinkEnvironmentContext from class org.apache.iceberg.flink.TableLoader$HadoopTableLoader [iceberg]

2024-10-27 Thread via GitHub
hashmapybx opened a new issue, #11408: URL: https://github.com/apache/iceberg/issues/11408 ### Apache Iceberg version 1.4.3 ### Query engine Flink ### Please describe the bug 🐞 I try to execution compaction small flie by flink-rewrite-action,https://iceberg

Re: [I] failed to access class org.apache.iceberg.flink.FlinkEnvironmentContext from class org.apache.iceberg.flink.TableLoader$HadoopTableLoader [iceberg]

2024-10-27 Thread via GitHub
hashmapybx commented on issue #11408: URL: https://github.com/apache/iceberg/issues/11408#issuecomment-2440059118 System.out.println("ζ‰§θ‘ŒεˆεΉΆζ“δ½œ"); LOG.info("===ζ‰§θ‘ŒεˆεΉΆζ“δ½œ="); ParameterTool tool = ParameterTool.fromArgs(args); St

Re: [I] failed to access class org.apache.iceberg.flink.FlinkEnvironmentContext from class org.apache.iceberg.flink.TableLoader$HadoopTableLoader [iceberg]

2024-10-27 Thread via GitHub
hashmapybx commented on issue #11408: URL: https://github.com/apache/iceberg/issues/11408#issuecomment-2440059316 https://github.com/user-attachments/assets/04c36422-557f-461c-aca8-c7a90127081b";> -- This is an automated message from the Apache Git Service. To respond to the message, p

Re: [PR] abort the whole table transaction if any updates in the transaction has failed [iceberg-python]

2024-10-27 Thread via GitHub
stevie9868 commented on PR #1246: URL: https://github.com/apache/iceberg-python/pull/1246#issuecomment-2440114506 > However, there may still be some concerns around this since Transaction is a public class. If this is the case, I think we can start from making the parameter "private" (autoc

Re: [PR] abort the whole table transaction if any updates in the transaction has failed [iceberg-python]

2024-10-27 Thread via GitHub
stevie9868 commented on PR #1246: URL: https://github.com/apache/iceberg-python/pull/1246#issuecomment-2440116074 I have also updated the PR based on existing comments, and thanks everyone for the inputs! -- This is an automated message from the Apache Git Service. To respond to the messa

Re: [PR] abort the whole table transaction if any updates in the transaction has failed [iceberg-python]

2024-10-27 Thread via GitHub
stevie9868 commented on code in PR #1246: URL: https://github.com/apache/iceberg-python/pull/1246#discussion_r1818151924 ## tests/catalog/test_base.py: ## @@ -265,6 +268,42 @@ def drop_view(self, identifier: Union[str, Identifier]) -> None: raise NotImplementedError

[PR] chore(deps): Bump crate-ci/typos from 1.26.0 to 1.26.8 [iceberg-rust]

2024-10-27 Thread via GitHub
dependabot[bot] opened a new pull request, #683: URL: https://github.com/apache/iceberg-rust/pull/683 Bumps [crate-ci/typos](https://github.com/crate-ci/typos) from 1.26.0 to 1.26.8. Release notes Sourced from https://github.com/crate-ci/typos/releases";>crate-ci/typos's releases.

Re: [PR] abort the whole table transaction if any updates in the transaction has failed [iceberg-python]

2024-10-27 Thread via GitHub
kevinjqliu commented on code in PR #1246: URL: https://github.com/apache/iceberg-python/pull/1246#discussion_r1818185830 ## pyiceberg/table/__init__.py: ## @@ -231,9 +233,13 @@ def __enter__(self) -> Transaction: """Start a transaction to update the table.""" r

Re: [PR] abort the whole table transaction if any updates in the transaction has failed [iceberg-python]

2024-10-27 Thread via GitHub
kevinjqliu commented on code in PR #1246: URL: https://github.com/apache/iceberg-python/pull/1246#discussion_r1818185562 ## pyiceberg/table/__init__.py: ## @@ -231,9 +233,13 @@ def __enter__(self) -> Transaction: """Start a transaction to update the table.""" r

Re: [PR] abort the whole table transaction if any updates in the transaction has failed [iceberg-python]

2024-10-27 Thread via GitHub
kevinjqliu commented on code in PR #1246: URL: https://github.com/apache/iceberg-python/pull/1246#discussion_r1818186164 ## tests/catalog/test_base.py: ## @@ -766,3 +766,26 @@ def test_table_properties_raise_for_none_value(catalog: InMemoryCatalog) -> None with pytest.rais

Re: [PR] Impl rest catalog + table updates & requirements [iceberg-go]

2024-10-27 Thread via GitHub
jwtryg commented on PR #146: URL: https://github.com/apache/iceberg-go/pull/146#issuecomment-2440181205 @zeroshade thank you for your patience :) I have implemented your feedback and made some other changes - specifically, being more mindful of what should be exported, and then undoin

Re: [PR] abort the whole table transaction if any updates in the transaction has failed [iceberg-python]

2024-10-27 Thread via GitHub
kevinjqliu commented on code in PR #1246: URL: https://github.com/apache/iceberg-python/pull/1246#discussion_r1818186598 ## tests/catalog/test_base.py: ## @@ -766,3 +766,26 @@ def test_table_properties_raise_for_none_value(catalog: InMemoryCatalog) -> None with pytest.rais

Re: [PR] abort the whole table transaction if any updates in the transaction has failed [iceberg-python]

2024-10-27 Thread via GitHub
kevinjqliu commented on code in PR #1246: URL: https://github.com/apache/iceberg-python/pull/1246#discussion_r1818186725 ## tests/catalog/test_base.py: ## @@ -766,3 +766,26 @@ def test_table_properties_raise_for_none_value(catalog: InMemoryCatalog) -> None with pytest.rais

[I] [discuss] `Transaction` API's `autocommit` [iceberg-python]

2024-10-27 Thread via GitHub
kevinjqliu opened a new issue, #1253: URL: https://github.com/apache/iceberg-python/issues/1253 ### Feature Request / Improvement Porting over from #1246 Can be a potential footgun (https://github.com/apache/iceberg-python/pull/1246#issuecomment-2439024843) Autocommit usage

Re: [PR] abort the whole table transaction if any updates in the transaction has failed [iceberg-python]

2024-10-27 Thread via GitHub
kevinjqliu commented on PR #1246: URL: https://github.com/apache/iceberg-python/pull/1246#issuecomment-2440184081 Thanks @HonahX @stevie9868! Glad we were able to get to the bottom of this important correctness issue. I started #1253 to continue the conversation on `autocommit`

Re: [I] [discuss] `Transaction` API's `autocommit` [iceberg-python]

2024-10-27 Thread via GitHub
kevinjqliu commented on issue #1253: URL: https://github.com/apache/iceberg-python/issues/1253#issuecomment-2440186317 > The idea is to make the code simpler if we only want to evolve schema/spec/... i.e. ``` with table.update_schema() as update: update.add_column("some_fiel

[PR] Revert "Core: Snapshot `summary` map must have `operation` key" [iceberg]

2024-10-27 Thread via GitHub
kevinjqliu opened a new pull request, #11409: URL: https://github.com/apache/iceberg/pull/11409 Reverts apache/iceberg#11354 -- 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.

Re: [PR] Core: Snapshot `summary` map must have `operation` key [iceberg]

2024-10-27 Thread via GitHub
kevinjqliu commented on code in PR #11354: URL: https://github.com/apache/iceberg/pull/11354#discussion_r1818193692 ## core/src/main/java/org/apache/iceberg/SnapshotParser.java: ## @@ -129,13 +129,12 @@ static Snapshot fromJson(JsonNode node) { "Cannot parse summary f

Re: [PR] Core: Snapshot `summary` map must have `operation` key [iceberg]

2024-10-27 Thread via GitHub
kevinjqliu commented on code in PR #11354: URL: https://github.com/apache/iceberg/pull/11354#discussion_r1818193692 ## core/src/main/java/org/apache/iceberg/SnapshotParser.java: ## @@ -129,13 +129,12 @@ static Snapshot fromJson(JsonNode node) { "Cannot parse summary f