Skip to content

Remove default predicate for cols {} in Selection DSL #1284

Open
@Jolanrensen

Description

@Jolanrensen

In the selection dsl you can write:
myGroup.cols() if you want to refer to all columns nested inside the group.
This is allowed, because the default argument makes it myGroup.cols { true }.

However, because we use a predicate, we cannot get compiler plugin support for that function.
It's better to let users use all() and allCols() instead. It's more expressive and can get compiler plugin backing :)

We need to simply create overloads for cols() without argument, deprecate them and point people to the all() family.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Compiler pluginAnything related to the DataFrame Compiler PluginenhancementNew feature or requestgood first issueGood issues to pick-up for newcomers

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions