zeroshade commented on PR #28:
URL: https://github.com/apache/arrow-experiments/pull/28#issuecomment-2069777958

   @kou so all of the things related directly to the protocol are in the 
`cudf-flight-poc.cc` file. Everything else is just wrapping the underlying UCX 
primitives with nicer classes so that I could abstract that away in the 
`cudf-flight-poc.cc` file to simplify it and make it easier to show off the 
protocol without getting bogged down by the UCX details directly. 
   
   I'm not sure what would be the best way to simplify this further outside of 
possibly shifting all of the ucx abstractions into a subdirectory rather than 
having it all in a single directory. That would at least make it easier to 
split `cudf-flight-poc.cc` into several smaller files to make it easier to see 
the "server" vs "client" implementations maybe?


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