----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49264/#review139908 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java (line 1774) <https://reviews.apache.org/r/49264/#comment205247> It is possible that p doesn't exist with above if condition. I don't think we should add p to result in this case. - Yibing Shi On June 29, 2016, 3:19 a.m., Aihua Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/49264/ > ----------------------------------------------------------- > > (Updated June 29, 2016, 3:19 a.m.) > > > Review request for hive. > > > Repository: hive-git > > > Description > ------- > > HIVE-14037: java.lang.ClassNotFoundException for the jar in > hive.reloadable.aux.jars.path in mapreduce > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 1d1306f > common/src/java/org/apache/hive/common/util/HiveStringUtils.java bba14e2 > ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 528d663 > ql/src/java/org/apache/hadoop/hive/ql/exec/mr/ExecDriver.java 8a6499b > ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapRedTask.java a42c2e9 > ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 96c826b > ql/src/test/org/apache/hadoop/hive/ql/exec/TestUtilities.java bb6a4e1 > ql/src/test/queries/clientpositive/reloadJar.q PRE-CREATION > ql/src/test/results/clientpositive/reloadJar.q.out PRE-CREATION > > Diff: https://reviews.apache.org/r/49264/diff/ > > > Testing > ------- > > > Thanks, > > Aihua Xu > >
