alamb commented on issue #8926:
URL:
https://github.com/apache/arrow-datafusion/issues/8926#issuecomment-1915277461
> I think this is good way to do this.
>
> ```
> pub struct SessionState {
> ...
> context : Option<Arc<dyn Any>>,
> ...
> }
> ```
>
> use SessionConfig need impl `ExtensionOptions` , somehow this is not a
`Options` normally.
I agree that looks like a reasonable way to add user defined context to
SessionState
As there are many things already called "context" , perhaps we could name it
something like `extension_context` instead?
```rust
pub struct SessionState {
...
extension_context : Option<Arc<dyn Any>>,
...
}
```
--
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]
