Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17854
Change subject: [common] small re-factoring on PartitionPruner ...................................................................... [common] small re-factoring on PartitionPruner This changelist simplifies the signature of the PartitionPruner::ConstructPartitionKeyRanges() method: the in-out parameter is removed and turned into the return value. I also moved RangeBoundsAndPartitionKeyRanges, RangeBounds, and PartitionKeyRange structures into the private section of the PartitionPruner class and removed default constructor declarations for them. There is no point of declaring default constructors for structures: the compiler creates them automatically. This patch doesn't contain any functional changes. Change-Id: I38c5c4809f251c8efff0de584bbbbbc03e9d952e --- M src/kudu/common/partition_pruner.cc M src/kudu/common/partition_pruner.h 2 files changed, 37 insertions(+), 48 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/54/17854/1 -- To view, visit http://gerrit.cloudera.org:8080/17854 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I38c5c4809f251c8efff0de584bbbbbc03e9d952e Gerrit-Change-Number: 17854 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
