> On Oct. 3, 2014, 7:06 p.m., Vaibhav Gumashta wrote:
> > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java, line 1726
> > <https://reviews.apache.org/r/26321/diff/1/?file=713544#file713544line1726>
> >
> >     Can you add a description for this new param (for why this is required)

For Windows OS, we need to pass HIVE_HADOOP_CLASSPATH java parameter while 
starting hiveserver2 using "-hiveconf hive.hadoop.classpath=%HIVE_LIB%".  This 
is where it's defined.


> On Oct. 3, 2014, 7:06 p.m., Vaibhav Gumashta wrote:
> > ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapredLocalTask.java, line 241
> > <https://reviews.apache.org/r/26321/diff/1/?file=713545#file713545line241>
> >
> >     Can you add a comment here explaining the intent of using 
> > HIVE_HADOOP_CLASSPATH?

For Windows OS, we need to pass HIVE_HADOOP_CLASSPATH java parameter while 
starting hiveserver2 using "-hiveconf hive.hadoop.classpath=%HIVE_LIB%". This 
is how it's used.


- XIAOBING


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


On Oct. 3, 2014, 6:27 p.m., XIAOBING ZHOU wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26321/
> -----------------------------------------------------------
> 
> (Updated Oct. 3, 2014, 6:27 p.m.)
> 
> 
> Review request for hive.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> On stopping the HS2 service from the services tab, it only kills the root 
> process and does not kill the child java process. As a result resources are 
> not freed and this throws an error on restarting from command line.
> 
> 
> Diffs
> -----
> 
>   bin/ext/hiveserver2.cmd a5f3bb5 
>   bin/hive.cmd c2e9853 
>   common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 3fe67b2 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapredLocalTask.java 79da5a0 
> 
> Diff: https://reviews.apache.org/r/26321/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> XIAOBING ZHOU
> 
>

Reply via email to