Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17754 )
Change subject: [common] simplify PartitionSchema methods' signatures ...................................................................... [common] simplify PartitionSchema methods' signatures While working on the key encoding code, I noticed that many methods return a Status object when in reality they cannot return anything but Status::OK(). This patch updates the signatures of those methods and corresponding call sites. This changelist doesn't contain any functional changes. Change-Id: I75bc4329e93f83fbad8f0120ee1d4979f15c0aed Reviewed-on: http://gerrit.cloudera.org:8080/17754 Reviewed-by: Andrew Wong <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Mahesh Reddy <[email protected]> --- M src/kudu/client/batcher.cc M src/kudu/client/partitioner-internal.cc M src/kudu/common/partition-test.cc M src/kudu/common/partition.cc M src/kudu/common/partition.h M src/kudu/common/scan_spec.cc M src/kudu/common/schema.h M src/kudu/tablet/tablet.cc M src/kudu/transactions/txn_system_client.cc 9 files changed, 131 insertions(+), 179 deletions(-) Approvals: Andrew Wong: Looks good to me, approved Kudu Jenkins: Verified Mahesh Reddy: Looks good to me, but someone else must approve -- To view, visit http://gerrit.cloudera.org:8080/17754 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I75bc4329e93f83fbad8f0120ee1d4979f15c0aed Gerrit-Change-Number: 17754 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]>
