-
Notifications
You must be signed in to change notification settings - Fork 906
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.
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
Release review for java tracing v1 #1465
Conversation
|
||
Public Examples: [Examples](https://github.com/open-telemetry/opentelemetry-java/tree/main/examples) | ||
|
||
Discovered Issues: [List of filed issues by the TC members during the review process] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A set of examples from the list, some PRs lead to changes in the specs like adding SpanLimits
, some PRs were internal inconsistency between packages, some were public classes/methods that could be hided.
open-telemetry/opentelemetry-java#2913
open-telemetry/opentelemetry-java#2894
open-telemetry/opentelemetry-java#2786
open-telemetry/opentelemetry-java#2784
open-telemetry/opentelemetry-java#2724
open-telemetry/opentelemetry-java#2664
open-telemetry/opentelemetry-java#2658
open-telemetry/opentelemetry-java#2629
open-telemetry/opentelemetry-java#2626
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
@bogdandrutu Java Tracing API and SDK have released their 1.0 version. Should we close this PR? |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
@jkwatson @bogdandrutu Ping |
I'm not sure what I'm supposed to do with this at this point. |
@jkwatson Guess adding the issues that Bogdan mentioned in his comment, and then we can merge, hopefully. Please confirm @bogdandrutu ;) |
7e3193a
to
7323271
Compare
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
Closed as inactive. Feel free to reopen if this PR is still being worked on. |
This is a draft of the release review template proposed in #1418 for the proposed release of the Java Tracing API and SDK for version 1.0.0.