bbeaudreault commented on PR #5853: URL: https://github.com/apache/hbase/pull/5853#issuecomment-2104353124
I agree with Nick. I also think it's a bit odd to require two separate configuration points for a working connection. So previously you had to specify a Configuration, and now you can use URL instead. You can audio combine them, but to _require_ both seems odd, and I find it very unlikely that a realistic user wouldn't need to specify a bunch of options, like timeouts, etc. So URL should be able to configure everything imo. But also, in trying to think what would be a security concern if it were compromised. It doesn't matter if someone specified hdfs settings because they won't be used by the client. -- 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
