[skip ci] Rename

Author: brfrn169

core/src/main/java/com/scalar/db/transaction/consensuscommit/ConsensusCommitUtils.java

@@ -319,7 +319,7 @@ public static int getNextTxVersion(@Nullable Integer currentTxVersion) {
     }
   }
 
-  static void extractAfterImageColumnsFromBeforeImage(
+  static void createAfterImageColumnsFromBeforeImage(
       Map<String, Column<?>> columns,
       TransactionResult result,
       Set<String> beforeImageColumnNames) {

core/src/main/java/com/scalar/db/transaction/consensuscommit/RecoveryExecutor.java

@@ -1,6 +1,6 @@
 package com.scalar.db.transaction.consensuscommit;
 
-import static com.scalar.db.transaction.consensuscommit.ConsensusCommitUtils.extractAfterImageColumnsFromBeforeImage;
+import static com.scalar.db.transaction.consensuscommit.ConsensusCommitUtils.createAfterImageColumnsFromBeforeImage;
 
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.util.concurrent.ThreadFactoryBuilder;
@@ -102,10 +102,10 @@ private Optional<TransactionResult> createRecoveredResult(
     throwUncommittedRecordExceptionIfTransactionNotExpired(state, selection, result, transactionId);
 
     if (!state.isPresent() || state.get().getState() == TransactionState.ABORTED) {
-      return createRolledBackRecord(selection, result, transactionId);
+      return createRecordFromBeforeImage(selection, result, transactionId);
     } else {
       assert state.get().getState() == TransactionState.COMMITTED;
-      return createRolledForwardResult(selection, result, transactionId);
+      return createResultFromAfterImage(selection, result, transactionId);
     }
   }
 
@@ -124,7 +124,7 @@ private void throwUncommittedRecordExceptionIfTransactionNotExpired(
     }
   }
 
-  private Optional<TransactionResult> createRolledBackRecord(
+  private Optional<TransactionResult> createRecordFromBeforeImage(
       Selection selection, TransactionResult result, String transactionId) throws CrudException {
     if (!result.hasBeforeImage()) {
       return Optional.empty();
@@ -138,7 +138,7 @@ private Optional<TransactionResult> createRolledBackRecord(
 
     Map<String, Column<?>> columns = new HashMap<>();
 
-    extractAfterImageColumnsFromBeforeImage(columns, result, beforeImageColumnNames);
+    createAfterImageColumnsFromBeforeImage(columns, result, beforeImageColumnNames);
 
     Key partitionKey = ScalarDbUtils.getPartitionKey(result, tableMetadata);
     partitionKey.getColumns().forEach(c -> columns.put(c.getName(), c));
@@ -151,7 +151,7 @@ private Optional<TransactionResult> createRolledBackRecord(
     return Optional.of(new TransactionResult(new ResultImpl(columns, tableMetadata)));
   }
 
-  private Optional<TransactionResult> createRolledForwardResult(
+  private Optional<TransactionResult> createResultFromAfterImage(
       Selection selection, TransactionResult result, String transactionId) throws CrudException {
     if (result.getState() == TransactionState.DELETED) {
       return Optional.empty();

core/src/main/java/com/scalar/db/transaction/consensuscommit/RollbackMutationComposer.java

@@ -5,7 +5,7 @@
 import static com.scalar.db.transaction.consensuscommit.Attribute.STATE;
 import static com.scalar.db.transaction.consensuscommit.Attribute.toIdValue;
 import static com.scalar.db.transaction.consensuscommit.Attribute.toStateValue;
-import static com.scalar.db.transaction.consensuscommit.ConsensusCommitUtils.extractAfterImageColumnsFromBeforeImage;
+import static com.scalar.db.transaction.consensuscommit.ConsensusCommitUtils.createAfterImageColumnsFromBeforeImage;
 import static com.scalar.db.transaction.consensuscommit.ConsensusCommitUtils.getTransactionTableMetadata;
 
 import com.scalar.db.api.ConditionBuilder;
@@ -106,7 +106,7 @@ private Put composePut(Operation base, TransactionResult result) throws Executio
     clusteringKey.ifPresent(putBuilder::clusteringKey);
 
     Map<String, Column<?>> columns = new HashMap<>();
-    extractAfterImageColumnsFromBeforeImage(columns, result, beforeImageColumnNames);
+    createAfterImageColumnsFromBeforeImage(columns, result, beforeImageColumnNames);
     columns.values().forEach(putBuilder::value);
 
     // Set before image columns to null

core/src/test/java/com/scalar/db/transaction/consensuscommit/ConsensusCommitUtilsTest.java

@@ -625,7 +625,7 @@ void getNextTxVersion_LargeValueGiven_shouldReturnNextVersion() {
   }
 
   @Test
-  public void extractAfterImageColumnsFromBeforeImage_shouldExtractCorrectly() {
+  public void createAfterImageColumnsFromBeforeImage_shouldExtractCorrectly() {
     // Arrange
     Map<String, Column<?>> columns = new HashMap<>();
     Set<String> beforeImageColumnNames = new HashSet<>();
@@ -641,7 +641,7 @@ public void extractAfterImageColumnsFromBeforeImage_shouldExtractCorrectly() {
     when(result.getColumns()).thenReturn(resultColumns);
 
     // Act
-    ConsensusCommitUtils.extractAfterImageColumnsFromBeforeImage(
+    ConsensusCommitUtils.createAfterImageColumnsFromBeforeImage(
         columns, result, beforeImageColumnNames);
 
     // Assert
@@ -655,7 +655,7 @@ public void extractAfterImageColumnsFromBeforeImage_shouldExtractCorrectly() {
 
   @Test
   public void
-      extractAfterImageColumnsFromBeforeImage_versionColumnWithZero_shouldCreateNullVersion() {
+      createAfterImageColumnsFromBeforeImage_versionColumnWithZero_shouldCreateNullVersion() {
     // Arrange
     Map<String, Column<?>> columns = new HashMap<>();
     Set<String> beforeImageColumnNames = new HashSet<>();
@@ -668,7 +668,7 @@ public void extractAfterImageColumnsFromBeforeImage_shouldExtractCorrectly() {
     when(result.getColumns()).thenReturn(resultColumns);
 
     // Act
-    ConsensusCommitUtils.extractAfterImageColumnsFromBeforeImage(
+    ConsensusCommitUtils.createAfterImageColumnsFromBeforeImage(
         columns, result, beforeImageColumnNames);
 
     // Assert
@@ -678,7 +678,7 @@ public void extractAfterImageColumnsFromBeforeImage_shouldExtractCorrectly() {
   }
 
   @Test
-  public void extractAfterImageColumnsFromBeforeImage_versionColumnWithNonZero_shouldCopyValue() {
+  public void createAfterImageColumnsFromBeforeImage_versionColumnWithNonZero_shouldCopyValue() {
     // Arrange
     Map<String, Column<?>> columns = new HashMap<>();
     Set<String> beforeImageColumnNames = new HashSet<>();
@@ -691,7 +691,7 @@ public void extractAfterImageColumnsFromBeforeImage_versionColumnWithNonZero_sho
     when(result.getColumns()).thenReturn(resultColumns);
 
     // Act
-    ConsensusCommitUtils.extractAfterImageColumnsFromBeforeImage(
+    ConsensusCommitUtils.createAfterImageColumnsFromBeforeImage(
         columns, result, beforeImageColumnNames);
 
     // Assert