Repository navigation
fix(store): bound jemalloc downloads - #3310
Merged
imbajin merged 1 commit intoOct 12, 2026
Merged
Conversation
Limit connection and total transfer time. Preserve verified cache and partial-file cleanup. Cover timeout fallback in launcher regressions. Document optional allocator startup behavior.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3310 +/- ##
============================================
+ Coverage 52.29% 52.30% +0.01%
+ Complexity 6052 2271 -3781
============================================
Files 762 762
Lines 67399 67399
Branches 9138 9138
============================================
+ Hits 35244 35251 +7
Misses 28601 28601
+ Partials 3554 3547 -7 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
This was referenced Oct 11, 2026
Pengzna
approved these changes
Oct 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Store startup downloads an optional jemalloc library before launching Java. A mirror that accepts a connection but stops responding could hold startup indefinitely. The download now allows 5 seconds to connect and 30 seconds in total, then follows the existing fallback and starts without adding the downloaded library to
LD_PRELOAD.Changes
Validation
The existing shell regression suite passed, including the new cases. With a real HTTP connection that never responded, the old transfer remained blocked; the new transfer timed out after about 30 seconds, cleaned its temporary file and took the optional fallback. Formatting and a clean Java 17 reactor compile passed. Three independent source reviews found no actionable issue.
The same org branch passed its selected CI, including launcher contracts, Server/RocksDB/HStore tests, container build and deployment checks, Compose smoke and platform compatibility. Local reproduction used scripts and loopback HTTP; the original release-container and Kubernetes acceptance scenarios remain separate follow-up checks.
Org validation: hugegraph#287
Documentation
Doc - Done:
hugegraph-store/README.md.Paired website documentation: ASF #517. Coordinate the documentation merge with this code change.