Skip to content

Add max queued requests docs#46778

Merged
edoakes merged 1 commit intomasterfrom
akshay-anyscale-patch-2
Jul 24, 2024
Merged

Add max queued requests docs#46778
edoakes merged 1 commit intomasterfrom
akshay-anyscale-patch-2

Conversation

@akshay-anyscale
Copy link
Contributor

Why are these changes needed?

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 :(

Signed-off-by: akshay-anyscale <122416226+akshay-anyscale@users.noreply.github.com>
Copy link
Collaborator

@edoakes edoakes left a comment

Choose a reason for hiding this comment

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

hm not ideal that these options are duplicated manually from the decorator docstring. makes it easy to get out of sync

@edoakes edoakes enabled auto-merge (squash) July 24, 2024 20:14
@github-actions github-actions bot added the go add ONLY when ready to merge, run all tests label Jul 24, 2024
@edoakes edoakes merged commit 2f4302a into master Jul 24, 2024
@edoakes edoakes deleted the akshay-anyscale-patch-2 branch July 24, 2024 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants