bkietz commented on pull request #10205:
URL: https://github.com/apache/arrow/pull/10205#issuecomment-839985461


   Instead of sniffing the class' template parameter, it'd be straightforward 
to check a callback's argument type.
   
   in `AddCallback(on_complete)`:
   - if `on_complete` takes a Status, let it be completed with just the status 
(ignoring the value)
   - otherwise, let it be completed with the Result
   
   in `Then(on_success, on_failure)`:
   - if `on_success` takes no args, let it be wrapped in an on_success which 
ignores the value
   - otherwise, let it receive the value


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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to