> On Nov. 23, 2015, 9:09 a.m., Jason Dere wrote:
> > - Why are 
> > alter_numbuckets_partitioned_table.q/alter_numbuckets_partitioned_table2.q 
> > removed?

Because same tests are also duplicated in 
alter_numbuckets_partitioned_table_h23.q/alter_numbuckets_partitioned_table2_h23.q
 When we used to support multiple hadoop versions those files (containing same 
tests) were kept to run for different hadoop versions. Now that we only support 
hadoop-2, we dont need this duplication.


- Ashutosh


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


On Nov. 5, 2015, 8:50 p.m., Ashutosh Chauhan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39960/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2015, 8:50 p.m.)
> 
> 
> Review request for hive and Gopal V.
> 
> 
> Bugs: HIVE-12331
>     https://issues.apache.org/jira/browse/HIVE-12331
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> Remove hive.enforce.bucketing & hive.enforce.sorting
> 
> 
> Diffs
> -----
> 
>   common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 3ab73ad 
>   ql/src/java/org/apache/hadoop/hive/ql/ErrorMsg.java 39a881a 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 02adf0c 
>   
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/SortedDynPartitionOptimizer.java
>  e2a0eae 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java d2c3a7c 
>   ql/src/test/org/apache/hadoop/hive/ql/TestTxnCommands.java e13e6eb 
>   ql/src/test/org/apache/hadoop/hive/ql/TestTxnCommands2.java 5aa2500 
>   ql/src/test/org/apache/hadoop/hive/ql/lockmgr/TestDbTxnManager2.java 
> 44ad8b0 
>   ql/src/test/queries/clientnegative/acid_overwrite.q 2e57a3c 
>   ql/src/test/queries/clientnegative/archive_corrupt.q 130b37b 
>   ql/src/test/queries/clientnegative/authorization_delete_nodeletepriv.q 
> f2de306 
>   ql/src/test/queries/clientnegative/authorization_update_noupdatepriv.q 
> c00c0eb 
>   ql/src/test/queries/clientnegative/delete_non_acid_table.q 6ae82ff 
>   ql/src/test/queries/clientnegative/delete_not_bucketed.q 80dffea 
>   ql/src/test/queries/clientnegative/delete_sorted.q fd8d579 
>   ql/src/test/queries/clientnegative/insert_sorted.q 18c942a 
>   ql/src/test/queries/clientnegative/insert_values_sorted.q 260e2fb 
>   ql/src/test/queries/clientnegative/merge_negative_3.q 6bc645e 
>   ql/src/test/queries/clientnegative/smb_bucketmapjoin.q 880323c 
>   ql/src/test/queries/clientnegative/smb_mapjoin_14.q 54bfba0 
>   ql/src/test/queries/clientnegative/sortmerge_mapjoin_mismatch_1.q 7d11f45 
>   ql/src/test/queries/clientnegative/update_bucket_col.q 515e024 
>   ql/src/test/queries/clientnegative/update_no_such_table.q 07239cf 
>   ql/src/test/queries/clientnegative/update_non_acid_table.q dd0b01e 
>   ql/src/test/queries/clientnegative/update_not_bucketed.q 8512fa7 
>   ql/src/test/queries/clientnegative/update_partition_col.q e9c60cc 
>   ql/src/test/queries/clientnegative/update_sorted.q 917c3b5 
>   ql/src/test/queries/clientpositive/acid_join.q 2e6aeae 
>   ql/src/test/queries/clientpositive/acid_vectorization.q 4b11412 
>   ql/src/test/queries/clientpositive/acid_vectorization_partition.q 00449bb 
>   ql/src/test/queries/clientpositive/acid_vectorization_project.q a44b57a 
>   ql/src/test/queries/clientpositive/alter_numbuckets_partitioned_table.q 
> 627fcc1 
>   ql/src/test/queries/clientpositive/alter_numbuckets_partitioned_table2.q 
> 2f26de8 
>   
> ql/src/test/queries/clientpositive/alter_numbuckets_partitioned_table2_h23.q 
> 2c2e184 
>   ql/src/test/queries/clientpositive/alter_numbuckets_partitioned_table_h23.q 
> 439f351 
>   ql/src/test/queries/clientpositive/archive_excludeHadoop20.q 316276a 
>   ql/src/test/queries/clientpositive/archive_multi.q 1004aca 
>   ql/src/test/queries/clientpositive/authorization_delete.q d96e6ab 
>   ql/src/test/queries/clientpositive/authorization_delete_own_table.q 7abdc12 
>   ql/src/test/queries/clientpositive/authorization_update.q da1054e 
>   ql/src/test/queries/clientpositive/authorization_update_own_table.q ace1ce2 
>   ql/src/test/queries/clientpositive/auto_smb_mapjoin_14.q 699777e 
>   ql/src/test/queries/clientpositive/auto_sortmerge_join_10.q c07dd23 
>   ql/src/test/queries/clientpositive/auto_sortmerge_join_13.q f35fec1 
>   ql/src/test/queries/clientpositive/auto_sortmerge_join_14.q eabeff0 
>   ql/src/test/queries/clientpositive/auto_sortmerge_join_15.q a553d93 
>   ql/src/test/queries/clientpositive/auto_sortmerge_join_16.q cb244cf 
>   ql/src/test/queries/clientpositive/auto_sortmerge_join_6.q 0ddf378 
>   ql/src/test/queries/clientpositive/auto_sortmerge_join_9.q 9eb85d3 
>   ql/src/test/queries/clientpositive/bucket1.q 0154b4e 
>   ql/src/test/queries/clientpositive/bucket2.q ecd7e53 
>   ql/src/test/queries/clientpositive/bucket3.q 7b7a9c3 
>   ql/src/test/queries/clientpositive/bucket4.q 1b49c7a 
>   ql/src/test/queries/clientpositive/bucket5.q 877f8a5 
>   ql/src/test/queries/clientpositive/bucket6.q fb55787 
>   ql/src/test/queries/clientpositive/bucket_many.q 1f0b795 
>   ql/src/test/queries/clientpositive/bucket_map_join_1.q 6bdb09e 
>   ql/src/test/queries/clientpositive/bucket_map_join_2.q 07f6d15 
>   ql/src/test/queries/clientpositive/bucket_map_join_spark4.q 1ca20e4 
>   ql/src/test/queries/clientpositive/bucket_map_join_tez1.q 8546e78 
>   ql/src/test/queries/clientpositive/bucket_map_join_tez2.q 2f968bd 
>   ql/src/test/queries/clientpositive/bucket_num_reducers.q 37ae6cc 
>   ql/src/test/queries/clientpositive/bucket_num_reducers2.q 8c64d60 
>   ql/src/test/queries/clientpositive/bucketmapjoin13.q f01c43e 
>   ql/src/test/queries/clientpositive/bucketmapjoin6.q a0ef371 
>   ql/src/test/queries/clientpositive/bucketsortoptimize_insert_1.q 8cc308f 
>   ql/src/test/queries/clientpositive/bucketsortoptimize_insert_2.q 9ecd2c4 
>   ql/src/test/queries/clientpositive/bucketsortoptimize_insert_3.q 91e97de 
>   ql/src/test/queries/clientpositive/bucketsortoptimize_insert_4.q 623b22b 
>   ql/src/test/queries/clientpositive/bucketsortoptimize_insert_5.q 205a450 
>   ql/src/test/queries/clientpositive/bucketsortoptimize_insert_6.q a4e84f8 
>   ql/src/test/queries/clientpositive/bucketsortoptimize_insert_7.q f597884 
>   ql/src/test/queries/clientpositive/bucketsortoptimize_insert_8.q 95a9a64 
>   ql/src/test/queries/clientpositive/cbo_rp_auto_join1.q 096ae10 
>   ql/src/test/queries/clientpositive/combine3.q c9afc91 
>   ql/src/test/queries/clientpositive/delete_all_non_partitioned.q 9110dcc 
>   ql/src/test/queries/clientpositive/delete_all_partitioned.q f082b6d 
>   ql/src/test/queries/clientpositive/delete_orig_table.q fd23f4b 
>   ql/src/test/queries/clientpositive/delete_tmp_table.q eb6c095 
>   ql/src/test/queries/clientpositive/delete_where_no_match.q 8ed979d 
>   ql/src/test/queries/clientpositive/delete_where_non_partitioned.q dac5375 
>   ql/src/test/queries/clientpositive/delete_where_partitioned.q f84f26a 
>   ql/src/test/queries/clientpositive/delete_whole_partition.q 8228a32 
>   ql/src/test/queries/clientpositive/disable_merge_for_bucketing.q d7f9ac8 
>   ql/src/test/queries/clientpositive/dynpart_sort_opt_bucketing.q 95ae6e3 
>   ql/src/test/queries/clientpositive/dynpart_sort_opt_vectorization.q 3d0cdcd 
>   ql/src/test/queries/clientpositive/dynpart_sort_optimization.q a1a87d8 
>   ql/src/test/queries/clientpositive/dynpart_sort_optimization2.q c18f1cc 
>   ql/src/test/queries/clientpositive/dynpart_sort_optimization_acid.q a2f2c77 
>   ql/src/test/queries/clientpositive/encryption_insert_partition_dynamic.q 
> 9556eed 
>   ql/src/test/queries/clientpositive/encryption_insert_partition_static.q 
> 69687df 
>   ql/src/test/queries/clientpositive/enforce_order.q 6a303c3 
>   ql/src/test/queries/clientpositive/explainuser_1.q 7ae1ed6 
>   ql/src/test/queries/clientpositive/explainuser_2.q 560a086 
>   ql/src/test/queries/clientpositive/explainuser_3.q f604d38 
>   ql/src/test/queries/clientpositive/groupby_sort_1.q ed888bb 
>   ql/src/test/queries/clientpositive/groupby_sort_10.q b3ddd42 
>   ql/src/test/queries/clientpositive/groupby_sort_11.q 19063f6 
>   ql/src/test/queries/clientpositive/groupby_sort_1_23.q a6e18c7 
>   ql/src/test/queries/clientpositive/groupby_sort_2.q 1574048 
>   ql/src/test/queries/clientpositive/groupby_sort_3.q b835f95 
>   ql/src/test/queries/clientpositive/groupby_sort_4.q a61c551 
>   ql/src/test/queries/clientpositive/groupby_sort_5.q 0d4ba42 
>   ql/src/test/queries/clientpositive/groupby_sort_6.q 752b927 
>   ql/src/test/queries/clientpositive/groupby_sort_7.q 3e3ba7a 
>   ql/src/test/queries/clientpositive/groupby_sort_8.q f53295e 
>   ql/src/test/queries/clientpositive/groupby_sort_9.q 296336d 
>   ql/src/test/queries/clientpositive/groupby_sort_skew_1.q 76a1725 
>   ql/src/test/queries/clientpositive/groupby_sort_skew_1_23.q 1b24aec 
>   ql/src/test/queries/clientpositive/groupby_sort_test_1.q 4ec138e 
>   ql/src/test/queries/clientpositive/infer_bucket_sort_bucketed_table.q 
> d69f49f 
>   ql/src/test/queries/clientpositive/infer_bucket_sort_map_operators.q 
> 16e8715 
>   ql/src/test/queries/clientpositive/insert_acid_dynamic_partition.q c544589 
>   ql/src/test/queries/clientpositive/insert_acid_not_bucketed.q a29b1e7 
>   ql/src/test/queries/clientpositive/insert_into_with_schema2.q a5352ec 
>   ql/src/test/queries/clientpositive/insert_nonacid_from_acid.q 639cb31 
>   ql/src/test/queries/clientpositive/insert_orig_table.q c38bd5a 
>   ql/src/test/queries/clientpositive/insert_update_delete.q 8dbb77c 
>   ql/src/test/queries/clientpositive/insert_values_acid_not_bucketed.q 
> fc0cb10 
>   ql/src/test/queries/clientpositive/insert_values_dynamic_partitioned.q 
> 71e0e73 
>   ql/src/test/queries/clientpositive/insert_values_non_partitioned.q d0e7b0f 
>   ql/src/test/queries/clientpositive/insert_values_orig_table.q 8fef549 
>   ql/src/test/queries/clientpositive/insert_values_partitioned.q c8223f7 
>   ql/src/test/queries/clientpositive/insert_values_tmp_table.q 4e4c39e 
>   ql/src/test/queries/clientpositive/insertoverwrite_bucket.q d939710 
>   ql/src/test/queries/clientpositive/join_nullsafe.q d6eda77 
>   ql/src/test/queries/clientpositive/load_dyn_part2.q e804971 
>   ql/src/test/queries/clientpositive/mergejoin.q 7550e09 
>   ql/src/test/queries/clientpositive/orc_empty_files.q d3cbc5a 
>   ql/src/test/queries/clientpositive/partition_wise_fileformat14.q f4d4d73 
>   ql/src/test/queries/clientpositive/quotedid_smb.q 38d1b99 
>   ql/src/test/queries/clientpositive/reduce_deduplicate.q 2e26adc 
>   ql/src/test/queries/clientpositive/sample10.q d9fe744 
>   ql/src/test/queries/clientpositive/smb_mapjoin9.q b959022 
>   ql/src/test/queries/clientpositive/smb_mapjoin_11.q 9300638 
>   ql/src/test/queries/clientpositive/smb_mapjoin_12.q 9f9748c 
>   ql/src/test/queries/clientpositive/smb_mapjoin_13.q 056bccd 
>   ql/src/test/queries/clientpositive/smb_mapjoin_14.q f03f92e 
>   ql/src/test/queries/clientpositive/smb_mapjoin_15.q 1e77a60 
>   ql/src/test/queries/clientpositive/smb_mapjoin_16.q 3a3a872 
>   ql/src/test/queries/clientpositive/smb_mapjoin_17.q 238f7e0 
>   ql/src/test/queries/clientpositive/smb_mapjoin_18.q 02e3fb5 
>   ql/src/test/queries/clientpositive/smb_mapjoin_19.q ca48f61 
>   ql/src/test/queries/clientpositive/smb_mapjoin_20.q f70e7d5 
>   ql/src/test/queries/clientpositive/smb_mapjoin_21.q 09edfc1 
>   ql/src/test/queries/clientpositive/smb_mapjoin_22.q 676f46a 
>   ql/src/test/queries/clientpositive/smb_mapjoin_25.q 683341b 
>   ql/src/test/queries/clientpositive/smb_mapjoin_6.q 2884a11 
>   ql/src/test/queries/clientpositive/smb_mapjoin_7.q ca1c749 
>   ql/src/test/queries/clientpositive/smb_mapjoin_8.q 4b4e167 
>   ql/src/test/queries/clientpositive/sort_merge_join_desc_1.q 8002ec5 
>   ql/src/test/queries/clientpositive/sort_merge_join_desc_2.q bd0cdb2 
>   ql/src/test/queries/clientpositive/sort_merge_join_desc_3.q a109878 
>   ql/src/test/queries/clientpositive/sort_merge_join_desc_4.q 0bc5071 
>   ql/src/test/queries/clientpositive/sort_merge_join_desc_5.q 3505db0 
>   ql/src/test/queries/clientpositive/sort_merge_join_desc_6.q 35b0535 
>   ql/src/test/queries/clientpositive/sort_merge_join_desc_7.q 65dc7f1 
>   ql/src/test/queries/clientpositive/sort_merge_join_desc_8.q 2ec0849 
>   ql/src/test/queries/clientpositive/stats10.q a3f375e 
>   ql/src/test/queries/clientpositive/tez_bmj_schema_evolution.q a06bb82 
>   ql/src/test/queries/clientpositive/tez_fsstat.q 90201b6 
>   ql/src/test/queries/clientpositive/tez_smb_1.q 03a1fea 
>   ql/src/test/queries/clientpositive/tez_smb_empty.q 196cc97 
>   ql/src/test/queries/clientpositive/tez_smb_main.q dff5112 
>   ql/src/test/queries/clientpositive/transform_acid.q 94782f1 
>   ql/src/test/queries/clientpositive/truncate_column_buckets.q a2ce215 
>   ql/src/test/queries/clientpositive/update_after_multiple_inserts.q eb8e5a1 
>   ql/src/test/queries/clientpositive/update_all_non_partitioned.q 3c01825 
>   ql/src/test/queries/clientpositive/update_all_partitioned.q e191d0a 
>   ql/src/test/queries/clientpositive/update_all_types.q 0229845 
>   ql/src/test/queries/clientpositive/update_orig_table.q 416c841 
>   ql/src/test/queries/clientpositive/update_tmp_table.q a896ac7 
>   ql/src/test/queries/clientpositive/update_two_cols.q b1972e5 
>   ql/src/test/queries/clientpositive/update_where_no_match.q d578862 
>   ql/src/test/queries/clientpositive/update_where_non_partitioned.q 06c688f 
>   ql/src/test/queries/clientpositive/update_where_partitioned.q 157712f 
>   ql/src/test/queries/clientpositive/vector_auto_smb_mapjoin_14.q 32be5ee 
>   ql/src/test/queries/clientpositive/vector_bucket.q 9360ce0 
>   ql/src/test/results/clientpositive/alter_numbuckets_partitioned_table.q.out 
> b1dfd7c 
>   
> ql/src/test/results/clientpositive/alter_numbuckets_partitioned_table2.q.out 
> e5f8e7f 
>   
> ql/src/test/results/clientpositive/alter_numbuckets_partitioned_table2_h23.q.out
>  f919f10 
>   ql/src/test/results/clientpositive/cp_sel.q.out a2c9fe0 
>   ql/src/test/results/clientpositive/index_auto_update.q.out 11af3f5 
>   ql/src/test/results/clientpositive/insert_into_with_schema2.q.out 32e6e92 
>   ql/src/test/results/clientpositive/orc_analyze.q.out bc46852 
>   ql/src/test/results/clientpositive/smb_mapjoin_11.q.out 7b54dbe 
>   ql/src/test/results/clientpositive/spark/smb_mapjoin_11.q.out 4d912ca 
>   ql/src/test/results/clientpositive/spark/smb_mapjoin_12.q.out eeb18b0 
>   ql/src/test/results/clientpositive/spark/stats9.q.out 7eae829 
>   ql/src/test/results/clientpositive/stats9.q.out e7c7743 
>   ql/src/test/results/clientpositive/tez/orc_analyze.q.out bc46852 
> 
> Diff: https://reviews.apache.org/r/39960/diff/
> 
> 
> Testing
> -------
> 
> Existing tests
> 
> 
> Thanks,
> 
> Ashutosh Chauhan
> 
>

Reply via email to