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

   `Cursor.adbc_read_partition()` reads through `AdbcConnectionReadPartition` 
and the returned stream belongs to the connection, but the cursor cancelled its 
statement instead: both from the SIGINT handler around the blocking calls and 
from `Cursor.adbc_cancel()`. A driver with no active query on the statement 
treats that as a no-op, leaving the partition read running.
   
   Track the current cancel target on the cursor: the statement after 
_clear()/execute(), the connection after `adbc_read_partition()`. Use it for 
`adbc_cancel()`, the blocking partition read, and the row iterator that 
consumes the result stream. Fix the `AdbcStatement.cancel` docstring, which 
claimed to cancel the connection.
   
   Test against the Flight SQL test server, whose "forever" query blocks 
ReadPartition until the client cancels the connection.
   
   AI disclaimer: Created with codex and gpt-6 astra. Iterated one and reviewed 
by me.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to