Re: [PR] AWS: Add parameter of excluding non-current fields in Glue [iceberg]

2025-05-07 Thread via GitHub
duoxoud commented on PR #12664: URL: https://github.com/apache/iceberg/pull/12664#issuecomment-2858422478 Hello @jackye1995 👋, do you happen to have time to check this PR? Many thanks 🫡 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] AWS: Add parameter of excluding non-current fields in Glue [iceberg]

2025-04-11 Thread via GitHub
xiaoxuandev commented on PR #12664: URL: https://github.com/apache/iceberg/pull/12664#issuecomment-2797404448 @duoxoud I still believe this change isn't necessary and could break LF integration. For your use case, could you explain why end users rely solely on the latest Glue table status?

Re: [PR] AWS: Add parameter of excluding non-current fields in Glue [iceberg]

2025-04-01 Thread via GitHub
duoxoud commented on PR #12664: URL: https://github.com/apache/iceberg/pull/12664#issuecomment-2769595436 > Displaying non-current columns is intentional in Glue, as users may use LakeFormation and need to access dropped columns. Users should not rely on Glue for the latest table status, Ic

Re: [PR] AWS: Add parameter of excluding non-current fields in Glue [iceberg]

2025-03-31 Thread via GitHub
xiaoxuandev commented on PR #12664: URL: https://github.com/apache/iceberg/pull/12664#issuecomment-2768177860 Displaying non-current columns is intentional in Glue, as users may use LakeFormation and need to access dropped columns. Users should not rely on Glue for the latest table status,

Re: [PR] AWS: Add parameter of excluding non-current fields in Glue [iceberg]

2025-03-27 Thread via GitHub
borjagonzal commented on PR #12664: URL: https://github.com/apache/iceberg/pull/12664#issuecomment-2758571505 This change fixes an issue we have faced during some time in our stack. Thank you for pushing this one @duoxoud! -- This is an automated message from the Apache Git Service. To