Skip to content

[Tests] Enable large test#10391

Merged
rkooo567 merged 5 commits intoray-project:masterfrom
rkooo567:enable-large-test
Aug 30, 2020
Merged

[Tests] Enable large test#10391
rkooo567 merged 5 commits intoray-project:masterfrom
rkooo567:enable-large-test

Conversation

@rkooo567
Copy link
Contributor

Why are these changes needed?

Enable large tests as we support object reconstruction now.

Related issue number

Checks

  • 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/latest/.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failure rates at https://ray-travis-tracker.herokuapp.com/.
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested (please justify below)

@rkooo567
Copy link
Contributor Author

NOTE: I don't know if tests will pass

Copy link
Contributor

@stephanie-wang stephanie-wang left a comment

Choose a reason for hiding this comment

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

Thanks! Some of the tests may no longer be relevant, so let me know if you're having trouble figuring out any test failures; it may be safe to remove some of them.

@ericl ericl removed their assignment Aug 28, 2020
@stephanie-wang stephanie-wang added the @author-action-required The PR author is responsible for the next step. Remove tag to send back to the reviewer. label Aug 28, 2020
@rkooo567 rkooo567 merged commit c8b14fd into ray-project:master Aug 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

@author-action-required The PR author is responsible for the next step. Remove tag to send back to the reviewer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants