Apply brfrn169 suggestion

Author: Torch3333

core/src/main/java/com/scalar/db/common/checker/OperationChecker.java

@@ -159,8 +159,7 @@ private void check(ScanAll scanAll) throws ExecutionException {
       throw new IllegalArgumentException(
           CoreError.OPERATION_CHECK_ERROR_CROSS_PARTITION_SCAN_ORDERING.buildMessage(scanAll));
     }
-    throwIfCrossPartitionScanOrderingOnBlobColumnNotSupported(scanAll, metadata);
-    checkOrderings(scanAll, metadata);
+    checkOrderingsForScanAll(scanAll, metadata);
 
     if (!config.isCrossPartitionScanFilteringEnabled() && !scanAll.getConjunctions().isEmpty()) {
       throw new IllegalArgumentException(
@@ -169,9 +168,6 @@ private void check(ScanAll scanAll) throws ExecutionException {
     checkConjunctions(scanAll, metadata);
   }
 
-  protected void throwIfCrossPartitionScanOrderingOnBlobColumnNotSupported(
-      ScanAll scanAll, TableMetadata metadata) {};
-
   private void checkProjections(Selection selection, TableMetadata metadata) {
     for (String projection : selection.getProjections()) {
       if (!metadata.getColumnNames().contains(projection)) {
@@ -262,7 +258,7 @@ private void checkOrderings(Scan scan, TableMetadata metadata) {
     }
   }
 
-  private void checkOrderings(ScanAll scanAll, TableMetadata metadata) {
+  protected void checkOrderingsForScanAll(ScanAll scanAll, TableMetadata metadata) {
     for (Scan.Ordering ordering : scanAll.getOrderings()) {
       if (!metadata.getColumnNames().contains(ordering.getColumnName())) {
         throw new IllegalArgumentException(

core/src/main/java/com/scalar/db/storage/jdbc/JdbcOperationChecker.java

@@ -22,8 +22,8 @@ public JdbcOperationChecker(
   }
 
   @Override
-  protected void throwIfCrossPartitionScanOrderingOnBlobColumnNotSupported(
-      ScanAll scanAll, TableMetadata metadata) {
+  protected void checkOrderingsForScanAll(ScanAll scanAll, TableMetadata metadata) {
+    super.checkOrderingsForScanAll(scanAll, metadata);
     rdbEngine.throwIfCrossPartitionScanOrderingOnBlobColumnNotSupported(scanAll, metadata);
   }
 }

core/src/test/java/com/scalar/db/common/checker/OperationCheckerTest.java

@@ -4,9 +4,8 @@
 import static org.assertj.core.api.Assertions.assertThatCode;
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.assertj.core.api.Assertions.catchException;
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.mockito.Mockito.any;
-import static org.mockito.Mockito.spy;
-import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
 import com.scalar.db.api.ConditionBuilder;
@@ -19,7 +18,6 @@
 import com.scalar.db.api.Put;
 import com.scalar.db.api.Scan;
 import com.scalar.db.api.Scan.Ordering;
-import com.scalar.db.api.ScanAll;
 import com.scalar.db.api.StorageInfo;
 import com.scalar.db.api.TableMetadata;
 import com.scalar.db.api.Update;
@@ -442,15 +440,10 @@ public void whenCheckingScanAllOperationWithCrossPartitionScanEnabledWithOrderin
     when(databaseConfig.isCrossPartitionScanEnabled()).thenReturn(true);
     when(databaseConfig.isCrossPartitionScanOrderingEnabled()).thenReturn(true);
 
-    operationChecker =
-        spy(new OperationChecker(databaseConfig, metadataManager, storageInfoProvider));
-
-    // Act
-    operationChecker.check(scan);
+    operationChecker = new OperationChecker(databaseConfig, metadataManager, storageInfoProvider);
 
-    // Assert
-    verify(operationChecker)
-        .throwIfCrossPartitionScanOrderingOnBlobColumnNotSupported((ScanAll) scan, metadata);
+    // Act Assert
+    assertDoesNotThrow(() -> operationChecker.check(scan));
   }
 
   @Test

core/src/test/java/com/scalar/db/storage/jdbc/JdbcOperationCheckerTest.java

@@ -32,12 +32,10 @@ public void setUp() throws Exception {
   }
 
   @Test
-  public void
-      throwIfCrossPartitionScanOrderingOnBlobColumnNotSupported_ShouldDelegateToRdbEngine() {
+  public void checkOrderingsForScanAll_ShouldInvokeFurtherCheckOnRdbEngine() {
     // Arrange
     // Act
-    operationChecker.throwIfCrossPartitionScanOrderingOnBlobColumnNotSupported(
-        scanAll, tableMetadata);
+    operationChecker.checkOrderingsForScanAll(scanAll, tableMetadata);
 
     // Assert
     verify(rdbEngine)