-
Notifications
You must be signed in to change notification settings - Fork 438
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Introduce Handwritten Bigtable Client #1392
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* mutate rows * Fixed type in comment
* add mutate rows optional configuration parameter * update order of append operation
* read rows * read rows acceptance test * fix type in parameter * fix param type in comment * Fix the state value * fix error in php5.5 * fix for protobuf extension * fix doc issue * review updates * fix rows->readAll * fix cs
* read rows filter * fix test case * Filter * add comments * add espace for literal regex * Fix failing test * add snippet test * Fix doc * code review updates * moved builder in to its own namespace * add snippet test * arrange use statement * fix failing test case * Fix documentation
* System test for filter * update doc * throw exception if rowKeys is not array in readRows * Simplified some fitler test * use dataprovider for filter system test * formatting * address feedback
* provide message for failed test * update message
* readModifyWriteRowRule model * read modify write row * read modify write row api * update comments * fix static initializer * fix exception check for php < 7 * fix test case * update pack and unpack for php5.5 * address feedback * fix php 5.5 snippet test case * update doc and check * address feedback
* sample row keys * sample row keys * fix system test case * address feedback * fix snippet test
* checkAndMutateRow api * fix doc * fix snippet test * change snippet test * address feedback * update note * update comment * apply note to truemutations * fix comment * fix snippet test
* refactor mutation model * fix doc comment * address feedback * use ternary operator * fix comment
* rename DataClient to Table and add generic constructor options * update documentation * update snippets * update and rename system tests * update unit tests * add projectId/credentials to client * update mutateRow * update mutateRow system test * add imports for documentation * fix tests * potential fix for hhvm * fix projectId
* prepare bigtable handwritten layer for release * fix trailing comma
* add note about differences in handwritten client * update projectID detection (we cannot reliably use the ClientTrait with the new client configuration options) * remove ClientTrait * add note on providing a keyfile * add note about authentication
googlebot
added
the
cla: no
This human has *not* signed the Contributor License Agreement.
label
Nov 5, 2018
dwsupplee
added
cla: yes
This human has signed the Contributor License Agreement.
and removed
cla: no
This human has *not* signed the Contributor License Agreement.
labels
Nov 5, 2018
A big thanks to @jdpedrie and @dwsupplee! |
ajaaym
approved these changes
Nov 5, 2018
* Add 64 or 32 bit test * remove test annotation
googlebot
added
cla: no
This human has *not* signed the Contributor License Agreement.
and removed
cla: yes
This human has signed the Contributor License Agreement.
labels
Nov 5, 2018
dwsupplee
added
cla: yes
This human has signed the Contributor License Agreement.
and removed
cla: no
This human has *not* signed the Contributor License Agreement.
labels
Nov 5, 2018
jdpedrie
approved these changes
Nov 5, 2018
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great work @dwsupplee @ajaaym.
Merged
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.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR introduces an alpha version of a Bigtable client which can be used for reading/writing rows. Big shoutout to @ajaaym for all his hard work on this. 👏
/cc @sduskis