Skip to content

BigTable: Add split keys on create table#5283

Closed
aneepct wants to merge 92 commits intogoogleapis:masterfrom
zakons:feature/add_splits_on_create_table
Closed

BigTable: Add split keys on create table#5283
aneepct wants to merge 92 commits intogoogleapis:masterfrom
zakons:feature/add_splits_on_create_table

Conversation

@aneepct
Copy link
Contributor

@aneepct aneepct commented May 2, 2018

Add ability to create initial splits during creation of a table. This feature branch is a fork from the GAPIC integration branch (PR #5178).

Luke Sneeringer and others added 30 commits January 4, 2018 10:29
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 2, 2018
@tseaver tseaver added the api: bigtable Issues related to the Bigtable API. label May 3, 2018
@theacodes
Copy link
Contributor

This also seems to need to be rebased.

@sduskis
Copy link
Contributor

sduskis commented May 18, 2018

Why are there app_profile changes here? Can you please remove them?

@sduskis
Copy link
Contributor

sduskis commented May 18, 2018

Also, please rebase on master. There are 90 commits here, and I doubt they are related to your change.

Copy link
Contributor

@sduskis sduskis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please add a system test for this new functionality?

@sduskis
Copy link
Contributor

sduskis commented May 22, 2018

Can you please rebase this PR?

@aneepct
Copy link
Contributor Author

aneepct commented May 23, 2018

@sduskis I created new PR #5370 to prevent few merge conflicts going to close this PR

@aneepct aneepct closed this May 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtable Issues related to the Bigtable API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants