This is an automated email from the ASF dual-hosted git repository.

michaelsmith pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git

commit 5f58545eff6e9aa1c9db8a89d89309713b38e080
Author: Surya Hebbar <[email protected]>
AuthorDate: Mon Feb 24 03:51:32 2025 +0530

    IMPALA-13787: Compensate for inaccuracies of EC2 instances in 
runtime-profile-test
    
    The resolution of the 'MonotonicStopwatch' is less on some machine
    configurations. On some EC2 instances, this can be 1 to 10 milliseconds.
    
    If events are recorded into the event sequence between this 'jiffy',
    the relation between the timestamps in the runtime-profile-test do not
    hold, as events will be recorded to occur at the same time.
    
    To properly simulate the events, either the test can be adapted to
    compensate for inaccuracies or provide higher sleep time between events.
    
    As this does not pose problems to the consistency of the recorded within
    the profile JSON, only the sleep timer has been increased for running
    tests on such machines.
    
    Change-Id: I8150b7e38e7e5e5294e2dc61c16c2b102032cca2
    Reviewed-on: http://gerrit.cloudera.org:8080/22526
    Reviewed-by: Impala Public Jenkins <[email protected]>
    Tested-by: Impala Public Jenkins <[email protected]>
---
 be/src/util/runtime-profile-test.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/be/src/util/runtime-profile-test.cc 
b/be/src/util/runtime-profile-test.cc
index 4839ce44d..46b91fe49 100644
--- a/be/src/util/runtime-profile-test.cc
+++ b/be/src/util/runtime-profile-test.cc
@@ -2002,7 +2002,7 @@ class AggregatedEventSequenceToJsonTest : public 
::testing::Test {
       seqs[i]->Start();
     }
 
-    int64_t dummy_event_duration = 5;
+    int64_t dummy_event_duration = 50;
 
     // Simulate parallel execution of events across instances
     if (events_completeness >= 1.0f) {

Reply via email to