sivakumar-mahalingam commented on PR #4333: URL: https://github.com/apache/amoro/pull/4333#issuecomment-5780416950
Could we avoid unconditional `icebergTable.refresh()` when HDFS impersonation is not enabled? `buildCommit()` now loads the table inside `withOptimizingCommitImpersonation()`, so the `TableOwnerResolver.resolve()` appears to refresh the Iceberg table on each optimizing commit before checking `hdfs.impersonation.enabled` It will add extra metadata I/O/failure point even when impersonation is disabled, which is the default behavior. Could we move the refresh after the `hdfs.impersonation.enabled` check so it will only run when impersonation is enabled? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
