lidavidm commented on issue #811:
URL: https://github.com/apache/arrow-adbc/issues/811#issuecomment-2123624086

   > sync_error is populated if a synchronous error happens before any 
asynchronous operations have begun.
   
   IMO this gets annoying, as you have to handle every error in two locations
   
   > the async callbacks return int rather than void so that the callbacks can 
indicate whether an error was encountered on their end and that the producer 
should cancel/stop calling callback methods.
   
   int being basically bool here? 
   
   Other questions
   
   - For cancellation, we'll continue to use the existing method?
   - How will this integrate with the rest of the Arrow ecosystem? It sounds 
like we'll have to convert everything back to synchronous interfaces in the 
end...


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