more

Author: trask

instrumentation/armeria/armeria-1.3/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/armeria/v1_3/ArmeriaInstrumentationModule.java

@@ -22,7 +22,7 @@ public ArmeriaInstrumentationModule() {
 
   @Override
   public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
-    // Unrelated class which was added in Armeria 1.3.0, the minimum version we support.
+    // added in 1.3.0
     return hasClassesNamed("com.linecorp.armeria.server.metric.PrometheusExpositionServiceBuilder");
   }
 

instrumentation/clickhouse/clickhouse-client-v1-0.5/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/clickhouse/clientv1/v0_5/ClickHouseClientV1InstrumentationModule.java

@@ -30,7 +30,7 @@ public boolean isHelperClass(String className) {
 
   @Override
   public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
-    // Unrelated class which was added in 0.5.0, the minimum version where tests pass
+    // added in 0.5.0
     return hasClassesNamed("com.clickhouse.client.config.ClickHouseProxyType");
   }
 

instrumentation/couchbase/couchbase-2.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/couchbase/v2_0/CouchbaseInstrumentationModule.java

@@ -46,7 +46,7 @@ public List<String> injectedClassNames() {
 
   @Override
   public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
-    // removed in 3.x
+    // removed in 3.0.0
     return hasClassesNamed("com.couchbase.client.java.CouchbaseAsyncBucket");
   }
 }

instrumentation/couchbase/couchbase-2.6/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/couchbase/v2_6/CouchbaseInstrumentationModule.java

@@ -25,7 +25,7 @@ public CouchbaseInstrumentationModule() {
 
   @Override
   public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
-    // introduced in java-client 2.6, removed in 3.x
+    // added in 2.6.0, removed in 3.0.0
     return hasClassesNamed("com.couchbase.client.java.auth.CertAuthenticator");
   }
 

instrumentation/elasticsearch/elasticsearch-api-client-7.16/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/apiclient/ElasticsearchApiClientInstrumentationModule.java

@@ -28,7 +28,7 @@ public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
     return hasClassesNamed(
             // added in 7.16
             "co.elastic.clients.elasticsearch.ElasticsearchClient")
-        // added in 8.10 (native OTel instrumentation)
+        // added in 8.10 (native OTel support)
         .and(not(hasClassesNamed("co.elastic.clients.transport.instrumentation.Instrumentation")));
   }
 

instrumentation/elasticsearch/elasticsearch-rest-5.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/rest/v5_0/ElasticsearchRest5InstrumentationModule.java

@@ -23,7 +23,7 @@ public ElasticsearchRest5InstrumentationModule() {
 
   @Override
   public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
-    // org.elasticsearch.client.Request was added in 6.4.0
+    // added in 6.4.0
     return not(hasClassesNamed("org.elasticsearch.client.Request"));
   }
 

instrumentation/elasticsearch/elasticsearch-rest-6.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/rest/v6_4/ElasticsearchRest6InstrumentationModule.java

@@ -23,9 +23,9 @@ public ElasticsearchRest6InstrumentationModule() {
 
   @Override
   public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
-    // class present in 6.4+ rest client
+    // added in 6.4.0
     return hasClassesNamed("org.elasticsearch.client.Request")
-        // class introduced in 7.0.0
+        // added in 7.0.0
         .and(not(hasClassesNamed("org.elasticsearch.client.RestClient$InternalRequest")));
   }
 

instrumentation/elasticsearch/elasticsearch-rest-7.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/elasticsearch/rest/v7_0/ElasticsearchRest7InstrumentationModule.java

@@ -27,7 +27,7 @@ public ElasticsearchRest7InstrumentationModule() {
   public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
     // added in 7.0.0
     return hasClassesNamed("org.elasticsearch.client.RestClient$InternalRequest")
-        // added in 8.10 (native OTel instrumentation)
+        // added in 8.10 (native OTel support)
         .and(not(hasClassesNamed("co.elastic.clients.transport.instrumentation.Instrumentation")));
   }
 

instrumentation/grizzly-2.3/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/grizzly/GrizzlyInstrumentationModule.java

@@ -22,7 +22,7 @@ public GrizzlyInstrumentationModule() {
 
   @Override
   public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
-    // class added in 2.3
+    // added in 2.3
     return hasClassesNamed("org.glassfish.grizzly.InputSource");
   }
 

instrumentation/log4j/log4j-context-data/log4j-context-data-2.17/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/log4j/contextdata/v2_17/Log4j2InstrumentationModule.java

@@ -41,9 +41,9 @@ public List<String> exposedClassNames() {
   @Override
   public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
     return hasClassesNamed(
-        // class added in 2.17.0 and backported to 2.12.3
+        // added in 2.17.0 and backported to 2.12.3
         "org.apache.logging.log4j.core.lookup.ConfigurationStrSubstitutor",
-        // class added in 2.15.0
+        // added in 2.15.0
         "org.apache.logging.log4j.core.config.arbiters.DefaultArbiter");
   }