Skip to content

[cherry-pick][core] Don't preload jemalloc for worker. (#39446)#39492

Merged
GeneDer merged 1 commit intoray-project:releases/2.7.0from
fishbone:cherry-pick-39446
Sep 9, 2023
Merged

[cherry-pick][core] Don't preload jemalloc for worker. (#39446)#39492
GeneDer merged 1 commit intoray-project:releases/2.7.0from
fishbone:cherry-pick-39446

Conversation

@fishbone
Copy link
Copy Markdown
Contributor

@fishbone fishbone commented Sep 8, 2023

Some library is not compatible with jemalloc. This PR disable jemalloc for python workers.

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

Some library is not compatible with jemalloc. This PR disable jemalloc for python workers.
Copy link
Copy Markdown
Member

@GeneDer GeneDer left a comment

Choose a reason for hiding this comment

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

Pick of #39446

Thanks for fixing it Yi!

@GeneDer
Copy link
Copy Markdown
Member

GeneDer commented Sep 8, 2023

@iycheng make sure to fix DCO

Copy link
Copy Markdown
Contributor

@zhe-thoughts zhe-thoughts left a comment

Choose a reason for hiding this comment

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

This PR resolves multiple release blockers ;) Let's pick

@GeneDer GeneDer merged commit a6c3c89 into ray-project:releases/2.7.0 Sep 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants