Anyone else got time to take a look at Dan's big patch here? It's pretty complex, so getting a second pair of eyes would be good. If no one has time, we can commit it and always improve the docs/clarity later, of course.
-Todd ---------- Forwarded message ---------- From: Dan Burkert (Code Review) <[email protected]> Date: Fri, Mar 18, 2016 at 5:21 PM Subject: [kudu-CR] KUDU-1165: Implement partition pruning for C++ client and server To: [email protected] Cc: Adar Dembo <[email protected]>, Todd Lipcon <[email protected]> Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2413 to look at the new patch set (#19). Change subject: KUDU-1165: Implement partition pruning for C++ client and server ...................................................................... KUDU-1165: Implement partition pruning for C++ client and server Change-Id: I9f4fc14b44af63a17b34cd9c2a9134127b1afe4d --- M src/kudu/client/client.cc M src/kudu/client/scanner-internal.cc M src/kudu/client/scanner-internal.h M src/kudu/common/CMakeLists.txt M src/kudu/common/partial_row.h M src/kudu/common/partition.cc M src/kudu/common/partition.h A src/kudu/common/partition_pruner-test.cc A src/kudu/common/partition_pruner.cc A src/kudu/common/partition_pruner.h M src/kudu/util/memory/arena.cc M src/kudu/util/memory/arena.h 12 files changed, 1,179 insertions(+), 88 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/13/2413/19 -- To view, visit http://gerrit.cloudera.org:8080/2413 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I9f4fc14b44af63a17b34cd9c2a9134127b1afe4d Gerrit-PatchSet: 19 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> -- Todd Lipcon Software Engineer, Cloudera
