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



ql/src/test/org/apache/hadoop/hive/ql/hooks/VerifyIsLocalModeHook.java
<https://reviews.apache.org/r/1132/#comment2210>

    We need a header for licensing.


- Siying


On 2011-07-15 02:16:34, Kevin Wilfong wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/1132/
> -----------------------------------------------------------
> 
> (Updated 2011-07-15 02:16:34)
> 
> 
> Review request for hive and Siying Dong.
> 
> 
> Summary
> -------
> 
> A query should run in local mode when block sampling is used and the sample 
> is small enough.  The size of the sample is currently being estimated, as it 
> is done to estimate the number of reducers.
> 
> 
> This addresses bug HIVE-2282.
>     https://issues.apache.org/jira/browse/HIVE-2282
> 
> 
> Diffs
> -----
> 
>   ql/src/test/queries/clientpositive/sample_islocalmode_hook.q PRE-CREATION 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/MapRedTask.java 53769a0 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java cd3de76 
>   ql/src/test/org/apache/hadoop/hive/ql/hooks/VerifyIsLocalModeHook.java 
> PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/1132/diff
> 
> 
> Testing
> -------
> 
> TestCliDriver TestNegativeCliDriver, manually tested
> 
> 
> Thanks,
> 
> Kevin
> 
>

Reply via email to