search

trinodb / trino-go-client

Go client for Trino
Apache License 2.0
135 stars 63 forks source link

Include error code and name on error message #98

Closed deigote closed 6 months ago

deigote commented 1 year ago

Ideally we'd expose the error info in a structured way like suggested in https://github.com/trinodb/trino-go-client/issues/97 . Since that's more involved as requires changing exported interfaces, in the meantime having the info in the error message might be already helpful for telemetry.

cla-bot[bot] commented 1 year ago

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to cla@trino.io. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

nineinchnick commented 6 months ago

@deigote I'll close this since we got #97 merged. Feel free to reopen this or create a new issue.