Apache9 commented on PR #5853: URL: https://github.com/apache/hbase/pull/5853#issuecomment-2076489882
@stoty @ndimiduk @bbeaudreault FYI This is another choice comparing to #5850 . In #5850, we map some configuration values between URI queries, so users could specify some connection registry specific configurations through URI queries. While in this PR, we just apply all the queries to our configuration, so users could set any configuration they like in the URI. The solution in this PR is more flexible, as there could lots of configurations which need to be specified when connecting to a hbase cluster, not only for connection registry. But there could also be potential security issues if we just apply all the quries to Configuration instance. So I think here we need to consider more, thoughts? Thanks. -- 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: issues-unsubscr...@hbase.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org