----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11036/#review20415 -----------------------------------------------------------
metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java <https://reviews.apache.org/r/11036/#comment42056> Hive depends on commons-lang v2.4 which supports join(Collection, String) and join(Collection, char). I don't think there's a need to call toArray(). metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java <https://reviews.apache.org/r/11036/#comment42055> We can avoid the oah.util.StringUtils dependency entirely by doing this: LOG.error("Converting exception to MetaException", e); - Carl Steinbach On May 10, 2013, 7:07 a.m., Thejas Nair wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11036/ > ----------------------------------------------------------- > > (Updated May 10, 2013, 7:07 a.m.) > > > Review request for hive. > > > Description > ------- > > ant package -Dhadoop.mr.rev=20 > leads to - > [javac] > /Users/thejas/hive_thejas_git/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java:382: > cannot find symbol > [javac] symbol : method > join(java.lang.String,java.util.List<java.lang.String>) > [javac] location: class org.apache.hadoop.util.StringUtils > [javac] StringUtils.join(",", incompatibleCols) > > import has hadoop StringUtils , but it does not have this function in hadoop > 0.20 libraries. > > > This addresses bug HIVE-4535. > https://issues.apache.org/jira/browse/HIVE-4535 > > > Diffs > ----- > > metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java > f9da781 > > Diff: https://reviews.apache.org/r/11036/diff/ > > > Testing > ------- > > > Thanks, > > Thejas Nair > >
