Alexey Serbin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/17740
Change subject: [common] introduce comparison operators for Slice
......................................................................
[common] introduce comparison operators for Slice
This patch introduces comparison operators for the Slice class.
In my opinion, that improves readability of the code. Compare:
if (mid_key.compare(target) < 0) {
...
}
vs
if (mid_key < target) {
...
}
CHECK(prev_key.compare(enc_key) < 0);
vs
CHECK(prev_key < enc_key);
if (key_a.compare(key_b) == 0) {
...
}
vs
if (key_a == key_b) {
...
}
I also did other minor code modifications in the code around.
This patch doesn't contain any significant functional changes.
Change-Id: I995803246fb3c21d8fe043b882fc93a87d9c2619
---
M src/kudu/cfile/binary_plain_block.cc
M src/kudu/cfile/binary_prefix_block.cc
M src/kudu/cfile/cfile_util.cc
M src/kudu/common/encoded_key.h
M src/kudu/common/partition.cc
M src/kudu/common/scan_spec.cc
M src/kudu/common/schema-test.cc
M src/kudu/common/types.h
M src/kudu/common/wire_protocol-test.cc
M src/kudu/fs/fs_manager-test.cc
M src/kudu/integration-tests/all_types-itest.cc
M src/kudu/rpc/rpc-test-base.h
M src/kudu/tablet/cbtree-test.cc
M src/kudu/tablet/cfile_set.cc
M src/kudu/tablet/deltamemstore-test.cc
M src/kudu/tablet/diskrowset.cc
M src/kudu/tablet/memrowset.h
M src/kudu/tablet/rowset_info.cc
M src/kudu/tablet/rowset_tree-test.cc
M src/kudu/tablet/tablet.cc
M src/kudu/tserver/tablet_service.cc
M src/kudu/util/memcmpable_varint-test.cc
M src/kudu/util/slice.h
23 files changed, 120 insertions(+), 78 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/40/17740/1
--
To view, visit http://gerrit.cloudera.org:8080/17740
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I995803246fb3c21d8fe043b882fc93a87d9c2619
Gerrit-Change-Number: 17740
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
