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



ivy/libraries.properties
<https://reviews.apache.org/r/12750/#comment47453>

    is connection pool removed intentionally? HIVE-4807 changed pooling enabled 
by default to solve some problem, I had trouble starting metastore with that 
setting without this jar. Could just be me but worth checking



metastore/src/model/package.jdo
<https://reviews.apache.org/r/12750/#comment47454>

    this (and other stuff above and below) will require schema upgrade script 
as far as I see


- Sergey Shelukhin


On July 18, 2013, 11:27 p.m., Xuefu Zhang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12750/
> -----------------------------------------------------------
> 
> (Updated July 18, 2013, 11:27 p.m.)
> 
> 
> Review request for hive and Ashutosh Chauhan.
> 
> 
> Bugs: HIVE-3632
>     https://issues.apache.org/jira/browse/HIVE-3632
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> 1. Upgrade DataNucleus to 3.2 stable version.
> 2. Fix a few issues that arose.
> 
> While the fix is motivated for JDK7 support, the patch is completely 
> applicable for running with JDK6.
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 11c3121 
>   conf/hive-default.xml.template 603b475 
>   ivy/libraries.properties 0e1b992 
>   metastore/build.xml 0e94611 
>   metastore/ivy.xml 356e19d 
>   metastore/src/model/package.jdo a84d2bf 
>   ql/ivy.xml bfb3116 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 9883659 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java cbda70b 
>   
> ql/src/test/results/clientpositive/alter_rename_partition_authorization.q.out 
> 4262b7c 
>   ql/src/test/results/clientpositive/authorization_2.q.out c934a2a 
>   ql/src/test/results/clientpositive/authorization_6.q.out b8483ca 
> 
> Diff: https://reviews.apache.org/r/12750/diff/
> 
> 
> Testing
> -------
> 
> 1. Ran all unit tests with 0.10.0 as the base, and all pased.
> 2. Ran all unit tests with trunk, and there was a test failure but seemingly 
> unrelated. Conducting further testing. (Second run was successful!)
> 3. Ran some basic end2end tests. Issues are fixed as they arise. The effort 
> will continue.
> 
> 
> Thanks,
> 
> Xuefu Zhang
> 
>

Reply via email to