You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "lidavidm (via GitHub)" <gi...@apache.org> on 2024/01/23 21:48:16 UTC
[PR] feat(python/adbc_driver_manager): handle KeyboardInterrupt [arrow-adbc]
lidavidm opened a new pull request, #1486:
URL: https://github.com/apache/arrow-adbc/pull/1486
Fixes #1484.
--
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] feat(python/adbc_driver_manager): handle KeyboardInterrupt [arrow-adbc]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm closed pull request #1486: feat(python/adbc_driver_manager): handle KeyboardInterrupt
URL: https://github.com/apache/arrow-adbc/pull/1486
--
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] feat(python/adbc_driver_manager): handle KeyboardInterrupt [arrow-adbc]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on PR #1486:
URL: https://github.com/apache/arrow-adbc/pull/1486#issuecomment-1906975818
@jorisvandenbossche how awful do you think the approach here is? :sweat_smile:
--
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] feat(python/adbc_driver_manager): handle KeyboardInterrupt [arrow-adbc]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on PR #1486:
URL: https://github.com/apache/arrow-adbc/pull/1486#issuecomment-1907001514
We could also implement the background-thread-and-self-pipe trick that Arrow C++ uses. That would be a good deal more complicated but would only need a single background thread per process
--
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] feat(python/adbc_driver_manager): handle KeyboardInterrupt [arrow-adbc]
Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on PR #1486:
URL: https://github.com/apache/arrow-adbc/pull/1486#issuecomment-1924320543
I'm going to implement the second option I sketched here and see how it works.
--
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