hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gunther Hagleitner" <ghagleit...@hortonworks.com>
Subject Re: Review Request 34713: Invalidate basic stats for insert queries if autogather=false
Date Fri, 29 May 2015 20:24:41 GMT

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



ql/src/test/queries/clientpositive/insert_into1.q
<https://reviews.apache.org/r/34713/#comment137571>

    ws issue



ql/src/test/results/clientpositive/stats12.q.out
<https://reviews.apache.org/r/34713/#comment137578>

    that's not correct, is it? why are the column stats true now? and there's more than 0
rows in the partition.



ql/src/test/results/clientpositive/stats5.q.out
<https://reviews.apache.org/r/34713/#comment137590>

    analyze_src is copy of src.. so num rows 1 and complete stats is incorrect no?


- Gunther Hagleitner


On May 28, 2015, 7:25 a.m., Ashutosh Chauhan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34713/
> -----------------------------------------------------------
> 
> (Updated May 28, 2015, 7:25 a.m.)
> 
> 
> Review request for hive and Gopal V.
> 
> 
> Bugs: HIVE-10807
>     https://issues.apache.org/jira/browse/HIVE-10807
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> Invalidate basic stats for insert queries if autogather=false
> 
> 
> Diffs
> -----
> 
>   ql/src/java/org/apache/hadoop/hive/ql/QueryProperties.java e8f7fba 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMRFileSink1.java e5b9c2b 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java acd9bf5 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/QBParseInfo.java 14a7e9c 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 7f355e5 
>   ql/src/test/queries/clientpositive/insert_into1.q f19506a 
>   ql/src/test/results/clientpositive/insert_into1.q.out 9e5f3bb 
>   ql/src/test/results/clientpositive/skewjoin_union_remove_1.q.out 1f21877 
>   ql/src/test/results/clientpositive/spark/insert_into1.q.out e7f7d8f 
>   ql/src/test/results/clientpositive/stats12.q.out d5ca59b 
>   ql/src/test/results/clientpositive/stats13.q.out e34f0b7 
>   ql/src/test/results/clientpositive/stats2.q.out 5e305d3 
>   ql/src/test/results/clientpositive/stats5.q.out 484e834 
>   ql/src/test/results/clientpositive/stats6.q.out b4435f2 
>   ql/src/test/results/clientpositive/stats9.q.out 8563f3b 
>   ql/src/test/results/clientpositive/stats_noscan_1.q.out 80c3092 
>   ql/src/test/results/clientpositive/stats_partscan_1_23.q.out 5488935 
>   ql/src/test/results/clientpositive/tez/insert_into1.q.out 359470b 
>   ql/src/test/results/clientpositive/tez/stats_noscan_1.q.out aa49526 
>   ql/src/test/results/clientpositive/union_remove_1.q.out 5fa8caf 
>   ql/src/test/results/clientpositive/union_remove_10.q.out 228ff9e 
>   ql/src/test/results/clientpositive/union_remove_11.q.out bf38f0a 
>   ql/src/test/results/clientpositive/union_remove_12.q.out 3ac1d44 
>   ql/src/test/results/clientpositive/union_remove_13.q.out b12d7cd 
>   ql/src/test/results/clientpositive/union_remove_14.q.out 94f2ff5 
>   ql/src/test/results/clientpositive/union_remove_15.q.out 67bdb1e 
>   ql/src/test/results/clientpositive/union_remove_16.q.out 2444337 
>   ql/src/test/results/clientpositive/union_remove_17.q.out a1b9474 
>   ql/src/test/results/clientpositive/union_remove_18.q.out 26e52e0 
>   ql/src/test/results/clientpositive/union_remove_19.q.out d8314f8 
>   ql/src/test/results/clientpositive/union_remove_2.q.out d3522f2 
>   ql/src/test/results/clientpositive/union_remove_20.q.out 4503726 
>   ql/src/test/results/clientpositive/union_remove_21.q.out 98e0166 
>   ql/src/test/results/clientpositive/union_remove_22.q.out 4f086f3 
>   ql/src/test/results/clientpositive/union_remove_23.q.out d0e21ba 
>   ql/src/test/results/clientpositive/union_remove_24.q.out f2f6bba 
>   ql/src/test/results/clientpositive/union_remove_25.q.out 3858bdc 
>   ql/src/test/results/clientpositive/union_remove_3.q.out 40e554a 
>   ql/src/test/results/clientpositive/union_remove_4.q.out 10a8e89 
>   ql/src/test/results/clientpositive/union_remove_5.q.out b59d6da 
>   ql/src/test/results/clientpositive/union_remove_6.q.out 67ba74c 
>   ql/src/test/results/clientpositive/union_remove_6_subq.q.out 0d1a436 
>   ql/src/test/results/clientpositive/union_remove_7.q.out 76c678f 
>   ql/src/test/results/clientpositive/union_remove_8.q.out fe67448 
>   ql/src/test/results/clientpositive/union_remove_9.q.out 684091b 
> 
> Diff: https://reviews.apache.org/r/34713/diff/
> 
> 
> Testing
> -------
> 
> Existing tests + added new tests
> 
> 
> Thanks,
> 
> Ashutosh Chauhan
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message