Quanlong Huang has posted comments on this change. ( http://gerrit.cloudera.org:8080/18777 )
Change subject: IMPALA-11454: part-1: use standard binaries path to start kudu test cluster ...................................................................... Patch Set 3: Code-Review+1 This makes sense to me. I think a simpler fix is modifying the toolchain build script to create symlinks pointing to sbin instead of copying binaries directly, as you mentioned in the JIRA: https://github.com/cloudera/native-toolchain/blob/2008e0658c1785ab7a78866e6f46ee6d9214c446/source/kudu/build.sh#L170-L176 Either way is ok for me. I'll see if other reviews have more comments. -- To view, visit http://gerrit.cloudera.org:8080/18777 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2276ee3e1650532770b856443f1087ad5036dc7a Gerrit-Change-Number: 18777 Gerrit-PatchSet: 3 Gerrit-Owner: Anonymous Coward <[email protected]> Gerrit-Reviewer: Anonymous Coward <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Wenzhe Zhou <[email protected]> Gerrit-Comment-Date: Fri, 29 Jul 2022 01:42:47 +0000 Gerrit-HasComments: No
