Casey Ching has posted comments on this change. Change subject: Add Kudu test helpers ......................................................................
Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/2855/4/infra/python/bootstrap_virtualenv.py File infra/python/bootstrap_virtualenv.py: Line 190: env["LD_LIBRARY_PATH"] = "{0}{1}lib{2}{0}{1}lib64".format( : kudu_client_dir, os.path.sep, os.path.pathsep) > Just a comment because this made me think: if whatever OS + toolchain versi I'm pretty sure this comes from Kudu itself. We'd have to make sure that Kudu always used "lib" instead of something else. That would be a good change though. -- To view, visit http://gerrit.cloudera.org:8080/2855 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8e5e22b38d5bd09a36238e66a69aa42d1a941de7 Gerrit-PatchSet: 4 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Casey Ching <[email protected]> Gerrit-Reviewer: Casey Ching <[email protected]> Gerrit-Reviewer: Michael Brown <[email protected]> Gerrit-HasComments: Yes
