Todd Lipcon has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/3722

Change subject: WIP: KUDU-1271: allow primary keys not at the start of the row
......................................................................

WIP: KUDU-1271: allow primary keys not at the start of the row

this is just some initial hackery, nowhere near complete:
- change Schema to represent PKs as a list of indexes, not a count
- add accessor for above new list of indexes
- hack around in EncodedKey to handle encoding/decoding non-contiguous keys
- change some wire protocol stuff to pass the PK column indexes in
  SchemaPB

Needs:
- _lots_ of new test coverage
- lots of places where we assume that the index in a key is the index
  in the schema -- particularly the partition pruning, predicate pushdown,
  etc

Change-Id: I35e0fd09e1fba9cfd0d1873759d7fbcaad7c5cb8
---
M src/kudu/client/client-test.cc
M src/kudu/client/client-unittest.cc
M src/kudu/client/scan_configuration.cc
M src/kudu/client/schema.cc
M src/kudu/client/schema.h
M src/kudu/client/write_op.cc
M src/kudu/codegen/codegen-test.cc
M src/kudu/common/common.proto
M src/kudu/common/encoded_key-test.cc
M src/kudu/common/encoded_key.cc
M src/kudu/common/partial_row.cc
M src/kudu/common/partition-test.cc
M src/kudu/common/partition.cc
M src/kudu/common/partition_pruner-test.cc
M src/kudu/common/partition_pruner.cc
M src/kudu/common/row_operations.cc
M src/kudu/common/scan_spec.cc
M src/kudu/common/schema-test.cc
M src/kudu/common/schema.cc
M src/kudu/common/schema.h
M src/kudu/common/wire_protocol-test.cc
M src/kudu/common/wire_protocol.cc
M src/kudu/common/wire_protocol.h
M src/kudu/gutil/port.h
M src/kudu/master/catalog_manager.cc
M src/kudu/tablet/deltamemstore-test.cc
M src/kudu/tablet/diskrowset-test-base.h
M src/kudu/tserver/tablet_server-test.cc
M src/kudu/tserver/tablet_service.cc
29 files changed, 374 insertions(+), 243 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/22/3722/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3722
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I35e0fd09e1fba9cfd0d1873759d7fbcaad7c5cb8
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>

Reply via email to