jorgecarleitao commented on pull request #9703:
URL: https://github.com/apache/arrow/pull/9703#issuecomment-804948544


   Got. Then fwiw, I think that we can build that with this trait, we just need 
a bit more work / investigation:
   
   I am not 100% sure, but I think that the path for libraries that want to 
expose a different `DataFrame` (datafusion-python needs this) and use 
datafusion's `ExecutionContext` is to make:
   
   `struct ExecutionContext<D: DataFrame>`
   
   For the case of Ballista, I think we need the opposite: Ballista has an 
`ExecutionContext`, and `DataFrame` should just have a way of absorving that 
implementation, e.g. via `DataFrame<C: ExecutionContext>` (we need a trait for 
`ExecutionContext` for this).
   
   (This is a hand-waving idea)


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


Reply via email to