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.

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

test: add spanner executor v1 #1140

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

test: add spanner executor v1 #1140

wants to merge 3 commits into from

Conversation

parthea
Copy link
Contributor

@parthea parthea commented May 17, 2024

For testing purposes
Googlers see cl/634783571 b/325234697

  1. In order to avoid a conflict between protobuf stubs and proto-plus, run the following commands to change the protobuf package name in the proto-plus library
sed -i -e 's/google.spanner.v1\"/google.spanner.v1.gapic_generated\"/g' google/cloud/spanner_v1/types/*.py

sed -i -e 's/google.spanner.admin.database.v1\"/google.spanner.admin.database.v1.gapic_generated\"/g' google/cloud/spanner_admin_database_v1/types/*.py

sed -i -e 's/google.spanner.admin.instance.v1\"/google.spanner.admin.instance.v1.gapic_generated\"/g' google/cloud/spanner_admin_instance_v1/types/*.py
  1. Re-install the spanner library
pip install .
  1. Run
cd test_proxy/protos
python3 cloud_executor_server.py 

You should see

(py392) partheniou@partheniou-vm-3:~/git/python-spanner/test_proxy/protos$ python3 cloud_executor_server.py 
Server started, listening on 50055
  1. Run the test suite. For example,
partheniou@partheniou-vm-3:~/git/spanner_test$ go run tests/client_main_mini.go 
2024/06/22 17:36:20 Getting feature for point
2024/06/22 17:36:20 Got output 0, <nil>
2024/06/22 17:36:25 client.RouteChat: stream.Send(action_id:6  action:{query:{sql:"SELECT name, user_id as id FROM Users ORDER BY name DESC"}}) failed: EOF
exit status 1

@product-auto-label product-auto-label bot added size: l Pull request size is large. api: spanner Issues related to the googleapis/python-spanner API. labels May 17, 2024
@product-auto-label product-auto-label bot added size: xl Pull request size is extra large. and removed size: l Pull request size is large. labels Jun 22, 2024
@parthea parthea force-pushed the add-spanner-executor-v1 branch from a9852c8 to 205e6dd Compare June 22, 2024 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: spanner Issues related to the googleapis/python-spanner API. size: xl Pull request size is extra large.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant