Tamas Mate has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17928 )

Change subject: IMPALA-10955: test_time_travel is flaky
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/17928/1/tests/query_test/test_iceberg.py
File tests/query_test/test_iceberg.py:

http://gerrit.cloudera.org:8080/#/c/17928/1/tests/query_test/test_iceberg.py@178
PS1, Line 178:       expect_results_t(cast_ts(ts_4) + " - interval 5 seconds", 
[])
> We use execute_query_ts which works the following way:
Got it, thank you for the explanation.

This might be an overkill, but do you think it would make sense calculating the 
time in L178, something like `now() - ts_3 + 1s`?



--
To view, visit http://gerrit.cloudera.org:8080/17928
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia270781dfdc4101b0d961db5aa20fc1678cdaaa8
Gerrit-Change-Number: 17928
Gerrit-PatchSet: 1
Gerrit-Owner: Zoltan Borok-Nagy <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tamas Mate <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Mon, 18 Oct 2021 13:43:14 +0000
Gerrit-HasComments: Yes

Reply via email to