You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by pengcheng xiong <px...@hortonworks.com> on 2016/04/08 01:58:18 UTC

Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

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

Review request for hive and Ashutosh Chauhan.


Repository: hive-git


Description
-------

HIVE-13341: Stats state is not captured correctly: differentiate load table and create table


Diffs
-----

  common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
  itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
  metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 866e1c3 
  ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java 4a546d1 
  ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java c27481f 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 549d24f 
  ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 03b4d8b 
  ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java eaeb66b 
  ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
  ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 

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


Testing
-------


Thanks,

pengcheng xiong


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

Posted by pengcheng xiong <px...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45911/#review129821
-----------------------------------------------------------




ql/src/test/results/clientpositive/stats3.q.out (lines 94 - 96)
<https://reviews.apache.org/r/45911/#comment193357>

    No. For false case, we do not have any flag.



ql/src/test/results/clientpositive/stats3.q.out (lines 238 - 241)
<https://reviews.apache.org/r/45911/#comment193356>

    No, an insert overwrite is before it.


- pengcheng xiong


On April 14, 2016, 11:52 p.m., pengcheng xiong wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45911/
> -----------------------------------------------------------
> 
> (Updated April 14, 2016, 11:52 p.m.)
> 
> 
> Review request for hive and Ashutosh Chauhan.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-13341: Stats state is not captured correctly: differentiate load table and create table
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
>   itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
>   metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java 54592cc 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
>   ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 4c9acce 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 549d24f 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 03b4d8b 
>   ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java eaeb66b 
>   ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
>   ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/join32.q.out 5cb124b 
>   ql/src/test/results/clientpositive/stats0.q.out 99326fc 
>   ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
>   ql/src/test/results/clientpositive/stats10.q.out d7519af 
>   ql/src/test/results/clientpositive/stats11.q.out 7598dc0 
>   ql/src/test/results/clientpositive/stats12.q.out 6a44d15 
>   ql/src/test/results/clientpositive/stats13.q.out f98753e 
>   ql/src/test/results/clientpositive/stats16.q.out 3b371a8 
>   ql/src/test/results/clientpositive/stats18.q.out 6971e44 
>   ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
>   ql/src/test/results/clientpositive/stats3.q.out 0d8cbbd 
>   ql/src/test/results/clientpositive/stats5.q.out 74ddadb 
>   ql/src/test/results/clientpositive/stats6.q.out f2a756f 
>   ql/src/test/results/clientpositive/stats_noscan_1.q.out 7382b31 
>   ql/src/test/results/clientpositive/stats_partscan_1_23.q.out ab2baaa 
> 
> Diff: https://reviews.apache.org/r/45911/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> pengcheng xiong
> 
>


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

Posted by pengcheng xiong <px...@hortonworks.com>.

> On May 6, 2016, 9:38 p.m., Ashutosh Chauhan wrote:
> > data/scripts/q_test_cleanup.sql, lines 2-12
> > <https://reviews.apache.org/r/45911/diff/5/?file=1375182#file1375182line2>
> >
> >     Is this needed or accidental?

Sorry, it is removed when I was debugging... I will restore and upload a new patch.


> On May 6, 2016, 9:38 p.m., Ashutosh Chauhan wrote:
> > data/scripts/q_test_init.sql, lines 11-16
> > <https://reviews.apache.org/r/45911/diff/5/?file=1375183#file1375183line11>
> >
> >     Is this needed or accidental?

Sorry, it is removed when I was debugging... I will restore and upload a new patch.


- pengcheng


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


On May 6, 2016, 9:29 p.m., pengcheng xiong wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45911/
> -----------------------------------------------------------
> 
> (Updated May 6, 2016, 9:29 p.m.)
> 
> 
> Review request for hive and Ashutosh Chauhan.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-13341: Stats state is not captured correctly: differentiate load table and create table
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
>   data/scripts/q_test_cleanup.sql 4620dcd 
>   data/scripts/q_test_init.sql 0c8668c 
>   itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
>   metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java c2c6c65 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
>   ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java f4a9772 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 4a6617f 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 500c7ed 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 2dc4e11 
>   ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java 667d5c2 
>   ql/src/test/queries/clientpositive/alter_table_add_partition.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/stats20.q 59701bd 
>   ql/src/test/results/clientnegative/alter_file_format.q.out 96f1bfb 
>   ql/src/test/results/clientnegative/unset_table_property.q.out 0510788 
>   ql/src/test/results/clientpositive/alter_file_format.q.out 5d83b23 
>   ql/src/test/results/clientpositive/alter_skewed_table.q.out a1caa99 
>   ql/src/test/results/clientpositive/alter_table_add_partition.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/alter_table_not_sorted.q.out 6e1ec59 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_1.q.out 0902556 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_11.q.out 81de2b0 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_12.q.out 26a11a7 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_2.q.out 78cd03d 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_3.q.out 7ddf6b3 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_4.q.out 5b68da2 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_5.q.out 07bcb89 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_7.q.out d12c1ca 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_8.q.out 23a3685 
>   ql/src/test/results/clientpositive/binary_output_format.q.out 608d475 
>   ql/src/test/results/clientpositive/bucket1.q.out 96b8d4e 
>   ql/src/test/results/clientpositive/bucket2.q.out b5572fd 
>   ql/src/test/results/clientpositive/bucket4.q.out c4baf72 
>   ql/src/test/results/clientpositive/bucket5.q.out dee79f9 
>   ql/src/test/results/clientpositive/bucket_many.q.out 90d9fb8 
>   ql/src/test/results/clientpositive/bucket_map_join_1.q.out 49bf2ca 
>   ql/src/test/results/clientpositive/bucket_map_join_2.q.out 9ec7033 
>   ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out 19937cb 
>   ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out 90528be 
>   ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out aede979 
>   ql/src/test/results/clientpositive/bucketcontext_1.q.out d1b7744 
>   ql/src/test/results/clientpositive/bucketcontext_2.q.out 60c021c 
>   ql/src/test/results/clientpositive/bucketcontext_3.q.out 12eaddf 
>   ql/src/test/results/clientpositive/bucketcontext_4.q.out 2573564 
>   ql/src/test/results/clientpositive/bucketcontext_5.q.out e1c6a80 
>   ql/src/test/results/clientpositive/bucketcontext_6.q.out 9deb64d 
>   ql/src/test/results/clientpositive/bucketcontext_7.q.out 621844e 
>   ql/src/test/results/clientpositive/bucketcontext_8.q.out 7f3e50a 
>   ql/src/test/results/clientpositive/bucketmapjoin1.q.out 9956fa7 
>   ql/src/test/results/clientpositive/bucketmapjoin10.q.out fcdbb09 
>   ql/src/test/results/clientpositive/bucketmapjoin11.q.out 968066a 
>   ql/src/test/results/clientpositive/bucketmapjoin12.q.out 9017871 
>   ql/src/test/results/clientpositive/bucketmapjoin2.q.out f2efb95 
>   ql/src/test/results/clientpositive/bucketmapjoin3.q.out 5659983 
>   ql/src/test/results/clientpositive/bucketmapjoin4.q.out 5794d47 
>   ql/src/test/results/clientpositive/bucketmapjoin5.q.out 1bcb382 
>   ql/src/test/results/clientpositive/bucketmapjoin7.q.out 76b13f3 
>   ql/src/test/results/clientpositive/bucketmapjoin8.q.out 097eaad 
>   ql/src/test/results/clientpositive/bucketmapjoin9.q.out 144d86d 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out d108575 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative2.q.out 9c6a226 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative3.q.out 3078854 
>   ql/src/test/results/clientpositive/columnStatsUpdateForStatsOptimizer_1.q.out 8c9664d 
>   ql/src/test/results/clientpositive/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
>   ql/src/test/results/clientpositive/columnstats_partlvl.q.out 4edf39a 
>   ql/src/test/results/clientpositive/columnstats_tbllvl.q.out 669807d 
>   ql/src/test/results/clientpositive/create_alter_list_bucketing_table1.q.out c4f51a3 
>   ql/src/test/results/clientpositive/create_like.q.out 9241b68 
>   ql/src/test/results/clientpositive/create_like_view.q.out e2dc2c4 
>   ql/src/test/results/clientpositive/create_skewed_table1.q.out 415bb77 
>   ql/src/test/results/clientpositive/database_location.q.out 797177d 
>   ql/src/test/results/clientpositive/default_file_format.q.out 3d5c20f 
>   ql/src/test/results/clientpositive/describe_comment_indent.q.out 3e0f45e 
>   ql/src/test/results/clientpositive/describe_comment_nonascii.q.out df0b65e 
>   ql/src/test/results/clientpositive/disable_merge_for_bucketing.q.out c4dbf17 
>   ql/src/test/results/clientpositive/display_colstats_tbllvl.q.out 1028c26 
>   ql/src/test/results/clientpositive/groupby_map_ppr.q.out 3d07b0d 
>   ql/src/test/results/clientpositive/groupby_map_ppr_multi_distinct.q.out 6689225 
>   ql/src/test/results/clientpositive/groupby_ppr.q.out 62fffbe 
>   ql/src/test/results/clientpositive/groupby_ppr_multi_distinct.q.out 2408056 
>   ql/src/test/results/clientpositive/groupby_sort_1_23.q.out a6e15ba 
>   ql/src/test/results/clientpositive/groupby_sort_6.q.out f523085 
>   ql/src/test/results/clientpositive/groupby_sort_skew_1_23.q.out 38b24a4 
>   ql/src/test/results/clientpositive/input_part1.q.out 2abaa3b 
>   ql/src/test/results/clientpositive/input_part2.q.out 2ce071c 
>   ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/join17.q.out 39c49a5 
>   ql/src/test/results/clientpositive/join26.q.out 0fdc403 
>   ql/src/test/results/clientpositive/join32.q.out aad5463 
>   ql/src/test/results/clientpositive/join32_lessSize.q.out a94b951 
>   ql/src/test/results/clientpositive/join33.q.out aad5463 
>   ql/src/test/results/clientpositive/join34.q.out 74fac5c 
>   ql/src/test/results/clientpositive/join35.q.out 6fe9cae 
>   ql/src/test/results/clientpositive/join9.q.out f41d153 
>   ql/src/test/results/clientpositive/join_map_ppr.q.out e720e65 
>   ql/src/test/results/clientpositive/list_bucket_dml_14.q.out c063dc1 
>   ql/src/test/results/clientpositive/list_bucket_dml_8.q.java1.7.out d40a693 
>   ql/src/test/results/clientpositive/mapjoin_memcheck.q.out 8f5492f 
>   ql/src/test/results/clientpositive/nullformat.q.out af91470 
>   ql/src/test/results/clientpositive/orc_create.q.out 3013fda 
>   ql/src/test/results/clientpositive/orc_llap.q.out 6fc73b7 
>   ql/src/test/results/clientpositive/orc_predicate_pushdown.q.out 7b361b7 
>   ql/src/test/results/clientpositive/parquet_array_null_element.q.out 387f01e 
>   ql/src/test/results/clientpositive/parquet_create.q.out c6d33ff 
>   ql/src/test/results/clientpositive/parquet_mixed_partition_formats.q.out d6affd6 
>   ql/src/test/results/clientpositive/parquet_serde.q.out c1e594a 
>   ql/src/test/results/clientpositive/pcr.q.out b53226e 
>   ql/src/test/results/clientpositive/rand_partitionpruner2.q.out d88c53d 
>   ql/src/test/results/clientpositive/rcfile_default_format.q.out c961231 
>   ql/src/test/results/clientpositive/reduce_deduplicate.q.out 379f884 
>   ql/src/test/results/clientpositive/sample1.q.out e188902 
>   ql/src/test/results/clientpositive/sample2.q.out 9653c41 
>   ql/src/test/results/clientpositive/sample4.q.out a9730b1 
>   ql/src/test/results/clientpositive/sample5.q.out d5ff403 
>   ql/src/test/results/clientpositive/sample6.q.out ea71ad6 
>   ql/src/test/results/clientpositive/sample7.q.out 94618e5 
>   ql/src/test/results/clientpositive/schema_evol_orc_nonvec_fetchwork_table.q.out 912463b 
>   ql/src/test/results/clientpositive/schema_evol_orc_nonvec_mapwork_table.q.out f007ba8 
>   ql/src/test/results/clientpositive/schema_evol_orc_vec_mapwork_table.q.out a305847 
>   ql/src/test/results/clientpositive/schema_evol_text_nonvec_mapwork_table.q.out 7a7e068 
>   ql/src/test/results/clientpositive/schema_evol_text_vec_mapwork_table.q.out 50328d3 
>   ql/src/test/results/clientpositive/schema_evol_text_vecrow_mapwork_table.q.out 50790f4 
>   ql/src/test/results/clientpositive/show_create_table_alter.q.out 32819ea 
>   ql/src/test/results/clientpositive/show_create_table_db_table.q.out 495f4b5 
>   ql/src/test/results/clientpositive/show_create_table_serde.q.out 2350d98 
>   ql/src/test/results/clientpositive/show_tblproperties.q.out 63bbe6d 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_1.q.out dfb571d 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_12.q.out 76ea0a8 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_3.q.out c942d11 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_4.q.out ba2547a 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_5.q.out 99fca25 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_7.q.out 00c601f 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_8.q.out 5564ceb 
>   ql/src/test/results/clientpositive/spark/bucket2.q.out aa47459 
>   ql/src/test/results/clientpositive/spark/bucket4.q.out 680cee7 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_1.q.out 299cedf 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_2.q.out f330bf2 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_spark1.q.out b32528c 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_spark2.q.out 7b304c5 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_spark3.q.out 4265d1a 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin1.q.out c6a7f0e 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin10.q.out 0e3b5c7 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin11.q.out 38f9785 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin12.q.out 55ded7d 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin2.q.out 48ff95f 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin3.q.out 95589ca 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin4.q.out fba617c 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin5.q.out 7214b42 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin7.q.out d344070 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin8.q.out a7ad6de 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin9.q.out 6e4c536 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin_negative.q.out 9fff7b2 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin_negative2.q.out b199ad9 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin_negative3.q.out 845e450 
>   ql/src/test/results/clientpositive/spark/disable_merge_for_bucketing.q.out e21b1b1 
>   ql/src/test/results/clientpositive/spark/groupby_sort_1_23.q.out 1b4822e 
>   ql/src/test/results/clientpositive/spark/groupby_sort_skew_1_23.q.out 0956771 
>   ql/src/test/results/clientpositive/spark/join17.q.out 0d92441 
>   ql/src/test/results/clientpositive/spark/mapjoin_memcheck.q.out 2f41847 
>   ql/src/test/results/clientpositive/spark/stats0.q.out e741320 
>   ql/src/test/results/clientpositive/spark/stats20.q.out d7e52b4 
>   ql/src/test/results/clientpositive/spark/stats3.q.out 4dbda94 
>   ql/src/test/results/clientpositive/spark/vectorized_ptf.q.out 7b56585 
>   ql/src/test/results/clientpositive/stats0.q.out 90a9a70 
>   ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
>   ql/src/test/results/clientpositive/stats11.q.out 8ca5b27 
>   ql/src/test/results/clientpositive/stats18.q.out 6971e44 
>   ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
>   ql/src/test/results/clientpositive/stats3.q.out 4dbda94 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_1.q.out 892539d 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_11.q.out 954fd96 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_12.q.out 578880d 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_2.q.out c283738 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_3.q.out 351fd20 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_4.q.out 2d9cdf8 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_5.q.out 8bc203a 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_7.q.out 18fc95c 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_8.q.out 9a1010c 
>   ql/src/test/results/clientpositive/tez/bucket2.q.out e9432a8 
>   ql/src/test/results/clientpositive/tez/bucket4.q.out a0f1177 
>   ql/src/test/results/clientpositive/tez/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
>   ql/src/test/results/clientpositive/tez/disable_merge_for_bucketing.q.out bb62e1f 
>   ql/src/test/results/clientpositive/tez/explainuser_1.q.out bdb8830 
>   ql/src/test/results/clientpositive/tez/sample1.q.out 009969e 
>   ql/src/test/results/clientpositive/tez/schema_evol_orc_nonvec_fetchwork_table.q.out 8a9481f 
>   ql/src/test/results/clientpositive/tez/schema_evol_orc_nonvec_mapwork_table.q.out 39954b8 
>   ql/src/test/results/clientpositive/tez/schema_evol_orc_vec_mapwork_table.q.out ca292b1 
>   ql/src/test/results/clientpositive/tez/schema_evol_text_nonvec_mapwork_table.q.out 14abca9 
>   ql/src/test/results/clientpositive/tez/schema_evol_text_vec_mapwork_table.q.out 2b799f9 
>   ql/src/test/results/clientpositive/tez/schema_evol_text_vecrow_mapwork_table.q.out 1b46fa7 
>   ql/src/test/results/clientpositive/tez/tez_join_result_complex.q.out 3e3b08c 
>   ql/src/test/results/clientpositive/tez/vectorized_ptf.q.out 1365626 
>   ql/src/test/results/clientpositive/truncate_column.q.out adbddfa 
>   ql/src/test/results/clientpositive/truncate_column_list_bucket.q.out 39ffc6c 
>   ql/src/test/results/clientpositive/unicode_notation.q.out 52da674 
>   ql/src/test/results/clientpositive/unset_table_view_property.q.out 8bbb9fe 
>   ql/src/test/results/clientpositive/vectorized_ptf.q.out fc4351d 
> 
> Diff: https://reviews.apache.org/r/45911/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> pengcheng xiong
> 
>


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

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




data/scripts/q_test_cleanup.sql 
<https://reviews.apache.org/r/45911/#comment196231>

    Is this needed or accidental?



data/scripts/q_test_init.sql 
<https://reviews.apache.org/r/45911/#comment196232>

    Is this needed or accidental?


- Ashutosh Chauhan


On May 6, 2016, 9:29 p.m., pengcheng xiong wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45911/
> -----------------------------------------------------------
> 
> (Updated May 6, 2016, 9:29 p.m.)
> 
> 
> Review request for hive and Ashutosh Chauhan.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-13341: Stats state is not captured correctly: differentiate load table and create table
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
>   data/scripts/q_test_cleanup.sql 4620dcd 
>   data/scripts/q_test_init.sql 0c8668c 
>   itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
>   metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java c2c6c65 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
>   ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java f4a9772 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 4a6617f 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 500c7ed 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 2dc4e11 
>   ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java 667d5c2 
>   ql/src/test/queries/clientpositive/alter_table_add_partition.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/stats20.q 59701bd 
>   ql/src/test/results/clientnegative/alter_file_format.q.out 96f1bfb 
>   ql/src/test/results/clientnegative/unset_table_property.q.out 0510788 
>   ql/src/test/results/clientpositive/alter_file_format.q.out 5d83b23 
>   ql/src/test/results/clientpositive/alter_skewed_table.q.out a1caa99 
>   ql/src/test/results/clientpositive/alter_table_add_partition.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/alter_table_not_sorted.q.out 6e1ec59 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_1.q.out 0902556 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_11.q.out 81de2b0 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_12.q.out 26a11a7 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_2.q.out 78cd03d 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_3.q.out 7ddf6b3 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_4.q.out 5b68da2 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_5.q.out 07bcb89 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_7.q.out d12c1ca 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_8.q.out 23a3685 
>   ql/src/test/results/clientpositive/binary_output_format.q.out 608d475 
>   ql/src/test/results/clientpositive/bucket1.q.out 96b8d4e 
>   ql/src/test/results/clientpositive/bucket2.q.out b5572fd 
>   ql/src/test/results/clientpositive/bucket4.q.out c4baf72 
>   ql/src/test/results/clientpositive/bucket5.q.out dee79f9 
>   ql/src/test/results/clientpositive/bucket_many.q.out 90d9fb8 
>   ql/src/test/results/clientpositive/bucket_map_join_1.q.out 49bf2ca 
>   ql/src/test/results/clientpositive/bucket_map_join_2.q.out 9ec7033 
>   ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out 19937cb 
>   ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out 90528be 
>   ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out aede979 
>   ql/src/test/results/clientpositive/bucketcontext_1.q.out d1b7744 
>   ql/src/test/results/clientpositive/bucketcontext_2.q.out 60c021c 
>   ql/src/test/results/clientpositive/bucketcontext_3.q.out 12eaddf 
>   ql/src/test/results/clientpositive/bucketcontext_4.q.out 2573564 
>   ql/src/test/results/clientpositive/bucketcontext_5.q.out e1c6a80 
>   ql/src/test/results/clientpositive/bucketcontext_6.q.out 9deb64d 
>   ql/src/test/results/clientpositive/bucketcontext_7.q.out 621844e 
>   ql/src/test/results/clientpositive/bucketcontext_8.q.out 7f3e50a 
>   ql/src/test/results/clientpositive/bucketmapjoin1.q.out 9956fa7 
>   ql/src/test/results/clientpositive/bucketmapjoin10.q.out fcdbb09 
>   ql/src/test/results/clientpositive/bucketmapjoin11.q.out 968066a 
>   ql/src/test/results/clientpositive/bucketmapjoin12.q.out 9017871 
>   ql/src/test/results/clientpositive/bucketmapjoin2.q.out f2efb95 
>   ql/src/test/results/clientpositive/bucketmapjoin3.q.out 5659983 
>   ql/src/test/results/clientpositive/bucketmapjoin4.q.out 5794d47 
>   ql/src/test/results/clientpositive/bucketmapjoin5.q.out 1bcb382 
>   ql/src/test/results/clientpositive/bucketmapjoin7.q.out 76b13f3 
>   ql/src/test/results/clientpositive/bucketmapjoin8.q.out 097eaad 
>   ql/src/test/results/clientpositive/bucketmapjoin9.q.out 144d86d 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out d108575 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative2.q.out 9c6a226 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative3.q.out 3078854 
>   ql/src/test/results/clientpositive/columnStatsUpdateForStatsOptimizer_1.q.out 8c9664d 
>   ql/src/test/results/clientpositive/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
>   ql/src/test/results/clientpositive/columnstats_partlvl.q.out 4edf39a 
>   ql/src/test/results/clientpositive/columnstats_tbllvl.q.out 669807d 
>   ql/src/test/results/clientpositive/create_alter_list_bucketing_table1.q.out c4f51a3 
>   ql/src/test/results/clientpositive/create_like.q.out 9241b68 
>   ql/src/test/results/clientpositive/create_like_view.q.out e2dc2c4 
>   ql/src/test/results/clientpositive/create_skewed_table1.q.out 415bb77 
>   ql/src/test/results/clientpositive/database_location.q.out 797177d 
>   ql/src/test/results/clientpositive/default_file_format.q.out 3d5c20f 
>   ql/src/test/results/clientpositive/describe_comment_indent.q.out 3e0f45e 
>   ql/src/test/results/clientpositive/describe_comment_nonascii.q.out df0b65e 
>   ql/src/test/results/clientpositive/disable_merge_for_bucketing.q.out c4dbf17 
>   ql/src/test/results/clientpositive/display_colstats_tbllvl.q.out 1028c26 
>   ql/src/test/results/clientpositive/groupby_map_ppr.q.out 3d07b0d 
>   ql/src/test/results/clientpositive/groupby_map_ppr_multi_distinct.q.out 6689225 
>   ql/src/test/results/clientpositive/groupby_ppr.q.out 62fffbe 
>   ql/src/test/results/clientpositive/groupby_ppr_multi_distinct.q.out 2408056 
>   ql/src/test/results/clientpositive/groupby_sort_1_23.q.out a6e15ba 
>   ql/src/test/results/clientpositive/groupby_sort_6.q.out f523085 
>   ql/src/test/results/clientpositive/groupby_sort_skew_1_23.q.out 38b24a4 
>   ql/src/test/results/clientpositive/input_part1.q.out 2abaa3b 
>   ql/src/test/results/clientpositive/input_part2.q.out 2ce071c 
>   ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/join17.q.out 39c49a5 
>   ql/src/test/results/clientpositive/join26.q.out 0fdc403 
>   ql/src/test/results/clientpositive/join32.q.out aad5463 
>   ql/src/test/results/clientpositive/join32_lessSize.q.out a94b951 
>   ql/src/test/results/clientpositive/join33.q.out aad5463 
>   ql/src/test/results/clientpositive/join34.q.out 74fac5c 
>   ql/src/test/results/clientpositive/join35.q.out 6fe9cae 
>   ql/src/test/results/clientpositive/join9.q.out f41d153 
>   ql/src/test/results/clientpositive/join_map_ppr.q.out e720e65 
>   ql/src/test/results/clientpositive/list_bucket_dml_14.q.out c063dc1 
>   ql/src/test/results/clientpositive/list_bucket_dml_8.q.java1.7.out d40a693 
>   ql/src/test/results/clientpositive/mapjoin_memcheck.q.out 8f5492f 
>   ql/src/test/results/clientpositive/nullformat.q.out af91470 
>   ql/src/test/results/clientpositive/orc_create.q.out 3013fda 
>   ql/src/test/results/clientpositive/orc_llap.q.out 6fc73b7 
>   ql/src/test/results/clientpositive/orc_predicate_pushdown.q.out 7b361b7 
>   ql/src/test/results/clientpositive/parquet_array_null_element.q.out 387f01e 
>   ql/src/test/results/clientpositive/parquet_create.q.out c6d33ff 
>   ql/src/test/results/clientpositive/parquet_mixed_partition_formats.q.out d6affd6 
>   ql/src/test/results/clientpositive/parquet_serde.q.out c1e594a 
>   ql/src/test/results/clientpositive/pcr.q.out b53226e 
>   ql/src/test/results/clientpositive/rand_partitionpruner2.q.out d88c53d 
>   ql/src/test/results/clientpositive/rcfile_default_format.q.out c961231 
>   ql/src/test/results/clientpositive/reduce_deduplicate.q.out 379f884 
>   ql/src/test/results/clientpositive/sample1.q.out e188902 
>   ql/src/test/results/clientpositive/sample2.q.out 9653c41 
>   ql/src/test/results/clientpositive/sample4.q.out a9730b1 
>   ql/src/test/results/clientpositive/sample5.q.out d5ff403 
>   ql/src/test/results/clientpositive/sample6.q.out ea71ad6 
>   ql/src/test/results/clientpositive/sample7.q.out 94618e5 
>   ql/src/test/results/clientpositive/schema_evol_orc_nonvec_fetchwork_table.q.out 912463b 
>   ql/src/test/results/clientpositive/schema_evol_orc_nonvec_mapwork_table.q.out f007ba8 
>   ql/src/test/results/clientpositive/schema_evol_orc_vec_mapwork_table.q.out a305847 
>   ql/src/test/results/clientpositive/schema_evol_text_nonvec_mapwork_table.q.out 7a7e068 
>   ql/src/test/results/clientpositive/schema_evol_text_vec_mapwork_table.q.out 50328d3 
>   ql/src/test/results/clientpositive/schema_evol_text_vecrow_mapwork_table.q.out 50790f4 
>   ql/src/test/results/clientpositive/show_create_table_alter.q.out 32819ea 
>   ql/src/test/results/clientpositive/show_create_table_db_table.q.out 495f4b5 
>   ql/src/test/results/clientpositive/show_create_table_serde.q.out 2350d98 
>   ql/src/test/results/clientpositive/show_tblproperties.q.out 63bbe6d 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_1.q.out dfb571d 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_12.q.out 76ea0a8 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_3.q.out c942d11 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_4.q.out ba2547a 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_5.q.out 99fca25 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_7.q.out 00c601f 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_8.q.out 5564ceb 
>   ql/src/test/results/clientpositive/spark/bucket2.q.out aa47459 
>   ql/src/test/results/clientpositive/spark/bucket4.q.out 680cee7 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_1.q.out 299cedf 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_2.q.out f330bf2 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_spark1.q.out b32528c 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_spark2.q.out 7b304c5 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_spark3.q.out 4265d1a 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin1.q.out c6a7f0e 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin10.q.out 0e3b5c7 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin11.q.out 38f9785 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin12.q.out 55ded7d 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin2.q.out 48ff95f 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin3.q.out 95589ca 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin4.q.out fba617c 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin5.q.out 7214b42 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin7.q.out d344070 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin8.q.out a7ad6de 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin9.q.out 6e4c536 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin_negative.q.out 9fff7b2 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin_negative2.q.out b199ad9 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin_negative3.q.out 845e450 
>   ql/src/test/results/clientpositive/spark/disable_merge_for_bucketing.q.out e21b1b1 
>   ql/src/test/results/clientpositive/spark/groupby_sort_1_23.q.out 1b4822e 
>   ql/src/test/results/clientpositive/spark/groupby_sort_skew_1_23.q.out 0956771 
>   ql/src/test/results/clientpositive/spark/join17.q.out 0d92441 
>   ql/src/test/results/clientpositive/spark/mapjoin_memcheck.q.out 2f41847 
>   ql/src/test/results/clientpositive/spark/stats0.q.out e741320 
>   ql/src/test/results/clientpositive/spark/stats20.q.out d7e52b4 
>   ql/src/test/results/clientpositive/spark/stats3.q.out 4dbda94 
>   ql/src/test/results/clientpositive/spark/vectorized_ptf.q.out 7b56585 
>   ql/src/test/results/clientpositive/stats0.q.out 90a9a70 
>   ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
>   ql/src/test/results/clientpositive/stats11.q.out 8ca5b27 
>   ql/src/test/results/clientpositive/stats18.q.out 6971e44 
>   ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
>   ql/src/test/results/clientpositive/stats3.q.out 4dbda94 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_1.q.out 892539d 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_11.q.out 954fd96 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_12.q.out 578880d 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_2.q.out c283738 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_3.q.out 351fd20 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_4.q.out 2d9cdf8 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_5.q.out 8bc203a 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_7.q.out 18fc95c 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_8.q.out 9a1010c 
>   ql/src/test/results/clientpositive/tez/bucket2.q.out e9432a8 
>   ql/src/test/results/clientpositive/tez/bucket4.q.out a0f1177 
>   ql/src/test/results/clientpositive/tez/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
>   ql/src/test/results/clientpositive/tez/disable_merge_for_bucketing.q.out bb62e1f 
>   ql/src/test/results/clientpositive/tez/explainuser_1.q.out bdb8830 
>   ql/src/test/results/clientpositive/tez/sample1.q.out 009969e 
>   ql/src/test/results/clientpositive/tez/schema_evol_orc_nonvec_fetchwork_table.q.out 8a9481f 
>   ql/src/test/results/clientpositive/tez/schema_evol_orc_nonvec_mapwork_table.q.out 39954b8 
>   ql/src/test/results/clientpositive/tez/schema_evol_orc_vec_mapwork_table.q.out ca292b1 
>   ql/src/test/results/clientpositive/tez/schema_evol_text_nonvec_mapwork_table.q.out 14abca9 
>   ql/src/test/results/clientpositive/tez/schema_evol_text_vec_mapwork_table.q.out 2b799f9 
>   ql/src/test/results/clientpositive/tez/schema_evol_text_vecrow_mapwork_table.q.out 1b46fa7 
>   ql/src/test/results/clientpositive/tez/tez_join_result_complex.q.out 3e3b08c 
>   ql/src/test/results/clientpositive/tez/vectorized_ptf.q.out 1365626 
>   ql/src/test/results/clientpositive/truncate_column.q.out adbddfa 
>   ql/src/test/results/clientpositive/truncate_column_list_bucket.q.out 39ffc6c 
>   ql/src/test/results/clientpositive/unicode_notation.q.out 52da674 
>   ql/src/test/results/clientpositive/unset_table_view_property.q.out 8bbb9fe 
>   ql/src/test/results/clientpositive/vectorized_ptf.q.out fc4351d 
> 
> Diff: https://reviews.apache.org/r/45911/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> pengcheng xiong
> 
>


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

Posted by pengcheng xiong <px...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45911/
-----------------------------------------------------------

(Updated May 6, 2016, 10:31 p.m.)


Review request for hive and Ashutosh Chauhan.


Repository: hive-git


Description
-------

HIVE-13341: Stats state is not captured correctly: differentiate load table and create table


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
  itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
  metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
  ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java c2c6c65 
  ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java f4a9772 
  ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 4a6617f 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 500c7ed 
  ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 2dc4e11 
  ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java 667d5c2 
  ql/src/test/queries/clientpositive/alter_table_add_partition.q PRE-CREATION 
  ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
  ql/src/test/queries/clientpositive/stats20.q 59701bd 
  ql/src/test/results/clientnegative/alter_file_format.q.out 96f1bfb 
  ql/src/test/results/clientnegative/unset_table_property.q.out 0510788 
  ql/src/test/results/clientpositive/alter_file_format.q.out 5d83b23 
  ql/src/test/results/clientpositive/alter_skewed_table.q.out a1caa99 
  ql/src/test/results/clientpositive/alter_table_add_partition.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/alter_table_not_sorted.q.out 6e1ec59 
  ql/src/test/results/clientpositive/auto_sortmerge_join_1.q.out 0902556 
  ql/src/test/results/clientpositive/auto_sortmerge_join_11.q.out 81de2b0 
  ql/src/test/results/clientpositive/auto_sortmerge_join_12.q.out 26a11a7 
  ql/src/test/results/clientpositive/auto_sortmerge_join_2.q.out 78cd03d 
  ql/src/test/results/clientpositive/auto_sortmerge_join_3.q.out 7ddf6b3 
  ql/src/test/results/clientpositive/auto_sortmerge_join_4.q.out 5b68da2 
  ql/src/test/results/clientpositive/auto_sortmerge_join_5.q.out 07bcb89 
  ql/src/test/results/clientpositive/auto_sortmerge_join_7.q.out d12c1ca 
  ql/src/test/results/clientpositive/auto_sortmerge_join_8.q.out 23a3685 
  ql/src/test/results/clientpositive/binary_output_format.q.out 608d475 
  ql/src/test/results/clientpositive/bucket1.q.out 96b8d4e 
  ql/src/test/results/clientpositive/bucket2.q.out b5572fd 
  ql/src/test/results/clientpositive/bucket4.q.out c4baf72 
  ql/src/test/results/clientpositive/bucket5.q.out dee79f9 
  ql/src/test/results/clientpositive/bucket_many.q.out 90d9fb8 
  ql/src/test/results/clientpositive/bucket_map_join_1.q.out 49bf2ca 
  ql/src/test/results/clientpositive/bucket_map_join_2.q.out 9ec7033 
  ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out 19937cb 
  ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out 90528be 
  ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out aede979 
  ql/src/test/results/clientpositive/bucketcontext_1.q.out d1b7744 
  ql/src/test/results/clientpositive/bucketcontext_2.q.out 60c021c 
  ql/src/test/results/clientpositive/bucketcontext_3.q.out 12eaddf 
  ql/src/test/results/clientpositive/bucketcontext_4.q.out 2573564 
  ql/src/test/results/clientpositive/bucketcontext_5.q.out e1c6a80 
  ql/src/test/results/clientpositive/bucketcontext_6.q.out 9deb64d 
  ql/src/test/results/clientpositive/bucketcontext_7.q.out 621844e 
  ql/src/test/results/clientpositive/bucketcontext_8.q.out 7f3e50a 
  ql/src/test/results/clientpositive/bucketmapjoin1.q.out 9956fa7 
  ql/src/test/results/clientpositive/bucketmapjoin10.q.out fcdbb09 
  ql/src/test/results/clientpositive/bucketmapjoin11.q.out 968066a 
  ql/src/test/results/clientpositive/bucketmapjoin12.q.out 9017871 
  ql/src/test/results/clientpositive/bucketmapjoin2.q.out f2efb95 
  ql/src/test/results/clientpositive/bucketmapjoin3.q.out 5659983 
  ql/src/test/results/clientpositive/bucketmapjoin4.q.out 5794d47 
  ql/src/test/results/clientpositive/bucketmapjoin5.q.out 1bcb382 
  ql/src/test/results/clientpositive/bucketmapjoin7.q.out 76b13f3 
  ql/src/test/results/clientpositive/bucketmapjoin8.q.out 097eaad 
  ql/src/test/results/clientpositive/bucketmapjoin9.q.out 144d86d 
  ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out d108575 
  ql/src/test/results/clientpositive/bucketmapjoin_negative2.q.out 9c6a226 
  ql/src/test/results/clientpositive/bucketmapjoin_negative3.q.out 3078854 
  ql/src/test/results/clientpositive/columnStatsUpdateForStatsOptimizer_1.q.out 8c9664d 
  ql/src/test/results/clientpositive/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
  ql/src/test/results/clientpositive/columnstats_partlvl.q.out 4edf39a 
  ql/src/test/results/clientpositive/columnstats_tbllvl.q.out 669807d 
  ql/src/test/results/clientpositive/create_alter_list_bucketing_table1.q.out c4f51a3 
  ql/src/test/results/clientpositive/create_like.q.out 9241b68 
  ql/src/test/results/clientpositive/create_like_view.q.out e2dc2c4 
  ql/src/test/results/clientpositive/create_skewed_table1.q.out 415bb77 
  ql/src/test/results/clientpositive/database_location.q.out 797177d 
  ql/src/test/results/clientpositive/default_file_format.q.out 3d5c20f 
  ql/src/test/results/clientpositive/describe_comment_indent.q.out 3e0f45e 
  ql/src/test/results/clientpositive/describe_comment_nonascii.q.out df0b65e 
  ql/src/test/results/clientpositive/disable_merge_for_bucketing.q.out c4dbf17 
  ql/src/test/results/clientpositive/display_colstats_tbllvl.q.out 1028c26 
  ql/src/test/results/clientpositive/groupby_map_ppr.q.out 3d07b0d 
  ql/src/test/results/clientpositive/groupby_map_ppr_multi_distinct.q.out 6689225 
  ql/src/test/results/clientpositive/groupby_ppr.q.out 62fffbe 
  ql/src/test/results/clientpositive/groupby_ppr_multi_distinct.q.out 2408056 
  ql/src/test/results/clientpositive/groupby_sort_1_23.q.out a6e15ba 
  ql/src/test/results/clientpositive/groupby_sort_6.q.out f523085 
  ql/src/test/results/clientpositive/groupby_sort_skew_1_23.q.out 38b24a4 
  ql/src/test/results/clientpositive/input_part1.q.out 2abaa3b 
  ql/src/test/results/clientpositive/input_part2.q.out 2ce071c 
  ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/join17.q.out 39c49a5 
  ql/src/test/results/clientpositive/join26.q.out 0fdc403 
  ql/src/test/results/clientpositive/join32.q.out aad5463 
  ql/src/test/results/clientpositive/join32_lessSize.q.out a94b951 
  ql/src/test/results/clientpositive/join33.q.out aad5463 
  ql/src/test/results/clientpositive/join34.q.out 74fac5c 
  ql/src/test/results/clientpositive/join35.q.out 6fe9cae 
  ql/src/test/results/clientpositive/join9.q.out f41d153 
  ql/src/test/results/clientpositive/join_map_ppr.q.out e720e65 
  ql/src/test/results/clientpositive/list_bucket_dml_14.q.out c063dc1 
  ql/src/test/results/clientpositive/list_bucket_dml_8.q.java1.7.out d40a693 
  ql/src/test/results/clientpositive/mapjoin_memcheck.q.out 8f5492f 
  ql/src/test/results/clientpositive/nullformat.q.out af91470 
  ql/src/test/results/clientpositive/orc_create.q.out 3013fda 
  ql/src/test/results/clientpositive/orc_llap.q.out 6fc73b7 
  ql/src/test/results/clientpositive/orc_predicate_pushdown.q.out 7b361b7 
  ql/src/test/results/clientpositive/parquet_array_null_element.q.out 387f01e 
  ql/src/test/results/clientpositive/parquet_create.q.out c6d33ff 
  ql/src/test/results/clientpositive/parquet_mixed_partition_formats.q.out d6affd6 
  ql/src/test/results/clientpositive/parquet_serde.q.out c1e594a 
  ql/src/test/results/clientpositive/pcr.q.out b53226e 
  ql/src/test/results/clientpositive/rand_partitionpruner2.q.out d88c53d 
  ql/src/test/results/clientpositive/rcfile_default_format.q.out c961231 
  ql/src/test/results/clientpositive/reduce_deduplicate.q.out 379f884 
  ql/src/test/results/clientpositive/sample1.q.out e188902 
  ql/src/test/results/clientpositive/sample2.q.out 9653c41 
  ql/src/test/results/clientpositive/sample4.q.out a9730b1 
  ql/src/test/results/clientpositive/sample5.q.out d5ff403 
  ql/src/test/results/clientpositive/sample6.q.out ea71ad6 
  ql/src/test/results/clientpositive/sample7.q.out 94618e5 
  ql/src/test/results/clientpositive/schema_evol_orc_nonvec_fetchwork_table.q.out 912463b 
  ql/src/test/results/clientpositive/schema_evol_orc_nonvec_mapwork_table.q.out f007ba8 
  ql/src/test/results/clientpositive/schema_evol_orc_vec_mapwork_table.q.out a305847 
  ql/src/test/results/clientpositive/schema_evol_text_nonvec_mapwork_table.q.out 7a7e068 
  ql/src/test/results/clientpositive/schema_evol_text_vec_mapwork_table.q.out 50328d3 
  ql/src/test/results/clientpositive/schema_evol_text_vecrow_mapwork_table.q.out 50790f4 
  ql/src/test/results/clientpositive/show_create_table_alter.q.out 32819ea 
  ql/src/test/results/clientpositive/show_create_table_db_table.q.out 495f4b5 
  ql/src/test/results/clientpositive/show_create_table_serde.q.out 2350d98 
  ql/src/test/results/clientpositive/show_tblproperties.q.out 63bbe6d 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_1.q.out dfb571d 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_12.q.out 76ea0a8 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_3.q.out c942d11 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_4.q.out ba2547a 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_5.q.out 99fca25 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_7.q.out 00c601f 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_8.q.out 5564ceb 
  ql/src/test/results/clientpositive/spark/bucket2.q.out aa47459 
  ql/src/test/results/clientpositive/spark/bucket4.q.out 680cee7 
  ql/src/test/results/clientpositive/spark/bucket_map_join_1.q.out 299cedf 
  ql/src/test/results/clientpositive/spark/bucket_map_join_2.q.out f330bf2 
  ql/src/test/results/clientpositive/spark/bucket_map_join_spark1.q.out b32528c 
  ql/src/test/results/clientpositive/spark/bucket_map_join_spark2.q.out 7b304c5 
  ql/src/test/results/clientpositive/spark/bucket_map_join_spark3.q.out 4265d1a 
  ql/src/test/results/clientpositive/spark/bucketmapjoin1.q.out c6a7f0e 
  ql/src/test/results/clientpositive/spark/bucketmapjoin10.q.out 0e3b5c7 
  ql/src/test/results/clientpositive/spark/bucketmapjoin11.q.out 38f9785 
  ql/src/test/results/clientpositive/spark/bucketmapjoin12.q.out 55ded7d 
  ql/src/test/results/clientpositive/spark/bucketmapjoin2.q.out 48ff95f 
  ql/src/test/results/clientpositive/spark/bucketmapjoin3.q.out 95589ca 
  ql/src/test/results/clientpositive/spark/bucketmapjoin4.q.out fba617c 
  ql/src/test/results/clientpositive/spark/bucketmapjoin5.q.out 7214b42 
  ql/src/test/results/clientpositive/spark/bucketmapjoin7.q.out d344070 
  ql/src/test/results/clientpositive/spark/bucketmapjoin8.q.out a7ad6de 
  ql/src/test/results/clientpositive/spark/bucketmapjoin9.q.out 6e4c536 
  ql/src/test/results/clientpositive/spark/bucketmapjoin_negative.q.out 9fff7b2 
  ql/src/test/results/clientpositive/spark/bucketmapjoin_negative2.q.out b199ad9 
  ql/src/test/results/clientpositive/spark/bucketmapjoin_negative3.q.out 845e450 
  ql/src/test/results/clientpositive/spark/disable_merge_for_bucketing.q.out e21b1b1 
  ql/src/test/results/clientpositive/spark/groupby_sort_1_23.q.out 1b4822e 
  ql/src/test/results/clientpositive/spark/groupby_sort_skew_1_23.q.out 0956771 
  ql/src/test/results/clientpositive/spark/join17.q.out 0d92441 
  ql/src/test/results/clientpositive/spark/mapjoin_memcheck.q.out 2f41847 
  ql/src/test/results/clientpositive/spark/stats0.q.out e741320 
  ql/src/test/results/clientpositive/spark/stats20.q.out d7e52b4 
  ql/src/test/results/clientpositive/spark/stats3.q.out 4dbda94 
  ql/src/test/results/clientpositive/spark/vectorized_ptf.q.out 7b56585 
  ql/src/test/results/clientpositive/stats0.q.out 90a9a70 
  ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
  ql/src/test/results/clientpositive/stats11.q.out 8ca5b27 
  ql/src/test/results/clientpositive/stats18.q.out 6971e44 
  ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
  ql/src/test/results/clientpositive/stats3.q.out 4dbda94 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_1.q.out 892539d 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_11.q.out 954fd96 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_12.q.out 578880d 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_2.q.out c283738 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_3.q.out 351fd20 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_4.q.out 2d9cdf8 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_5.q.out 8bc203a 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_7.q.out 18fc95c 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_8.q.out 9a1010c 
  ql/src/test/results/clientpositive/tez/bucket2.q.out e9432a8 
  ql/src/test/results/clientpositive/tez/bucket4.q.out a0f1177 
  ql/src/test/results/clientpositive/tez/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
  ql/src/test/results/clientpositive/tez/disable_merge_for_bucketing.q.out bb62e1f 
  ql/src/test/results/clientpositive/tez/explainuser_1.q.out bdb8830 
  ql/src/test/results/clientpositive/tez/sample1.q.out 009969e 
  ql/src/test/results/clientpositive/tez/schema_evol_orc_nonvec_fetchwork_table.q.out 8a9481f 
  ql/src/test/results/clientpositive/tez/schema_evol_orc_nonvec_mapwork_table.q.out 39954b8 
  ql/src/test/results/clientpositive/tez/schema_evol_orc_vec_mapwork_table.q.out ca292b1 
  ql/src/test/results/clientpositive/tez/schema_evol_text_nonvec_mapwork_table.q.out 14abca9 
  ql/src/test/results/clientpositive/tez/schema_evol_text_vec_mapwork_table.q.out 2b799f9 
  ql/src/test/results/clientpositive/tez/schema_evol_text_vecrow_mapwork_table.q.out 1b46fa7 
  ql/src/test/results/clientpositive/tez/tez_join_result_complex.q.out 3e3b08c 
  ql/src/test/results/clientpositive/tez/vectorized_ptf.q.out 1365626 
  ql/src/test/results/clientpositive/truncate_column.q.out adbddfa 
  ql/src/test/results/clientpositive/truncate_column_list_bucket.q.out 39ffc6c 
  ql/src/test/results/clientpositive/unicode_notation.q.out 52da674 
  ql/src/test/results/clientpositive/unset_table_view_property.q.out 8bbb9fe 
  ql/src/test/results/clientpositive/vectorized_ptf.q.out fc4351d 

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


Testing
-------


Thanks,

pengcheng xiong


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

Posted by pengcheng xiong <px...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45911/
-----------------------------------------------------------

(Updated May 6, 2016, 9:29 p.m.)


Review request for hive and Ashutosh Chauhan.


Changes
-------

more golden file updates.


Repository: hive-git


Description
-------

HIVE-13341: Stats state is not captured correctly: differentiate load table and create table


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
  data/scripts/q_test_cleanup.sql 4620dcd 
  data/scripts/q_test_init.sql 0c8668c 
  itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
  metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
  ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java c2c6c65 
  ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java f4a9772 
  ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java 4a6617f 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 500c7ed 
  ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 2dc4e11 
  ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java 667d5c2 
  ql/src/test/queries/clientpositive/alter_table_add_partition.q PRE-CREATION 
  ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
  ql/src/test/queries/clientpositive/stats20.q 59701bd 
  ql/src/test/results/clientnegative/alter_file_format.q.out 96f1bfb 
  ql/src/test/results/clientnegative/unset_table_property.q.out 0510788 
  ql/src/test/results/clientpositive/alter_file_format.q.out 5d83b23 
  ql/src/test/results/clientpositive/alter_skewed_table.q.out a1caa99 
  ql/src/test/results/clientpositive/alter_table_add_partition.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/alter_table_not_sorted.q.out 6e1ec59 
  ql/src/test/results/clientpositive/auto_sortmerge_join_1.q.out 0902556 
  ql/src/test/results/clientpositive/auto_sortmerge_join_11.q.out 81de2b0 
  ql/src/test/results/clientpositive/auto_sortmerge_join_12.q.out 26a11a7 
  ql/src/test/results/clientpositive/auto_sortmerge_join_2.q.out 78cd03d 
  ql/src/test/results/clientpositive/auto_sortmerge_join_3.q.out 7ddf6b3 
  ql/src/test/results/clientpositive/auto_sortmerge_join_4.q.out 5b68da2 
  ql/src/test/results/clientpositive/auto_sortmerge_join_5.q.out 07bcb89 
  ql/src/test/results/clientpositive/auto_sortmerge_join_7.q.out d12c1ca 
  ql/src/test/results/clientpositive/auto_sortmerge_join_8.q.out 23a3685 
  ql/src/test/results/clientpositive/binary_output_format.q.out 608d475 
  ql/src/test/results/clientpositive/bucket1.q.out 96b8d4e 
  ql/src/test/results/clientpositive/bucket2.q.out b5572fd 
  ql/src/test/results/clientpositive/bucket4.q.out c4baf72 
  ql/src/test/results/clientpositive/bucket5.q.out dee79f9 
  ql/src/test/results/clientpositive/bucket_many.q.out 90d9fb8 
  ql/src/test/results/clientpositive/bucket_map_join_1.q.out 49bf2ca 
  ql/src/test/results/clientpositive/bucket_map_join_2.q.out 9ec7033 
  ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out 19937cb 
  ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out 90528be 
  ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out aede979 
  ql/src/test/results/clientpositive/bucketcontext_1.q.out d1b7744 
  ql/src/test/results/clientpositive/bucketcontext_2.q.out 60c021c 
  ql/src/test/results/clientpositive/bucketcontext_3.q.out 12eaddf 
  ql/src/test/results/clientpositive/bucketcontext_4.q.out 2573564 
  ql/src/test/results/clientpositive/bucketcontext_5.q.out e1c6a80 
  ql/src/test/results/clientpositive/bucketcontext_6.q.out 9deb64d 
  ql/src/test/results/clientpositive/bucketcontext_7.q.out 621844e 
  ql/src/test/results/clientpositive/bucketcontext_8.q.out 7f3e50a 
  ql/src/test/results/clientpositive/bucketmapjoin1.q.out 9956fa7 
  ql/src/test/results/clientpositive/bucketmapjoin10.q.out fcdbb09 
  ql/src/test/results/clientpositive/bucketmapjoin11.q.out 968066a 
  ql/src/test/results/clientpositive/bucketmapjoin12.q.out 9017871 
  ql/src/test/results/clientpositive/bucketmapjoin2.q.out f2efb95 
  ql/src/test/results/clientpositive/bucketmapjoin3.q.out 5659983 
  ql/src/test/results/clientpositive/bucketmapjoin4.q.out 5794d47 
  ql/src/test/results/clientpositive/bucketmapjoin5.q.out 1bcb382 
  ql/src/test/results/clientpositive/bucketmapjoin7.q.out 76b13f3 
  ql/src/test/results/clientpositive/bucketmapjoin8.q.out 097eaad 
  ql/src/test/results/clientpositive/bucketmapjoin9.q.out 144d86d 
  ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out d108575 
  ql/src/test/results/clientpositive/bucketmapjoin_negative2.q.out 9c6a226 
  ql/src/test/results/clientpositive/bucketmapjoin_negative3.q.out 3078854 
  ql/src/test/results/clientpositive/columnStatsUpdateForStatsOptimizer_1.q.out 8c9664d 
  ql/src/test/results/clientpositive/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
  ql/src/test/results/clientpositive/columnstats_partlvl.q.out 4edf39a 
  ql/src/test/results/clientpositive/columnstats_tbllvl.q.out 669807d 
  ql/src/test/results/clientpositive/create_alter_list_bucketing_table1.q.out c4f51a3 
  ql/src/test/results/clientpositive/create_like.q.out 9241b68 
  ql/src/test/results/clientpositive/create_like_view.q.out e2dc2c4 
  ql/src/test/results/clientpositive/create_skewed_table1.q.out 415bb77 
  ql/src/test/results/clientpositive/database_location.q.out 797177d 
  ql/src/test/results/clientpositive/default_file_format.q.out 3d5c20f 
  ql/src/test/results/clientpositive/describe_comment_indent.q.out 3e0f45e 
  ql/src/test/results/clientpositive/describe_comment_nonascii.q.out df0b65e 
  ql/src/test/results/clientpositive/disable_merge_for_bucketing.q.out c4dbf17 
  ql/src/test/results/clientpositive/display_colstats_tbllvl.q.out 1028c26 
  ql/src/test/results/clientpositive/groupby_map_ppr.q.out 3d07b0d 
  ql/src/test/results/clientpositive/groupby_map_ppr_multi_distinct.q.out 6689225 
  ql/src/test/results/clientpositive/groupby_ppr.q.out 62fffbe 
  ql/src/test/results/clientpositive/groupby_ppr_multi_distinct.q.out 2408056 
  ql/src/test/results/clientpositive/groupby_sort_1_23.q.out a6e15ba 
  ql/src/test/results/clientpositive/groupby_sort_6.q.out f523085 
  ql/src/test/results/clientpositive/groupby_sort_skew_1_23.q.out 38b24a4 
  ql/src/test/results/clientpositive/input_part1.q.out 2abaa3b 
  ql/src/test/results/clientpositive/input_part2.q.out 2ce071c 
  ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/join17.q.out 39c49a5 
  ql/src/test/results/clientpositive/join26.q.out 0fdc403 
  ql/src/test/results/clientpositive/join32.q.out aad5463 
  ql/src/test/results/clientpositive/join32_lessSize.q.out a94b951 
  ql/src/test/results/clientpositive/join33.q.out aad5463 
  ql/src/test/results/clientpositive/join34.q.out 74fac5c 
  ql/src/test/results/clientpositive/join35.q.out 6fe9cae 
  ql/src/test/results/clientpositive/join9.q.out f41d153 
  ql/src/test/results/clientpositive/join_map_ppr.q.out e720e65 
  ql/src/test/results/clientpositive/list_bucket_dml_14.q.out c063dc1 
  ql/src/test/results/clientpositive/list_bucket_dml_8.q.java1.7.out d40a693 
  ql/src/test/results/clientpositive/mapjoin_memcheck.q.out 8f5492f 
  ql/src/test/results/clientpositive/nullformat.q.out af91470 
  ql/src/test/results/clientpositive/orc_create.q.out 3013fda 
  ql/src/test/results/clientpositive/orc_llap.q.out 6fc73b7 
  ql/src/test/results/clientpositive/orc_predicate_pushdown.q.out 7b361b7 
  ql/src/test/results/clientpositive/parquet_array_null_element.q.out 387f01e 
  ql/src/test/results/clientpositive/parquet_create.q.out c6d33ff 
  ql/src/test/results/clientpositive/parquet_mixed_partition_formats.q.out d6affd6 
  ql/src/test/results/clientpositive/parquet_serde.q.out c1e594a 
  ql/src/test/results/clientpositive/pcr.q.out b53226e 
  ql/src/test/results/clientpositive/rand_partitionpruner2.q.out d88c53d 
  ql/src/test/results/clientpositive/rcfile_default_format.q.out c961231 
  ql/src/test/results/clientpositive/reduce_deduplicate.q.out 379f884 
  ql/src/test/results/clientpositive/sample1.q.out e188902 
  ql/src/test/results/clientpositive/sample2.q.out 9653c41 
  ql/src/test/results/clientpositive/sample4.q.out a9730b1 
  ql/src/test/results/clientpositive/sample5.q.out d5ff403 
  ql/src/test/results/clientpositive/sample6.q.out ea71ad6 
  ql/src/test/results/clientpositive/sample7.q.out 94618e5 
  ql/src/test/results/clientpositive/schema_evol_orc_nonvec_fetchwork_table.q.out 912463b 
  ql/src/test/results/clientpositive/schema_evol_orc_nonvec_mapwork_table.q.out f007ba8 
  ql/src/test/results/clientpositive/schema_evol_orc_vec_mapwork_table.q.out a305847 
  ql/src/test/results/clientpositive/schema_evol_text_nonvec_mapwork_table.q.out 7a7e068 
  ql/src/test/results/clientpositive/schema_evol_text_vec_mapwork_table.q.out 50328d3 
  ql/src/test/results/clientpositive/schema_evol_text_vecrow_mapwork_table.q.out 50790f4 
  ql/src/test/results/clientpositive/show_create_table_alter.q.out 32819ea 
  ql/src/test/results/clientpositive/show_create_table_db_table.q.out 495f4b5 
  ql/src/test/results/clientpositive/show_create_table_serde.q.out 2350d98 
  ql/src/test/results/clientpositive/show_tblproperties.q.out 63bbe6d 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_1.q.out dfb571d 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_12.q.out 76ea0a8 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_3.q.out c942d11 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_4.q.out ba2547a 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_5.q.out 99fca25 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_7.q.out 00c601f 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_8.q.out 5564ceb 
  ql/src/test/results/clientpositive/spark/bucket2.q.out aa47459 
  ql/src/test/results/clientpositive/spark/bucket4.q.out 680cee7 
  ql/src/test/results/clientpositive/spark/bucket_map_join_1.q.out 299cedf 
  ql/src/test/results/clientpositive/spark/bucket_map_join_2.q.out f330bf2 
  ql/src/test/results/clientpositive/spark/bucket_map_join_spark1.q.out b32528c 
  ql/src/test/results/clientpositive/spark/bucket_map_join_spark2.q.out 7b304c5 
  ql/src/test/results/clientpositive/spark/bucket_map_join_spark3.q.out 4265d1a 
  ql/src/test/results/clientpositive/spark/bucketmapjoin1.q.out c6a7f0e 
  ql/src/test/results/clientpositive/spark/bucketmapjoin10.q.out 0e3b5c7 
  ql/src/test/results/clientpositive/spark/bucketmapjoin11.q.out 38f9785 
  ql/src/test/results/clientpositive/spark/bucketmapjoin12.q.out 55ded7d 
  ql/src/test/results/clientpositive/spark/bucketmapjoin2.q.out 48ff95f 
  ql/src/test/results/clientpositive/spark/bucketmapjoin3.q.out 95589ca 
  ql/src/test/results/clientpositive/spark/bucketmapjoin4.q.out fba617c 
  ql/src/test/results/clientpositive/spark/bucketmapjoin5.q.out 7214b42 
  ql/src/test/results/clientpositive/spark/bucketmapjoin7.q.out d344070 
  ql/src/test/results/clientpositive/spark/bucketmapjoin8.q.out a7ad6de 
  ql/src/test/results/clientpositive/spark/bucketmapjoin9.q.out 6e4c536 
  ql/src/test/results/clientpositive/spark/bucketmapjoin_negative.q.out 9fff7b2 
  ql/src/test/results/clientpositive/spark/bucketmapjoin_negative2.q.out b199ad9 
  ql/src/test/results/clientpositive/spark/bucketmapjoin_negative3.q.out 845e450 
  ql/src/test/results/clientpositive/spark/disable_merge_for_bucketing.q.out e21b1b1 
  ql/src/test/results/clientpositive/spark/groupby_sort_1_23.q.out 1b4822e 
  ql/src/test/results/clientpositive/spark/groupby_sort_skew_1_23.q.out 0956771 
  ql/src/test/results/clientpositive/spark/join17.q.out 0d92441 
  ql/src/test/results/clientpositive/spark/mapjoin_memcheck.q.out 2f41847 
  ql/src/test/results/clientpositive/spark/stats0.q.out e741320 
  ql/src/test/results/clientpositive/spark/stats20.q.out d7e52b4 
  ql/src/test/results/clientpositive/spark/stats3.q.out 4dbda94 
  ql/src/test/results/clientpositive/spark/vectorized_ptf.q.out 7b56585 
  ql/src/test/results/clientpositive/stats0.q.out 90a9a70 
  ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
  ql/src/test/results/clientpositive/stats11.q.out 8ca5b27 
  ql/src/test/results/clientpositive/stats18.q.out 6971e44 
  ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
  ql/src/test/results/clientpositive/stats3.q.out 4dbda94 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_1.q.out 892539d 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_11.q.out 954fd96 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_12.q.out 578880d 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_2.q.out c283738 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_3.q.out 351fd20 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_4.q.out 2d9cdf8 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_5.q.out 8bc203a 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_7.q.out 18fc95c 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_8.q.out 9a1010c 
  ql/src/test/results/clientpositive/tez/bucket2.q.out e9432a8 
  ql/src/test/results/clientpositive/tez/bucket4.q.out a0f1177 
  ql/src/test/results/clientpositive/tez/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
  ql/src/test/results/clientpositive/tez/disable_merge_for_bucketing.q.out bb62e1f 
  ql/src/test/results/clientpositive/tez/explainuser_1.q.out bdb8830 
  ql/src/test/results/clientpositive/tez/sample1.q.out 009969e 
  ql/src/test/results/clientpositive/tez/schema_evol_orc_nonvec_fetchwork_table.q.out 8a9481f 
  ql/src/test/results/clientpositive/tez/schema_evol_orc_nonvec_mapwork_table.q.out 39954b8 
  ql/src/test/results/clientpositive/tez/schema_evol_orc_vec_mapwork_table.q.out ca292b1 
  ql/src/test/results/clientpositive/tez/schema_evol_text_nonvec_mapwork_table.q.out 14abca9 
  ql/src/test/results/clientpositive/tez/schema_evol_text_vec_mapwork_table.q.out 2b799f9 
  ql/src/test/results/clientpositive/tez/schema_evol_text_vecrow_mapwork_table.q.out 1b46fa7 
  ql/src/test/results/clientpositive/tez/tez_join_result_complex.q.out 3e3b08c 
  ql/src/test/results/clientpositive/tez/vectorized_ptf.q.out 1365626 
  ql/src/test/results/clientpositive/truncate_column.q.out adbddfa 
  ql/src/test/results/clientpositive/truncate_column_list_bucket.q.out 39ffc6c 
  ql/src/test/results/clientpositive/unicode_notation.q.out 52da674 
  ql/src/test/results/clientpositive/unset_table_view_property.q.out 8bbb9fe 
  ql/src/test/results/clientpositive/vectorized_ptf.q.out fc4351d 

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


Testing
-------


Thanks,

pengcheng xiong


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

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



Patch looks good. Checked golden files, looks good.


ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java (line 815)
<https://reviews.apache.org/r/45911/#comment195696>

    We need similiar logic for alter Table add partition.


- Ashutosh Chauhan


On May 3, 2016, 8:40 p.m., pengcheng xiong wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45911/
> -----------------------------------------------------------
> 
> (Updated May 3, 2016, 8:40 p.m.)
> 
> 
> Review request for hive and Ashutosh Chauhan.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-13341: Stats state is not captured correctly: differentiate load table and create table
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
>   itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
>   metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java 54592cc 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
>   ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 4d9c3d2 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 500c7ed 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 2dc4e11 
>   ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java 667d5c2 
>   ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/stats20.q 59701bd 
>   ql/src/test/results/clientpositive/alter_file_format.q.out 5d83b23 
>   ql/src/test/results/clientpositive/alter_skewed_table.q.out a1caa99 
>   ql/src/test/results/clientpositive/alter_table_not_sorted.q.out 6e1ec59 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_1.q.out 0902556 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_11.q.out 81de2b0 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_12.q.out 26a11a7 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_2.q.out 78cd03d 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_3.q.out 7ddf6b3 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_4.q.out 5b68da2 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_5.q.out 07bcb89 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_7.q.out d12c1ca 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_8.q.out 23a3685 
>   ql/src/test/results/clientpositive/binary_output_format.q.out 608d475 
>   ql/src/test/results/clientpositive/bucket1.q.out 96b8d4e 
>   ql/src/test/results/clientpositive/bucket2.q.out b5572fd 
>   ql/src/test/results/clientpositive/bucket4.q.out c4baf72 
>   ql/src/test/results/clientpositive/bucket5.q.out dee79f9 
>   ql/src/test/results/clientpositive/bucket_many.q.out 90d9fb8 
>   ql/src/test/results/clientpositive/bucket_map_join_1.q.out 49bf2ca 
>   ql/src/test/results/clientpositive/bucket_map_join_2.q.out 9ec7033 
>   ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out 19937cb 
>   ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out 90528be 
>   ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out aede979 
>   ql/src/test/results/clientpositive/bucketcontext_1.q.out d1b7744 
>   ql/src/test/results/clientpositive/bucketcontext_2.q.out 60c021c 
>   ql/src/test/results/clientpositive/bucketcontext_3.q.out 12eaddf 
>   ql/src/test/results/clientpositive/bucketcontext_4.q.out 2573564 
>   ql/src/test/results/clientpositive/bucketcontext_5.q.out e1c6a80 
>   ql/src/test/results/clientpositive/bucketcontext_6.q.out 9deb64d 
>   ql/src/test/results/clientpositive/bucketcontext_7.q.out 621844e 
>   ql/src/test/results/clientpositive/bucketcontext_8.q.out 7f3e50a 
>   ql/src/test/results/clientpositive/bucketmapjoin1.q.out 9956fa7 
>   ql/src/test/results/clientpositive/bucketmapjoin10.q.out fcdbb09 
>   ql/src/test/results/clientpositive/bucketmapjoin11.q.out 968066a 
>   ql/src/test/results/clientpositive/bucketmapjoin12.q.out 9017871 
>   ql/src/test/results/clientpositive/bucketmapjoin2.q.out f2efb95 
>   ql/src/test/results/clientpositive/bucketmapjoin3.q.out 5659983 
>   ql/src/test/results/clientpositive/bucketmapjoin4.q.out 5794d47 
>   ql/src/test/results/clientpositive/bucketmapjoin5.q.out 1bcb382 
>   ql/src/test/results/clientpositive/bucketmapjoin7.q.out 76b13f3 
>   ql/src/test/results/clientpositive/bucketmapjoin8.q.out 097eaad 
>   ql/src/test/results/clientpositive/bucketmapjoin9.q.out 144d86d 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out d108575 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative2.q.out 9c6a226 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative3.q.out 3078854 
>   ql/src/test/results/clientpositive/columnStatsUpdateForStatsOptimizer_1.q.out 8c9664d 
>   ql/src/test/results/clientpositive/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
>   ql/src/test/results/clientpositive/columnstats_partlvl.q.out 4edf39a 
>   ql/src/test/results/clientpositive/columnstats_tbllvl.q.out 669807d 
>   ql/src/test/results/clientpositive/create_alter_list_bucketing_table1.q.out c4f51a3 
>   ql/src/test/results/clientpositive/create_like.q.out 9241b68 
>   ql/src/test/results/clientpositive/create_like_view.q.out e2dc2c4 
>   ql/src/test/results/clientpositive/create_skewed_table1.q.out 415bb77 
>   ql/src/test/results/clientpositive/database_location.q.out 797177d 
>   ql/src/test/results/clientpositive/default_file_format.q.out 3d5c20f 
>   ql/src/test/results/clientpositive/describe_comment_indent.q.out 3e0f45e 
>   ql/src/test/results/clientpositive/describe_comment_nonascii.q.out df0b65e 
>   ql/src/test/results/clientpositive/disable_merge_for_bucketing.q.out c4dbf17 
>   ql/src/test/results/clientpositive/display_colstats_tbllvl.q.out 1028c26 
>   ql/src/test/results/clientpositive/groupby_map_ppr.q.out 3d07b0d 
>   ql/src/test/results/clientpositive/groupby_map_ppr_multi_distinct.q.out 6689225 
>   ql/src/test/results/clientpositive/groupby_ppr.q.out 62fffbe 
>   ql/src/test/results/clientpositive/groupby_ppr_multi_distinct.q.out 2408056 
>   ql/src/test/results/clientpositive/groupby_sort_1_23.q.out a6e15ba 
>   ql/src/test/results/clientpositive/groupby_sort_6.q.out f523085 
>   ql/src/test/results/clientpositive/groupby_sort_skew_1_23.q.out 38b24a4 
>   ql/src/test/results/clientpositive/input_part1.q.out 2abaa3b 
>   ql/src/test/results/clientpositive/input_part2.q.out 2ce071c 
>   ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/join17.q.out 39c49a5 
>   ql/src/test/results/clientpositive/join26.q.out 0fdc403 
>   ql/src/test/results/clientpositive/join32.q.out aad5463 
>   ql/src/test/results/clientpositive/join32_lessSize.q.out a94b951 
>   ql/src/test/results/clientpositive/join33.q.out aad5463 
>   ql/src/test/results/clientpositive/join34.q.out 74fac5c 
>   ql/src/test/results/clientpositive/join35.q.out 6fe9cae 
>   ql/src/test/results/clientpositive/join9.q.out f41d153 
>   ql/src/test/results/clientpositive/join_map_ppr.q.out e720e65 
>   ql/src/test/results/clientpositive/list_bucket_dml_14.q.out c063dc1 
>   ql/src/test/results/clientpositive/list_bucket_dml_8.q.java1.7.out d40a693 
>   ql/src/test/results/clientpositive/mapjoin_memcheck.q.out 8f5492f 
>   ql/src/test/results/clientpositive/nullformat.q.out af91470 
>   ql/src/test/results/clientpositive/orc_create.q.out 3013fda 
>   ql/src/test/results/clientpositive/orc_llap.q.out 6fc73b7 
>   ql/src/test/results/clientpositive/orc_predicate_pushdown.q.out 7b361b7 
>   ql/src/test/results/clientpositive/parquet_array_null_element.q.out 387f01e 
>   ql/src/test/results/clientpositive/parquet_create.q.out c6d33ff 
>   ql/src/test/results/clientpositive/parquet_mixed_partition_formats.q.out d6affd6 
>   ql/src/test/results/clientpositive/parquet_serde.q.out c1e594a 
>   ql/src/test/results/clientpositive/pcr.q.out b53226e 
>   ql/src/test/results/clientpositive/rand_partitionpruner2.q.out d88c53d 
>   ql/src/test/results/clientpositive/rcfile_default_format.q.out c961231 
>   ql/src/test/results/clientpositive/reduce_deduplicate.q.out 379f884 
>   ql/src/test/results/clientpositive/sample1.q.out e188902 
>   ql/src/test/results/clientpositive/sample2.q.out 9653c41 
>   ql/src/test/results/clientpositive/sample4.q.out a9730b1 
>   ql/src/test/results/clientpositive/sample5.q.out d5ff403 
>   ql/src/test/results/clientpositive/sample6.q.out ea71ad6 
>   ql/src/test/results/clientpositive/sample7.q.out 94618e5 
>   ql/src/test/results/clientpositive/show_create_table_alter.q.out 32819ea 
>   ql/src/test/results/clientpositive/show_create_table_db_table.q.out 495f4b5 
>   ql/src/test/results/clientpositive/show_create_table_serde.q.out 2350d98 
>   ql/src/test/results/clientpositive/show_tblproperties.q.out 63bbe6d 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_12.q.out 76ea0a8 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_8.q.out 5564ceb 
>   ql/src/test/results/clientpositive/spark/stats20.q.out d7e52b4 
>   ql/src/test/results/clientpositive/stats0.q.out 90a9a70 
>   ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
>   ql/src/test/results/clientpositive/stats11.q.out 8ca5b27 
>   ql/src/test/results/clientpositive/stats18.q.out 6971e44 
>   ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
>   ql/src/test/results/clientpositive/stats3.q.out 4dbda94 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_1.q.out 892539d 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_11.q.out 954fd96 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_12.q.out 578880d 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_2.q.out c283738 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_3.q.out 351fd20 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_4.q.out 2d9cdf8 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_5.q.out 8bc203a 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_7.q.out 18fc95c 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_8.q.out 9a1010c 
>   ql/src/test/results/clientpositive/tez/bucket2.q.out e9432a8 
>   ql/src/test/results/clientpositive/tez/bucket4.q.out a0f1177 
>   ql/src/test/results/clientpositive/tez/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
>   ql/src/test/results/clientpositive/tez/disable_merge_for_bucketing.q.out bb62e1f 
>   ql/src/test/results/clientpositive/tez/explainuser_1.q.out bdb8830 
>   ql/src/test/results/clientpositive/tez/sample1.q.out 009969e 
>   ql/src/test/results/clientpositive/tez/tez_join_result_complex.q.out 3e3b08c 
>   ql/src/test/results/clientpositive/tez/vectorized_ptf.q.out 1365626 
>   ql/src/test/results/clientpositive/truncate_column.q.out adbddfa 
>   ql/src/test/results/clientpositive/truncate_column_list_bucket.q.out 39ffc6c 
>   ql/src/test/results/clientpositive/unicode_notation.q.out 52da674 
>   ql/src/test/results/clientpositive/unset_table_view_property.q.out 8bbb9fe 
>   ql/src/test/results/clientpositive/vectorized_ptf.q.out fc4351d 
> 
> Diff: https://reviews.apache.org/r/45911/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> pengcheng xiong
> 
>


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

Posted by pengcheng xiong <px...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45911/
-----------------------------------------------------------

(Updated May 3, 2016, 8:40 p.m.)


Review request for hive and Ashutosh Chauhan.


Repository: hive-git


Description
-------

HIVE-13341: Stats state is not captured correctly: differentiate load table and create table


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
  itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
  metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
  ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java 54592cc 
  ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 4d9c3d2 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 500c7ed 
  ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 2dc4e11 
  ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java 667d5c2 
  ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
  ql/src/test/queries/clientpositive/stats20.q 59701bd 
  ql/src/test/results/clientpositive/alter_file_format.q.out 5d83b23 
  ql/src/test/results/clientpositive/alter_skewed_table.q.out a1caa99 
  ql/src/test/results/clientpositive/alter_table_not_sorted.q.out 6e1ec59 
  ql/src/test/results/clientpositive/auto_sortmerge_join_1.q.out 0902556 
  ql/src/test/results/clientpositive/auto_sortmerge_join_11.q.out 81de2b0 
  ql/src/test/results/clientpositive/auto_sortmerge_join_12.q.out 26a11a7 
  ql/src/test/results/clientpositive/auto_sortmerge_join_2.q.out 78cd03d 
  ql/src/test/results/clientpositive/auto_sortmerge_join_3.q.out 7ddf6b3 
  ql/src/test/results/clientpositive/auto_sortmerge_join_4.q.out 5b68da2 
  ql/src/test/results/clientpositive/auto_sortmerge_join_5.q.out 07bcb89 
  ql/src/test/results/clientpositive/auto_sortmerge_join_7.q.out d12c1ca 
  ql/src/test/results/clientpositive/auto_sortmerge_join_8.q.out 23a3685 
  ql/src/test/results/clientpositive/binary_output_format.q.out 608d475 
  ql/src/test/results/clientpositive/bucket1.q.out 96b8d4e 
  ql/src/test/results/clientpositive/bucket2.q.out b5572fd 
  ql/src/test/results/clientpositive/bucket4.q.out c4baf72 
  ql/src/test/results/clientpositive/bucket5.q.out dee79f9 
  ql/src/test/results/clientpositive/bucket_many.q.out 90d9fb8 
  ql/src/test/results/clientpositive/bucket_map_join_1.q.out 49bf2ca 
  ql/src/test/results/clientpositive/bucket_map_join_2.q.out 9ec7033 
  ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out 19937cb 
  ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out 90528be 
  ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out aede979 
  ql/src/test/results/clientpositive/bucketcontext_1.q.out d1b7744 
  ql/src/test/results/clientpositive/bucketcontext_2.q.out 60c021c 
  ql/src/test/results/clientpositive/bucketcontext_3.q.out 12eaddf 
  ql/src/test/results/clientpositive/bucketcontext_4.q.out 2573564 
  ql/src/test/results/clientpositive/bucketcontext_5.q.out e1c6a80 
  ql/src/test/results/clientpositive/bucketcontext_6.q.out 9deb64d 
  ql/src/test/results/clientpositive/bucketcontext_7.q.out 621844e 
  ql/src/test/results/clientpositive/bucketcontext_8.q.out 7f3e50a 
  ql/src/test/results/clientpositive/bucketmapjoin1.q.out 9956fa7 
  ql/src/test/results/clientpositive/bucketmapjoin10.q.out fcdbb09 
  ql/src/test/results/clientpositive/bucketmapjoin11.q.out 968066a 
  ql/src/test/results/clientpositive/bucketmapjoin12.q.out 9017871 
  ql/src/test/results/clientpositive/bucketmapjoin2.q.out f2efb95 
  ql/src/test/results/clientpositive/bucketmapjoin3.q.out 5659983 
  ql/src/test/results/clientpositive/bucketmapjoin4.q.out 5794d47 
  ql/src/test/results/clientpositive/bucketmapjoin5.q.out 1bcb382 
  ql/src/test/results/clientpositive/bucketmapjoin7.q.out 76b13f3 
  ql/src/test/results/clientpositive/bucketmapjoin8.q.out 097eaad 
  ql/src/test/results/clientpositive/bucketmapjoin9.q.out 144d86d 
  ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out d108575 
  ql/src/test/results/clientpositive/bucketmapjoin_negative2.q.out 9c6a226 
  ql/src/test/results/clientpositive/bucketmapjoin_negative3.q.out 3078854 
  ql/src/test/results/clientpositive/columnStatsUpdateForStatsOptimizer_1.q.out 8c9664d 
  ql/src/test/results/clientpositive/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
  ql/src/test/results/clientpositive/columnstats_partlvl.q.out 4edf39a 
  ql/src/test/results/clientpositive/columnstats_tbllvl.q.out 669807d 
  ql/src/test/results/clientpositive/create_alter_list_bucketing_table1.q.out c4f51a3 
  ql/src/test/results/clientpositive/create_like.q.out 9241b68 
  ql/src/test/results/clientpositive/create_like_view.q.out e2dc2c4 
  ql/src/test/results/clientpositive/create_skewed_table1.q.out 415bb77 
  ql/src/test/results/clientpositive/database_location.q.out 797177d 
  ql/src/test/results/clientpositive/default_file_format.q.out 3d5c20f 
  ql/src/test/results/clientpositive/describe_comment_indent.q.out 3e0f45e 
  ql/src/test/results/clientpositive/describe_comment_nonascii.q.out df0b65e 
  ql/src/test/results/clientpositive/disable_merge_for_bucketing.q.out c4dbf17 
  ql/src/test/results/clientpositive/display_colstats_tbllvl.q.out 1028c26 
  ql/src/test/results/clientpositive/groupby_map_ppr.q.out 3d07b0d 
  ql/src/test/results/clientpositive/groupby_map_ppr_multi_distinct.q.out 6689225 
  ql/src/test/results/clientpositive/groupby_ppr.q.out 62fffbe 
  ql/src/test/results/clientpositive/groupby_ppr_multi_distinct.q.out 2408056 
  ql/src/test/results/clientpositive/groupby_sort_1_23.q.out a6e15ba 
  ql/src/test/results/clientpositive/groupby_sort_6.q.out f523085 
  ql/src/test/results/clientpositive/groupby_sort_skew_1_23.q.out 38b24a4 
  ql/src/test/results/clientpositive/input_part1.q.out 2abaa3b 
  ql/src/test/results/clientpositive/input_part2.q.out 2ce071c 
  ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/join17.q.out 39c49a5 
  ql/src/test/results/clientpositive/join26.q.out 0fdc403 
  ql/src/test/results/clientpositive/join32.q.out aad5463 
  ql/src/test/results/clientpositive/join32_lessSize.q.out a94b951 
  ql/src/test/results/clientpositive/join33.q.out aad5463 
  ql/src/test/results/clientpositive/join34.q.out 74fac5c 
  ql/src/test/results/clientpositive/join35.q.out 6fe9cae 
  ql/src/test/results/clientpositive/join9.q.out f41d153 
  ql/src/test/results/clientpositive/join_map_ppr.q.out e720e65 
  ql/src/test/results/clientpositive/list_bucket_dml_14.q.out c063dc1 
  ql/src/test/results/clientpositive/list_bucket_dml_8.q.java1.7.out d40a693 
  ql/src/test/results/clientpositive/mapjoin_memcheck.q.out 8f5492f 
  ql/src/test/results/clientpositive/nullformat.q.out af91470 
  ql/src/test/results/clientpositive/orc_create.q.out 3013fda 
  ql/src/test/results/clientpositive/orc_llap.q.out 6fc73b7 
  ql/src/test/results/clientpositive/orc_predicate_pushdown.q.out 7b361b7 
  ql/src/test/results/clientpositive/parquet_array_null_element.q.out 387f01e 
  ql/src/test/results/clientpositive/parquet_create.q.out c6d33ff 
  ql/src/test/results/clientpositive/parquet_mixed_partition_formats.q.out d6affd6 
  ql/src/test/results/clientpositive/parquet_serde.q.out c1e594a 
  ql/src/test/results/clientpositive/pcr.q.out b53226e 
  ql/src/test/results/clientpositive/rand_partitionpruner2.q.out d88c53d 
  ql/src/test/results/clientpositive/rcfile_default_format.q.out c961231 
  ql/src/test/results/clientpositive/reduce_deduplicate.q.out 379f884 
  ql/src/test/results/clientpositive/sample1.q.out e188902 
  ql/src/test/results/clientpositive/sample2.q.out 9653c41 
  ql/src/test/results/clientpositive/sample4.q.out a9730b1 
  ql/src/test/results/clientpositive/sample5.q.out d5ff403 
  ql/src/test/results/clientpositive/sample6.q.out ea71ad6 
  ql/src/test/results/clientpositive/sample7.q.out 94618e5 
  ql/src/test/results/clientpositive/show_create_table_alter.q.out 32819ea 
  ql/src/test/results/clientpositive/show_create_table_db_table.q.out 495f4b5 
  ql/src/test/results/clientpositive/show_create_table_serde.q.out 2350d98 
  ql/src/test/results/clientpositive/show_tblproperties.q.out 63bbe6d 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_12.q.out 76ea0a8 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_8.q.out 5564ceb 
  ql/src/test/results/clientpositive/spark/stats20.q.out d7e52b4 
  ql/src/test/results/clientpositive/stats0.q.out 90a9a70 
  ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
  ql/src/test/results/clientpositive/stats11.q.out 8ca5b27 
  ql/src/test/results/clientpositive/stats18.q.out 6971e44 
  ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
  ql/src/test/results/clientpositive/stats3.q.out 4dbda94 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_1.q.out 892539d 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_11.q.out 954fd96 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_12.q.out 578880d 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_2.q.out c283738 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_3.q.out 351fd20 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_4.q.out 2d9cdf8 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_5.q.out 8bc203a 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_7.q.out 18fc95c 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_8.q.out 9a1010c 
  ql/src/test/results/clientpositive/tez/bucket2.q.out e9432a8 
  ql/src/test/results/clientpositive/tez/bucket4.q.out a0f1177 
  ql/src/test/results/clientpositive/tez/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
  ql/src/test/results/clientpositive/tez/disable_merge_for_bucketing.q.out bb62e1f 
  ql/src/test/results/clientpositive/tez/explainuser_1.q.out bdb8830 
  ql/src/test/results/clientpositive/tez/sample1.q.out 009969e 
  ql/src/test/results/clientpositive/tez/tez_join_result_complex.q.out 3e3b08c 
  ql/src/test/results/clientpositive/tez/vectorized_ptf.q.out 1365626 
  ql/src/test/results/clientpositive/truncate_column.q.out adbddfa 
  ql/src/test/results/clientpositive/truncate_column_list_bucket.q.out 39ffc6c 
  ql/src/test/results/clientpositive/unicode_notation.q.out 52da674 
  ql/src/test/results/clientpositive/unset_table_view_property.q.out 8bbb9fe 
  ql/src/test/results/clientpositive/vectorized_ptf.q.out fc4351d 

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


Testing
-------


Thanks,

pengcheng xiong


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

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




ql/src/test/results/clientnegative/stats_partialscan_autogether.q.out (lines 71 - 72)
<https://reviews.apache.org/r/45911/#comment194200>

    This looks incorrect.



ql/src/test/results/clientpositive/allcolref_in_udf.q.out (line 190)
<https://reviews.apache.org/r/45911/#comment194262>

    seems like basic stats were not retrieved in this case?



ql/src/test/results/clientpositive/auto_sortmerge_join_1.q.out (line 201)
<https://reviews.apache.org/r/45911/#comment194263>

    This doesn't look correct.



ql/src/test/results/clientpositive/auto_sortmerge_join_11.q.out (line 169)
<https://reviews.apache.org/r/45911/#comment194264>

    Doesn't look correct.



ql/src/test/results/clientpositive/bucket_map_join_1.q.out (line 186)
<https://reviews.apache.org/r/45911/#comment194265>

    Is this correct?



ql/src/test/results/clientpositive/bucketcontext_1.q.out (line 170)
<https://reviews.apache.org/r/45911/#comment194266>

    Is this correct?



ql/src/test/results/clientpositive/bucketmapjoin1.q.out (line 557)
<https://reviews.apache.org/r/45911/#comment194267>

    Is this correct?



ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out (line 168)
<https://reviews.apache.org/r/45911/#comment194268>

    Is this correct?



ql/src/test/results/clientpositive/cbo_SortUnionTransposeRule.q.out (line 35)
<https://reviews.apache.org/r/45911/#comment194269>

    seems like earlier stats were correct.



ql/src/test/results/clientpositive/cbo_rp_cross_product_check_2.q.out (line 56)
<https://reviews.apache.org/r/45911/#comment194343>

    Incorrect basic stats?



ql/src/test/results/clientpositive/columnStatsUpdateForStatsOptimizer_1.q.out (line 912)
<https://reviews.apache.org/r/45911/#comment194344>

    Incorrect basic stats ?



ql/src/test/results/clientpositive/columnstats_partlvl.q.out (line 152)
<https://reviews.apache.org/r/45911/#comment194345>

    Incorrect stats ?



ql/src/test/results/clientpositive/columnstats_tbllvl.q.out (line 146)
<https://reviews.apache.org/r/45911/#comment194346>

    Incorrect stats?



ql/src/test/results/clientpositive/cross_product_check_1.q.out (line 49)
<https://reviews.apache.org/r/45911/#comment194347>

    Incorrect basic stats?



ql/src/test/results/clientpositive/display_colstats_tbllvl.q.out (line 162)
<https://reviews.apache.org/r/45911/#comment194348>

    Incorrect basic stats?



ql/src/test/results/clientpositive/explain_ddl.q.out (line 162)
<https://reviews.apache.org/r/45911/#comment194349>

    Incorrect basic stats?



ql/src/test/results/clientpositive/lateral_view_outer.q.out (line 213)
<https://reviews.apache.org/r/45911/#comment194350>

    incorrect basic stats state?



ql/src/test/results/clientpositive/mapjoin_memcheck.q.out (line 47)
<https://reviews.apache.org/r/45911/#comment194351>

    incorrect basic stats state?



ql/src/test/results/clientpositive/masking_1.q.out (line 27)
<https://reviews.apache.org/r/45911/#comment194352>

    incorrect basic stats state?



ql/src/test/results/clientpositive/masking_3.q.out (line 27)
<https://reviews.apache.org/r/45911/#comment194353>

    incorrect basic stats state?



ql/src/test/results/clientpositive/masking_4.q.out (line 89)
<https://reviews.apache.org/r/45911/#comment194354>

    Incorrect basic stats state?



ql/src/test/results/clientpositive/masking_5.q.out (line 28)
<https://reviews.apache.org/r/45911/#comment194355>

    Incorrect basic stats state?



ql/src/test/results/clientpositive/masking_disablecbo_1.q.out (line 27)
<https://reviews.apache.org/r/45911/#comment194356>

    Incorrect basic stats state?



ql/src/test/results/clientpositive/masking_disablecbo_3.q.out (line 27)
<https://reviews.apache.org/r/45911/#comment194357>

    Incorrect basic stats state?



ql/src/test/results/clientpositive/masking_disablecbo_4.q.out (line 89)
<https://reviews.apache.org/r/45911/#comment194385>

    incorrect basic stats state?



ql/src/test/results/clientpositive/merge3.q.out (line 113)
<https://reviews.apache.org/r/45911/#comment194386>

    incorrect basic stats state?



ql/src/test/results/clientpositive/multi_join_union.q.out (line 85)
<https://reviews.apache.org/r/45911/#comment194388>

    incorrect basic stats state?



ql/src/test/results/clientpositive/orc_createas1.q.out (line 163)
<https://reviews.apache.org/r/45911/#comment194389>

    incorrect basic stats state?



ql/src/test/results/clientpositive/parquet_mixed_partition_formats.q.out (line 129)
<https://reviews.apache.org/r/45911/#comment194390>

    incorrect basic stats state?



ql/src/test/results/clientpositive/sample_islocalmode_hook_use_metadata.q.out (line 116)
<https://reviews.apache.org/r/45911/#comment194391>

    stats optimization got turned off ?



ql/src/test/results/clientpositive/stats11.q.out (lines 91 - 92)
<https://reviews.apache.org/r/45911/#comment194392>

    incorrect basic stats state?



ql/src/test/results/clientpositive/union27.q.out (line 53)
<https://reviews.apache.org/r/45911/#comment194393>

    incorrect basic stats state?


- Ashutosh Chauhan


On April 25, 2016, 5:53 a.m., pengcheng xiong wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45911/
> -----------------------------------------------------------
> 
> (Updated April 25, 2016, 5:53 a.m.)
> 
> 
> Review request for hive and Ashutosh Chauhan.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-13341: Stats state is not captured correctly: differentiate load table and create table
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
>   itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
>   metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java 54592cc 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
>   ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 4c9acce 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 549d24f 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 03b4d8b 
>   ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java eaeb66b 
>   ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
>   ql/src/test/queries/clientpositive/stats20.q 59701bd 
>   ql/src/test/results/clientnegative/alter_file_format.q.out 96f1bfb 
>   ql/src/test/results/clientnegative/stats_partialscan_autogether.q.out 4faf327 
>   ql/src/test/results/clientnegative/unset_table_property.q.out 0510788 
>   ql/src/test/results/clientpositive/allcolref_in_udf.q.out eda49ed 
>   ql/src/test/results/clientpositive/alter_file_format.q.out 5d83b23 
>   ql/src/test/results/clientpositive/alter_skewed_table.q.out a1caa99 
>   ql/src/test/results/clientpositive/alter_table_not_sorted.q.out 6e1ec59 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_1.q.out f768ea4 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_11.q.out f4bbfd0 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_12.q.out e5ff904 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_2.q.out 0de079d 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_3.q.out a161f1c 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_4.q.out 3421ab1 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_5.q.out 9c2663c 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_7.q.out a25db01 
>   ql/src/test/results/clientpositive/auto_sortmerge_join_8.q.out d22ba3b 
>   ql/src/test/results/clientpositive/binary_output_format.q.out 32b2feb 
>   ql/src/test/results/clientpositive/bucket1.q.out 0680176 
>   ql/src/test/results/clientpositive/bucket2.q.out 958e556 
>   ql/src/test/results/clientpositive/bucket4.q.out dd9ef00 
>   ql/src/test/results/clientpositive/bucket5.q.out fa926aa 
>   ql/src/test/results/clientpositive/bucket_many.q.out a9e2f0b 
>   ql/src/test/results/clientpositive/bucket_map_join_1.q.out 05fb434 
>   ql/src/test/results/clientpositive/bucket_map_join_2.q.out 1173c93 
>   ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out 3b1912a 
>   ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out 9a6aef7 
>   ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out 1d2166f 
>   ql/src/test/results/clientpositive/bucketcontext_1.q.out 78a0c8c 
>   ql/src/test/results/clientpositive/bucketcontext_2.q.out bc08fd2 
>   ql/src/test/results/clientpositive/bucketcontext_3.q.out 0cba2ca 
>   ql/src/test/results/clientpositive/bucketcontext_4.q.out 652b377 
>   ql/src/test/results/clientpositive/bucketcontext_5.q.out ee4532a 
>   ql/src/test/results/clientpositive/bucketcontext_6.q.out e280b95 
>   ql/src/test/results/clientpositive/bucketcontext_7.q.out b44ae95 
>   ql/src/test/results/clientpositive/bucketcontext_8.q.out eee7b57 
>   ql/src/test/results/clientpositive/bucketmapjoin1.q.out ac8a76f 
>   ql/src/test/results/clientpositive/bucketmapjoin10.q.out 041f2d0 
>   ql/src/test/results/clientpositive/bucketmapjoin11.q.out 5e9fbe4 
>   ql/src/test/results/clientpositive/bucketmapjoin12.q.out 3a38d7b 
>   ql/src/test/results/clientpositive/bucketmapjoin2.q.out d9c4463 
>   ql/src/test/results/clientpositive/bucketmapjoin3.q.out 77b19ba 
>   ql/src/test/results/clientpositive/bucketmapjoin4.q.out 5858160 
>   ql/src/test/results/clientpositive/bucketmapjoin5.q.out 728c0b7 
>   ql/src/test/results/clientpositive/bucketmapjoin7.q.out 7f96a23 
>   ql/src/test/results/clientpositive/bucketmapjoin8.q.out 804c882 
>   ql/src/test/results/clientpositive/bucketmapjoin9.q.out cc0672e 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out 36b1cfe 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative2.q.out 21138d6 
>   ql/src/test/results/clientpositive/bucketmapjoin_negative3.q.out c95681a 
>   ql/src/test/results/clientpositive/cbo_SortUnionTransposeRule.q.out 7542882 
>   ql/src/test/results/clientpositive/cbo_rp_cross_product_check_2.q.out 89f1746 
>   ql/src/test/results/clientpositive/columnStatsUpdateForStatsOptimizer_1.q.out 8c9664d 
>   ql/src/test/results/clientpositive/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
>   ql/src/test/results/clientpositive/columnstats_partlvl.q.out bfdc3b6 
>   ql/src/test/results/clientpositive/columnstats_tbllvl.q.out 96cf8fd 
>   ql/src/test/results/clientpositive/create_alter_list_bucketing_table1.q.out c4f51a3 
>   ql/src/test/results/clientpositive/create_like.q.out 9241b68 
>   ql/src/test/results/clientpositive/create_like_view.q.out e2dc2c4 
>   ql/src/test/results/clientpositive/create_skewed_table1.q.out 415bb77 
>   ql/src/test/results/clientpositive/cross_product_check_1.q.out 4feb798 
>   ql/src/test/results/clientpositive/cross_product_check_2.q.out f34f2b5 
>   ql/src/test/results/clientpositive/ctas.q.out c9676de 
>   ql/src/test/results/clientpositive/ctas_colname.q.out 2622676 
>   ql/src/test/results/clientpositive/ctas_uses_database_location.q.out 39d8bcf 
>   ql/src/test/results/clientpositive/database_location.q.out 797177d 
>   ql/src/test/results/clientpositive/decimal_join2.q.out 3a0b327 
>   ql/src/test/results/clientpositive/decimal_serde.q.out e0f5c74 
>   ql/src/test/results/clientpositive/default_file_format.q.out 3d5c20f 
>   ql/src/test/results/clientpositive/describe_comment_indent.q.out 3e0f45e 
>   ql/src/test/results/clientpositive/describe_comment_nonascii.q.out df0b65e 
>   ql/src/test/results/clientpositive/disable_merge_for_bucketing.q.out 366bd7a 
>   ql/src/test/results/clientpositive/display_colstats_tbllvl.q.out b79095c 
>   ql/src/test/results/clientpositive/empty_join.q.out 9a47c1f 
>   ql/src/test/results/clientpositive/explain_ddl.q.out fa73d99 
>   ql/src/test/results/clientpositive/groupby_map_ppr.q.out 556995c 
>   ql/src/test/results/clientpositive/groupby_map_ppr_multi_distinct.q.out 393e6fa 
>   ql/src/test/results/clientpositive/groupby_ppr.q.out 2e160b3 
>   ql/src/test/results/clientpositive/groupby_ppr_multi_distinct.q.out 93fbc0e 
>   ql/src/test/results/clientpositive/groupby_sort_1_23.q.out efb13a7 
>   ql/src/test/results/clientpositive/groupby_sort_6.q.out 844a2a4 
>   ql/src/test/results/clientpositive/groupby_sort_skew_1_23.q.out 860a880 
>   ql/src/test/results/clientpositive/input_part1.q.out 501f7a9 
>   ql/src/test/results/clientpositive/input_part2.q.out 9eba510 
>   ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/join17.q.out 13260ef 
>   ql/src/test/results/clientpositive/join26.q.out 5533ce4 
>   ql/src/test/results/clientpositive/join32.q.out 5cb124b 
>   ql/src/test/results/clientpositive/join32_lessSize.q.out 544e814 
>   ql/src/test/results/clientpositive/join33.q.out 5cb124b 
>   ql/src/test/results/clientpositive/join34.q.out e2c2b1a 
>   ql/src/test/results/clientpositive/join35.q.out 96f7cc8 
>   ql/src/test/results/clientpositive/join41.q.out 2ad59f5 
>   ql/src/test/results/clientpositive/join42.q.out 462e49e 
>   ql/src/test/results/clientpositive/join9.q.out e904b31 
>   ql/src/test/results/clientpositive/join_filters_overlap.q.out dede6b7 
>   ql/src/test/results/clientpositive/join_map_ppr.q.out 444fd7f 
>   ql/src/test/results/clientpositive/lateral_view_outer.q.out 994945a 
>   ql/src/test/results/clientpositive/list_bucket_dml_14.q.out 790801f 
>   ql/src/test/results/clientpositive/list_bucket_dml_8.q.java1.7.out 672e5ac 
>   ql/src/test/results/clientpositive/llap/dynamic_partition_pruning.q.out fe2049f 
>   ql/src/test/results/clientpositive/llap/hybridgrace_hashjoin_1.q.out f281d44 
>   ql/src/test/results/clientpositive/llap/llap_nullscan.q.out e17d721 
>   ql/src/test/results/clientpositive/llap/llap_udf.q.out 6197cf0 
>   ql/src/test/results/clientpositive/llap/llapdecider.q.out df90835 
>   ql/src/test/results/clientpositive/llap/tez_dml.q.out f620010 
>   ql/src/test/results/clientpositive/llap/tez_join_result_complex.q.out 440466c 
>   ql/src/test/results/clientpositive/llap/vectorized_dynamic_partition_pruning.q.out 5f52822 
>   ql/src/test/results/clientpositive/mapjoin_memcheck.q.out 8f5492f 
>   ql/src/test/results/clientpositive/masking_1.q.out 3b63550 
>   ql/src/test/results/clientpositive/masking_3.q.out 1925dce 
>   ql/src/test/results/clientpositive/masking_4.q.out 7e923e8 
>   ql/src/test/results/clientpositive/masking_5.q.out acb6471 
>   ql/src/test/results/clientpositive/masking_disablecbo_1.q.out 6717527 
>   ql/src/test/results/clientpositive/masking_disablecbo_3.q.out 6aaab20 
>   ql/src/test/results/clientpositive/masking_disablecbo_4.q.out 698c797 
>   ql/src/test/results/clientpositive/merge3.q.out 5b581db 
>   ql/src/test/results/clientpositive/multi_insert_lateral_view.q.out 7964405 
>   ql/src/test/results/clientpositive/multi_insert_union_src.q.out 2036e63 
>   ql/src/test/results/clientpositive/multi_join_union.q.out b361a1a 
>   ql/src/test/results/clientpositive/nullformat.q.out af91470 
>   ql/src/test/results/clientpositive/nullformatCTAS.q.out 7686419 
>   ql/src/test/results/clientpositive/orc_create.q.out 3013fda 
>   ql/src/test/results/clientpositive/orc_createas1.q.out 506f39d 
>   ql/src/test/results/clientpositive/orc_llap.q.out 6fc73b7 
>   ql/src/test/results/clientpositive/orc_predicate_pushdown.q.out 7b361b7 
>   ql/src/test/results/clientpositive/parallel_orderby.q.out 2991122 
>   ql/src/test/results/clientpositive/parquet_array_null_element.q.out 387f01e 
>   ql/src/test/results/clientpositive/parquet_create.q.out c6d33ff 
>   ql/src/test/results/clientpositive/parquet_join.q.out 86fb64a 
>   ql/src/test/results/clientpositive/parquet_mixed_partition_formats.q.out d6affd6 
>   ql/src/test/results/clientpositive/parquet_serde.q.out c1e594a 
>   ql/src/test/results/clientpositive/pcr.q.out 684d4d7 
>   ql/src/test/results/clientpositive/query_result_fileformat.q.out bce3e22 
>   ql/src/test/results/clientpositive/rand_partitionpruner2.q.out 4559b39 
>   ql/src/test/results/clientpositive/rcfile_default_format.q.out c961231 
>   ql/src/test/results/clientpositive/reduce_deduplicate.q.out 96b51d7 
>   ql/src/test/results/clientpositive/sample1.q.out caf62f4 
>   ql/src/test/results/clientpositive/sample2.q.out e70edd7 
>   ql/src/test/results/clientpositive/sample4.q.out 90dfc26 
>   ql/src/test/results/clientpositive/sample5.q.out 04aa8cc 
>   ql/src/test/results/clientpositive/sample6.q.out c2be9d6 
>   ql/src/test/results/clientpositive/sample7.q.out e1897ee 
>   ql/src/test/results/clientpositive/sample_islocalmode_hook_use_metadata.q.out d268837 
>   ql/src/test/results/clientpositive/semijoin.q.out 25f62a2 
>   ql/src/test/results/clientpositive/semijoin3.q.out aea7e45 
>   ql/src/test/results/clientpositive/show_create_table_alter.q.out 32819ea 
>   ql/src/test/results/clientpositive/show_create_table_db_table.q.out 495f4b5 
>   ql/src/test/results/clientpositive/show_create_table_serde.q.out 2350d98 
>   ql/src/test/results/clientpositive/show_tblproperties.q.out 63bbe6d 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_1.q.out c32a0dd 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_12.q.out d004ff3 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_3.q.out 00d5138 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_4.q.out b06df04 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_5.q.out 18ab5c8 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_7.q.out dd01e69 
>   ql/src/test/results/clientpositive/spark/auto_sortmerge_join_8.q.out f0f27f0 
>   ql/src/test/results/clientpositive/spark/bucket2.q.out f4f87c2 
>   ql/src/test/results/clientpositive/spark/bucket4.q.out 4049f22 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_1.q.out fd17425 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_2.q.out 4884907 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_spark1.q.out 3dbfddd 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_spark2.q.out 216c0bd 
>   ql/src/test/results/clientpositive/spark/bucket_map_join_spark3.q.out 5bc2ec4 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin1.q.out cee6601 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin10.q.out 8764bc3 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin11.q.out 58ac20c 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin12.q.out bde9085 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin2.q.out cbcca57 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin3.q.out 3b306db 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin4.q.out 11a7d3a 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin5.q.out 188e4e7 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin7.q.out d89bd58 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin8.q.out 80dd175 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin9.q.out 2c3dd41 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin_negative.q.out b07af54 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin_negative2.q.out 98c19b4 
>   ql/src/test/results/clientpositive/spark/bucketmapjoin_negative3.q.out dd59399 
>   ql/src/test/results/clientpositive/spark/cross_product_check_1.q.out 65f0c22 
>   ql/src/test/results/clientpositive/spark/cross_product_check_2.q.out 26bee4e 
>   ql/src/test/results/clientpositive/spark/ctas.q.out 086ad73 
>   ql/src/test/results/clientpositive/spark/disable_merge_for_bucketing.q.out fc2ff7b 
>   ql/src/test/results/clientpositive/spark/groupby_map_ppr.q.out b5eae57 
>   ql/src/test/results/clientpositive/spark/groupby_map_ppr_multi_distinct.q.out ada2f91 
>   ql/src/test/results/clientpositive/spark/groupby_ppr.q.out 97a9a6d 
>   ql/src/test/results/clientpositive/spark/groupby_ppr_multi_distinct.q.out a8724f5 
>   ql/src/test/results/clientpositive/spark/groupby_sort_1_23.q.out 4e7e72e 
>   ql/src/test/results/clientpositive/spark/groupby_sort_skew_1_23.q.out b8f1bbd 
>   ql/src/test/results/clientpositive/spark/input_part2.q.out 6129b1c 
>   ql/src/test/results/clientpositive/spark/join17.q.out 026bf1a 
>   ql/src/test/results/clientpositive/spark/join26.q.out b0449f6 
>   ql/src/test/results/clientpositive/spark/join32.q.out 3771217 
>   ql/src/test/results/clientpositive/spark/join32_lessSize.q.out c4ac2dd 
>   ql/src/test/results/clientpositive/spark/join33.q.out 3771217 
>   ql/src/test/results/clientpositive/spark/join34.q.out bb55191 
>   ql/src/test/results/clientpositive/spark/join35.q.out 60ceca0 
>   ql/src/test/results/clientpositive/spark/join41.q.out 2c2f128 
>   ql/src/test/results/clientpositive/spark/join9.q.out 5f26aaf 
>   ql/src/test/results/clientpositive/spark/join_filters_overlap.q.out cde7213 
>   ql/src/test/results/clientpositive/spark/join_map_ppr.q.out 8e28715 
>   ql/src/test/results/clientpositive/spark/mapjoin_memcheck.q.out 2f41847 
>   ql/src/test/results/clientpositive/spark/multi_insert_lateral_view.q.out c3a3511 
>   ql/src/test/results/clientpositive/spark/multi_join_union.q.out 52b3c74 
>   ql/src/test/results/clientpositive/spark/parquet_join.q.out a8d98e8 
>   ql/src/test/results/clientpositive/spark/pcr.q.out 6345eff 
>   ql/src/test/results/clientpositive/spark/sample1.q.out 1d574cc 
>   ql/src/test/results/clientpositive/spark/sample2.q.out 6c070f0 
>   ql/src/test/results/clientpositive/spark/sample4.q.out d6a0808 
>   ql/src/test/results/clientpositive/spark/sample5.q.out 928b673 
>   ql/src/test/results/clientpositive/spark/sample6.q.out 42dc25b 
>   ql/src/test/results/clientpositive/spark/sample7.q.out 4b87cbd 
>   ql/src/test/results/clientpositive/spark/semijoin.q.out 085257e 
>   ql/src/test/results/clientpositive/spark/stats0.q.out 6f881b4 
>   ql/src/test/results/clientpositive/spark/stats1.q.out 588a33d 
>   ql/src/test/results/clientpositive/spark/stats18.q.out 6971e44 
>   ql/src/test/results/clientpositive/spark/stats20.q.out d7e52b4 
>   ql/src/test/results/clientpositive/spark/stats3.q.out 0d8cbbd 
>   ql/src/test/results/clientpositive/spark/stats5.q.out 95293c0 
>   ql/src/test/results/clientpositive/spark/temp_table.q.out c2ec3b4 
>   ql/src/test/results/clientpositive/spark/temp_table_join1.q.out b5742ad 
>   ql/src/test/results/clientpositive/spark/union24.q.out 3bdc503 
>   ql/src/test/results/clientpositive/spark/union27.q.out bccbbb1 
>   ql/src/test/results/clientpositive/spark/union31.q.out bbe49aa 
>   ql/src/test/results/clientpositive/spark/union32.q.out 1ec7e64 
>   ql/src/test/results/clientpositive/spark/vector_between_in.q.out fbb43c4 
>   ql/src/test/results/clientpositive/spark/vector_decimal_aggregate.q.out cfdfce1 
>   ql/src/test/results/clientpositive/spark/vectorization_short_regress.q.out 7caa50d 
>   ql/src/test/results/clientpositive/spark/vectorized_ptf.q.out f0a4444 
>   ql/src/test/results/clientpositive/stats0.q.out 99326fc 
>   ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
>   ql/src/test/results/clientpositive/stats11.q.out 7598dc0 
>   ql/src/test/results/clientpositive/stats18.q.out 6971e44 
>   ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
>   ql/src/test/results/clientpositive/stats3.q.out 0d8cbbd 
>   ql/src/test/results/clientpositive/stats5.q.out 74ddadb 
>   ql/src/test/results/clientpositive/temp_table.q.out 8aedfa1 
>   ql/src/test/results/clientpositive/temp_table_join1.q.out aafc263 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_1.q.out bd6db7f 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_11.q.out 04b48a2 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_12.q.out 8664435 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_2.q.out a0f2be1 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_3.q.out e6fb5dc 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_4.q.out b24c2769 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_5.q.out db7b805 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_7.q.out faf20fa 
>   ql/src/test/results/clientpositive/tez/auto_sortmerge_join_8.q.out 7b51ffc 
>   ql/src/test/results/clientpositive/tez/bucket2.q.out 151b0ce 
>   ql/src/test/results/clientpositive/tez/bucket4.q.out f2030e8 
>   ql/src/test/results/clientpositive/tez/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
>   ql/src/test/results/clientpositive/tez/cross_product_check_1.q.out 470590a 
>   ql/src/test/results/clientpositive/tez/cross_product_check_2.q.out 68df37d 
>   ql/src/test/results/clientpositive/tez/ctas.q.out 52ea981 
>   ql/src/test/results/clientpositive/tez/disable_merge_for_bucketing.q.out 65f750b 
>   ql/src/test/results/clientpositive/tez/dynamic_partition_pruning.q.out 2626768 
>   ql/src/test/results/clientpositive/tez/empty_join.q.out 8a0c040 
>   ql/src/test/results/clientpositive/tez/explainuser_1.q.out c70f104 
>   ql/src/test/results/clientpositive/tez/hybridgrace_hashjoin_1.q.out a8eb0d3 
>   ql/src/test/results/clientpositive/tez/llap_nullscan.q.out 39f04ea 
>   ql/src/test/results/clientpositive/tez/llapdecider.q.out df90835 
>   ql/src/test/results/clientpositive/tez/sample1.q.out 9663218 
>   ql/src/test/results/clientpositive/tez/temp_table.q.out 1c10a4e 
>   ql/src/test/results/clientpositive/tez/tez_dml.q.out 43996cd 
>   ql/src/test/results/clientpositive/tez/tez_join_result_complex.q.out 180bcc6 
>   ql/src/test/results/clientpositive/tez/unionDistinct_1.q.out bef2365 
>   ql/src/test/results/clientpositive/tez/union_fast_stats.q.out 578205e 
>   ql/src/test/results/clientpositive/tez/vector_between_columns.q.out d8f9c8b 
>   ql/src/test/results/clientpositive/tez/vector_between_in.q.out a4cf61a 
>   ql/src/test/results/clientpositive/tez/vector_char_mapjoin1.q.out 4ee8150 
>   ql/src/test/results/clientpositive/tez/vector_decimal_10_0.q.out 7e6638e 
>   ql/src/test/results/clientpositive/tez/vector_decimal_aggregate.q.out efc77ff 
>   ql/src/test/results/clientpositive/tez/vector_decimal_expressions.q.out e5e5b4b 
>   ql/src/test/results/clientpositive/tez/vector_grouping_sets.q.out 86c7306 
>   ql/src/test/results/clientpositive/tez/vector_interval_mapjoin.q.out 4775167 
>   ql/src/test/results/clientpositive/tez/vector_join30.q.out dd5b5aa 
>   ql/src/test/results/clientpositive/tez/vector_leftsemi_mapjoin.q.out 92ad7b9 
>   ql/src/test/results/clientpositive/tez/vector_nullsafe_join.q.out 045b687 
>   ql/src/test/results/clientpositive/tez/vector_outer_join6.q.out bdcdc42 
>   ql/src/test/results/clientpositive/tez/vector_partitioned_date_time.q.out 13e70dd 
>   ql/src/test/results/clientpositive/tez/vector_reduce_groupby_decimal.q.out 8c58232 
>   ql/src/test/results/clientpositive/tez/vector_varchar_mapjoin1.q.out d50d875 
>   ql/src/test/results/clientpositive/tez/vectorization_short_regress.q.out 70f8d1b 
>   ql/src/test/results/clientpositive/tez/vectorized_dynamic_partition_pruning.q.out a790b97 
>   ql/src/test/results/clientpositive/tez/vectorized_ptf.q.out 3d1f22f 
>   ql/src/test/results/clientpositive/truncate_column.q.out adbddfa 
>   ql/src/test/results/clientpositive/truncate_column_list_bucket.q.out 1fa437b 
>   ql/src/test/results/clientpositive/unicode_notation.q.out 52da674 
>   ql/src/test/results/clientpositive/union24.q.out 993b838 
>   ql/src/test/results/clientpositive/union27.q.out f023360 
>   ql/src/test/results/clientpositive/union31.q.out bb35d5c 
>   ql/src/test/results/clientpositive/union32.q.out a3fefa8 
>   ql/src/test/results/clientpositive/unionDistinct_1.q.out 61bfa74 
>   ql/src/test/results/clientpositive/union_fast_stats.q.out f0879af 
>   ql/src/test/results/clientpositive/unset_table_view_property.q.out 8bbb9fe 
>   ql/src/test/results/clientpositive/updateBasicStats.q.out 596de00 
>   ql/src/test/results/clientpositive/vector_between_columns.q.out a4e8d64 
>   ql/src/test/results/clientpositive/vector_between_in.q.out 9f351b2 
>   ql/src/test/results/clientpositive/vector_char_mapjoin1.q.out 000a501 
>   ql/src/test/results/clientpositive/vector_decimal_10_0.q.out 2ee396b 
>   ql/src/test/results/clientpositive/vector_decimal_aggregate.q.out b022435 
>   ql/src/test/results/clientpositive/vector_decimal_expressions.q.out 9244efd 
>   ql/src/test/results/clientpositive/vector_decimal_math_funcs.q.out 0b70d4c 
>   ql/src/test/results/clientpositive/vector_grouping_sets.q.out 4207c19 
>   ql/src/test/results/clientpositive/vector_interval_mapjoin.q.out 2223e81 
>   ql/src/test/results/clientpositive/vector_join30.q.out 45ed894 
>   ql/src/test/results/clientpositive/vector_leftsemi_mapjoin.q.out 485e352 
>   ql/src/test/results/clientpositive/vector_nullsafe_join.q.out 2090c24 
>   ql/src/test/results/clientpositive/vector_number_compare_projection.q.out 85f7cc8 
>   ql/src/test/results/clientpositive/vector_outer_join6.q.out 8c09716 
>   ql/src/test/results/clientpositive/vector_partitioned_date_time.q.out 09dd873 
>   ql/src/test/results/clientpositive/vector_reduce_groupby_decimal.q.out 19a3d50 
>   ql/src/test/results/clientpositive/vector_varchar_mapjoin1.q.out 5ca6d70 
>   ql/src/test/results/clientpositive/vectorization_decimal_date.q.out 6cae52c 
>   ql/src/test/results/clientpositive/vectorization_short_regress.q.out 7691dda 
>   ql/src/test/results/clientpositive/vectorized_ptf.q.out 3b17591 
> 
> Diff: https://reviews.apache.org/r/45911/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> pengcheng xiong
> 
>


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

Posted by pengcheng xiong <px...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45911/
-----------------------------------------------------------

(Updated April 25, 2016, 5:53 a.m.)


Review request for hive and Ashutosh Chauhan.


Repository: hive-git


Description
-------

HIVE-13341: Stats state is not captured correctly: differentiate load table and create table


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
  itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
  metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
  ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java 54592cc 
  ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 4c9acce 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 549d24f 
  ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 03b4d8b 
  ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java eaeb66b 
  ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
  ql/src/test/queries/clientpositive/stats20.q 59701bd 
  ql/src/test/results/clientnegative/alter_file_format.q.out 96f1bfb 
  ql/src/test/results/clientnegative/stats_partialscan_autogether.q.out 4faf327 
  ql/src/test/results/clientnegative/unset_table_property.q.out 0510788 
  ql/src/test/results/clientpositive/allcolref_in_udf.q.out eda49ed 
  ql/src/test/results/clientpositive/alter_file_format.q.out 5d83b23 
  ql/src/test/results/clientpositive/alter_skewed_table.q.out a1caa99 
  ql/src/test/results/clientpositive/alter_table_not_sorted.q.out 6e1ec59 
  ql/src/test/results/clientpositive/auto_sortmerge_join_1.q.out f768ea4 
  ql/src/test/results/clientpositive/auto_sortmerge_join_11.q.out f4bbfd0 
  ql/src/test/results/clientpositive/auto_sortmerge_join_12.q.out e5ff904 
  ql/src/test/results/clientpositive/auto_sortmerge_join_2.q.out 0de079d 
  ql/src/test/results/clientpositive/auto_sortmerge_join_3.q.out a161f1c 
  ql/src/test/results/clientpositive/auto_sortmerge_join_4.q.out 3421ab1 
  ql/src/test/results/clientpositive/auto_sortmerge_join_5.q.out 9c2663c 
  ql/src/test/results/clientpositive/auto_sortmerge_join_7.q.out a25db01 
  ql/src/test/results/clientpositive/auto_sortmerge_join_8.q.out d22ba3b 
  ql/src/test/results/clientpositive/binary_output_format.q.out 32b2feb 
  ql/src/test/results/clientpositive/bucket1.q.out 0680176 
  ql/src/test/results/clientpositive/bucket2.q.out 958e556 
  ql/src/test/results/clientpositive/bucket4.q.out dd9ef00 
  ql/src/test/results/clientpositive/bucket5.q.out fa926aa 
  ql/src/test/results/clientpositive/bucket_many.q.out a9e2f0b 
  ql/src/test/results/clientpositive/bucket_map_join_1.q.out 05fb434 
  ql/src/test/results/clientpositive/bucket_map_join_2.q.out 1173c93 
  ql/src/test/results/clientpositive/bucket_map_join_spark1.q.out 3b1912a 
  ql/src/test/results/clientpositive/bucket_map_join_spark2.q.out 9a6aef7 
  ql/src/test/results/clientpositive/bucket_map_join_spark3.q.out 1d2166f 
  ql/src/test/results/clientpositive/bucketcontext_1.q.out 78a0c8c 
  ql/src/test/results/clientpositive/bucketcontext_2.q.out bc08fd2 
  ql/src/test/results/clientpositive/bucketcontext_3.q.out 0cba2ca 
  ql/src/test/results/clientpositive/bucketcontext_4.q.out 652b377 
  ql/src/test/results/clientpositive/bucketcontext_5.q.out ee4532a 
  ql/src/test/results/clientpositive/bucketcontext_6.q.out e280b95 
  ql/src/test/results/clientpositive/bucketcontext_7.q.out b44ae95 
  ql/src/test/results/clientpositive/bucketcontext_8.q.out eee7b57 
  ql/src/test/results/clientpositive/bucketmapjoin1.q.out ac8a76f 
  ql/src/test/results/clientpositive/bucketmapjoin10.q.out 041f2d0 
  ql/src/test/results/clientpositive/bucketmapjoin11.q.out 5e9fbe4 
  ql/src/test/results/clientpositive/bucketmapjoin12.q.out 3a38d7b 
  ql/src/test/results/clientpositive/bucketmapjoin2.q.out d9c4463 
  ql/src/test/results/clientpositive/bucketmapjoin3.q.out 77b19ba 
  ql/src/test/results/clientpositive/bucketmapjoin4.q.out 5858160 
  ql/src/test/results/clientpositive/bucketmapjoin5.q.out 728c0b7 
  ql/src/test/results/clientpositive/bucketmapjoin7.q.out 7f96a23 
  ql/src/test/results/clientpositive/bucketmapjoin8.q.out 804c882 
  ql/src/test/results/clientpositive/bucketmapjoin9.q.out cc0672e 
  ql/src/test/results/clientpositive/bucketmapjoin_negative.q.out 36b1cfe 
  ql/src/test/results/clientpositive/bucketmapjoin_negative2.q.out 21138d6 
  ql/src/test/results/clientpositive/bucketmapjoin_negative3.q.out c95681a 
  ql/src/test/results/clientpositive/cbo_SortUnionTransposeRule.q.out 7542882 
  ql/src/test/results/clientpositive/cbo_rp_cross_product_check_2.q.out 89f1746 
  ql/src/test/results/clientpositive/columnStatsUpdateForStatsOptimizer_1.q.out 8c9664d 
  ql/src/test/results/clientpositive/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
  ql/src/test/results/clientpositive/columnstats_partlvl.q.out bfdc3b6 
  ql/src/test/results/clientpositive/columnstats_tbllvl.q.out 96cf8fd 
  ql/src/test/results/clientpositive/create_alter_list_bucketing_table1.q.out c4f51a3 
  ql/src/test/results/clientpositive/create_like.q.out 9241b68 
  ql/src/test/results/clientpositive/create_like_view.q.out e2dc2c4 
  ql/src/test/results/clientpositive/create_skewed_table1.q.out 415bb77 
  ql/src/test/results/clientpositive/cross_product_check_1.q.out 4feb798 
  ql/src/test/results/clientpositive/cross_product_check_2.q.out f34f2b5 
  ql/src/test/results/clientpositive/ctas.q.out c9676de 
  ql/src/test/results/clientpositive/ctas_colname.q.out 2622676 
  ql/src/test/results/clientpositive/ctas_uses_database_location.q.out 39d8bcf 
  ql/src/test/results/clientpositive/database_location.q.out 797177d 
  ql/src/test/results/clientpositive/decimal_join2.q.out 3a0b327 
  ql/src/test/results/clientpositive/decimal_serde.q.out e0f5c74 
  ql/src/test/results/clientpositive/default_file_format.q.out 3d5c20f 
  ql/src/test/results/clientpositive/describe_comment_indent.q.out 3e0f45e 
  ql/src/test/results/clientpositive/describe_comment_nonascii.q.out df0b65e 
  ql/src/test/results/clientpositive/disable_merge_for_bucketing.q.out 366bd7a 
  ql/src/test/results/clientpositive/display_colstats_tbllvl.q.out b79095c 
  ql/src/test/results/clientpositive/empty_join.q.out 9a47c1f 
  ql/src/test/results/clientpositive/explain_ddl.q.out fa73d99 
  ql/src/test/results/clientpositive/groupby_map_ppr.q.out 556995c 
  ql/src/test/results/clientpositive/groupby_map_ppr_multi_distinct.q.out 393e6fa 
  ql/src/test/results/clientpositive/groupby_ppr.q.out 2e160b3 
  ql/src/test/results/clientpositive/groupby_ppr_multi_distinct.q.out 93fbc0e 
  ql/src/test/results/clientpositive/groupby_sort_1_23.q.out efb13a7 
  ql/src/test/results/clientpositive/groupby_sort_6.q.out 844a2a4 
  ql/src/test/results/clientpositive/groupby_sort_skew_1_23.q.out 860a880 
  ql/src/test/results/clientpositive/input_part1.q.out 501f7a9 
  ql/src/test/results/clientpositive/input_part2.q.out 9eba510 
  ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/join17.q.out 13260ef 
  ql/src/test/results/clientpositive/join26.q.out 5533ce4 
  ql/src/test/results/clientpositive/join32.q.out 5cb124b 
  ql/src/test/results/clientpositive/join32_lessSize.q.out 544e814 
  ql/src/test/results/clientpositive/join33.q.out 5cb124b 
  ql/src/test/results/clientpositive/join34.q.out e2c2b1a 
  ql/src/test/results/clientpositive/join35.q.out 96f7cc8 
  ql/src/test/results/clientpositive/join41.q.out 2ad59f5 
  ql/src/test/results/clientpositive/join42.q.out 462e49e 
  ql/src/test/results/clientpositive/join9.q.out e904b31 
  ql/src/test/results/clientpositive/join_filters_overlap.q.out dede6b7 
  ql/src/test/results/clientpositive/join_map_ppr.q.out 444fd7f 
  ql/src/test/results/clientpositive/lateral_view_outer.q.out 994945a 
  ql/src/test/results/clientpositive/list_bucket_dml_14.q.out 790801f 
  ql/src/test/results/clientpositive/list_bucket_dml_8.q.java1.7.out 672e5ac 
  ql/src/test/results/clientpositive/llap/dynamic_partition_pruning.q.out fe2049f 
  ql/src/test/results/clientpositive/llap/hybridgrace_hashjoin_1.q.out f281d44 
  ql/src/test/results/clientpositive/llap/llap_nullscan.q.out e17d721 
  ql/src/test/results/clientpositive/llap/llap_udf.q.out 6197cf0 
  ql/src/test/results/clientpositive/llap/llapdecider.q.out df90835 
  ql/src/test/results/clientpositive/llap/tez_dml.q.out f620010 
  ql/src/test/results/clientpositive/llap/tez_join_result_complex.q.out 440466c 
  ql/src/test/results/clientpositive/llap/vectorized_dynamic_partition_pruning.q.out 5f52822 
  ql/src/test/results/clientpositive/mapjoin_memcheck.q.out 8f5492f 
  ql/src/test/results/clientpositive/masking_1.q.out 3b63550 
  ql/src/test/results/clientpositive/masking_3.q.out 1925dce 
  ql/src/test/results/clientpositive/masking_4.q.out 7e923e8 
  ql/src/test/results/clientpositive/masking_5.q.out acb6471 
  ql/src/test/results/clientpositive/masking_disablecbo_1.q.out 6717527 
  ql/src/test/results/clientpositive/masking_disablecbo_3.q.out 6aaab20 
  ql/src/test/results/clientpositive/masking_disablecbo_4.q.out 698c797 
  ql/src/test/results/clientpositive/merge3.q.out 5b581db 
  ql/src/test/results/clientpositive/multi_insert_lateral_view.q.out 7964405 
  ql/src/test/results/clientpositive/multi_insert_union_src.q.out 2036e63 
  ql/src/test/results/clientpositive/multi_join_union.q.out b361a1a 
  ql/src/test/results/clientpositive/nullformat.q.out af91470 
  ql/src/test/results/clientpositive/nullformatCTAS.q.out 7686419 
  ql/src/test/results/clientpositive/orc_create.q.out 3013fda 
  ql/src/test/results/clientpositive/orc_createas1.q.out 506f39d 
  ql/src/test/results/clientpositive/orc_llap.q.out 6fc73b7 
  ql/src/test/results/clientpositive/orc_predicate_pushdown.q.out 7b361b7 
  ql/src/test/results/clientpositive/parallel_orderby.q.out 2991122 
  ql/src/test/results/clientpositive/parquet_array_null_element.q.out 387f01e 
  ql/src/test/results/clientpositive/parquet_create.q.out c6d33ff 
  ql/src/test/results/clientpositive/parquet_join.q.out 86fb64a 
  ql/src/test/results/clientpositive/parquet_mixed_partition_formats.q.out d6affd6 
  ql/src/test/results/clientpositive/parquet_serde.q.out c1e594a 
  ql/src/test/results/clientpositive/pcr.q.out 684d4d7 
  ql/src/test/results/clientpositive/query_result_fileformat.q.out bce3e22 
  ql/src/test/results/clientpositive/rand_partitionpruner2.q.out 4559b39 
  ql/src/test/results/clientpositive/rcfile_default_format.q.out c961231 
  ql/src/test/results/clientpositive/reduce_deduplicate.q.out 96b51d7 
  ql/src/test/results/clientpositive/sample1.q.out caf62f4 
  ql/src/test/results/clientpositive/sample2.q.out e70edd7 
  ql/src/test/results/clientpositive/sample4.q.out 90dfc26 
  ql/src/test/results/clientpositive/sample5.q.out 04aa8cc 
  ql/src/test/results/clientpositive/sample6.q.out c2be9d6 
  ql/src/test/results/clientpositive/sample7.q.out e1897ee 
  ql/src/test/results/clientpositive/sample_islocalmode_hook_use_metadata.q.out d268837 
  ql/src/test/results/clientpositive/semijoin.q.out 25f62a2 
  ql/src/test/results/clientpositive/semijoin3.q.out aea7e45 
  ql/src/test/results/clientpositive/show_create_table_alter.q.out 32819ea 
  ql/src/test/results/clientpositive/show_create_table_db_table.q.out 495f4b5 
  ql/src/test/results/clientpositive/show_create_table_serde.q.out 2350d98 
  ql/src/test/results/clientpositive/show_tblproperties.q.out 63bbe6d 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_1.q.out c32a0dd 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_12.q.out d004ff3 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_3.q.out 00d5138 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_4.q.out b06df04 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_5.q.out 18ab5c8 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_7.q.out dd01e69 
  ql/src/test/results/clientpositive/spark/auto_sortmerge_join_8.q.out f0f27f0 
  ql/src/test/results/clientpositive/spark/bucket2.q.out f4f87c2 
  ql/src/test/results/clientpositive/spark/bucket4.q.out 4049f22 
  ql/src/test/results/clientpositive/spark/bucket_map_join_1.q.out fd17425 
  ql/src/test/results/clientpositive/spark/bucket_map_join_2.q.out 4884907 
  ql/src/test/results/clientpositive/spark/bucket_map_join_spark1.q.out 3dbfddd 
  ql/src/test/results/clientpositive/spark/bucket_map_join_spark2.q.out 216c0bd 
  ql/src/test/results/clientpositive/spark/bucket_map_join_spark3.q.out 5bc2ec4 
  ql/src/test/results/clientpositive/spark/bucketmapjoin1.q.out cee6601 
  ql/src/test/results/clientpositive/spark/bucketmapjoin10.q.out 8764bc3 
  ql/src/test/results/clientpositive/spark/bucketmapjoin11.q.out 58ac20c 
  ql/src/test/results/clientpositive/spark/bucketmapjoin12.q.out bde9085 
  ql/src/test/results/clientpositive/spark/bucketmapjoin2.q.out cbcca57 
  ql/src/test/results/clientpositive/spark/bucketmapjoin3.q.out 3b306db 
  ql/src/test/results/clientpositive/spark/bucketmapjoin4.q.out 11a7d3a 
  ql/src/test/results/clientpositive/spark/bucketmapjoin5.q.out 188e4e7 
  ql/src/test/results/clientpositive/spark/bucketmapjoin7.q.out d89bd58 
  ql/src/test/results/clientpositive/spark/bucketmapjoin8.q.out 80dd175 
  ql/src/test/results/clientpositive/spark/bucketmapjoin9.q.out 2c3dd41 
  ql/src/test/results/clientpositive/spark/bucketmapjoin_negative.q.out b07af54 
  ql/src/test/results/clientpositive/spark/bucketmapjoin_negative2.q.out 98c19b4 
  ql/src/test/results/clientpositive/spark/bucketmapjoin_negative3.q.out dd59399 
  ql/src/test/results/clientpositive/spark/cross_product_check_1.q.out 65f0c22 
  ql/src/test/results/clientpositive/spark/cross_product_check_2.q.out 26bee4e 
  ql/src/test/results/clientpositive/spark/ctas.q.out 086ad73 
  ql/src/test/results/clientpositive/spark/disable_merge_for_bucketing.q.out fc2ff7b 
  ql/src/test/results/clientpositive/spark/groupby_map_ppr.q.out b5eae57 
  ql/src/test/results/clientpositive/spark/groupby_map_ppr_multi_distinct.q.out ada2f91 
  ql/src/test/results/clientpositive/spark/groupby_ppr.q.out 97a9a6d 
  ql/src/test/results/clientpositive/spark/groupby_ppr_multi_distinct.q.out a8724f5 
  ql/src/test/results/clientpositive/spark/groupby_sort_1_23.q.out 4e7e72e 
  ql/src/test/results/clientpositive/spark/groupby_sort_skew_1_23.q.out b8f1bbd 
  ql/src/test/results/clientpositive/spark/input_part2.q.out 6129b1c 
  ql/src/test/results/clientpositive/spark/join17.q.out 026bf1a 
  ql/src/test/results/clientpositive/spark/join26.q.out b0449f6 
  ql/src/test/results/clientpositive/spark/join32.q.out 3771217 
  ql/src/test/results/clientpositive/spark/join32_lessSize.q.out c4ac2dd 
  ql/src/test/results/clientpositive/spark/join33.q.out 3771217 
  ql/src/test/results/clientpositive/spark/join34.q.out bb55191 
  ql/src/test/results/clientpositive/spark/join35.q.out 60ceca0 
  ql/src/test/results/clientpositive/spark/join41.q.out 2c2f128 
  ql/src/test/results/clientpositive/spark/join9.q.out 5f26aaf 
  ql/src/test/results/clientpositive/spark/join_filters_overlap.q.out cde7213 
  ql/src/test/results/clientpositive/spark/join_map_ppr.q.out 8e28715 
  ql/src/test/results/clientpositive/spark/mapjoin_memcheck.q.out 2f41847 
  ql/src/test/results/clientpositive/spark/multi_insert_lateral_view.q.out c3a3511 
  ql/src/test/results/clientpositive/spark/multi_join_union.q.out 52b3c74 
  ql/src/test/results/clientpositive/spark/parquet_join.q.out a8d98e8 
  ql/src/test/results/clientpositive/spark/pcr.q.out 6345eff 
  ql/src/test/results/clientpositive/spark/sample1.q.out 1d574cc 
  ql/src/test/results/clientpositive/spark/sample2.q.out 6c070f0 
  ql/src/test/results/clientpositive/spark/sample4.q.out d6a0808 
  ql/src/test/results/clientpositive/spark/sample5.q.out 928b673 
  ql/src/test/results/clientpositive/spark/sample6.q.out 42dc25b 
  ql/src/test/results/clientpositive/spark/sample7.q.out 4b87cbd 
  ql/src/test/results/clientpositive/spark/semijoin.q.out 085257e 
  ql/src/test/results/clientpositive/spark/stats0.q.out 6f881b4 
  ql/src/test/results/clientpositive/spark/stats1.q.out 588a33d 
  ql/src/test/results/clientpositive/spark/stats18.q.out 6971e44 
  ql/src/test/results/clientpositive/spark/stats20.q.out d7e52b4 
  ql/src/test/results/clientpositive/spark/stats3.q.out 0d8cbbd 
  ql/src/test/results/clientpositive/spark/stats5.q.out 95293c0 
  ql/src/test/results/clientpositive/spark/temp_table.q.out c2ec3b4 
  ql/src/test/results/clientpositive/spark/temp_table_join1.q.out b5742ad 
  ql/src/test/results/clientpositive/spark/union24.q.out 3bdc503 
  ql/src/test/results/clientpositive/spark/union27.q.out bccbbb1 
  ql/src/test/results/clientpositive/spark/union31.q.out bbe49aa 
  ql/src/test/results/clientpositive/spark/union32.q.out 1ec7e64 
  ql/src/test/results/clientpositive/spark/vector_between_in.q.out fbb43c4 
  ql/src/test/results/clientpositive/spark/vector_decimal_aggregate.q.out cfdfce1 
  ql/src/test/results/clientpositive/spark/vectorization_short_regress.q.out 7caa50d 
  ql/src/test/results/clientpositive/spark/vectorized_ptf.q.out f0a4444 
  ql/src/test/results/clientpositive/stats0.q.out 99326fc 
  ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
  ql/src/test/results/clientpositive/stats11.q.out 7598dc0 
  ql/src/test/results/clientpositive/stats18.q.out 6971e44 
  ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
  ql/src/test/results/clientpositive/stats3.q.out 0d8cbbd 
  ql/src/test/results/clientpositive/stats5.q.out 74ddadb 
  ql/src/test/results/clientpositive/temp_table.q.out 8aedfa1 
  ql/src/test/results/clientpositive/temp_table_join1.q.out aafc263 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_1.q.out bd6db7f 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_11.q.out 04b48a2 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_12.q.out 8664435 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_2.q.out a0f2be1 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_3.q.out e6fb5dc 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_4.q.out b24c2769 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_5.q.out db7b805 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_7.q.out faf20fa 
  ql/src/test/results/clientpositive/tez/auto_sortmerge_join_8.q.out 7b51ffc 
  ql/src/test/results/clientpositive/tez/bucket2.q.out 151b0ce 
  ql/src/test/results/clientpositive/tez/bucket4.q.out f2030e8 
  ql/src/test/results/clientpositive/tez/column_names_with_leading_and_trailing_spaces.q.out 46c285e 
  ql/src/test/results/clientpositive/tez/cross_product_check_1.q.out 470590a 
  ql/src/test/results/clientpositive/tez/cross_product_check_2.q.out 68df37d 
  ql/src/test/results/clientpositive/tez/ctas.q.out 52ea981 
  ql/src/test/results/clientpositive/tez/disable_merge_for_bucketing.q.out 65f750b 
  ql/src/test/results/clientpositive/tez/dynamic_partition_pruning.q.out 2626768 
  ql/src/test/results/clientpositive/tez/empty_join.q.out 8a0c040 
  ql/src/test/results/clientpositive/tez/explainuser_1.q.out c70f104 
  ql/src/test/results/clientpositive/tez/hybridgrace_hashjoin_1.q.out a8eb0d3 
  ql/src/test/results/clientpositive/tez/llap_nullscan.q.out 39f04ea 
  ql/src/test/results/clientpositive/tez/llapdecider.q.out df90835 
  ql/src/test/results/clientpositive/tez/sample1.q.out 9663218 
  ql/src/test/results/clientpositive/tez/temp_table.q.out 1c10a4e 
  ql/src/test/results/clientpositive/tez/tez_dml.q.out 43996cd 
  ql/src/test/results/clientpositive/tez/tez_join_result_complex.q.out 180bcc6 
  ql/src/test/results/clientpositive/tez/unionDistinct_1.q.out bef2365 
  ql/src/test/results/clientpositive/tez/union_fast_stats.q.out 578205e 
  ql/src/test/results/clientpositive/tez/vector_between_columns.q.out d8f9c8b 
  ql/src/test/results/clientpositive/tez/vector_between_in.q.out a4cf61a 
  ql/src/test/results/clientpositive/tez/vector_char_mapjoin1.q.out 4ee8150 
  ql/src/test/results/clientpositive/tez/vector_decimal_10_0.q.out 7e6638e 
  ql/src/test/results/clientpositive/tez/vector_decimal_aggregate.q.out efc77ff 
  ql/src/test/results/clientpositive/tez/vector_decimal_expressions.q.out e5e5b4b 
  ql/src/test/results/clientpositive/tez/vector_grouping_sets.q.out 86c7306 
  ql/src/test/results/clientpositive/tez/vector_interval_mapjoin.q.out 4775167 
  ql/src/test/results/clientpositive/tez/vector_join30.q.out dd5b5aa 
  ql/src/test/results/clientpositive/tez/vector_leftsemi_mapjoin.q.out 92ad7b9 
  ql/src/test/results/clientpositive/tez/vector_nullsafe_join.q.out 045b687 
  ql/src/test/results/clientpositive/tez/vector_outer_join6.q.out bdcdc42 
  ql/src/test/results/clientpositive/tez/vector_partitioned_date_time.q.out 13e70dd 
  ql/src/test/results/clientpositive/tez/vector_reduce_groupby_decimal.q.out 8c58232 
  ql/src/test/results/clientpositive/tez/vector_varchar_mapjoin1.q.out d50d875 
  ql/src/test/results/clientpositive/tez/vectorization_short_regress.q.out 70f8d1b 
  ql/src/test/results/clientpositive/tez/vectorized_dynamic_partition_pruning.q.out a790b97 
  ql/src/test/results/clientpositive/tez/vectorized_ptf.q.out 3d1f22f 
  ql/src/test/results/clientpositive/truncate_column.q.out adbddfa 
  ql/src/test/results/clientpositive/truncate_column_list_bucket.q.out 1fa437b 
  ql/src/test/results/clientpositive/unicode_notation.q.out 52da674 
  ql/src/test/results/clientpositive/union24.q.out 993b838 
  ql/src/test/results/clientpositive/union27.q.out f023360 
  ql/src/test/results/clientpositive/union31.q.out bb35d5c 
  ql/src/test/results/clientpositive/union32.q.out a3fefa8 
  ql/src/test/results/clientpositive/unionDistinct_1.q.out 61bfa74 
  ql/src/test/results/clientpositive/union_fast_stats.q.out f0879af 
  ql/src/test/results/clientpositive/unset_table_view_property.q.out 8bbb9fe 
  ql/src/test/results/clientpositive/updateBasicStats.q.out 596de00 
  ql/src/test/results/clientpositive/vector_between_columns.q.out a4e8d64 
  ql/src/test/results/clientpositive/vector_between_in.q.out 9f351b2 
  ql/src/test/results/clientpositive/vector_char_mapjoin1.q.out 000a501 
  ql/src/test/results/clientpositive/vector_decimal_10_0.q.out 2ee396b 
  ql/src/test/results/clientpositive/vector_decimal_aggregate.q.out b022435 
  ql/src/test/results/clientpositive/vector_decimal_expressions.q.out 9244efd 
  ql/src/test/results/clientpositive/vector_decimal_math_funcs.q.out 0b70d4c 
  ql/src/test/results/clientpositive/vector_grouping_sets.q.out 4207c19 
  ql/src/test/results/clientpositive/vector_interval_mapjoin.q.out 2223e81 
  ql/src/test/results/clientpositive/vector_join30.q.out 45ed894 
  ql/src/test/results/clientpositive/vector_leftsemi_mapjoin.q.out 485e352 
  ql/src/test/results/clientpositive/vector_nullsafe_join.q.out 2090c24 
  ql/src/test/results/clientpositive/vector_number_compare_projection.q.out 85f7cc8 
  ql/src/test/results/clientpositive/vector_outer_join6.q.out 8c09716 
  ql/src/test/results/clientpositive/vector_partitioned_date_time.q.out 09dd873 
  ql/src/test/results/clientpositive/vector_reduce_groupby_decimal.q.out 19a3d50 
  ql/src/test/results/clientpositive/vector_varchar_mapjoin1.q.out 5ca6d70 
  ql/src/test/results/clientpositive/vectorization_decimal_date.q.out 6cae52c 
  ql/src/test/results/clientpositive/vectorization_short_regress.q.out 7691dda 
  ql/src/test/results/clientpositive/vectorized_ptf.q.out 3b17591 

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


Testing
-------


Thanks,

pengcheng xiong


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

Posted by pengcheng xiong <px...@hortonworks.com>.

> On April 19, 2016, 6:23 p.m., Ashutosh Chauhan wrote:
> > ql/src/test/results/clientpositive/stats20.q.out, lines 47-50
> > <https://reviews.apache.org/r/45911/diff/2/?file=1345987#file1345987line47>
> >
> >     This doesnt look right. Create table followed by insert overwrite should get us correct stats for all these

After carefully check, it is right... We should specify the partition for testing partition table...But, I agree that for clarification purpose, we should not have stats for partition table itself.


> On April 19, 2016, 6:23 p.m., Ashutosh Chauhan wrote:
> > ql/src/test/results/clientpositive/stats20.q.out, lines 101-104
> > <https://reviews.apache.org/r/45911/diff/2/?file=1345987#file1345987line101>
> >
> >     Doesnt look right.

The same as before.


> On April 19, 2016, 6:23 p.m., Ashutosh Chauhan wrote:
> > ql/src/test/results/clientpositive/stats6.q.out, lines 165-166
> > <https://reviews.apache.org/r/45911/diff/2/?file=1345990#file1345990line165>
> >
> >     Wrong ?

agree.


- pengcheng


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


On April 14, 2016, 11:52 p.m., pengcheng xiong wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45911/
> -----------------------------------------------------------
> 
> (Updated April 14, 2016, 11:52 p.m.)
> 
> 
> Review request for hive and Ashutosh Chauhan.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-13341: Stats state is not captured correctly: differentiate load table and create table
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
>   itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
>   metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java 54592cc 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
>   ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 4c9acce 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 549d24f 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 03b4d8b 
>   ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java eaeb66b 
>   ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
>   ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/join32.q.out 5cb124b 
>   ql/src/test/results/clientpositive/stats0.q.out 99326fc 
>   ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
>   ql/src/test/results/clientpositive/stats10.q.out d7519af 
>   ql/src/test/results/clientpositive/stats11.q.out 7598dc0 
>   ql/src/test/results/clientpositive/stats12.q.out 6a44d15 
>   ql/src/test/results/clientpositive/stats13.q.out f98753e 
>   ql/src/test/results/clientpositive/stats16.q.out 3b371a8 
>   ql/src/test/results/clientpositive/stats18.q.out 6971e44 
>   ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
>   ql/src/test/results/clientpositive/stats3.q.out 0d8cbbd 
>   ql/src/test/results/clientpositive/stats5.q.out 74ddadb 
>   ql/src/test/results/clientpositive/stats6.q.out f2a756f 
>   ql/src/test/results/clientpositive/stats_noscan_1.q.out 7382b31 
>   ql/src/test/results/clientpositive/stats_partscan_1_23.q.out ab2baaa 
> 
> Diff: https://reviews.apache.org/r/45911/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> pengcheng xiong
> 
>


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

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




ql/src/test/results/clientpositive/stats20.q.out (lines 47 - 50)
<https://reviews.apache.org/r/45911/#comment193015>

    This doesnt look right. Create table followed by insert overwrite should get us correct stats for all these



ql/src/test/results/clientpositive/stats20.q.out (lines 101 - 104)
<https://reviews.apache.org/r/45911/#comment193016>

    Doesnt look right.



ql/src/test/results/clientpositive/stats3.q.out (lines 94 - 96)
<https://reviews.apache.org/r/45911/#comment193023>

    This should also print COLUMN_STATS_ACCURATE {"BASIC_STATS":"false"}, no?



ql/src/test/results/clientpositive/stats3.q.out (lines 238 - 241)
<https://reviews.apache.org/r/45911/#comment193026>

    Looks wrong.



ql/src/test/results/clientpositive/stats6.q.out (lines 165 - 166)
<https://reviews.apache.org/r/45911/#comment193027>

    Wrong ?



ql/src/test/results/clientpositive/stats_noscan_1.q.out (line 106)
<https://reviews.apache.org/r/45911/#comment193028>

    Wrong.



ql/src/test/results/clientpositive/stats_partscan_1_23.q.out (line 81)
<https://reviews.apache.org/r/45911/#comment193030>

    wrong.


- Ashutosh Chauhan


On April 14, 2016, 11:52 p.m., pengcheng xiong wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45911/
> -----------------------------------------------------------
> 
> (Updated April 14, 2016, 11:52 p.m.)
> 
> 
> Review request for hive and Ashutosh Chauhan.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-13341: Stats state is not captured correctly: differentiate load table and create table
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
>   itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
>   metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java 54592cc 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
>   ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 4c9acce 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 549d24f 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 03b4d8b 
>   ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java eaeb66b 
>   ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
>   ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/join32.q.out 5cb124b 
>   ql/src/test/results/clientpositive/stats0.q.out 99326fc 
>   ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
>   ql/src/test/results/clientpositive/stats10.q.out d7519af 
>   ql/src/test/results/clientpositive/stats11.q.out 7598dc0 
>   ql/src/test/results/clientpositive/stats12.q.out 6a44d15 
>   ql/src/test/results/clientpositive/stats13.q.out f98753e 
>   ql/src/test/results/clientpositive/stats16.q.out 3b371a8 
>   ql/src/test/results/clientpositive/stats18.q.out 6971e44 
>   ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
>   ql/src/test/results/clientpositive/stats3.q.out 0d8cbbd 
>   ql/src/test/results/clientpositive/stats5.q.out 74ddadb 
>   ql/src/test/results/clientpositive/stats6.q.out f2a756f 
>   ql/src/test/results/clientpositive/stats_noscan_1.q.out 7382b31 
>   ql/src/test/results/clientpositive/stats_partscan_1_23.q.out ab2baaa 
> 
> Diff: https://reviews.apache.org/r/45911/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> pengcheng xiong
> 
>


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

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



Can you update RB with full patch?


ql/src/test/results/clientpositive/stats12.q.out (lines 327 - 330)
<https://reviews.apache.org/r/45911/#comment193374>

    This loooks incorrect. We ran analyze statement for column stats but not on this partition.



ql/src/test/results/clientpositive/stats13.q.out (lines 239 - 241)
<https://reviews.apache.org/r/45911/#comment193375>

    This also looks incorrect.


- Ashutosh Chauhan


On April 14, 2016, 11:52 p.m., pengcheng xiong wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/45911/
> -----------------------------------------------------------
> 
> (Updated April 14, 2016, 11:52 p.m.)
> 
> 
> Review request for hive and Ashutosh Chauhan.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-13341: Stats state is not captured correctly: differentiate load table and create table
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
>   itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
>   metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java 54592cc 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
>   ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 4c9acce 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 549d24f 
>   ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 03b4d8b 
>   ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java eaeb66b 
>   ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
>   ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
>   ql/src/test/results/clientpositive/join32.q.out 5cb124b 
>   ql/src/test/results/clientpositive/stats0.q.out 99326fc 
>   ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
>   ql/src/test/results/clientpositive/stats10.q.out d7519af 
>   ql/src/test/results/clientpositive/stats11.q.out 7598dc0 
>   ql/src/test/results/clientpositive/stats12.q.out 6a44d15 
>   ql/src/test/results/clientpositive/stats13.q.out f98753e 
>   ql/src/test/results/clientpositive/stats16.q.out 3b371a8 
>   ql/src/test/results/clientpositive/stats18.q.out 6971e44 
>   ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
>   ql/src/test/results/clientpositive/stats3.q.out 0d8cbbd 
>   ql/src/test/results/clientpositive/stats5.q.out 74ddadb 
>   ql/src/test/results/clientpositive/stats6.q.out f2a756f 
>   ql/src/test/results/clientpositive/stats_noscan_1.q.out 7382b31 
>   ql/src/test/results/clientpositive/stats_partscan_1_23.q.out ab2baaa 
> 
> Diff: https://reviews.apache.org/r/45911/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> pengcheng xiong
> 
>


Re: Review Request 45911: HIVE-13341: Stats state is not captured correctly: differentiate load table and create table

Posted by pengcheng xiong <px...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45911/
-----------------------------------------------------------

(Updated April 14, 2016, 11:52 p.m.)


Review request for hive and Ashutosh Chauhan.


Repository: hive-git


Description
-------

HIVE-13341: Stats state is not captured correctly: differentiate load table and create table


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/common/StatsSetupConst.java 41d150c 
  itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/history/TestHiveHistory.java c046708 
  metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java 76220f4 
  ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java 54592cc 
  ql/src/java/org/apache/hadoop/hive/ql/exec/StatsTask.java 87a7667 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 4c9acce 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 549d24f 
  ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java 03b4d8b 
  ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java eaeb66b 
  ql/src/test/queries/clientpositive/insert_values_orig_table_use_metadata.q PRE-CREATION 
  ql/src/test/results/clientpositive/insert_values_orig_table_use_metadata.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/join32.q.out 5cb124b 
  ql/src/test/results/clientpositive/stats0.q.out 99326fc 
  ql/src/test/results/clientpositive/stats1.q.out 72c53e3 
  ql/src/test/results/clientpositive/stats10.q.out d7519af 
  ql/src/test/results/clientpositive/stats11.q.out 7598dc0 
  ql/src/test/results/clientpositive/stats12.q.out 6a44d15 
  ql/src/test/results/clientpositive/stats13.q.out f98753e 
  ql/src/test/results/clientpositive/stats16.q.out 3b371a8 
  ql/src/test/results/clientpositive/stats18.q.out 6971e44 
  ql/src/test/results/clientpositive/stats20.q.out d7e52b4 
  ql/src/test/results/clientpositive/stats3.q.out 0d8cbbd 
  ql/src/test/results/clientpositive/stats5.q.out 74ddadb 
  ql/src/test/results/clientpositive/stats6.q.out f2a756f 
  ql/src/test/results/clientpositive/stats_noscan_1.q.out 7382b31 
  ql/src/test/results/clientpositive/stats_partscan_1_23.q.out ab2baaa 

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


Testing
-------


Thanks,

pengcheng xiong