Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12351 )
Change subject: KUDU-2411: Set SASL_PATH if needed when starting MiniCluster ...................................................................... KUDU-2411: Set SASL_PATH if needed when starting MiniCluster When using the binary artifact on Linux, the MiniCluster should check to see if the sasl2 libraries are included in the artifact and if should set the SASL_PATH environment variable to the location of the sasl2 modules when starting the MiniCluster. This is required because the sasl2 modules are dynamically loaded at runtime. Some refactoring was required to provide the relevant information to set the SASL_PATH when starting the mini cluster process. Additionally, this patch includes some miscellaneous cleanup. Change-Id: Iaaeb30781f4483910c35a20c6d7c76f7f85aa4ce Reviewed-on: http://gerrit.cloudera.org:8080/12351 Reviewed-by: Grant Henke <[email protected]> Tested-by: Kudu Jenkins --- A java/kudu-test-utils/src/main/java/org/apache/kudu/test/cluster/KuduBinaryInfo.java M java/kudu-test-utils/src/main/java/org/apache/kudu/test/cluster/KuduBinaryJarExtractor.java M java/kudu-test-utils/src/main/java/org/apache/kudu/test/cluster/KuduBinaryLocator.java M java/kudu-test-utils/src/main/java/org/apache/kudu/test/cluster/MiniKuduCluster.java M java/kudu-test-utils/src/test/java/org/apache/kudu/test/cluster/TestKuduBinaryJarExtractor.java 5 files changed, 133 insertions(+), 36 deletions(-) Approvals: Grant Henke: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/12351 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iaaeb30781f4483910c35a20c6d7c76f7f85aa4ce Gerrit-Change-Number: 12351 Gerrit-PatchSet: 6 Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Brian McDevitt <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
