ferdelyi commented on PR #5638: URL: https://github.com/apache/hadoop/pull/5638#issuecomment-1548089533
Thank you Szilard for the CR. The change was exclusively tested with the unit test, which is a kind of integration test, as a ZK Server was brought up and the communication tested between the client and the server. This code change is in the common code base and there is no component calling it yet. Once YARN-11468 [Zookeeper SSL/TLS support] is implemented, we can test it in a real cluster environment. Wondering if we should update the https://hadoop.apache.org/docs/stable/hadoop-project-dist/hadoop-hdfs-rbf/dependency-analysis.html page with the Netty dependency? The parameter descriptions are added to the commit to the core-default.xml. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
