You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "CurtHagenlocher (via GitHub)" <gi...@apache.org> on 2024/05/03 20:43:33 UTC
[PR] fix(csharp/src/Apache.Arrow.Adbc/C): imported errors don't return a native error or SQL state [arrow-adbc]
CurtHagenlocher opened a new pull request, #1815:
URL: https://github.com/apache/arrow-adbc/pull/1815
Creates subclass of AdbcException for imported drivers which reads the native error code and SQLState from the C error structure.
Closes #1813
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(csharp/src/Apache.Arrow.Adbc/C): imported errors don't return a native error or SQL state [arrow-adbc]
Posted by "CurtHagenlocher (via GitHub)" <gi...@apache.org>.
CurtHagenlocher commented on PR #1815:
URL: https://github.com/apache/arrow-adbc/pull/1815#issuecomment-2093737010
It turns out that neither DuckDB nor Snowflake are setting these. Maybe PostgreSQL or sqlite would work; something to look at in the future.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(csharp/src/Apache.Arrow.Adbc/C): imported errors don't return a native error or SQL state [arrow-adbc]
Posted by "CurtHagenlocher (via GitHub)" <gi...@apache.org>.
CurtHagenlocher merged PR #1815:
URL: https://github.com/apache/arrow-adbc/pull/1815
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(csharp/src/Apache.Arrow.Adbc/C): imported errors don't return a native error or SQL state [arrow-adbc]
Posted by "CurtHagenlocher (via GitHub)" <gi...@apache.org>.
CurtHagenlocher commented on PR #1815:
URL: https://github.com/apache/arrow-adbc/pull/1815#issuecomment-2094146627
Ah, I've only started to implement 1.1 functionality so I'd forgotten about that use of the vendor code.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix(csharp/src/Apache.Arrow.Adbc/C): imported errors don't return a native error or SQL state [arrow-adbc]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on PR #1815:
URL: https://github.com/apache/arrow-adbc/pull/1815#issuecomment-2094127507
- I think basically nothing uses the error code so far (also see https://github.com/apache/arrow-adbc/issues/1576)
- PostgreSQL should set SQLSTATE
- I would like Flight SQL to be able to set SQLSTATE but it would need a protocol enhancement
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org