Apply suggestions from code review

Co-authored-by: Trask Stalnaker <[email protected]>

Author: laurit

instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/internal/Experimental.java

@@ -55,7 +55,7 @@ public static <REQUEST> void setUrlTemplateExtractor(
     }
   }
 
-  @SuppressWarnings({"rawtypes", "unchecked"}) // we loose the generic type information
+  @SuppressWarnings({"rawtypes", "unchecked"}) // we lose the generic type information
   public static <REQUEST> void internalSetUrlTemplateExtractor(
       BiConsumer<HttpSpanNameExtractorBuilder<REQUEST>, Function<REQUEST, String>>
           urlTemplateExtractorSetter) {
@@ -76,7 +76,7 @@ public static <REQUEST, RESPONSE> void addOperationListenerAttributesExtractor(
     }
   }
 
-  @SuppressWarnings({"rawtypes", "unchecked"}) // we loose the generic type information
+  @SuppressWarnings({"rawtypes", "unchecked"}) // we lose the generic type information
   public static <REQUEST, RESPONSE> void internalAddOperationListenerAttributesExtractor(
       BiConsumer<
               InstrumenterBuilder<REQUEST, RESPONSE>,

instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/internal/RuntimeVirtualFieldSupplier.java

@@ -51,7 +51,7 @@ private static final class CacheBasedVirtualFieldSupplier implements VirtualFiel
         ownerToFieldToImplementationMap = Cache.weak();
 
     @Override
-    // storing VirtualField instances in a map looses the generic types
+    // storing VirtualField instances in a map loses the generic types
     @SuppressWarnings("unchecked")
     public <U extends T, V extends F, T, F> VirtualField<U, V> find(
         Class<T> type, Class<F> fieldType) {

instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/internal/ServiceLoaderUtil.java

@@ -18,7 +18,7 @@ public final class ServiceLoaderUtil {
 
   private ServiceLoaderUtil() {}
 
-  // we loose the generic type information because of using the loader function
+  // we lose the generic type information because of using the loader function
   @SuppressWarnings("unchecked")
   public static <T> Iterable<T> load(Class<T> clazz) {
     return (Iterable<T>) loadFunction.apply(clazz);

instrumentation/grpc-1.6/library/src/main/java/io/opentelemetry/instrumentation/grpc/v1_6/TracingClientInterceptor.java

@@ -37,11 +37,11 @@ final class TracingClientInterceptor implements ClientInterceptor {
   private static final String SENT = "SENT";
   private static final String RECEIVED = "RECEIVED";
 
-  @SuppressWarnings("rawtypes") // AtomicLongFieldUpdater.newUpdate looses generic type
+  @SuppressWarnings("rawtypes") // AtomicLongFieldUpdater.newUpdate loses generic type
   private static final AtomicLongFieldUpdater<TracingClientCall> SENT_MESSAGE_ID_UPDATER =
       AtomicLongFieldUpdater.newUpdater(TracingClientCall.class, "sentMessageId");
 
-  @SuppressWarnings("rawtypes") // AtomicLongFieldUpdater.newUpdate looses generic type
+  @SuppressWarnings("rawtypes") // AtomicLongFieldUpdater.newUpdate loses generic type
   private static final AtomicLongFieldUpdater<TracingClientCall> RECEIVED_MESSAGE_ID_UPDATER =
       AtomicLongFieldUpdater.newUpdater(TracingClientCall.class, "receivedMessageId");
 

instrumentation/grpc-1.6/library/src/main/java/io/opentelemetry/instrumentation/grpc/v1_6/TracingServerInterceptor.java

@@ -37,11 +37,11 @@ final class TracingServerInterceptor implements ServerInterceptor {
   private static final String SENT = "SENT";
   private static final String RECEIVED = "RECEIVED";
 
-  @SuppressWarnings("rawtypes") // AtomicLongFieldUpdater.newUpdate looses generic type
+  @SuppressWarnings("rawtypes") // AtomicLongFieldUpdater.newUpdate loses generic type
   private static final AtomicLongFieldUpdater<TracingServerCall> SENT_MESSAGE_ID_UPDATER =
       AtomicLongFieldUpdater.newUpdater(TracingServerCall.class, "sentMessageId");
 
-  @SuppressWarnings("rawtypes") // AtomicLongFieldUpdater.newUpdate looses generic type
+  @SuppressWarnings("rawtypes") // AtomicLongFieldUpdater.newUpdate loses generic type
   private static final AtomicLongFieldUpdater<TracingServerCall> RECEIVED_MESSAGE_ID_UPDATER =
       AtomicLongFieldUpdater.newUpdater(TracingServerCall.class, "receivedMessageId");
 

instrumentation/jedis/jedis-common/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/JedisRequestContext.java

@@ -53,7 +53,7 @@ public static <T> void endIfNotAttached(
     }
   }
 
-  @SuppressWarnings("unchecked") // we loose the generic type in ThreadLocal
+  @SuppressWarnings("unchecked") // we lose the generic type in ThreadLocal
   private static <T> JedisRequestContext<T> current() {
     return (JedisRequestContext<T>) contextThreadLocal.get();
   }

instrumentation/opentelemetry-api/opentelemetry-api-1.50/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_50/incubator/logs/ApplicationLogRecordBuilder150Incubator.java

@@ -117,7 +117,7 @@ public <T> ExtendedLogRecordBuilder setAttribute(AttributeKey<T> key, @Nullable
   }
 
   @Override
-  @SuppressWarnings("unchecked") // converting ExtendedAttributeKey looses generic type
+  @SuppressWarnings("unchecked") // converting ExtendedAttributeKey loses generic type
   public <T> ExtendedLogRecordBuilder setAttribute(ExtendedAttributeKey<T> key, T value) {
     io.opentelemetry.api.incubator.common.ExtendedAttributeKey<T> agentKey =
         convertExtendedAttributeKey(key);
@@ -140,7 +140,7 @@ public ExtendedLogRecordBuilder setException(Throwable throwable) {
   @SuppressWarnings({
     "unchecked",
     "rawtypes"
-  }) // converting ExtendedAttributeKey looses generic type
+  }) // converting ExtendedAttributeKey loses generic type
   private static io.opentelemetry.api.incubator.common.ExtendedAttributes convertExtendedAttributes(
       ExtendedAttributes applicationAttributes) {
     io.opentelemetry.api.incubator.common.ExtendedAttributesBuilder agentAttributes =
@@ -159,7 +159,7 @@ private static io.opentelemetry.api.incubator.common.ExtendedAttributes convertE
     return agentAttributes.build();
   }
 
-  @SuppressWarnings("rawtypes") // converting ExtendedAttributeKey looses generic type
+  @SuppressWarnings("rawtypes") // converting ExtendedAttributeKey loses generic type
   private static io.opentelemetry.api.incubator.common.ExtendedAttributeKey
       convertExtendedAttributeKey(ExtendedAttributeKey applicationKey) {
     switch (applicationKey.getType()) {

instrumentation/resources/library/src/main/java/io/opentelemetry/instrumentation/resources/AttributeResourceProvider.java

@@ -37,7 +37,7 @@ private AttributeBuilder() {}
 
     @CanIgnoreReturnValue
     @Override
-    @SuppressWarnings({"unchecked", "rawtypes"}) // we loose generic types when storing in map
+    @SuppressWarnings({"unchecked", "rawtypes"}) // we lose generic types when storing in map
     public <T> AttributeBuilder add(AttributeKey<T> key, Function<D, Optional<T>> getter) {
       attributeGetters.put((AttributeKey) key, Objects.requireNonNull((Function) getter));
       return this;

instrumentation/spring/spring-boot-autoconfigure/src/main/java/io/opentelemetry/instrumentation/spring/autoconfigure/internal/properties/SpringConfigProperties.java

@@ -193,7 +193,7 @@ public Double getDouble(String name) {
         otelSdkProperties.getDouble(name));
   }
 
-  @SuppressWarnings("unchecked") // reading list form environment looses generic type
+  @SuppressWarnings("unchecked") // reading list form environment loses generic type
   @Override
   public List<String> getList(String name) {
     String normalizedName = ConfigUtil.normalizeEnvironmentVariableKey(name);
@@ -224,7 +224,7 @@ public Duration getDuration(String name) {
         .getDuration(name);
   }
 
-  @SuppressWarnings("unchecked") // reading map looses generic type
+  @SuppressWarnings("unchecked") // reading map loses generic type
   @Override
   public Map<String, String> getMap(String name) {
     Map<String, String> otelSdkMap = otelSdkProperties.getMap(name);

instrumentation/undertow-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/undertow/UndertowHelper.java

@@ -67,7 +67,7 @@ public void exchangeCompleted(Context context, HttpServerExchange exchange) {
     }
   }
 
-  @SuppressWarnings("unchecked") // we loose type info with attachments
+  @SuppressWarnings("unchecked") // we lose type info with attachments
   @Nullable
   public Context getServerContext(HttpServerExchange exchange) {
     AttachmentKey<Context> contextKey =
@@ -78,7 +78,7 @@ public Context getServerContext(HttpServerExchange exchange) {
     return exchange.getAttachment(contextKey);
   }
 
-  @SuppressWarnings("unchecked") // we loose type info with attachments
+  @SuppressWarnings("unchecked") // we lose type info with attachments
   private static void attachServerContext(Context context, HttpServerExchange exchange) {
     AttachmentKey<Context> contextKey =
         (AttachmentKey<Context>)