pr review

Author: zeitlinger

javaagent-extension-api/src/main/java/io/opentelemetry/javaagent/extension/internal/ConfigPropertiesTranslator.java

@@ -7,6 +7,7 @@
 
 import java.util.LinkedHashMap;
 import java.util.Map;
+import javax.annotation.Nullable;
 
 @SuppressWarnings("NonApiType")
 class ConfigPropertiesTranslator {
@@ -29,8 +30,9 @@ String translateProperty(String property) {
     return property;
   }
 
+  @SuppressWarnings("unchecked")
   @Nullable
-  public Object get(String propertyName) {
-    return fixedValues.get(propertyName);
+  public <T> T get(String propertyName) {
+    return (T) fixedValues.get(propertyName);
   }
 }

javaagent-extension-api/src/main/java/io/opentelemetry/javaagent/extension/internal/DeclarativeConfigPropertiesBridge.java

@@ -126,12 +126,11 @@ public Duration getDuration(String propertyName) {
     return Duration.ofMillis(millis);
   }
 
-  @SuppressWarnings("unchecked")
   @Override
   public List<String> getList(String propertyName) {
-    Object value = translator.get(propertyName);
+    List<String> value = translator.get(propertyName);
     if (value != null) {
-      return (List<String>) value;
+      return value;
     }
     List<String> propertyValue =
         getPropertyValue(
@@ -140,12 +139,11 @@ public List<String> getList(String propertyName) {
     return propertyValue == null ? Collections.emptyList() : propertyValue;
   }
 
-  @SuppressWarnings("unchecked")
   @Override
   public Map<String, String> getMap(String propertyName) {
-    Object fixed = translator.get(propertyName);
+    Map<String, String> fixed = translator.get(propertyName);
     if (fixed != null) {
-      return (Map<String, String>) fixed;
+      return fixed;
     }
     DeclarativeConfigProperties propertyValue =
         getPropertyValue(propertyName, DeclarativeConfigProperties::getStructured);