Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13569 )
Change subject: IMPALA-8642: krpc should depend on thrift-deps in CMake ...................................................................... IMPALA-8642: krpc should depend on thrift-deps in CMake When build with IMPALA_BUILD_THREADS=1, the compiling of krpc will fail since kudu_version.cc includes "gen-cpp/Status_types.h" which will be built later. We should add dependency on thrift-deps for krpc to avoid such a race condition in compiling. Change-Id: Ieaf9d992340f07fdf88b35fb967553fa2e35b628 Reviewed-on: http://gerrit.cloudera.org:8080/13569 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/kudu/rpc/CMakeLists.txt 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/13569 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ieaf9d992340f07fdf88b35fb967553fa2e35b628 Gerrit-Change-Number: 13569 Gerrit-PatchSet: 3 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
