song bruce zhang has posted comments on this change.

Change subject: KUDU-1542: in some cases apply will hang.
......................................................................


Patch Set 6:

> > Lemme put it more clear:
 > > 1 the "looping forever" happened when i am using
 > "AUTO_FLUSH_BACKGROUND"
 > > flush_mode.
 > > 2 the timeout exception in log is like 
 > > "com.stumbleupon.async.TimeoutException:
 > > Timed out after 10000ms when joining Defered@"
 > > 3 this "looping forever" can hang quite a while ,maybe  several
 > > hours . Reconnct kudu-client recover it.
 > 
 > That's very interesting; I wouldn't expect a background flush to
 > hang for hours unless there's something broken in the client.
 > 
 > In KUDU-1542, JD asked if you've had a chance to retest with his
 > newer patches. Did you have a chance to do that? I suspect that
 > your fix works around a hang that was fixed.

 > > Lemme put it more clear:
 > > 1 the "looping forever" happened when i am using
 > "AUTO_FLUSH_BACKGROUND"
 > > flush_mode.
 > > 2 the timeout exception in log is like 
 > > "com.stumbleupon.async.TimeoutException:
 > > Timed out after 10000ms when joining Defered@"
 > > 3 this "looping forever" can hang quite a while ,maybe  several
 > > hours . Reconnct kudu-client recover it.
 > 
 > That's very interesting; I wouldn't expect a background flush to
 > hang for hours unless there's something broken in the client.
 > 
 > In KUDU-1542, JD asked if you've had a chance to retest with his
 > newer patches. Did you have a chance to do that? I suspect that
 > your fix works around a hang that was fixed.

Haven't met this kind of hang recently. And didn't test that JD's patch either. 
From code i think https://gerrit.cloudera.org/#/c/3724/ may fix the root cause 
of hang: incorrect behavior of readTimeoutHandler . Will try that patch .

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iacd33cdc5316e294e613d1b2273ef12e6b1cf687
Gerrit-PatchSet: 6
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: song bruce zhang <zsyuyizh...@gmail.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: song bruce zhang <zsyuyizh...@gmail.com>
Gerrit-HasComments: No

Reply via email to