NihalJain commented on PR #5623: URL: https://github.com/apache/hbase/pull/5623#issuecomment-1890593311
Thanks for the detailed answers. > However since this TLS support is as-yet unreleased, we have freedom to decide the default behavior for the first release. (2.6.0) Ah I have no objections on defaults. Just called this out as I was not sure if this was intentional or by mistake. > Another option I thought of is to create a new core module hbase-extensions (or something). It could have a sub module hbase-openssl-support which lists tcnative as a dependency. I am +1 on this idea, this would be useful and serve for feature toggle kind of use cases, like this one. -- 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]
