Internal Jenkins has submitted this change and it was merged. Change subject: Kudu: Fix warnings from clang ......................................................................
Kudu: Fix warnings from clang Changes: 1) Several places in the tests didn't check return statuses. KUDU_ASSERT_OK can only be used in functions that return void, KUDU_CHECK_OK is used otherwise. 2) The forward declared "class ColumnType" should have actually been a struct. Now there aren't any more Kudu related warnings from clang. Change-Id: Id3e2f5ec9925c3cf81c7f4048decc6a5f97eee66 Reviewed-on: http://gerrit.cloudera.org:8080/3062 Reviewed-by: Dan Hecht <[email protected]> Tested-by: Internal Jenkins --- M be/src/exec/kudu-scan-node-test.cc M be/src/exec/kudu-table-sink-test.cc M be/src/exec/kudu-testutil.h M be/src/exec/kudu-util.h 4 files changed, 9 insertions(+), 9 deletions(-) Approvals: Internal Jenkins: Verified Dan Hecht: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/3062 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id3e2f5ec9925c3cf81c7f4048decc6a5f97eee66 Gerrit-PatchSet: 2 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Casey Ching <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Internal Jenkins
