Address review comment

Author: steverao

instrumentation/apache-elasticjob-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apacheelasticjob/v3_0/ElasticJobExperimentalAttributeExtractor.java

@@ -18,8 +18,8 @@ class ElasticJobExperimentalAttributeExtractor
       AttributeKey.stringKey("scheduling.apache-elasticjob.job.name");
   private static final AttributeKey<String> ELASTICJOB_TASK_ID =
       AttributeKey.stringKey("scheduling.apache-elasticjob.task.id");
-  private static final AttributeKey<Long> ELASTICJOB_ITEM =
-      AttributeKey.longKey("scheduling.apache-elasticjob.item");
+  private static final AttributeKey<Long> ELASTICJOB_SHARDING_ITEM_INDEX =
+      AttributeKey.longKey("scheduling.apache-elasticjob.sharding.item.index");
   private static final AttributeKey<Long> ELASTICJOB_SHARDING_TOTAL_COUNT =
       AttributeKey.longKey("scheduling.apache-elasticjob.sharding.total.count");
   private static final AttributeKey<String> ELASTICJOB_SHARDING_ITEM_PARAMETERS =
@@ -32,7 +32,7 @@ public void onStart(
       ElasticJobProcessRequest elasticJobProcessRequest) {
     attributes.put(ELASTICJOB_JOB_NAME, elasticJobProcessRequest.getJobName());
     attributes.put(ELASTICJOB_TASK_ID, elasticJobProcessRequest.getTaskId());
-    attributes.put(ELASTICJOB_ITEM, elasticJobProcessRequest.getItem());
+    attributes.put(ELASTICJOB_SHARDING_ITEM_INDEX, elasticJobProcessRequest.getShardingItemIndex());
     attributes.put(
         ELASTICJOB_SHARDING_TOTAL_COUNT, elasticJobProcessRequest.getShardingTotalCount());
     if (elasticJobProcessRequest.getShardingItemParameters() != null) {

instrumentation/apache-elasticjob-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/apacheelasticjob/v3_0/ElasticJobProcessRequest.java

@@ -8,7 +8,7 @@
 public final class ElasticJobProcessRequest {
   private String jobName;
   private String taskId;
-  private int item;
+  private int shardingItemIndex;
   private int shardingTotalCount;
   private String shardingItemParameters;
   private boolean failed;
@@ -19,14 +19,14 @@ public final class ElasticJobProcessRequest {
   public static ElasticJobProcessRequest create(
       String jobName,
       String taskId,
-      int item,
+      int shardingItemIndex,
       int shardingTotalCount,
       String shardingItemParameters,
       String jobType) {
     ElasticJobProcessRequest request = new ElasticJobProcessRequest();
     request.jobName = jobName;
     request.taskId = taskId;
-    request.item = item;
+    request.shardingItemIndex = shardingItemIndex;
     request.shardingTotalCount = shardingTotalCount;
     request.shardingItemParameters = shardingItemParameters;
     request.jobType = jobType;
@@ -36,7 +36,7 @@ public static ElasticJobProcessRequest create(
   public static ElasticJobProcessRequest createWithUserJobInfo(
       String jobName,
       String taskId,
-      int item,
+      int shardingItemIndex,
       int shardingTotalCount,
       String shardingItemParameters,
       String jobType,
@@ -45,7 +45,7 @@ public static ElasticJobProcessRequest createWithUserJobInfo(
     ElasticJobProcessRequest request = new ElasticJobProcessRequest();
     request.jobName = jobName;
     request.taskId = taskId;
-    request.item = item;
+    request.shardingItemIndex = shardingItemIndex;
     request.shardingTotalCount = shardingTotalCount;
     request.shardingItemParameters = shardingItemParameters;
     request.jobType = jobType;
@@ -70,8 +70,8 @@ public String getTaskId() {
     return this.taskId;
   }
 
-  public int getItem() {
-    return this.item;
+  public int getShardingItemIndex() {
+    return this.shardingItemIndex;
   }
 
   public int getShardingTotalCount() {

instrumentation/apache-elasticjob-3.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/apacheelasticjob/v3_0/ElasticJobTest.java

@@ -94,7 +94,7 @@ void testHttpJob() {
       bootstrap.schedule();
 
       testing.waitAndAssertSortedTraces(
-          comparingRootSpanAttribute(longKey("scheduling.apache-elasticjob.item")),
+          comparingRootSpanAttribute(longKey("scheduling.apache-elasticjob.sharding.item.index")),
           trace ->
               trace.hasSpansSatisfyingExactly(
                   span ->
@@ -140,7 +140,7 @@ void testSimpleJob() {
       bootstrap.schedule();
 
       testing.waitAndAssertSortedTraces(
-          comparingRootSpanAttribute(longKey("scheduling.apache-elasticjob.item")),
+          comparingRootSpanAttribute(longKey("scheduling.apache-elasticjob.sharding.item.index")),
           trace ->
               trace.hasSpansSatisfyingExactly(
                   span ->
@@ -188,7 +188,7 @@ void testDataflowJob() {
       bootstrap.schedule();
 
       testing.waitAndAssertSortedTraces(
-          comparingRootSpanAttribute(longKey("scheduling.apache-elasticjob.item")),
+          comparingRootSpanAttribute(longKey("scheduling.apache-elasticjob.sharding.item.index")),
           trace ->
               trace.hasSpansSatisfyingExactly(
                   span ->
@@ -335,7 +335,7 @@ private static List<AttributeAssertion> elasticJobBaseAttributes(
     List<AttributeAssertion> assertions = new ArrayList<>();
     assertions.add(equalTo(stringKey("job.system"), "elasticjob"));
     assertions.add(equalTo(stringKey("scheduling.apache-elasticjob.job.name"), jobName));
-    assertions.add(equalTo(longKey("scheduling.apache-elasticjob.item"), item));
+    assertions.add(equalTo(longKey("scheduling.apache-elasticjob.sharding.item.index"), item));
     assertions.add(
         equalTo(longKey("scheduling.apache-elasticjob.sharding.total.count"), totalCount));
     assertions.add(