Skip to content
This repository was archived by the owner on Mar 31, 2026. It is now read-only.

test: add spanner executor v1#1140

Closed
parthea wants to merge 3 commits intomainfrom
add-spanner-executor-v1
Closed

test: add spanner executor v1#1140
parthea wants to merge 3 commits intomainfrom
add-spanner-executor-v1

Conversation

@parthea
Copy link
Copy Markdown
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
@chalmerlowe
Copy link
Copy Markdown
Contributor

Migration Notice: This library is moving to the google-cloud-python monorepo soon.

We closed this PR due to inactivity to ensure a clean migration. Please re-open this work in the new monorepo once the migration is complete!

@chalmerlowe chalmerlowe closed this Mar 2, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

2 participants