Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

Fokko merged PR #12334: URL: https://github.com/apache/iceberg/pull/12334 -- 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.apa

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

Fokko commented on PR #12334: URL: https://github.com/apache/iceberg/pull/12334#issuecomment-2703056127 See https://lists.apache.org/thread/54r4nm7qmr4vxhdpwmbx5rntynspskl7 Thanks everyone! -- This is an automated message from the Apache Git Service. To respond to the message, pleas

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

Fokko commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1978969371 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

szehon-ho commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1978283986 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

szehon-ho commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1978283986 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

szehon-ho commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1978283986 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

Fokko commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1978270259 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

Fokko commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1978271846 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

danielcweeks commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1968935239 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spa

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

liurenjie1024 commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1968725930 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "sp

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

szehon-ho commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1968535875 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

rdblue commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1968529807 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

Fokko commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1967996821 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

emkornfield commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1966610820 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spar

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

danielcweeks commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1966349930 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spa

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

danielcweeks commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1966349930 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spa

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

danielcweeks commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1966349930 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spa

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

emkornfield commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1966257174 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spar

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

liurenjie1024 commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1965226276 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "sp

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

wgtmac commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1965136812 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

wgtmac commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1965136812 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

Fokko commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1965123936 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

Fokko commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1965123936 ## format/spec.md: ## @@ -1754,6 +1754,14 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

szehon-ho commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1962368522 ## format/spec.md: ## @@ -1754,6 +1754,10 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

Fokko commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1962342015 ## format/spec.md: ## @@ -1754,6 +1754,10 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

Fokko commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1962342015 ## format/spec.md: ## @@ -1754,6 +1754,10 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

szehon-ho commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1962283684 ## format/spec.md: ## @@ -1754,6 +1754,10 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

szehon-ho commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1962283684 ## format/spec.md: ## @@ -1754,6 +1754,10 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

Fokko commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1962224337 ## format/spec.md: ## @@ -1754,6 +1754,10 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "spark"

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

RussellSpitzer commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1961967419 ## format/spec.md: ## @@ -1754,6 +1754,10 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "s

Re: [PR] Spec: Add implementation note on `current-snapshot-id` [iceberg]

RussellSpitzer commented on code in PR #12334: URL: https://github.com/apache/iceberg/pull/12334#discussion_r1961967419 ## format/spec.md: ## @@ -1754,6 +1754,10 @@ Snapshot summary can include metrics fields to track numeric stats of the snapsh | **`engine-name`**| "s