[ 
https://issues.apache.org/jira/browse/ARTEMIS-2466?focusedWorklogId=332815&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-332815
 ]

ASF GitHub Bot logged work on ARTEMIS-2466:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 23/Oct/19 19:16
            Start Date: 23/Oct/19 19:16
    Worklog Time Spent: 10m 
      Work Description: brusdev commented on pull request #2868: ARTEMIS-2466 
PageSyncTimer::timeSync isn't configurable using ASYNCIO
URL: https://github.com/apache/activemq-artemis/pull/2868#discussion_r338231386
 
 

 ##########
 File path: 
artemis-cli/src/main/java/org/apache/activemq/artemis/cli/commands/Create.java
 ##########
 @@ -1002,43 +1003,51 @@ private void applyAddressesAndQueues(HashMap<String, 
String> filters) {
    private void performAutoTune(HashMap<String, String> filters, JournalType 
journalType, File dataFolder) {
       if (noAutoTune) {
          filters.put("${journal-buffer.settings}", "");
+         filters.put("${page-sync.settings}", "");
       } else {
          try {
             int writes = 250;
             System.out.println("");
             System.out.println("Auto tuning journal ...");
 
+            long time = SyncCalculation.syncTest(dataFolder, 4096, writes, 5, 
verbose, !noJournalSync, false, "journal-test.tmp", 
ActiveMQDefaultConfiguration.getDefaultJournalMaxIoAio(), journalType);
+            long nanoseconds = SyncCalculation.toNanos(time, writes, verbose);
+            double writesPerMillisecond = (double) writes / (double) time;
+            String writesPerMillisecondStr = new 
DecimalFormat("###.##").format(writesPerMillisecond);
+
+            System.out.println("done! Your system can make " + 
writesPerMillisecondStr + " writes per millisecond");
+
             if (mapped && noJournalSync) {
                HashMap<String, String> syncFilter = new HashMap<>();
                syncFilter.put("${nanoseconds}", "0");
                syncFilter.put("${writesPerMillisecond}", "0");
                syncFilter.put("${maxaio}", journalType == JournalType.ASYNCIO 
? "" + ActiveMQDefaultConfiguration.getDefaultJournalMaxIoAio() : "1");
 
-               System.out.println("...Since you disabled sync and are using 
MAPPED journal, we are diabling buffer times");
+               System.out.println("Since you disabled sync and are using 
MAPPED journal, we are disabling buffer times");
 
                filters.put("${journal-buffer.settings}", 
readTextFile(ETC_JOURNAL_BUFFER_SETTINGS, syncFilter));
-
             } else {
-               long time = SyncCalculation.syncTest(dataFolder, 4096, writes, 
5, verbose, !noJournalSync, false, "journal-test.tmp", 
ActiveMQDefaultConfiguration.getDefaultJournalMaxIoAio(), journalType);
-               long nanoseconds = SyncCalculation.toNanos(time, writes, 
verbose);
-               double writesPerMillisecond = (double) writes / (double) time;
-
-               String writesPerMillisecondStr = new 
DecimalFormat("###.##").format(writesPerMillisecond);
-
                HashMap<String, String> syncFilter = new HashMap<>();
                syncFilter.put("${nanoseconds}", Long.toString(nanoseconds));
                syncFilter.put("${writesPerMillisecond}", 
writesPerMillisecondStr);
                syncFilter.put("${maxaio}", journalType == JournalType.ASYNCIO 
? "" + ActiveMQDefaultConfiguration.getDefaultJournalMaxIoAio() : "1");
 
-               System.out.println("done! Your system can make " + 
writesPerMillisecondStr +
-                                     " writes per millisecond, your 
journal-buffer-timeout will be " + nanoseconds);
+               System.out.println("Your journal-buffer-timeout will be " + 
nanoseconds);
 
 Review comment:
   I moved that println at line 1018, because the test is always executed.
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 332815)
    Time Spent: 1h 20m  (was: 1h 10m)

> PageSyncTimer::timeSync isn't configurable using ASYNCIO journal
> ----------------------------------------------------------------
>
>                 Key: ARTEMIS-2466
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-2466
>             Project: ActiveMQ Artemis
>          Issue Type: Bug
>          Components: Broker
>    Affects Versions: 2.10.0
>            Reporter: Francesco Nigro
>            Priority: Major
>          Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> If the broker is configured to use ASYNCIO journal, PageSyncTimer::timeSync 
> is being defaulted with the default journal buffer timeout of NIO ie 3333333 
> or 300 writes/sec



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to