Skip to content

enable rocksdb jemalloc only for make build-release#2404

Closed
tzemanovic wants to merge 2 commits intomainfrom
tomas/opt-jemalloc
Closed

enable rocksdb jemalloc only for make build-release#2404
tzemanovic wants to merge 2 commits intomainfrom
tomas/opt-jemalloc

Conversation

@tzemanovic
Copy link
Copy Markdown
Collaborator

@tzemanovic tzemanovic commented Jan 16, 2024 •
edited
Loading

Describe your changes

RocksDB tikv-jemalloc-sys takes a long time to build in rust-analyzer. This change disable RocksDB jemalloc feature by default and only uses it for make build-release

Indicate on which release or other PRs this topic is based on

0.30.0

Checklist before merging to draft

  • I have added a changelog
  • Git history is in acceptable state

@tzemanovic tzemanovic requested a review from Fraccaman January 16, 2024 14:04
@tzemanovic tzemanovic marked this pull request as ready for review January 16, 2024 14:05
@tzemanovic
Copy link
Copy Markdown
Collaborator Author

tzemanovic commented Jan 16, 2024

unfortunately this is not yet supported rust-lang/cargo#1197

@tzemanovic tzemanovic closed this Jan 16, 2024
tzemanovic added a commit that referenced this pull request Jan 24, 2024
@tzemanovic
changelog: add #2404
4d5b435
tzemanovic added a commit that referenced this pull request Jan 24, 2024
@tzemanovic
changelog: add #2404
87963ae
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Fraccaman Fraccaman Awaiting requested review from Fraccaman

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@tzemanovic