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

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

                Author: ASF GitHub Bot
            Created on: 06/Apr/22 15:42
            Start Date: 06/Apr/22 15:42
    Worklog Time Spent: 10m 
      Work Description: clebertsuconic commented on code in PR #4017:
URL: https://github.com/apache/activemq-artemis/pull/4017#discussion_r844102697


##########
artemis-server/src/main/resources/schema/artemis-configuration.xsd:
##########
@@ -2928,6 +2928,13 @@
                </xsd:documentation>
             </xsd:annotation>
          </xsd:element>
+         <xsd:element name="data-source-time-margin" type="xsd:string" 
minOccurs="0" maxOccurs="1">
+            <xsd:annotation>
+               <xsd:documentation>
+                  The maximal time offset between the broker and the database 
in milliseconds.

Review Comment:
   it would be nice to have more information here.. at least say this is for 
locking.





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

    Worklog Id:     (was: 753470)
    Time Spent: 0.5h  (was: 20m)

> Use configurable time margin for checking offset between database server and 
> broker
> -----------------------------------------------------------------------------------
>
>                 Key: ARTEMIS-3763
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-3763
>             Project: ActiveMQ Artemis
>          Issue Type: Improvement
>          Components: Configuration
>    Affects Versions: 2.21.0
>            Reporter: Marlon Müller
>            Priority: Minor
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> When using JDBC persistence artemis checks for time differences between the 
> broker and the database. Currently this check requests the time of the 
> database and compares it to the system time before and after sending this 
> query by removing the milliseconds and verifying that the database time is 
> between the start and the end time. 
> This leads to somehow random warnings in the log depending on the current 
> milliseconds when this check is performed. 
> For example if the time on your artemis server is 00:00:01.000 before the 
> check and the time on the database is 00:00:00.999 it will log a warning 
> although there is only 1 millisecond difference. On the other hand if your 
> artemis server time is 00:00:01.000 after sending the query and the database 
> time is 00:00:01.999 there will be no warning although the time is 999 
> milliseconds after the end time.
> Therefore it would be better to use a consistent margin for the database time 
> during this check. To avoid problems in setups where the system times are not 
> well synchronized, this time margin should be configurable to allow the user 
> to fine tune it.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to