Skip to content

Accept legacy const sampler type configuration (27.x) - #12368

Merged
tomas-langer merged 8 commits into
helidon-io:mainfrom
tomas-langer:12367-accept-legacy-const-sampler-type
Sep 2, 2026
Merged

Accept legacy const sampler type configuration (27.x)#12368
tomas-langer merged 8 commits into
helidon-io:mainfrom
tomas-langer:12367-accept-legacy-const-sampler-type

Conversation

@tomas-langer

Copy link
Copy Markdown
Member

Resolves #12367

This is a source-faithful carry-forward of 4.x PR #11988 and commit 49edbfa0 to main.

Adds a configuration factory so the legacy const sampler type maps to SamplerType.CONSTANT without adding another public enum value. Covers lowercase and uppercase const and constant configurations.

This carry-forward intentionally preserves inherited source behavior, including existing invalid-value diagnostics and generated config-metadata representation; it is not an attempt to resolve those findings.

@tomas-langer tomas-langer added tracing backport Issues that are merged into a single branch, but missing in either master or previous release 27.x Issues for 27.x version branch labels Aug 27, 2026
@tomas-langer tomas-langer self-assigned this Aug 27, 2026
@oracle-contributor-agreement oracle-contributor-agreement Bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Aug 27, 2026
tvallin
tvallin previously approved these changes Aug 28, 2026
danielkec
danielkec previously approved these changes Aug 28, 2026
@tomas-langer
tomas-langer dismissed stale reviews from danielkec and tvallin via 96c063a August 28, 2026 22:06
@tomas-langer
tomas-langer requested a review from danielkec August 28, 2026 22:07

@danielkec danielkec left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Two current-head review findings.

Comment thread builder/codegen/src/main/java/io/helidon/builder/codegen/SchemaGenerator.java Outdated
@tomas-langer
tomas-langer requested a review from danielkec August 31, 2026 21:55
@tomas-langer
tomas-langer force-pushed the 12367-accept-legacy-const-sampler-type branch from 0f30617 to f1c8038 Compare September 1, 2026 07:19
@tomas-langer
tomas-langer merged commit 5344108 into helidon-io:main Sep 2, 2026
44 checks passed
@tomas-langer
tomas-langer deleted the 12367-accept-legacy-const-sampler-type branch September 2, 2026 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

27.x Issues for 27.x version branch backport Issues that are merged into a single branch, but missing in either master or previous release OCA Verified All contributors have signed the Oracle Contributor Agreement. tracing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Accept legacy const sampler type configuration (27.x)

3 participants