----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25976/#review54862 -----------------------------------------------------------
contrib/native/client/src/clientlib/drillClientImpl.cpp <https://reviews.apache.org/r/25976/#comment95179> It seems that isOK is always true in the patch. Did we miss somthing? - Xiao Meng On Sept. 24, 2014, 1:24 a.m., Parth Chandra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25976/ > ----------------------------------------------------------- > > (Updated Sept. 24, 2014, 1:24 a.m.) > > > Review request for drill. > > > Repository: drill-git > > > Description > ------- > > C++ client library was sending an invalid Cancel response instead of a Cancel > request to the server > > > Diffs > ----- > > contrib/native/client/example/querySubmitter.cpp 17bec3c > contrib/native/client/src/clientlib/drillClientImpl.hpp 5ac158f > contrib/native/client/src/clientlib/drillClientImpl.cpp f1a165d > > Diff: https://reviews.apache.org/r/25976/diff/ > > > Testing > ------- > > Updated querySubmitter test program to send a cancel and tested both sync and > async apis > > > Thanks, > > Parth Chandra > >
