You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "jacobmarble (via GitHub)" <gi...@apache.org> on 2023/05/02 20:27:06 UTC

[GitHub] [arrow-adbc] jacobmarble opened a new pull request, #641: fix(sqldriver/flightsql): do not swallow array.RecordReader error

jacobmarble opened a new pull request, #641:
URL: https://github.com/apache/arrow-adbc/pull/641

   In the `database/sql` `Rows.Next()` implementation, errors occurring within `array.RecordReader` were ignored, so that an error result appeared as an empty result.


-- 
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


[GitHub] [arrow-adbc] jacobmarble commented on pull request #641: fix(go/adbc/sqldriver): do not swallow array.RecordReader error

Posted by "jacobmarble (via GitHub)" <gi...@apache.org>.
jacobmarble commented on PR #641:
URL: https://github.com/apache/arrow-adbc/pull/641#issuecomment-1532176076

   Thanks @lidavidm !


-- 
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


[GitHub] [arrow-adbc] lidavidm merged pull request #641: fix(go/adbc/sqldriver): do not swallow array.RecordReader error

Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm merged PR #641:
URL: https://github.com/apache/arrow-adbc/pull/641


-- 
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


[GitHub] [arrow-adbc] jacobmarble commented on pull request #641: fix(sqldriver/flightsql): do not swallow array.RecordReader error

Posted by "jacobmarble (via GitHub)" <gi...@apache.org>.
jacobmarble commented on PR #641:
URL: https://github.com/apache/arrow-adbc/pull/641#issuecomment-1532110025

   @lidavidm do you think this change could be added to v0.4.0?


-- 
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