dask-sql icon indicating copy to clipboard operation
dask-sql copied to clipboard

[DF] Support multiple LogicalPlan types/names per plugin

Open jdye64 opened this issue 2 years ago • 0 comments

Is your feature request related to a problem? Please describe. The current plugin model allows for a single name/type to be used for each plugin. While straightforward this prevents the same plugin from being used for multiple LogicalPlan types. For example, aggregate.py could be used for both the LogicalPlan type of Aggregate but also for Distinct. It is currently not possible to support that however.

Describe the solution you'd like Modify the utils.py and existing plugins to provide a List[str] of names/types they support. For example change aggregate.py line class_name = "Aggregate" to class_name = ["Aggregate", "Distinct"]. Therefore allowing the plugin to be used for both types.

Describe alternatives you've considered None

Additional context None

jdye64 avatar Jul 18 '22 13:07 jdye64