-
Couldn't load subscription status.
- Fork 1.7k
Closed
Labels
api changeChanges the API exposed to users of the crateChanges the API exposed to users of the crateenhancementNew feature or requestNew feature or request
Description
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
We currently have three Configurations: ExecutionConfig, PhysicalPlanConfig and RuntimeConfig. They all share batch_size, which is only meaningful for query execution.
As we get RuntimeEnv plumbing through all execute() methods, we've got RuntimeConfig available to all executing code. Therefore, we'd better consolidate these configs and better separate their responsibilities.
Describe the solution you'd like
- Keep the
batch_sizeonly inRuntimeConfig, remove all unrelatedbatch_sizepassing during query planning. - Rename
PhysicalPlanConfigtoFileScanConfigthat clarifies its usage scenario.
Describe alternatives you've considered
Additional context
#1526 (comment) as well for context.
Metadata
Metadata
Assignees
Labels
api changeChanges the API exposed to users of the crateChanges the API exposed to users of the crateenhancementNew feature or requestNew feature or request