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

feat: add progress field to UpdateDatabaseDdlMetadata #361

Merged
merged 7 commits into from Jun 7, 2021
Merged

Conversation

parthea
Copy link
Contributor

@parthea parthea commented Jun 3, 2021

This PR includes the latest generated client from googleapis-gen. I also modified the owlbot.py script to properly copy multiple versions from googleapis-gen here.

This PR also includes the following features and fixes:

feat: move import of CommitResponse
fix: add async client to %name_%version/init.py
chore: add autogenerated snippets
chore: remove auth, policy, and options from the reserved names list
feat: support self-signed JWT flow for service accounts
chore: enable GAPIC metadata generation
chore: sort subpackages in %namespace/%name/init.py

@parthea parthea requested a review from larkee Jun 3, 2021
@parthea parthea requested review from as code owners Jun 3, 2021
@parthea parthea requested review from crwilcox and removed request for Jun 3, 2021
@product-auto-label product-auto-label bot added the api: spanner label Jun 3, 2021
@google-cla google-cla bot added the cla: yes label Jun 3, 2021
@parthea
Copy link
Contributor Author

@parthea parthea commented Jun 3, 2021

The build has the following error: ImportError: cannot import name 'CommitResponse' . I'm looking into it.

==================================== ERRORS ====================================
_________________ ERROR collecting tests/system/test_system.py _________________
ImportError while importing test module '/home/runner/work/python-spanner/python-spanner/tests/system/test_system.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/opt/hostedtoolcache/Python/3.8.10/x64/lib/python3.8/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
tests/system/test_system.py:33: in <module>
    from google.cloud.spanner_v1 import param_types
google/cloud/spanner_v1/__init__.py:37: in <module>
    from .types.spanner import CommitResponse
E   ImportError: cannot import name 'CommitResponse' from 'google.cloud.spanner_v1.types.spanner' (/home/runner/work/python-spanner/python-spanner/google/cloud/spanner_v1/types/spanner.py)
______________ ERROR collecting tests/system/test_system_dbapi.py ______________
ImportError while importing test module '/home/runner/work/python-spanner/python-spanner/tests/system/test_system_dbapi.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/opt/hostedtoolcache/Python/3.8.10/x64/lib/python3.8/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
tests/system/test_system_dbapi.py:23: in <module>
    from google.cloud.spanner_v1 import BurstyPool
google/cloud/spanner_v1/__init__.py:37: in <module>
    from .types.spanner import CommitResponse
E   ImportError: cannot import name 'CommitResponse' from 'google.cloud.spanner_v1.types.spanner' (/home/runner/work/python-spanner/python-spanner/google/cloud/spanner_v1/types/spanner.py)

@parthea parthea marked this pull request as draft Jun 3, 2021
@parthea
Copy link
Contributor Author

@parthea parthea commented Jun 3, 2021

There is an issue with the docs. I was able to resolve the issue by removing the headings in this docstring .

sphinx.errors.SphinxWarning: /usr/local/google/home/partheniou/git/python-spanner/google/cloud/spanner_v1/types/transaction.py:docstring of google.cloud.spanner_v1.types.transaction.TransactionOptions:4:Unexpected section title.

I'll add a workaround in owlbot.py . I've suggested an update to the source proto to remove the headings based on AIP-192. Googlers see cl/377290854

Headings and tables must not be used, as these cause problems for several tools, and are unsuitable for client library reference documentation.

@parthea parthea marked this pull request as ready for review Jun 3, 2021
@parthea parthea added the do not merge label Jun 3, 2021
@parthea
Copy link
Contributor Author

@parthea parthea commented Jun 4, 2021

I'm going to switch this back to draft. It should be ready for review tomorrow.

@parthea parthea marked this pull request as draft Jun 4, 2021
@parthea parthea removed the do not merge label Jun 6, 2021
@parthea parthea marked this pull request as ready for review Jun 6, 2021
larkee
larkee approved these changes Jun 7, 2021
@larkee larkee merged commit 1c03dcc into master Jun 7, 2021
11 checks passed
@larkee larkee deleted the fix-owlbot-py branch Jun 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: spanner cla: yes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants