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



metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
<https://reviews.apache.org/r/34443/#comment135668>

    - 2 spaces btw "partitions" and "separately"
    - trailing space if isSeparateListTxn==false



metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
<https://reviews.apache.org/r/34443/#comment135669>

    maybe use "if (hasTxn)" instead


- Alexander Pivovarov


On May 19, 2015, 11:23 p.m., Sergey Shelukhin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34443/
> -----------------------------------------------------------
> 
> (Updated May 19, 2015, 11:23 p.m.)
> 
> 
> Review request for hive, Alexander Pivovarov, Sushanth Sowmyan, and Thejas 
> Nair.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> see jira
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/conf/HiveConf.java eff4d30 
>   metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java 
> e796048 
>   metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java 
> fd61333 
>   
> metastore/src/java/org/apache/hadoop/hive/metastore/TServerSocketKeepAlive.java
>  9ac18dc 
>   
> metastore/src/java/org/apache/hadoop/hive/metastore/TSetIpAddressProcessor.java
>  4a56bfa 
>   metastore/src/java/org/apache/hadoop/hive/metastore/TUGIBasedProcessor.java 
> ec8d608 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/DefaultFetchFormatter.java 
> 37852ef 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveInputFormat.java 20d0304 
> 
> Diff: https://reviews.apache.org/r/34443/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Sergey Shelukhin
> 
>

Reply via email to