Address PR review comments

Author: andresbeckruiz

server/src/main/java/org/apache/cassandra/sidecar/job/OperationalJob.java

@@ -239,10 +239,11 @@ public String name()
         String simpleName = this.getClass().getSimpleName();
         return simpleName.isEmpty() ? this.getClass().getName() : simpleName;
     }
-    
-    /** 
-     * @return the {@link OperationType} of this job
+
+    /**
+     * {@inheritDoc}
      */
+    @Override
     public abstract OperationType operationType();
 
     /**

server/src/main/java/org/apache/cassandra/sidecar/job/OperationalJobManager.java

@@ -121,7 +121,7 @@ public void trySubmitJob(OperationalJob job,
             OperationalJob tracked = jobTracker.computeIfAbsent(job.jobId(), jobId -> job);
             if (tracked == job)
             {
-                checkCoordination(job);
+                tryCoordination(job);
                 internalExecutorPool.executeBlocking(job::execute);
             }
         }
@@ -158,7 +158,7 @@ private void checkConflict(OperationalJob job) throws OperationalJobConflictExce
      * @param job instance of the job to coordinate
      * @throws OperationalJobConflictException when the coordinator cannot activate the operation
      */
-    private void checkCoordination(OperationalJob job) throws OperationalJobConflictException
+    private void tryCoordination(OperationalJob job) throws OperationalJobConflictException
     {
         if (job.requiresCoordination())
         {

…ob/StorageOperationalJobCoordinator.java …rageBackedOperationalJobCoordinator.javaserver/src/main/java/org/apache/cassandra/sidecar/job/StorageOperationalJobCoordinator.java renamed to server/src/main/java/org/apache/cassandra/sidecar/job/StorageBackedOperationalJobCoordinator.java server/src/main/java/org/apache/cassandra/sidecar/job/StorageOperationalJobCoordinator.java renamed to server/src/main/java/org/apache/cassandra/sidecar/job/StorageBackedOperationalJobCoordinator.java

@@ -33,12 +33,12 @@
  * for coordination of active operations.
  */
 @Singleton
-public class StorageOperationalJobCoordinator implements OperationalJobCoordinator
+public class StorageBackedOperationalJobCoordinator implements OperationalJobCoordinator
 {
     private final StorageProvider storageProvider;
 
     @Inject
-    public StorageOperationalJobCoordinator(StorageProvider storageProvider)
+    public StorageBackedOperationalJobCoordinator(StorageProvider storageProvider)
     {
         this.storageProvider = storageProvider;
     }

server/src/main/java/org/apache/cassandra/sidecar/modules/CassandraOperationsModule.java

@@ -64,7 +64,7 @@
 import org.apache.cassandra.sidecar.job.InMemoryOperationalJobTracker;
 import org.apache.cassandra.sidecar.job.OperationalJobCoordinator;
 import org.apache.cassandra.sidecar.job.OperationalJobTracker;
-import org.apache.cassandra.sidecar.job.StorageOperationalJobCoordinator;
+import org.apache.cassandra.sidecar.job.StorageBackedOperationalJobCoordinator;
 import org.apache.cassandra.sidecar.modules.multibindings.KeyClassMapKey;
 import org.apache.cassandra.sidecar.modules.multibindings.TableSchemaMapKeys;
 import org.apache.cassandra.sidecar.modules.multibindings.VertxRouteMapKeys;
@@ -85,7 +85,7 @@ public class CassandraOperationsModule extends AbstractModule
     protected void configure()
     {
         bind(OperationalJobTracker.class).to(InMemoryOperationalJobTracker.class);
-        bind(OperationalJobCoordinator.class).to(StorageOperationalJobCoordinator.class);
+        bind(OperationalJobCoordinator.class).to(StorageBackedOperationalJobCoordinator.class);
     }
 
     @ProvidesIntoMap