-
Notifications
You must be signed in to change notification settings - Fork 292
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
expose cassandra contact points and use for peer.service #5375
Conversation
9d86db4
to
b8468ff
Compare
BenchmarksParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 22 cases. |
b8468ff
to
564ab81
Compare
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.
Few inline questions.
About the tests, there is some System.setProperty("java.net.preferIPv4Stack", "true")
but nothing related to IPv6. Should it be tested too?
} | ||
return inetSocketAddress.getHostString(); | ||
}) | ||
.collect(Collectors.joining(",")); |
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.
Are they implicitly distinct?
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.
I think so
...dra-4/src/main/java/datadog/trace/instrumentation/datastax/cassandra4/ContactPointsUtil.java
Show resolved
Hide resolved
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.
Actually, I request to change a package name 😓
oups I have to clean it up (I used locally and forgot) thanks! |
564ab81
to
d0da813
Compare
d0da813
to
de87c2c
Compare
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.
LGTM!
de87c2c
to
02b467c
Compare
What Does This Do
Surface cassandra contact points on a tag
db.cassandra.contact.points
and use it as a privileged precursor for cassandra peer.serviceMotivation
Additional Notes