Skip to content

Conversation

owenowenisme
Copy link
Member

@owenowenisme owenowenisme commented Oct 13, 2025

Why are these changes needed?

In #57035 we deprecate concurrency params and use compute instead in map, map_batches, flat_map and filter so the related docs should be changed to use it as well so user won't use deprecated params.

Related issue number

Follow up for in #57035

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run pre-commit jobs to lint the changes in this PR. (pre-commit setup)
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

Signed-off-by: You-Cheng Lin (Owen) <[email protected]>
@owenowenisme owenowenisme added the go add ONLY when ready to merge, run all tests label Oct 13, 2025
return self.aggregate(*aggs)

@PublicAPI(api_group=FA_API_GROUP)
def map_groups(
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missed updating this API in #57035, so I’m including the follow-up fix in this PR.

@owenowenisme owenowenisme marked this pull request as ready for review October 13, 2025 05:33
@owenowenisme owenowenisme requested review from a team as code owners October 13, 2025 05:33
@ray-gardener ray-gardener bot added docs An issue or change related to documentation data Ray Data-related issues labels Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data Ray Data-related issues docs An issue or change related to documentation go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant