-
Notifications
You must be signed in to change notification settings - Fork 58
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
perf: optimize row merging #628
Merged
Merged
Commits on Aug 5, 2022
-
test: improve the row merging tests
- extract read-rows-acceptance-test.json based tests into own file - update the json to match the latest available in https://github.com/googleapis/conformance-tests/tree/main/bigtable/v2 - use parameterized pytest test to run all of the scenarios (instead of creating a function for each json blob) - use json protobufs to parse the file I left a TODO to allow easy updates of the file, unfortunately its not straight forward as the canonical protos get renamed for python gapic Next PR will extract row merging functionality from row_data to make it easier to maintain
Configuration menu - View commit details
-
Copy full SHA for e3bd84c - Browse repository at this point
Copy the full SHA e3bd84cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 32c0551 - Browse repository at this point
Copy the full SHA 32c0551View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0dcdec4 - Browse repository at this point
Copy the full SHA 0dcdec4View commit details -
Configuration menu - View commit details
-
Copy full SHA for a6345e0 - Browse repository at this point
Copy the full SHA a6345e0View commit details -
Configuration menu - View commit details
-
Copy full SHA for 36e29ca - Browse repository at this point
Copy the full SHA 36e29caView commit details -
Configuration menu - View commit details
-
Copy full SHA for 5c89bc7 - Browse repository at this point
Copy the full SHA 5c89bc7View commit details
Commits on Aug 8, 2022
-
Configuration menu - View commit details
-
Copy full SHA for b33f5ad - Browse repository at this point
Copy the full SHA b33f5adView commit details -
Configuration menu - View commit details
-
Copy full SHA for 894d3da - Browse repository at this point
Copy the full SHA 894d3daView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6465a9d - Browse repository at this point
Copy the full SHA 6465a9dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6b70004 - Browse repository at this point
Copy the full SHA 6b70004View commit details -
Configuration menu - View commit details
-
Copy full SHA for 1fd7e46 - Browse repository at this point
Copy the full SHA 1fd7e46View commit details -
Configuration menu - View commit details
-
Copy full SHA for 01f4680 - Browse repository at this point
Copy the full SHA 01f4680View commit details
Commits on Aug 10, 2022
-
Configuration menu - View commit details
-
Copy full SHA for a93efbc - Browse repository at this point
Copy the full SHA a93efbcView commit details -
Configuration menu - View commit details
-
Copy full SHA for a848082 - Browse repository at this point
Copy the full SHA a848082View commit details -
Configuration menu - View commit details
-
Copy full SHA for 60f7fee - Browse repository at this point
Copy the full SHA 60f7feeView commit details -
Configuration menu - View commit details
-
Copy full SHA for 1ca5745 - Browse repository at this point
Copy the full SHA 1ca5745View commit details -
Configuration menu - View commit details
-
Copy full SHA for b73ec7d - Browse repository at this point
Copy the full SHA b73ec7dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 71f4513 - Browse repository at this point
Copy the full SHA 71f4513View commit details -
Configuration menu - View commit details
-
Copy full SHA for 62a6c19 - Browse repository at this point
Copy the full SHA 62a6c19View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2ca3e62 - Browse repository at this point
Copy the full SHA 2ca3e62View commit details
Commits on Aug 17, 2022
-
Merge remote-tracking branch 'origin/main' into row_merger
# Conflicts: # google/cloud/bigtable/row_data.py
Configuration menu - View commit details
-
Copy full SHA for 9db0027 - Browse repository at this point
Copy the full SHA 9db0027View commit details -
Merge remote-tracking branch 'igor/row_merger' into row_merger
# Conflicts: # google/cloud/bigtable/row_merger.py
Configuration menu - View commit details
-
Copy full SHA for ef8cf3f - Browse repository at this point
Copy the full SHA ef8cf3fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 37d1ced - Browse repository at this point
Copy the full SHA 37d1cedView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9012af1 - Browse repository at this point
Copy the full SHA 9012af1View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2b28101 - Browse repository at this point
Copy the full SHA 2b28101View commit details
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.