Re: [PR] Spec: Deprecate the file system table scheme. [iceberg]

2024-08-05 Thread via GitHub
manuzhang commented on code in PR #10833: URL: https://github.com/apache/iceberg/pull/10833#discussion_r1704850668 ## format/spec.md: ## @@ -1393,4 +1395,4 @@ This section covers topics not required by the specification but recommendations Iceberg supports two types of histori

Re: [PR] Spec: Deprecate the file system table scheme. [iceberg]

2024-08-05 Thread via GitHub
amogh-jahagirdar commented on code in PR #10833: URL: https://github.com/apache/iceberg/pull/10833#discussion_r1704357131 ## format/spec.md: ## @@ -1393,4 +1395,4 @@ This section covers topics not required by the specification but recommendations Iceberg supports two types of

Re: [PR] Spec: Deprecate the file system table scheme. [iceberg]

2024-08-05 Thread via GitHub
ajantha-bhat commented on code in PR #10833: URL: https://github.com/apache/iceberg/pull/10833#discussion_r1704346921 ## format/spec.md: ## @@ -1393,4 +1395,4 @@ This section covers topics not required by the specification but recommendations Iceberg supports two types of hist

Re: [PR] Spec: Deprecate the file system table scheme. [iceberg]

2024-08-05 Thread via GitHub
ajantha-bhat commented on code in PR #10833: URL: https://github.com/apache/iceberg/pull/10833#discussion_r1704346921 ## format/spec.md: ## @@ -1393,4 +1395,4 @@ This section covers topics not required by the specification but recommendations Iceberg supports two types of hist

Re: [PR] Spec: Deprecate the file system table scheme. [iceberg]

2024-08-04 Thread via GitHub
rdblue merged PR #10833: URL: https://github.com/apache/iceberg/pull/10833 -- 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: Deprecate the file system table scheme. [iceberg]

2024-08-04 Thread via GitHub
rdblue commented on code in PR #10833: URL: https://github.com/apache/iceberg/pull/10833#discussion_r1703399862 ## format/spec.md: ## @@ -779,7 +779,9 @@ When two commits happen at the same time and are based on the same version, only File System Tables -An atomic swa

Re: [PR] Spec: Deprecate the file system table scheme. [iceberg]

2024-08-01 Thread via GitHub
danielcweeks commented on code in PR #10833: URL: https://github.com/apache/iceberg/pull/10833#discussion_r1700400015 ## format/spec.md: ## @@ -779,7 +779,9 @@ When two commits happen at the same time and are based on the same version, only File System Tables -An atom

Re: [PR] Spec: Deprecate the file system table scheme. [iceberg]

2024-08-01 Thread via GitHub
nastra commented on code in PR #10833: URL: https://github.com/apache/iceberg/pull/10833#discussion_r1699610745 ## format/spec.md: ## @@ -1393,4 +1395,4 @@ This section covers topics not required by the specification but recommendations Iceberg supports two types of histories

Re: [PR] Spec: Deprecate the file system table scheme. [iceberg]

2024-07-31 Thread via GitHub
manuzhang commented on PR #10833: URL: https://github.com/apache/iceberg/pull/10833#issuecomment-2261960418 We can hide whitespace in review https://github.com/apache/iceberg/pull/10833/files?diff=split&w=1. -- This is an automated message from the Apache Git Service. To respond to the me

Re: [PR] Spec: Deprecate the file system table scheme. [iceberg]

2024-07-31 Thread via GitHub
dimas-b commented on code in PR #10833: URL: https://github.com/apache/iceberg/pull/10833#discussion_r1699236415 ## format/spec.md: ## @@ -1393,4 +1395,4 @@ This section covers topics not required by the specification but recommendations Iceberg supports two types of histories

Re: [PR] Spec: Deprecate the file system table scheme. [iceberg]

2024-07-31 Thread via GitHub
rdblue commented on code in PR #10833: URL: https://github.com/apache/iceberg/pull/10833#discussion_r1699235451 ## format/spec.md: ## @@ -1393,4 +1395,4 @@ This section covers topics not required by the specification but recommendations Iceberg supports two types of histories

[PR] Spec: Deprecate the file system table scheme. [iceberg]

2024-07-31 Thread via GitHub
rdblue opened a new pull request, #10833: URL: https://github.com/apache/iceberg/pull/10833 In the [discuss thread on the dev list](https://lists.apache.org/thread/oohcjfp1vpo005h2r0f6gfpsp6op0qps), there was agreement to deprecate the "File System Tables" scheme for committing metadata. T