Skip to content
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

Add missing equals and hashCode method into datadog.trace.instrumentation.opentracing32.OTSpan #1995

Merged
merged 1 commit into from
Oct 15, 2020

Conversation

ygree
Copy link
Contributor

@ygree ygree commented Oct 15, 2020

It's not possible to check equality of opentracing32.OTSpan intances because it doesn't implement the equals and hashCode methods for the internal agent span. Also the opentracing32.OTSpan class is not part of public API and the getDelegate method is not assessible via the Span or MutableSpan inteface.

At the same time datadog.opentracing.OTSpan has proper implementations of the equals and hashCode methods.

…tion.opentracing32.OTSpan

It's not possible to check equality of opentracing32.OTSpan intances because it doesn't implement the equals and hashCode methods for the internal agent span. Also the opentracing32.OTSpan class is not part of public API and the getDelegate method is not assessible via the Span or MutableSpan inteface.

At the same time datadog.opentracing.OTSpan has proper implementations of the equals and hashCode methods.
@ygree ygree requested a review from a team as a code owner October 15, 2020 17:51
Copy link
Contributor

@devinsba devinsba left a comment

Choose a reason for hiding this comment

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

Great, thanks for contributing

@randomanderson randomanderson added the tag: community Community contribution label Oct 15, 2020
@randomanderson randomanderson added this to the 0.66.0 milestone Oct 15, 2020
@tylerbenson tylerbenson merged commit 471f490 into DataDog:master Oct 15, 2020
@ygree ygree deleted the otspan-equals branch December 16, 2021 22:26
@ygree ygree self-assigned this Jan 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tag: community Community contribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants