jerqi commented on PR #9827: URL: https://github.com/apache/gravitino/pull/9827#issuecomment-3822081150
> 1. Is there any reason to add `ValidatingCache`, seems not tightly bound to this PR? > 2. In your another PR, you shadow the security catalog properties, could the dynamic catalog provider works after you doing that? 1. We should validate whether the catalog config is expired. If the catalog is expired, we should reload the catalog config and create a new catalog wrapper. 2. It can work. I hide the sensitive catalog properties in catalog operations. We use dispatcher methods in the IRC dynamic config provider. -- 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]
