Skip to content

Revert "Revert "Unhandled exception handler based on local ref counti…#14113

Merged
ericl merged 4 commits intoray-project:masterfrom
ericl:revert-2
Feb 15, 2021
Merged

Revert "Revert "Unhandled exception handler based on local ref counti…#14113
ericl merged 4 commits intoray-project:masterfrom
ericl:revert-2

Conversation

@ericl
Copy link
Copy Markdown
Contributor

@ericl ericl commented Feb 15, 2021

…ng (#14049)" (#14099)"

This reverts commit b45ae76.

Why are these changes needed?

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/master/.
  • 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 :(

@ericl ericl added the tests-ok The tagger certifies test failures are unrelated and assumes personal liability. label Feb 15, 2021
@ericl ericl merged commit e457872 into ray-project:master Feb 15, 2021
fishbone pushed a commit to fishbone/ray that referenced this pull request Feb 16, 2021
ray-project#14113)

* Revert "Revert "Unhandled exception handler based on local ref counting (ray-project#14049)" (ray-project#14099)"

This reverts commit b45ae76.

* reomve test

* fix

* fix
fishbone added a commit to fishbone/ray that referenced this pull request Feb 16, 2021
architkulkarni added a commit that referenced this pull request Feb 16, 2021
ericl pushed a commit that referenced this pull request Feb 16, 2021
ericl added a commit to ericl/ray that referenced this pull request Feb 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests-ok The tagger certifies test failures are unrelated and assumes personal liability.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants