jerqi commented on PR #9827: URL: https://github.com/apache/gravitino/pull/9827#issuecomment-3822535214
> > 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. > > I would suggest to implement this cache in separate PR, or we can talk about it in issues about the neccessary to introduce it. > > > 2. It can work. I hide the sensitive catalog properties in catalog operations. We use dispatcher methods in the IRC dynamic config provider. > > Could you provide more context about how to resolve this issue? 1. OK 2. Because dynamic config provider won't call the HTTP request to load catalog if authorization is enabled. We only hidden the sensitive information for HTTP requests. We don't hidden the sensitive information. -- 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]
