----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45551/#review126361 -----------------------------------------------------------
Ship it! Ship It! - Bruce Schuchardt On March 31, 2016, 5:25 p.m., Hitesh Khamesra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45551/ > ----------------------------------------------------------- > > (Updated March 31, 2016, 5:25 p.m.) > > > Review request for geode, Bruce Schuchardt, Darrel Schneider, Jianxia Chen, > and Udo Kohlmeyer. > > > Repository: geode > > > Description > ------- > > GEODE-1031 Now pool closes queueMge before connectionMgr > > At cache close, there is possibility that queueMgr is trying to > recover queue but cache/pool trying to close cache with keep-alive > false flag. During this time there is possibility that client(queueMgr) > may be unable to close socket properly as it doesn't know about > keep-alive flag. Thats why closing connectionMge after queueMgr > so that it should be able to send keep-alive flage properly > to server. > > > Diffs > ----- > > > geode-core/src/main/java/com/gemstone/gemfire/cache/client/internal/PoolImpl.java > 84debc8 > > Diff: https://reviews.apache.org/r/45551/diff/ > > > Testing > ------- > > > Thanks, > > Hitesh Khamesra > >
