Skip to content

feat(java-agent): add release telemetry comparison workflow#503

Closed
suhr25 wants to merge 2 commits into
open-telemetry:mainfrom
suhr25:feat/java-release-comparison-page
Closed

feat(java-agent): add release telemetry comparison workflow#503
suhr25 wants to merge 2 commits into
open-telemetry:mainfrom
suhr25:feat/java-release-comparison-page

Conversation

@suhr25
Copy link
Copy Markdown
Contributor

@suhr25 suhr25 commented May 17, 2026

SUMMARY

Implemented the Java Agent Release Comparison page to let users compare two agent versions and quickly see instrumentation and telemetry changes across releases.

FIX

Replaced the existing placeholder page with a working release comparison experience using the existing telemetry diff engine. Added version selection with URL sync, release summary stats, and a searchable/filterable results table showing added, removed, and changed instrumentations. Also added tests for the new hook and UI components, and enabled the feature flag locally.

Feature remains gated behind JAVA_RELEASE_COMPARISON.

CHECKLIST

  • Feature implemented with minimal surface-area changes
  • Existing filter state API reused
  • Added unit test coverage
  • No breaking API changes
  • UI behavior verified locally
  • Lint + typecheck passing
  • Ready for review

Replace the placeholder skeleton at /java-agent/releases with a working
cross-version diff view. Adds useReleaseDiff hook, ReleaseSummaryStats,
and ReleaseResultsTable; enables JAVA_RELEASE_COMPARISON flag locally.

Signed-off-by: Suhrid Marwah <suhridmarwah07@gmail.com>
@suhr25 suhr25 requested review from a team as code owners May 17, 2026 10:16
@netlify
Copy link
Copy Markdown

netlify Bot commented May 17, 2026

Deploy Preview for otel-ecosystem-explorer ready!

Name Link
🔨 Latest commit 4de986d
🔍 Latest deploy log https://app.netlify.com/projects/otel-ecosystem-explorer/deploys/6a09989ab535080008922410
😎 Deploy Preview https://deploy-preview-503--otel-ecosystem-explorer.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Signed-off-by: Suhrid Marwah <suhridmarwah07@gmail.com>
@suhr25
Copy link
Copy Markdown
Contributor Author

suhr25 commented May 17, 2026

Hi @jaydeluca,
Kindly review.
Thanks!

@jaydeluca
Copy link
Copy Markdown
Member

@suhr25 im a bit confused as to why you are submitting this. @Pittu-Sharma was working on this previously and the issue is assigned to them, are the two of you coordinating/communicating in some way?

@jaydeluca
Copy link
Copy Markdown
Member

@suhr25 Also as a side note, I’d appreciate if you could avoid directly pinging me or other maintainers on issues and PRs unless something is truly urgent. We all receive notifications for activity in the repository and review things as time permits. There are only a few maintainers managing a fairly large volume of issues and PRs, so a bit of patience and understanding goes a long way. Thanks for keeping that in mind.

@suhr25
Copy link
Copy Markdown
Contributor Author

suhr25 commented May 17, 2026

Thanks for the clarification.
Will open another PR with a fresh issue.

@Pittu-Sharma
Copy link
Copy Markdown
Contributor

Pittu-Sharma commented May 17, 2026

Thanks for the clarification. Will open another PR with a fresh issue.

@suhr25, I am working on this, I think there is some misunderstanding .

@suhr
Copy link
Copy Markdown

suhr commented May 17, 2026

I'm not @suhr25.

@suhr25
Copy link
Copy Markdown
Contributor Author

suhr25 commented May 17, 2026

@Pittu-Sharma
Yeah the mentor informed me about it , that was clear to me.

@jaydeluca
Copy link
Copy Markdown
Member

going to close this one. Thank you @suhr25 for the attempted contribution, in the future just try and avoid issues that are already assigned to someone

@jaydeluca jaydeluca closed this May 18, 2026
@suhr25 suhr25 deleted the feat/java-release-comparison-page branch May 21, 2026 10:30
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.

4 participants