milenkovicm opened a new pull request, #20249:
URL: https://github.com/apache/datafusion/pull/20249

   ## Which issue does this PR close?
   
   - Closes #.
   
   ## Rationale for this change
   
   This is WIP which  exposes datafusion query planner as FFI interface,
   I hope to split this PR into two separate once implementation is nailed
   
   ## What changes are included in this PR?
   
   - query planner moved to different module and interface changed (this will 
be subject of separate PR, once query planner part gets green light) there is 
question at which point Session should be downcast to SessionState. There are 
two approaches, non of which i particularly like 
   - FFI query planner implementation
   
   There are few open questions 
   
   - is ffi planner module correct
   - can we expose FFI_SessionRef as public 
   
   ## Are these changes tested?
   
   There is one test, integration test to follow if this makes sese
   
   ## Are there any user-facing changes?
   
   yes query planner is moved to different module and signature changed


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to