-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Description
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
The name MergeExec has long confused me.
The name MergeExec implies to me that its purpose is to "merge sorted streams" into another sorted stream
However, MergeExec does something more like UNION ALL of streams into a single stream, combining the input streams in some arbitrary output order. Something that looks to me like an actual "merge" operation is described in #362.
Describe the solution you'd like
I propose renaming MergeExec to UnionToSingleStreamExec to make it clearer
It could also potentially combined with the existing UnionExec (which preserves partitioning) as another flag
Describe alternatives you've considered
As this would be a fairly invasive (and backwards incompatible change) leaving the current naming is also a possibility.