Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24976 )

Change subject: IMPALA-15463: Use Kudu live row count stats
......................................................................


Patch Set 5: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/24976/4/testdata/workloads/functional-query/queries/QueryTest/kudu_stats.test
File testdata/workloads/functional-query/queries/QueryTest/kudu_stats.test:

http://gerrit.cloudera.org:8080/#/c/24976/4/testdata/workloads/functional-query/queries/QueryTest/kudu_stats.test@24
PS4, Line 24: # Kudu takes time to commit the rows and update its live row 
count.
            : select sleep(1000);
> That was the point of the RETRY=10, it'll try 10 times until stats are upda
I see.  Yes, that makes sense -- having a retry should cover for TS-->master 
heartbeat timing irregularities.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I17d806fa9f849bb66fac5dfb500f83fc6d115b58
Gerrit-Change-Number: 24976
Gerrit-PatchSet: 5
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: David Rorke <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]>
Gerrit-Comment-Date: Thu, 01 Oct 2026 18:33:19 +0000
Gerrit-HasComments: Yes

Reply via email to