Skip to content

[IMP] Consider deprecating apply_boolean_mask with apply_retention_mask #22219

Description

@mhaseeb123

Description

Currently, we have two public masking APIs called apply_boolean_mask and apply_deletion_mask. These API internally call the detail apply_mask API with a mask_type enum which has values: RETENTION, DELETION.

We should consider deprecating the apply_boolean_mask with apply_retention_mask for consistency.

Alternatives

N/A. Current state of APIs

Considerations.

cudf::apply_boolean_mask is among the most used public APIs and its deprecation would affect a lot of partners.

Metadata

Metadata

Assignees

No one assigned

    Labels

    improvementImprovement / enhancement to an existing functionlibcudfAffects libcudf (C++/CUDA) code.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions