docs: rewrite IMPLEMENTING_JRES.md for BaseJRE pattern#1339
Open
stokpop wants to merge 4 commits into
Open
Conversation
PR cloudfoundry#1288 refactored all standard JREs to embed BaseJRE (14 lines each), but the guide still showed the old 100+ line manual implementation with explicit Supply()/Finalize() boilerplate. Rewrite to document the actual pattern: newBaseJRE() + optional extraFinalizeOpts. Removes sections on manual memory calculator and JVMKill integration (now internal to BaseJRE). Adds variation-point table and examples for all three patterns (standard, exact-dir, error-hint).
…ooting Rebalances the IMPLEMENTING_JRES.md rewrite — adds back lean sections for: - Helper function reference table (GetJREVersion, WriteJavaOpts, etc.) - profile.d script output example - Memory calculator: runtime output and user customization env vars - JVMKill: purpose, heap dump / volume service binding - Troubleshooting: JAVA_HOME, memory calculator, version resolution
- Use Java 21 in version examples and profile.d output - Prefer JBP_CONFIG_<JRE>_JRE for memory calculator config (covers stack_threads, class_count, headroom) - Note MEMORY_CALCULATOR_* env vars take precedence but don't cover class_count - Add stack_threads/class_count/headroom descriptions
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
1 participant
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.
Summary
IMPLEMENTING_JRES.mdto reflect theBaseJREtemplate-method pattern introduced in PR refactor: extract BaseJRE to eliminate duplication across standard JRE providers #1288What changed
BaseJREautomatically)BaseJREvariation points table, 3-step implementation guide, lean examples for standard/exact-dir/error-hint patternsJBP_CONFIG_<JRE>_JREas preferred (coversstack_threads,class_count,headroom) with a note thatMEMORY_CALCULATOR_*env vars override but don't coverclass_countRelated
Follows #1288 (
BaseJRErefactor that this doc now describes)Please review if complete or must be extended.