yuqi1129 commented on PR #7548: URL: https://github.com/apache/gravitino/pull/7548#issuecomment-3030239108
> Re SqlSessionFactoryHelper is it really needed? A trusted person is going to supply the config, there's a lot more they could do than this if they wanted to. It would be better to add some checks for it. I also share the same view as you initially, after giving it some thought, I believe those checks will provide more security guarantees, even though they are mostly set in the trusted environment for Gravitino server configurations. -- 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]
