Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-09-02 Thread via GitHub
ajantha-bhat commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1740747491 ## format/spec.md: ## @@ -113,9 +122,9 @@ Tables do not require random-access writes. Once written, data and metadata file Tables do not require rename, except

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-22 Thread via GitHub
rdblue merged PR #10948: URL: https://github.com/apache/iceberg/pull/10948 -- 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.ap

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-19 Thread via GitHub
rdblue commented on PR #10948: URL: https://github.com/apache/iceberg/pull/10948#issuecomment-2297312087 Thanks for reviewing, everyone. I've opened a thread on the dev list to merge this. -- This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
emkornfield commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1720475417 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, ve

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
emkornfield commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1720474856 ## format/spec.md: ## @@ -113,9 +122,9 @@ Tables do not require random-access writes. Once written, data and metadata file Tables do not require rename, except

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
rdblue commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1720451537 ## format/spec.md: ## @@ -1308,7 +1321,7 @@ Default values are added to struct fields in v3. Types `timestamp_ns` and `timestamptz_ns` are added in v3. -All reade

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
rdblue commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1720345226 ## format/spec.md: ## @@ -193,16 +204,14 @@ Supported primitive types are defined in the table below. Primitive types added Notes: -1. Decimal scale is fixed and

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
rdblue commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1720443701 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, version

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
rdblue commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1720346551 ## format/spec.md: ## @@ -1308,7 +1321,7 @@ Default values are added to struct fields in v3. Types `timestamp_ns` and `timestamptz_ns` are added in v3. -All reade

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
rdblue commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1720345226 ## format/spec.md: ## @@ -193,16 +204,14 @@ Supported primitive types are defined in the table below. Primitive types added Notes: -1. Decimal scale is fixed and

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
rdblue commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1720183537 ## format/spec.md: ## @@ -113,9 +122,9 @@ Tables do not require random-access writes. Once written, data and metadata file Tables do not require rename, except for t

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
rdblue commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1720181535 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, version

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
rdblue commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1720179715 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, version

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
rdblue commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1720178070 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, version

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
rdblue commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1720176739 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, version

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
Fokko commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1719458102 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, version

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
Fokko commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1719454561 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, version

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
Fokko commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1719442481 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, version

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
Fokko commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1719445797 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, version

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-16 Thread via GitHub
Fokko commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1719442481 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, version

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-15 Thread via GitHub
emkornfield commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1719268387 ## format/spec.md: ## @@ -1308,7 +1321,7 @@ Default values are added to struct fields in v3. Types `timestamp_ns` and `timestamptz_ns` are added in v3. -All

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-15 Thread via GitHub
emkornfield commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1719262786 ## format/spec.md: ## @@ -193,16 +204,14 @@ Supported primitive types are defined in the table below. Primitive types added Notes: -1. Decimal scale is fixe

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-15 Thread via GitHub
emkornfield commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1719261154 ## format/spec.md: ## @@ -113,9 +122,9 @@ Tables do not require random-access writes. Once written, data and metadata file Tables do not require rename, except

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-15 Thread via GitHub
emkornfield commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1719260843 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, ve

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-15 Thread via GitHub
emkornfield commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1719260575 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, ve

Re: [PR] Spec: Minor modifications for v3 [iceberg]

2024-08-15 Thread via GitHub
emkornfield commented on code in PR #10948: URL: https://github.com/apache/iceberg/pull/10948#discussion_r1719259618 ## format/spec.md: ## @@ -44,6 +44,15 @@ The primary change in version 2 adds delete files to encode rows that are delete In addition to row-level deletes, ve

[PR] Spec: Minor modifications for v3 [iceberg]

2024-08-15 Thread via GitHub
rdblue opened a new pull request, #10948: URL: https://github.com/apache/iceberg/pull/10948 This adds some minor changes to prepare for v3 spec work: * Adds a section to summarize the high-level v3 changes * Reduces some heading levels to leave room for better organization. The previou