Skip to content

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