You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Ashutosh Chauhan <ha...@apache.org> on 2015/05/27 17:40:26 UTC

Review Request 34713: Invalidate basic stats for insert queries if autogather=false

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

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/exec/StatsTask.java 2a8167a 
  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/clientnegative/stats_partialscan_autogether.q.out 321ebe5 
  ql/src/test/results/clientpositive/auto_join_filters.q.out a6720d9 
  ql/src/test/results/clientpositive/auto_join_nulls.q.out 4416f3e 
  ql/src/test/results/clientpositive/auto_sortmerge_join_1.q.out 5114038 
  ql/src/test/results/clientpositive/auto_sortmerge_join_11.q.out e6e7ef3 
  ql/src/test/results/clientpositive/auto_sortmerge_join_12.q.out b2e782f 
  ql/src/test/results/clientpositive/auto_sortmerge_join_2.q.out 210f1ab 
  ql/src/test/results/clientpositive/auto_sortmerge_join_3.q.out a307b13 
  ql/src/test/results/clientpositive/auto_sortmerge_join_4.q.out f4ceee7 
  ql/src/test/results/clientpositive/auto_sortmerge_join_5.q.out 3c2951a 
  ql/src/test/results/clientpositive/auto_sortmerge_join_7.q.out e1f3888 
  ql/src/test/results/clientpositive/auto_sortmerge_join_8.q.out 38ecdbe 
  ql/src/test/results/clientpositive/bucket_map_join_1.q.out 42e6a3f 
  ql/src/test/results/clientpositive/bucket_map_join_2.q.out af73309 
  ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out 870ecdd 
  ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out 33f5c46 
  ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out 067d1ff 
  ql/src/test/results/clientpositive/bucketcontext_1.q.out 77bfcf9 
  ql/src/test/results/clientpositive/bucketcontext_2.q.out a9db13d 
  ql/src/test/results/clientpositive/bucketcontext_3.q.out 9ba3e0c 
  ql/src/test/results/clientpositive/bucketcontext_4.q.out a2b37a8 
  ql/src/test/results/clientpositive/bucketcontext_5.q.out 3ee1f0e 
  ql/src/test/results/clientpositive/bucketcontext_6.q.out d2304fa 
  ql/src/test/results/clientpositive/bucketcontext_7.q.out 1a105ed 
  ql/src/test/results/clientpositive/bucketcontext_8.q.out 138e415 
  ql/src/test/results/clientpositive/bucketmapjoin1.q.out 471ff73 
  ql/src/test/results/clientpositive/bucketmapjoin10.q.out b0e849d 
  ql/src/test/results/clientpositive/bucketmapjoin11.q.out 4263cab 
  ql/src/test/results/clientpositive/bucketmapjoin12.q.out bcd7394 
  ql/src/test/results/clientpositive/bucketmapjoin2.q.out a8d9e9d 
  ql/src/test/results/clientpositive/bucketmapjoin3.q.out c759f05 
  ql/src/test/results/clientpositive/bucketmapjoin4.q.out f61500c 
  ql/src/test/results/clientpositive/bucketmapjoin5.q.out 0cb2825 
  ql/src/test/results/clientpositive/bucketmapjoin7.q.out 667a9db 
  ql/src/test/results/clientpositive/bucketmapjoin8.q.out 252b377 
  ql/src/test/results/clientpositive/bucketmapjoin9.q.out 5e28dc3 
  ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out 6ae127d 
  ql/src/test/results/clientpositive/bucketmapjoin_negative2.q.out 4c9f54a 
  ql/src/test/results/clientpositive/bucketmapjoin_negative3.q.out 9a0bfc4 
  ql/src/test/results/clientpositive/columnstats_partlvl.q.out e0c4cfe 
  ql/src/test/results/clientpositive/columnstats_tbllvl.q.out 19283bb 
  ql/src/test/results/clientpositive/display_colstats_tbllvl.q.out 7c91248 
  ql/src/test/results/clientpositive/encrypted/encryption_insert_partition_dynamic.q.out 939e206 
  ql/src/test/results/clientpositive/encrypted/encryption_insert_partition_static.q.out fd7932e 
  ql/src/test/results/clientpositive/encrypted/encryption_join_unencrypted_tbl.q.out 9b6f750 
  ql/src/test/results/clientpositive/groupby_sort_6.q.out 0169430 
  ql/src/test/results/clientpositive/insert_into1.q.out 9e5f3bb 
  ql/src/test/results/clientpositive/join_filters.q.out 4f112bd 
  ql/src/test/results/clientpositive/join_nulls.q.out 46e0170 
  ql/src/test/results/clientpositive/list_bucket_dml_8.q.java1.7.out a9522e0 
  ql/src/test/results/clientpositive/parquet_serde.q.out e753180 
  ql/src/test/results/clientpositive/ql_rewrite_gbtoidx_cbo_2.q.out 3ee2e0f 
  ql/src/test/results/clientpositive/skewjoin_union_remove_1.q.out 1f21877 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_1.q.out 09d2692 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_12.q.out a70b161 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_2.q.out 2ea0a65 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_3.q.out 6281929 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_4.q.out 31e9d86 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_5.q.out 3eceb0b 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_7.q.out ddbca05 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_8.q.out 88d4dcb 
  ql/src/test/results/clientpositive/spark/bucket_map_join_1.q.out 7570ebe 
  ql/src/test/results/clientpositive/spark/bucket_map_join_2.q.out 80b44e9 
  ql/src/test/results/clientpositive/spark/bucket_map_join_spark1.q.out 6230bef 
  ql/src/test/results/clientpositive/spark/bucket_map_join_spark2.q.out 1a33625 
  ql/src/test/results/clientpositive/spark/bucket_map_join_spark3.q.out fed923c 
  ql/src/test/results/clientpositive/spark/bucketmapjoin1.q.out d4a9c98 
  ql/src/test/results/clientpositive/spark/bucketmapjoin10.q.out 678ad54 
  ql/src/test/results/clientpositive/spark/bucketmapjoin11.q.out 95606f0 
  ql/src/test/results/clientpositive/spark/bucketmapjoin12.q.out d6c25e4 
  ql/src/test/results/clientpositive/spark/bucketmapjoin2.q.out d82480e 
  ql/src/test/results/clientpositive/spark/bucketmapjoin3.q.out 39552c1 
  ql/src/test/results/clientpositive/spark/bucketmapjoin4.q.out ad2762d 
  ql/src/test/results/clientpositive/spark/bucketmapjoin5.q.out f7c3d4d 
  ql/src/test/results/clientpositive/spark/bucketmapjoin7.q.out 7bfe440 
  ql/src/test/results/clientpositive/spark/bucketmapjoin8.q.out 4601eb1 
  ql/src/test/results/clientpositive/spark/bucketmapjoin9.q.out 60bd103 
  ql/src/test/results/clientpositive/spark/bucketmapjoin_negative.q.out 031c46c 
  ql/src/test/results/clientpositive/spark/bucketmapjoin_negative2.q.out 4a8f46d 
  ql/src/test/results/clientpositive/spark/bucketmapjoin_negative3.q.out a09904e 
  ql/src/test/results/clientpositive/spark/count.q.out cb9eda5 
  ql/src/test/results/clientpositive/spark/insert_into1.q.out e7f7d8f 
  ql/src/test/results/clientpositive/spark/skewjoin_union_remove_1.q.out f6acc34 
  ql/src/test/results/clientpositive/spark/stats1.q.out f00db10 
  ql/src/test/results/clientpositive/spark/stats12.q.out db575df 
  ql/src/test/results/clientpositive/spark/stats13.q.out f38f876 
  ql/src/test/results/clientpositive/spark/stats18.q.out a061846 
  ql/src/test/results/clientpositive/spark/stats2.q.out ddc8226 
  ql/src/test/results/clientpositive/spark/stats3.q.out dd3a95b 
  ql/src/test/results/clientpositive/spark/stats6.q.out b4435f2 
  ql/src/test/results/clientpositive/spark/stats_noscan_1.q.out 80c3092 
  ql/src/test/results/clientpositive/spark/stats_noscan_2.q.out 238727d 
  ql/src/test/results/clientpositive/spark/stats_partscan_1_23.q.out adcf150 
  ql/src/test/results/clientpositive/spark/union_remove_1.q.out bf0fc20 
  ql/src/test/results/clientpositive/spark/union_remove_10.q.out a9d4b51 
  ql/src/test/results/clientpositive/spark/union_remove_11.q.out be65741 
  ql/src/test/results/clientpositive/spark/union_remove_12.q.out 62d9d7d 
  ql/src/test/results/clientpositive/spark/union_remove_13.q.out 995d180 
  ql/src/test/results/clientpositive/spark/union_remove_14.q.out b28bb75 
  ql/src/test/results/clientpositive/spark/union_remove_15.q.out 58abd21 
  ql/src/test/results/clientpositive/spark/union_remove_16.q.out 356e79a 
  ql/src/test/results/clientpositive/spark/union_remove_17.q.out 74a5b23 
  ql/src/test/results/clientpositive/spark/union_remove_18.q.out 2c01a5c 
  ql/src/test/results/clientpositive/spark/union_remove_19.q.out d1e2312 
  ql/src/test/results/clientpositive/spark/union_remove_2.q.out 59d88cb 
  ql/src/test/results/clientpositive/spark/union_remove_20.q.out f57353f 
  ql/src/test/results/clientpositive/spark/union_remove_21.q.out 48867fb 
  ql/src/test/results/clientpositive/spark/union_remove_22.q.out c41e12f 
  ql/src/test/results/clientpositive/spark/union_remove_23.q.out d9875f3 
  ql/src/test/results/clientpositive/spark/union_remove_24.q.out e989ed2 
  ql/src/test/results/clientpositive/spark/union_remove_25.q.out a5a0126 
  ql/src/test/results/clientpositive/spark/union_remove_3.q.out a14a8b4 
  ql/src/test/results/clientpositive/spark/union_remove_4.q.out 7d94d1d 
  ql/src/test/results/clientpositive/spark/union_remove_5.q.out b31b3e6 
  ql/src/test/results/clientpositive/spark/union_remove_6.q.out cd36189 
  ql/src/test/results/clientpositive/spark/union_remove_6_subq.q.out 988973f 
  ql/src/test/results/clientpositive/spark/union_remove_7.q.out 2a5eccf 
  ql/src/test/results/clientpositive/spark/union_remove_8.q.out 7aee2af 
  ql/src/test/results/clientpositive/spark/union_remove_9.q.out 87ceca2 
  ql/src/test/results/clientpositive/stats1.q.out 6f560d3 
  ql/src/test/results/clientpositive/stats11.q.out e51f049 
  ql/src/test/results/clientpositive/stats12.q.out d5ca59b 
  ql/src/test/results/clientpositive/stats13.q.out e34f0b7 
  ql/src/test/results/clientpositive/stats18.q.out a061846 
  ql/src/test/results/clientpositive/stats2.q.out 5e305d3 
  ql/src/test/results/clientpositive/stats3.q.out dd3a95b 
  ql/src/test/results/clientpositive/stats6.q.out b4435f2 
  ql/src/test/results/clientpositive/stats_noscan_1.q.out 80c3092 
  ql/src/test/results/clientpositive/stats_noscan_2.q.out 238727d 
  ql/src/test/results/clientpositive/stats_partscan_1_23.q.out 5488935 
  ql/src/test/results/clientpositive/temp_table_display_colstats_tbllvl.q.out cfa88ab 
  ql/src/test/results/clientpositive/tez/auto_join_filters.q.out 8fde41d 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_1.q.out a275d27 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_11.q.out 6ac74ca 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_12.q.out e90af15 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_2.q.out 2cb8416 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_3.q.out abeceb8 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_4.q.out 8eb9ce5 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_5.q.out adcc1fa 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_7.q.out 2562cb0 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_8.q.out 31b0a97 
  ql/src/test/results/clientpositive/tez/explainuser_1.q.out dadcec1 
  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/truncate_column.q.out 2d3e378 
  ql/src/test/results/clientpositive/truncate_column_list_bucket.q.out 5ff6607 
  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


Re: Review Request 34713: Invalidate basic stats for insert queries if autogather=false

Posted by Ashutosh Chauhan <ha...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34713/
-----------------------------------------------------------

(Updated Oct. 22, 2015, 7:53 p.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 (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/QueryProperties.java e8f7fba 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 4e3be0d 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMapRedUtils.java 109b938 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/StatsOptimizer.java 5a21e6b 
  ql/src/java/org/apache/hadoop/hive/ql/parse/QBParseInfo.java 14a7e9c 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 3262887 
  ql/src/test/queries/clientpositive/insert_into1.q af82e3a 
  ql/src/test/queries/clientpositive/insert_into2.q 7183c75 
  ql/src/test/results/clientpositive/bucket_map_join_1.q.out ce0df01 
  ql/src/test/results/clientpositive/bucket_map_join_2.q.out 791e606 
  ql/src/test/results/clientpositive/encrypted/encryption_insert_partition_dynamic.q.out 3ed1fdb 
  ql/src/test/results/clientpositive/encrypted/encryption_join_unencrypted_tbl.q.out 7997fcb 
  ql/src/test/results/clientpositive/insert_into1.q.out cf627a6 
  ql/src/test/results/clientpositive/insert_into2.q.out cc7e135 
  ql/src/test/results/clientpositive/spark/bucket_map_join_1.q.out d3f433d 
  ql/src/test/results/clientpositive/spark/bucket_map_join_2.q.out 3d850db 
  ql/src/test/results/clientpositive/spark/insert_into1.q.out 38134a1 
  ql/src/test/results/clientpositive/spark/insert_into2.q.out 578fae2 
  ql/src/test/results/clientpositive/spark/stats3.q.out 2afb76e 
  ql/src/test/results/clientpositive/stats3.q.out 2afb76e 
  ql/src/test/results/clientpositive/tez/insert_into1.q.out b24b407 
  ql/src/test/results/clientpositive/tez/insert_into2.q.out 2c7ec63 

Diff: https://reviews.apache.org/r/34713/diff/


Testing
-------

Existing tests + added new tests


Thanks,

Ashutosh Chauhan


Re: Review Request 34713: Invalidate basic stats for insert queries if autogather=false

Posted by Gunther Hagleitner <gh...@hortonworks.com>.
-----------------------------------------------------------
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
> 
>


Re: Review Request 34713: Invalidate basic stats for insert queries if autogather=false

Posted by Ashutosh Chauhan <ha...@apache.org>.
-----------------------------------------------------------
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 (updated)
-----

  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