Skip to content

chore: Add SessionState to MockContextProvider just like SessionContextProvider #11531

@dharanad

Description

@dharanad

Is your feature request related to a problem or challenge?

          I prefer not to re-introduce `with_expr_planner` back for context provider.

It is better to use session state in context, and add planner into session state.

Similar to SessionContextProvider

struct SessionContextProvider<'a> {
    state: &'a SessionState,
    ...
}

Originally posted by @jayzhan211 in #11487 (comment)

This is cleanup task to remove functions like with_udaf with_expr_planner from MockContextProvider and make it consistent with public interfaces. We need to update all the testcases which are affected by this change.

Describe the solution you'd like

No response

Describe alternatives you've considered

No response

Additional context

No response

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions