Skip to content

Fix ReadOnlyArrayMap.containsValue to handle null #8652

Open
NithinU2802 wants to merge 1 commit into
open-telemetry:mainfrom
NithinU2802:fix/contains-value-null-handling
Open

Fix ReadOnlyArrayMap.containsValue to handle null #8652
NithinU2802 wants to merge 1 commit into
open-telemetry:mainfrom
NithinU2802:fix/contains-value-null-handling

Conversation

@NithinU2802

Copy link
Copy Markdown

Previously, containsValue() assumed stored values were non-null when comparing values, which could result in incorrect behavior when the map contained a null value. The comparison has been updated to be null-safe while preserving the existing behavior for non-null values.

I'm happy to make any changes if needed.

Closes #8651

@NithinU2802
NithinU2802 requested a review from a team as a code owner July 20, 2026 17:45
@linux-foundation-easycla

linux-foundation-easycla Bot commented Jul 20, 2026

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: NithinU2802 / name: NithinU2802 (464a043)

@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Jul 20, 2026

Copy link
Copy Markdown

Pull request dashboard status

Status last refreshed: 2026-07-21 20:56:26 UTC.

  • Waiting on: Author
  • Next steps:
    • Investigate required status check failures.
    • Address or respond to 1 review feedback item:
      • Top-level feedback: 1
      • For each item, link to the commit that addresses it, explain why no change is needed, or ask a follow-up question.

This automated status or its linked feedback items may be incorrect. If something looks wrong, report it with the result you expected.

@jack-berg

Copy link
Copy Markdown
Member

Looks like a formatting failure. Please run ./gradlew spotlessApply and commit / push.

https://github.com/open-telemetry/opentelemetry-java/blob/main/docs/knowledge/build.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix ReadOnlyArrayMap null value handling in containsValue

2 participants