Skip to content

[Java] remove resetLibraryPath #10671

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
raulchen merged 1 commit into from
Sep 9, 2020
Merged

[Java] remove resetLibraryPath #10671

raulchen merged 1 commit into from
Sep 9, 2020

Conversation

@chaokunyang
Copy link
Member

@chaokunyang chaokunyang commented Sep 9, 2020 •
edited
Loading

Why are these changes needed?

resetLibraryPath is useless and doesn't work in higher version jdk. This PR remove resetLibraryPath code

Related issue number

#10673

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)

@raulchen raulchen merged commit 72ac85c into ray-project:master Sep 9, 2020
@raulchen raulchen deleted the remove_reset_library_path branch September 9, 2020 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@raulchen raulchen raulchen approved these changes

@kfstorm kfstorm kfstorm approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@chaokunyang @raulchen @kfstorm