Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Data] Update outdated ExecutionPlan docstring #46638

Merged
merged 2 commits into from
Jul 16, 2024

Conversation

scottjlee
Copy link
Contributor

@scottjlee scottjlee commented Jul 15, 2024

Why are these changes needed?

Docstring only update, no code changes. Update ExecutionPlan docstring with updated implementation details.

Related issue number

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • 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 :(

Scott Lee added 2 commits July 15, 2024 12:05
Signed-off-by: Scott Lee <sjl@anyscale.com>
Signed-off-by: Scott Lee <sjl@anyscale.com>
Copy link
Member

@bveeramani bveeramani left a comment

Choose a reason for hiding this comment

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

Nice, ty

@scottjlee scottjlee added the go add ONLY when ready to merge, run all tests label Jul 15, 2024
@anyscalesam anyscalesam added data Ray Data-related issues triage Needs triage (eg: priority, bug/not-bug, and owning component) labels Jul 16, 2024
@bveeramani bveeramani merged commit 17dfcfd into ray-project:master Jul 16, 2024
6 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
data Ray Data-related issues go add ONLY when ready to merge, run all tests triage Needs triage (eg: priority, bug/not-bug, and owning component)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants