fix: use Commit as method name for per-attempt metrics

The per-attempt transaction count should use METHOD_COMMIT (Commit) since
each attempt records a commit operation. The overall transaction latency
continues to use METHOD_TRANSACTION_RUN (Transaction.Run).

Author: lqiu96

java-datastore/google-cloud-datastore/src/main/java/com/google/cloud/datastore/DatastoreImpl.java

@@ -240,7 +240,7 @@ private void recordAttempt(String status) {
       attributes.put(TelemetryConstants.ATTRIBUTES_KEY_STATUS, status);
       attributes.put(
           TelemetryConstants.ATTRIBUTES_KEY_METHOD_NAME,
-          TelemetryConstants.METHOD_TRANSACTION_RUN);
+          TelemetryConstants.METHOD_COMMIT);
       metricsRecorder.recordTransactionAttemptCount(1, attributes);
     }
 

java-datastore/google-cloud-datastore/src/test/java/com/google/cloud/datastore/DatastoreImplMetricsTest.java

@@ -142,7 +142,7 @@ public void runInTransaction_recordsPerAttemptCountOnSuccess() {
     assertThat(point).isNotNull();
     assertThat(point.getValue()).isEqualTo(1);
     assertThat(point.getAttributes().get(AttributeKey.stringKey("method_name")))
-        .isEqualTo("Transaction.Run");
+        .isEqualTo("Commit");
 
     EasyMock.verify(rpcFactoryMock, rpcMock);
   }

java-datastore/google-cloud-datastore/src/test/java/com/google/cloud/datastore/telemetry/OpenTelemetryMetricsRecorderTest.java

@@ -91,7 +91,7 @@ public void recordTransactionAttemptCount_recordsCounterWithAttributes() {
     attributes.put(TelemetryConstants.ATTRIBUTES_KEY_STATUS, "ABORTED");
     attributes.put(
         TelemetryConstants.ATTRIBUTES_KEY_METHOD_NAME,
-        TelemetryConstants.METHOD_TRANSACTION_RUN);
+        TelemetryConstants.METHOD_COMMIT);
 
     recorder.recordTransactionAttemptCount(1, attributes);
 
@@ -116,7 +116,7 @@ public void recordTransactionAttemptCount_recordsCounterWithAttributes() {
     assertThat(point.getValue()).isEqualTo(1);
     assertThat(point.getAttributes().get(AttributeKey.stringKey("status"))).isEqualTo("ABORTED");
     assertThat(point.getAttributes().get(AttributeKey.stringKey("method_name")))
-        .isEqualTo("Transaction.Run");
+        .isEqualTo("Commit");
   }
 
   @Test
@@ -125,13 +125,13 @@ public void recordTransactionAttemptCount_multipleAttempts_accumulates() {
     abortedAttributes.put(TelemetryConstants.ATTRIBUTES_KEY_STATUS, "ABORTED");
     abortedAttributes.put(
         TelemetryConstants.ATTRIBUTES_KEY_METHOD_NAME,
-        TelemetryConstants.METHOD_TRANSACTION_RUN);
+        TelemetryConstants.METHOD_COMMIT);
 
     Map<String, String> okAttributes = new HashMap<>();
     okAttributes.put(TelemetryConstants.ATTRIBUTES_KEY_STATUS, "OK");
     okAttributes.put(
         TelemetryConstants.ATTRIBUTES_KEY_METHOD_NAME,
-        TelemetryConstants.METHOD_TRANSACTION_RUN);
+        TelemetryConstants.METHOD_COMMIT);
 
     // Simulate a retry scenario: first attempt ABORTED, second attempt OK
     recorder.recordTransactionAttemptCount(1, abortedAttributes);