Skip to content

Conversation

@owenowenisme
Copy link
Member

@owenowenisme owenowenisme commented Oct 21, 2025

Description

Add ApproximateTopK aggregator.

Reason:

  • Enables efficient support for the summary API with categorical aggregations.

Related issues

Additional information

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
@owenowenisme owenowenisme added the go add ONLY when ready to merge, run all tests label Oct 21, 2025
Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
@owenowenisme owenowenisme marked this pull request as ready for review October 21, 2025 13:39
@owenowenisme owenowenisme requested a review from a team as a code owner October 21, 2025 13:39
Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
@owenowenisme owenowenisme changed the title [Data] Add ApproximateTopK to aggregraters [Data] Add ApproximateTopK to aggregators Oct 21, 2025
@ray-gardener ray-gardener bot added the data Ray Data-related issues label Oct 21, 2025
Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
@alexeykudinkin alexeykudinkin enabled auto-merge (squash) October 28, 2025 02:19
@alexeykudinkin alexeykudinkin merged commit 55aa6c3 into ray-project:master Oct 28, 2025
7 checks passed
YoussefEssDS pushed a commit to YoussefEssDS/ray that referenced this pull request Nov 8, 2025
## Description
Add ApproximateTopK aggregator.

Reason:
- Enables efficient support for the summary API with categorical
aggregations.
## Related issues

## Additional information

---------

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
landscapepainter pushed a commit to landscapepainter/ray that referenced this pull request Nov 17, 2025
## Description
Add ApproximateTopK aggregator.

Reason:
- Enables efficient support for the summary API with categorical
aggregations.
## Related issues

## Additional information

---------

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
Aydin-ab pushed a commit to Aydin-ab/ray-aydin that referenced this pull request Nov 19, 2025
## Description
Add ApproximateTopK aggregator.

Reason:
- Enables efficient support for the summary API with categorical
aggregations.
## Related issues

## Additional information

---------

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
Signed-off-by: Aydin Abiar <aydin@anyscale.com>
Future-Outlier pushed a commit to Future-Outlier/ray that referenced this pull request Dec 7, 2025
## Description
Add ApproximateTopK aggregator.

Reason:
- Enables efficient support for the summary API with categorical
aggregations.
## Related issues

## Additional information

---------

Signed-off-by: You-Cheng Lin <mses010108@gmail.com>
Signed-off-by: Future-Outlier <eric901201@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data Ray Data-related issues go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants