search

trinodb / trino-python-client

Python client for Trino
Apache License 2.0
311 stars 154 forks source link

Add support for SET ROLE statement #200

Closed ulisesojeda closed 1 year ago

ulisesojeda commented 2 years ago

Add support for SET ROLE statement

Fixes https://github.com/trinodb/trino-python-client/issues/7

Follow-up of closed PR https://github.com/trinodb/trino-python-client/pull/196.

cla-bot[bot] commented 2 years ago

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla.

cla-bot[bot] commented 2 years ago

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla.

cla-bot[bot] commented 2 years ago

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@trino.io. For more information, see https://github.com/trinodb/cla.

mdesmet commented 2 years ago

Please note that you should not merge master into your PR, that way existing changes are re-introduced in your PR with a new commit id. Instead you should rebase your branch on master (pull latest changes from upstream repo on master and then git rebase master on your PR branch).

ulisesojeda commented 2 years ago

LGTM. Please squash the commits

Ready. Thanks

ebyhr commented 1 year ago

Merged, thanks!

ulisesojeda commented 1 year ago

Merged, thanks!

Thanks!