apply review feedback

Author: jasonstack

src/java/org/apache/cassandra/db/compaction/BackgroundCompactionRunner.java

@@ -463,7 +463,7 @@ public static void handleCompactionError(Throwable t, ColumnFamilyStore cfs)
             // we might have to rely on error message parsing...
             t = t instanceof FSError ? t : new FSWriteError(t);
             JVMStabilityInspector.inspectThrowable(t);
-            CompactionManager.instance.incrementAborted();
+            CompactionManager.instance.incrementFailed();
         }
         // No-Space-Left IO exception is thrown by JDK when disk has reached its capacity. The key difference between this
         // and the earlier case with `FSDiskFullWriteError` is that here we have definitively run out of disk space, and

test/unit/org/apache/cassandra/db/compaction/BackgroundCompactionRunnerTest.java

@@ -586,13 +586,13 @@ public void handleFSWriteError()
                 diskErrorEncountered.set(err);
             });
 
-            long before = CompactionManager.instance.getMetrics().compactionsAborted.getCount();
+            long before = CompactionManager.instance.getMetrics().totalCompactionsFailed.getCount();
 
             FSWriteError fsWriteError = new FSWriteError(null, "file");
             BackgroundCompactionRunner.handleCompactionError(fsWriteError, cfs);
 
             assertThat(diskErrorEncountered.get()).isSameAs(fsWriteError);
-            assertThat(CompactionManager.instance.getMetrics().compactionsAborted.getCount()).isEqualTo(before + 1);
+            assertThat(CompactionManager.instance.getMetrics().totalCompactionsFailed.getCount()).isEqualTo(before + 1);
         }
         finally
         {