Yingchun Lai has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13936 )
Change subject: [common] add FindColumn for Schema ...................................................................... [common] add FindColumn for Schema There are several implementations of FindColumn, now add it as a member of Schema. Change-Id: Iede12b95b774754f914295cecd2f6797008fed46 Reviewed-on: http://gerrit.cloudera.org:8080/13936 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/client/scan_batch.cc M src/kudu/common/partial_row.cc M src/kudu/common/row_operations-test.cc M src/kudu/common/schema-test.cc M src/kudu/common/schema.cc M src/kudu/common/schema.h M src/kudu/master/master_service.cc 7 files changed, 49 insertions(+), 50 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/13936 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iede12b95b774754f914295cecd2f6797008fed46 Gerrit-Change-Number: 13936 Gerrit-PatchSet: 4 Gerrit-Owner: Yingchun Lai <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <[email protected]>
