westonpace opened a new pull request #10205: URL: https://github.com/apache/arrow/pull/10205
Per the JIRA `Future<>::AddCallback` callbacks receive a `Status`. `Future<T>::AddCallback` callbacks receive a `Result<T>` `Future<>::Then` callbacks receive nothing `Future<T>::Then` callbacks receive `const T&` To achieve this I had to explicitly specialize the empty `Future` but I introduced `FutureBase` to reduce the amount of duplicated code. `detail::Empty` is still around (although it got renamed to `internal::Empty` as a side effect of moving into `functional.h`). It could be removed if one wanted to create a specialized `FutureImpl` but I that doesn't seem to be needed at the moment. Draft while still in cleanup/WIP but all tests pass on my machine. -- 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
