Skip to content

Make Telemetry Tags Immutable #20788

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
msfroh merged 12 commits into from
Mar 9, 2026
Merged

Make Telemetry Tags Immutable #20788

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
cf4e273
Refactor Tags to immutable sorted-array implementation with precomput…
akrahdan Mar 5, 2026
cf503ce
Addressed javadoc styling
akrahdan Mar 5, 2026
6f1b1eb
retrigger CI
akrahdan Mar 6, 2026
6e1ff74
retrigger CI
akrahdan Mar 6, 2026
f8f21db
Address review feedback: improve Tags API clarity and compatibility
akrahdan Mar 6, 2026
3a7cda7
retrigger CI
akrahdan Mar 7, 2026
aa323b0
retrigger CI
akrahdan Mar 8, 2026
fa2424a
retrigger CI
akrahdan Mar 8, 2026
078f77f
Remove @Deprecated annotations from experimental Tags API
akrahdan Mar 8, 2026
08b48c1
retrigger CI
akrahdan Mar 8, 2026
ed54255
Validate tag value types to String, Long, Double, or Boolean
akrahdan Mar 8, 2026
47bebbd
Added method overloads
akrahdan Mar 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,12 +39,14 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
- Implement FieldMappingIngestionMessageMapper for pull-based ingestion ([#20729](https://github.com/opensearch-project/OpenSearch/pull/20729))

### Changed
- Make telemetry `Tags` immutable ([#20788](https://github.com/opensearch-project/OpenSearch/pull/20788))
- Move Randomness from server to libs/common ([#20570](https://github.com/opensearch-project/OpenSearch/pull/20570))
- Use env variable (OPENSEARCH_FIPS_MODE) to enable opensearch to run in FIPS enforced mode instead of checking for existence of bcFIPS jars ([#20625](https://github.com/opensearch-project/OpenSearch/pull/20625))
- Update streaming flag to use search request context ([#20530](https://github.com/opensearch-project/OpenSearch/pull/20530))
- Move pull-based ingestion classes from experimental to publicAPI ([#20704](https://github.com/opensearch-project/OpenSearch/pull/20704))

### Fixed
- Fix `AutoForceMergeMetrics` silently dropping tags due to unreassigned `addTag()` return value ([#20788](https://github.com/opensearch-project/OpenSearch/pull/20788))
- Fix flaky test failures in ShardsLimitAllocationDeciderIT ([#20375](https://github.com/opensearch-project/OpenSearch/pull/20375))
- Prevent criteria update for context aware indices ([#20250](https://github.com/opensearch-project/OpenSearch/pull/20250))
- Update EncryptedBlobContainer to adhere limits while listing blobs in specific sort order if wrapped blob container supports ([#20514](https://github.com/opensearch-project/OpenSearch/pull/20514))
Expand Down
Loading
Loading