You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@shardingsphere.apache.org by GitBox <gi...@apache.org> on 2022/09/05 02:07:13 UTC

[GitHub] [shardingsphere] TeslaCN commented on pull request #20620: add md5 support for openGauss

TeslaCN commented on PR #20620:
URL: https://github.com/apache/shardingsphere/pull/20620#issuecomment-1236478266

   > > Hi @RaigorJiang Could you provide some suggestion about this PR?
   > 
   > @TeslaCN Authority is a kernel configuration, and I do agree with what @justbk2015 said, when changing API, should consider compatibility with older version. But there are several factors that drive us to consider new APIs:
   > 
   > 1. Specify different authentication methods for users
   > 2. Users can have no password
   >    When the second scenario occurs, the configuration of this PR will not support it.
   > 
   > So I propose that the PMCs discuss this and give advice?
   
   Technical proposal could be discuss in dev mailing list. Every one could start or participant in discussion.


-- 
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: notifications-unsubscribe@shardingsphere.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org