You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@kudu.apache.org by "Zoltan Martonka (Code Review)" <ge...@cloudera.org> on 2025/12/04 12:56:02 UTC

[kudu-CR] KUDU-1973 Enable some unit tests for heartbeat batching.

Zoltan Martonka has posted comments on this change. ( http://gerrit.cloudera.org:8080/23529 )

Change subject: KUDU-1973 Enable some unit tests for heartbeat batching.
......................................................................


Patch Set 6:

> (2 comments)
 > 
 > Overall looks good. Just a couple of nits.

TSAN takes:
 real	20m17.783s
user	60m25.114s
sys	5m43.909s

Debug:
real    12m46.252s
user    20m11.526s
sys     2m28.377s

If you think adding this much for test is ok, then I can add make all tests run with both state,
but I found it a bit too much, that's why I picked 5.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifd442b36b46c038fbf836f48feaa21b1ab296d82
Gerrit-Change-Number: 23529
Gerrit-PatchSet: 6
Gerrit-Owner: Zoltan Martonka <zm...@cloudera.com>
Gerrit-Reviewer: Ashwani Raina <ar...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <gr...@gmail.com>
Gerrit-Reviewer: Zoltan Chovan <zc...@cloudera.com>
Gerrit-Reviewer: Zoltan Martonka <zm...@cloudera.com>
Gerrit-Comment-Date: Thu, 04 Dec 2025 12:56:02 +0000
Gerrit-HasComments: No