> On Sept. 21, 2016, 9:33 a.m., Marta Kuczora wrote: > > Thanks a lot for the patch. I had just some minor hints.
Thanks for you comment. I have fixed it. - Ke ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52111/#review149801 ----------------------------------------------------------- On Oct. 20, 2016, 3:02 a.m., Ke Jia wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52111/ > ----------------------------------------------------------- > > (Updated Oct. 20, 2016, 3:02 a.m.) > > > Review request for hive and cheng xu. > > > Repository: hive-git > > > Description > ------- > > Currently, when you run insert command on beeline, it returns a message > saying "No rows affected .." > A better and more intuitive msg would be "xxx rows inserted (26.068 seconds)" > > > Diffs > ----- > > jdbc/src/java/org/apache/hive/jdbc/HiveStatement.java a242501 > ql/src/java/org/apache/hadoop/hive/ql/Driver.java 9e5fd37 > ql/src/java/org/apache/hadoop/hive/ql/MapRedStats.java 4b60514 > ql/src/java/org/apache/hadoop/hive/ql/exec/mr/HadoopJobExecHelper.java > 41887d7 > ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 71aea3a > service-rpc/if/TCLIService.thrift a4fa7b0 > service/src/java/org/apache/hive/service/cli/CLIService.java 6d07d0c > service/src/java/org/apache/hive/service/cli/OperationStatus.java b0a26e3 > service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java > 2938338 > > Diff: https://reviews.apache.org/r/52111/diff/ > > > Testing > ------- > > > Thanks, > > Ke Jia > >
