The change happened in a commercial release between grants. The ping case had a deprecated long read that was not compatible with the server Geode 1.0. After removing the read the code path was merged with the default as there was no diff.
On Mon, Feb 13, 2017 at 10:06 AM Michael William Dodge <mdo...@pivotal.io> wrote: > I'm not sure exactly what happened as my git fu isn't that great but could > it be as part of commit 4fa64db926f51d4b12d6e4040c703cc69a9832fe? In that > commit I see a block under case TcrMessage::PING: being removed so that > execution falls through to default: but I'm unsure that I've pieced the > output from git diff together properly so that may not be a change that > happened in TcrMessage::handleByteArrayResponse. > > Sarge > > > On 12 Feb, 2017, at 09:25, Avital Amity <avital.am...@amdocs.com> wrote: > > > > Hi, > > > > I'm trying to track the history of TcrMessage.cpp but I can find it only > in the new client release where it moved under cppcache/src > > In particular I'm searching for the change where in function > TcrMessage::handleByteArrayResponse > > Where the case of PING message was merged with the case of the default > message > > > > Thanks > > Avital > > This message and the information contained herein is proprietary and > confidential and subject to the Amdocs policy statement, > > > > you may review at http://www.amdocs.com/email_disclaimer.asp > >
