hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ke Jia <ke.a....@intel.com>
Subject Re: Review Request 52111: HIVE-14388:Add number of rows inserted message after insert command in Beeline
Date Thu, 22 Sep 2016 01:53:57 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52111/
-----------------------------------------------------------

(Updated Sept. 22, 2016, 1:53 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 (updated)
-----

  jdbc/src/java/org/apache/hive/jdbc/HiveStatement.java a242501 
  ql/src/java/org/apache/hadoop/hive/ql/Driver.java 42d398d 
  ql/src/java/org/apache/hadoop/hive/ql/MapRedStats.java 4b60514 
  ql/src/java/org/apache/hadoop/hive/ql/exec/mr/HadoopJobExecHelper.java bb6ed84 
  ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 408c92e 
  service-rpc/if/TCLIService.thrift a4fa7b0 
  service/src/java/org/apache/hive/service/cli/CLIService.java 662e55c 
  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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message