You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Dan Burkert (Code Review)" <ge...@cloudera.org> on 2016/05/18 22:38:05 UTC

[kudu-CR] [c++-client] remove default table partitioning

Dan Burkert has uploaded a new change for review.

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

Change subject: [c++-client] remove default table partitioning
......................................................................

[c++-client] remove default table partitioning

This commit changes the C++ client to remove the default range partitioning.
When creating tables either hash, range, or a combination of hash and range
partitioning must be specified. The previous default was determined to be a
poor default, and removing it and forcing users to specify the partitioning is
less error prone.

Change-Id: I7021d7950f8dbb4918503ea6fab2e6ee35076064
---
M src/kudu/client/client-test.cc
M src/kudu/client/client.cc
M src/kudu/client/client.h
M src/kudu/client/predicate-test.cc
M src/kudu/client/samples/sample.cc
M src/kudu/integration-tests/alter_table-randomized-test.cc
M src/kudu/integration-tests/alter_table-test.cc
7 files changed, 34 insertions(+), 11 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I7021d7950f8dbb4918503ea6fab2e6ee35076064
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <da...@cloudera.com>