Refactoring: naming

JAVA-5956, JAVA-6117, JAVA-6113, JAVA-6119, JAVA-6141

Author: stIncMale

driver-core/src/main/com/mongodb/internal/operation/AsyncOperationHelper.java

@@ -335,7 +335,7 @@ static <D, T> void createReadCommandAndExecuteAsync(
     static <R> AsyncCallbackSupplier<R> decorateReadWithRetriesAsync(final RetryState retryState, final OperationContext operationContext,
             final AsyncCallbackSupplier<R> asyncReadFunction) {
         return new RetryingAsyncCallbackSupplier<>(retryState, onRetryableReadAttemptFailure(operationContext),
-                CommandOperationHelper::shouldAttemptToRetryRead, callback -> {
+                CommandOperationHelper::loggingShouldAttemptToRetryRead, callback -> {
             logRetryExecute(retryState, operationContext);
             asyncReadFunction.get(callback);
         });

driver-core/src/main/com/mongodb/internal/operation/CommandOperationHelper.java

@@ -166,7 +166,7 @@ static boolean isNamespaceError(final Throwable t) {
         }
     }
 
-    static boolean shouldAttemptToRetryRead(final RetryState retryState, final Throwable attemptFailure) {
+    static boolean loggingShouldAttemptToRetryRead(final RetryState retryState, final Throwable attemptFailure) {
         assertFalse(attemptFailure instanceof ResourceSupplierInternalException);
         boolean decision = isRetryableException(attemptFailure)
                 || (attemptFailure instanceof MongoSecurityException
@@ -178,7 +178,7 @@ static boolean shouldAttemptToRetryRead(final RetryState retryState, final Throw
     }
 
     static boolean loggingShouldAttemptToRetryWriteAndAddRetryableLabel(final RetryState retryState, final Throwable attemptFailure) {
-        Throwable attemptFailureNotToBeRetried = getAttemptFailureNotToRetryOrAddRetryableLabel(retryState, attemptFailure);
+        Throwable attemptFailureNotToBeRetried = getWriteAttemptFailureNotToBeRetriedOrAddRetryableLabel(retryState, attemptFailure);
         boolean decision = attemptFailureNotToBeRetried == null;
         if (!decision && retryState.attachment(AttachmentKeys.retryableCommandFlag()).orElse(false)) {
             logUnableToRetry(
@@ -189,14 +189,14 @@ static boolean loggingShouldAttemptToRetryWriteAndAddRetryableLabel(final RetryS
     }
 
     static boolean shouldAttemptToRetryWriteAndAddRetryableLabel(final RetryState retryState, final Throwable attemptFailure) {
-        return getAttemptFailureNotToRetryOrAddRetryableLabel(retryState, attemptFailure) != null;
+        return getWriteAttemptFailureNotToBeRetriedOrAddRetryableLabel(retryState, attemptFailure) != null;
     }
 
     /**
      * @return {@code null} if the decision is {@code true}. Otherwise, returns the {@link Throwable} that must not be retried.
      */
     @Nullable
-    private static Throwable getAttemptFailureNotToRetryOrAddRetryableLabel(final RetryState retryState, final Throwable attemptFailure) {
+    private static Throwable getWriteAttemptFailureNotToBeRetriedOrAddRetryableLabel(final RetryState retryState, final Throwable attemptFailure) {
         Throwable failure = attemptFailure instanceof ResourceSupplierInternalException ? attemptFailure.getCause() : attemptFailure;
         boolean decision = false;
         MongoException exceptionRetryableRegardlessOfCommand = null;

driver-core/src/main/com/mongodb/internal/operation/SyncOperationHelper.java

@@ -332,7 +332,7 @@ static <R> Supplier<R> decorateWriteWithRetries(final RetryState retryState,
     static <R> Supplier<R> decorateReadWithRetries(final RetryState retryState, final OperationContext operationContext,
             final Supplier<R> readFunction) {
         return new RetryingSyncSupplier<>(retryState, onRetryableReadAttemptFailure(operationContext),
-                CommandOperationHelper::shouldAttemptToRetryRead, () -> {
+                CommandOperationHelper::loggingShouldAttemptToRetryRead, () -> {
             logRetryExecute(retryState, operationContext);
             return readFunction.get();
         });