alexanderbianchi commented on PR #27210:
URL: https://github.com/apache/datafusion/pull/27210#issuecomment-6836133808
I took a quick look at the flow and went through it with the same agent that
helped with my initial brainstorming. I think this is very much on the right
track. The preparation hook and unified cache look like the key pieces we
needed.
The remaining extension support seems like a straightforward next step,
following DataFusion’s existing extension patterns. Something like:
pub enum TableSelector {
Version(ScalarValue),
Timestamp(i64),
Extension(Arc<dyn TableSelectorExtension>),
}
With appropriate equality/hash and serialization behavior, that should let
downstream implementations pass their own table-selection arguments through to
the schema provider without adding more special cases to SessionState.
Thanks for exploring this! Ping me when you want a deeper review!
--
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]
