tristaZero commented on pull request #10899:
URL: https://github.com/apache/shardingsphere/pull/10899#issuecomment-865682691


   Hi @lwclover ,
   
   Thanks for your PR, which attracts my interest. Let firstly set aside your 
coding, and target your idea and plan.
   
   It basically looks nice to me, but there is an oversight, i.e. when users 
change the map of user and privileges, `refresh()` just refresh the runtime. 
All these changes will lose once users restart ShardingProxy. What do you think?
   


-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to