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

   ## Which issue does this PR close?
   
   - Addresses part of https://github.com/apache/datafusion/issues/18671 but 
does not close it.
   
   Note: This is based on top of 
https://github.com/apache/datafusion/pull/18681. I will rebase after that one 
merges to remove the parts of the code that are not relevant to this PR.
   
   
   ## Rationale for this change
   
   We have improved the ability to de/serialize expressions across the FFI 
boundary by passing around a `TaskContextProvider`, but we still have the case 
where we may want to use a logical or physical extension codec. This PR adds 
the support for these, but does not yet include it in the processing path. A 
follow on PR will add these codecs to the existing FFI infrastructure.
   
   ## What changes are included in this PR?
   
   - Adds FFI variants for Logical and Physical Extension Codecs
   - Adds unit tests
   
   ## Are these changes tested?
   
   Yes, `cargo tarpaulin` reports 92% coverage.
   
   ## Are there any user-facing changes?
   
   No, this is a new addition.


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