[GitHub] [iceberg] jackye1995 commented on pull request #4280: [WIP] AWS: Add support for LakeFormation database and table access control

[GitBox <gi...@apache.org>]

jackye1995 commented on pull request #4280:
URL: https://github.com/apache/iceberg/pull/4280#issuecomment-1068358957


   Read the PR for a few rounds, I think let's first have a minimum PR that:
   1. does not have any caching features, just assume we always fetch new credentials and we do not cache anything like table name, FileIO, etc. caching can always come later.
   2. there is no need to have a new `LakeFormationCatalog`, I think the feature can be turned on as a part of `GlueCatalog`, using a flag like `glue.lakeformation-enabled` and we can update directly in side `GlueCatalog` and `GlueTableOperations` to make this work.


-- 
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-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org