hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "cheng xu" <>
Subject Re: Review Request 35107: HIVE-6791 Support variable substition for Beeline shell command
Date Fri, 26 Jun 2015 08:02:50 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated June 26, 2015, 4:02 p.m.)

Review request for hive, chinna and Xuefu Zhang.


1) rebase code
2) add some java doc
3) do some code clean work

Bugs: HIVE-6791

Repository: hive-git


1) move the beeline-cli convertor to the place where cli is executed(class **Commands**)
2) support substitution for source command
3) add some unit test for substitution
4) add one way to get the configuration from HS2

Diffs (updated)

  beeline/src/java/org/apache/hive/beeline/ 66fe322 
  beeline/src/java/org/apache/hive/beeline/ aaf6aec 
  beeline/src/test/org/apache/hive/beeline/cli/ 6cbb030 
  cli/src/java/org/apache/hadoop/hive/cli/ d62fd5c 
  common/src/java/org/apache/hadoop/hive/conf/ PRE-CREATION 
  common/src/java/org/apache/hadoop/hive/conf/ PRE-CREATION 
  common/src/test/org/apache/hadoop/hive/conf/ PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/ 669e6be 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ a5f0a7f 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ e8b1d96 
  ql/src/java/org/apache/hadoop/hive/ql/processors/ 0558c53 
  ql/src/java/org/apache/hadoop/hive/ql/processors/ 25ce168 
  ql/src/java/org/apache/hadoop/hive/ql/processors/ 9052c82 
  ql/src/java/org/apache/hadoop/hive/ql/processors/ cc0414d 
  ql/src/java/org/apache/hadoop/hive/ql/processors/ d271d6d 
  service/src/java/org/apache/hive/service/cli/operation/ cc9df76 



Unit test passed


cheng xu

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