-
Notifications
You must be signed in to change notification settings - Fork 199
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
Update smoke tests and fix Kafka running locally #2460
Conversation
350d229
to
2534f10
Compare
2534f10
to
506ccd2
Compare
506ccd2
to
10f5755
Compare
@@ -192,7 +192,7 @@ public static class Instrumentation { | |||
|
|||
public EnabledByDefaultInstrumentation azureSdk = new EnabledByDefaultInstrumentation(); | |||
public EnabledByDefaultInstrumentation cassandra = new EnabledByDefaultInstrumentation(); | |||
public EnabledByDefaultInstrumentation jdbc = new EnabledByDefaultInstrumentation(); | |||
public JdbcInstrumentation jdbc = new JdbcInstrumentation(); |
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.
It seems possible to make the fields final. It may be converted into constants and the Instrumentation class could be an interface or the class could own a private constructor.
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.
sorry, this code change is part of #2453, this PR is built on top of that one, and I forgot to set the target branch for it to reflect, will rebase this PR after that one is approved/merged
No description provided.