Re: [PR] Spec: Add cross-region bucket access property to config [iceberg]

2025-01-13 Thread via GitHub
RussellSpitzer commented on PR #11260: URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2587600228 Approved and Merged, thanks @munendrasn for the PR and @Fokko and @nastra for reviewing -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] Spec: Add cross-region bucket access property to config [iceberg]

2025-01-13 Thread via GitHub
RussellSpitzer merged PR #11260: URL: https://github.com/apache/iceberg/pull/11260 -- 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] Spec: Add cross-region bucket access property to config [iceberg]

2025-01-13 Thread via GitHub
munendrasn commented on PR #11260: URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2587128406 Reached over to iceberg-slack to get the additional review -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

Re: [PR] Spec: Add cross-region bucket access property to config [iceberg]

2025-01-11 Thread via GitHub
github-actions[bot] commented on PR #11260: URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2585490645 This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pul

Re: [PR] Spec: Add cross-region bucket access property to config [iceberg]

2024-12-12 Thread via GitHub
nastra commented on PR #11260: URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2538464248 the changes LGTM and given this is only a doc change (by stating a new property that's being supported) I don't think this PR would have to go through the normal voting process on the ma

Re: [PR] Spec: Add cross-region bucket access property to config [iceberg]

2024-12-12 Thread via GitHub
munendrasn commented on PR #11260: URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2538452754 @nastra @Fokko are the changes good to be merged? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

Re: [PR] Spec: Add cross-region bucket access property to config [iceberg]

2024-11-25 Thread via GitHub
munendrasn commented on PR #11260: URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2497810049 @nastra Could you please review again? -- 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

Re: [PR] Spec: Add cross-region bucket access property to config [iceberg]

2024-11-15 Thread via GitHub
munendrasn commented on PR #11260: URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2478133424 @nastra Resolved conflicts, please 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 above to go

Re: [PR] Spec: Add cross-region bucket access property to config [iceberg]

2024-11-14 Thread via GitHub
munendrasn commented on PR #11260: URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2476506543 @nastra Could you please take a look.. would this be correct place to add this info? -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Spec: Add cross-region bucket access property to config [iceberg]

2024-10-17 Thread via GitHub
munendrasn commented on PR #11260: URL: https://github.com/apache/iceberg/pull/11260#issuecomment-2419824121 @nastra Please review, If this looks good, post PR #10722 and https://github.com/apache/iceberg/pull/10576, will move this change to fileIo properties (https://github.com/apache/iceb