add a few comments for later improvements

custom/src/main/java/co/elastic/otel/ElasticSpanExporter.java

@@ -67,6 +67,10 @@ public Attributes getAttributes() {
               @Override
               public Resource getResource() {
                 Resource original = span.getResource();
+                // TODO once we implement asynchronous cloud resource loading, this is the place where
+                // we can merge them to make it as if they were available through the otel SDK
+                // if the original resource is immutable, we can probably keep a map and cache to prevent too
+                // much allocation
                 return Resource.create(original.getAttributes(), original.getSchemaUrl());
               }
             });

resources/build.gradle.kts

@@ -12,6 +12,9 @@ dependencies {
     }
     dependencies {
       // AWS cloud resource providers transitive dependencies
+      // TODO remove the static version dependencies, either by automatically getting the
+      // transitive dependencies of the shaded artifacts, or by reusing the common versions that
+      // are very likely provided through transitive dependencies
       implementation("com.fasterxml.jackson.core:jackson-core:2.15.2")
       implementation("com.squareup.okhttp3:okhttp:4.11.0")
 

smoke-tests/src/test/java/com/example/javaagent/smoketest/AwsResourceProvidersTest.java

@@ -214,6 +214,8 @@ private void testResourceProvider(ResourceAttributesCheck check) {
   }
 
   private static void mockEc2Metadata() {
+    // TODO when testing async resource loading, we need to configure extra delay on response, which
+    // should not slow down the app startup if the async loading works as expected
 
     mockServerClient
         .when(HttpRequest.request().withMethod("PUT").withPath("/latest/api/token"))