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 <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: song bruce zhang <[email protected]> Gerrit-HasComments: No
