Sorry for being a bit vague. The python test client expects the error message to be whatever was passed as the argument, so prior to this patch, it was impossible to successfully run the python test client against the C++ test server.
Greg Stein wrote: > Hunh? I don't see how that log message corresponds to the actual change. > > ?? > > On Sat, Jan 31, 2009 at 22:39, <[email protected]> wrote: >> Author: dreiss >> Date: Sat Jan 31 21:39:11 2009 >> New Revision: 739629 >> >> URL: http://svn.apache.org/viewvc?rev=739629&view=rev >> Log: >> cpp: Make the test server compatible with the Python test client >> >> Modified: >> incubator/thrift/trunk/test/cpp/src/TestServer.cpp >> >> Modified: incubator/thrift/trunk/test/cpp/src/TestServer.cpp >> URL: >> http://svn.apache.org/viewvc/incubator/thrift/trunk/test/cpp/src/TestServer.cpp?rev=739629&r1=739628&r2=739629&view=diff >> ============================================================================== >> --- incubator/thrift/trunk/test/cpp/src/TestServer.cpp (original) >> +++ incubator/thrift/trunk/test/cpp/src/TestServer.cpp Sat Jan 31 21:39:11 >> 2009 >> @@ -229,7 +229,7 @@ >> if (arg.compare("Xception") == 0) { >> Xception e; >> e.errorCode = 1001; >> - e.message = "This is an Xception"; >> + e.message = arg; >> throw e; >> } else if (arg.compare("ApplicationException") == 0) { >> facebook::thrift::TException e; >> >> >>
