Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[DRAFT] feat: add new sync data client #957

Draft
wants to merge 55 commits into
base: main
Choose a base branch
from
Draft

[DRAFT] feat: add new sync data client #957

wants to merge 55 commits into from

Conversation

daniel-sanche
Copy link
Contributor

WIP

This PR generates a sync counterpart to the new async data client

@daniel-sanche
pulled in sync generator from previous branch
51d3557
@daniel-sanche
simplified generator
fb2ae39
@daniel-sanche
refactoring
e788933
@daniel-sanche
adding better support for yaml config
7c740ef
@daniel-sanche
updated import maps
ff0458d
@daniel-sanche
added back name_replacements
fcf2a2f
@daniel-sanche
added configs for all async classes
51d9c7f
@daniel-sanche
removed concrete class map
3fefea4
@daniel-sanche
improced handling of unexpected asyncio tasks
b6a89dd
@daniel-sanche
added concrete classes
4d18e58
@daniel-sanche
keep docstrings on pass
70fd731
@daniel-sanche
improved import replacement code
5c96cd3
@daniel-sanche
renamed the two types of replacements
86b2e25
@daniel-sanche
use concrete names in generator
db16d0f
@daniel-sanche
fixed import paths
8c9676d
@daniel-sanche
avoid circular import
d4ca864
@daniel-sanche
improved string typing
2d7b88f
@daniel-sanche
got client and table working
2c950f2
@daniel-sanche
moved config into yaml file
f73a225
@daniel-sanche
moved save_path into yaml
92d5608
@daniel-sanche
fixed sync emulator channel
7b40f27
@daniel-sanche
moved acceptance tests to async folder
b3546d8
@daniel-sanche
broke module_replacements into asyncio and imports
cbec8f2
@daniel-sanche
update retries_async
1af60be
@daniel-sanche
replaced asyncio.Task
e486ad5
@daniel-sanche
removed use of async generator
1974a50
@daniel-sanche
fixed type issue
64ec3d5
@daniel-sanche
got _mutate_rows tests passing
f67ebc4
@daniel-sanche
got sync read_rows tests passing
2ebbb72
@daniel-sanche
got flow control tests passing
0bfb257
@daniel-sanche
made some fixes to mutations batcher
5b9e967
@daniel-sanche
improved modularity of mutation batcher tests
717c401
@daniel-sanche
fixed TableAssync references
aa472c1
@daniel-sanche
added event for batcher lock
06f0586
@daniel-sanche
got mutation batcher tests passing
d097aa1
@daniel-sanche
did some refactoring of client tests
8df5920
@daniel-sanche
added sharded read rows sync
dcc0ab6
@daniel-sanche
made tweaks to read rows sharded
10c565f
@daniel-sanche
got client tests passing
2cdf0ab
@daniel-sanche
moved more logic into generated portion
ef892bc
@daniel-sanche
got table tests passing
fe572bc
@daniel-sanche
got all client tests passing
1d428a6
@daniel-sanche
fixed slow test
751c676
@daniel-sanche
got async read_rows acceptance working again
97f4797
@daniel-sanche
got acceptance tests working
92f2132
@daniel-sanche
ran blacken
c006482
@daniel-sanche
fixed some lint issues
c39d26a
@daniel-sanche
fixed mypy issues
d5903e2
@daniel-sanche
refactored system tests into class
f9862a2
@daniel-sanche
fixed retry type
5cb4e82
@daniel-sanche
renamed test file
46ef676
@daniel-sanche
got system tests generated
44840f3
@daniel-sanche
fixed possible flake
3c33954
@daniel-sanche
changed class order to match async
7c9c3bd
@product-auto-label product-auto-label bot added size: xl Pull request size is extra large. api: bigtable Issues related to the googleapis/python-bigtable API. labels Apr 18, 2024
@daniel-sanche daniel-sanche mentioned this pull request Apr 18, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
api: bigtable Issues related to the googleapis/python-bigtable API. size: xl Pull request size is extra large.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@daniel-sanche