Huaisi Xu has posted comments on this change.

Change subject: IMPALA-3613: Do not update unregistered subscribers
......................................................................


Patch Set 2:

(1 comment)

testing is done by setting hearbeat interval to 10s, and subscriber's timeout 
to be 1s. after ~1 hour everything looks good to me on a 3 nodes cluster(local 
dev).

http://gerrit.cloudera.org:8080/#/c/3320/2/be/src/statestore/statestore.cc
File be/src/statestore/statestore.cc:

Line 714:       LOG(INFO) << "Next " << (is_heartbeat ? "heartbeat" : "update") 
<< " deadline for: "
I will rever this...


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic573d257b8171bfcab33a1f72de1399be7db10fd
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Huaisi Xu <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Huaisi Xu <[email protected]>
Gerrit-HasComments: Yes

Reply via email to