Github user michaelandrepearce commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1605#discussion_r146618435
  
    --- Diff: 
artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/util/SyncCalculation.java
 ---
    @@ -54,92 +172,94 @@ public static long syncTest(File datafolder,
                                    String fileName,
                                    int maxAIO,
                                    JournalType journalType) throws Exception {
    -      SequentialFileFactory factory = newFactory(datafolder, fsync, 
journalType, blockSize * blocks, maxAIO);
    -
    +      final Supplier<? extends SyncIOCompletion> ioCallbackFactory;
    --- End diff --
    
    essentially i would be expecting to see no diff in the code, except an if 
(verbose) statement with all the change only in that section.


---

Reply via email to