Re: Review Request 72491: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 4

2020-05-13 Thread Zoltan Haindrich 

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




ql/src/test/results/clientpositive/llap/union35.q.out
Line 18 (original)


resultset order change



ql/src/test/results/clientpositive/llap/union_pos_alias.q.out
Line 461 (original), 462 (patched)


huge resultset change



ql/src/test/results/clientpositive/llap/union_pos_alias.q.out
Lines 1895 (patched)


huge resultset change


- Zoltan Haindrich


On May 11, 2020, 9:05 p.m., Miklos Gergely wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72491/
> ---
> 
> (Updated May 11, 2020, 9:05 p.m.)
> 
> 
> Review request for hive, Jesús Camacho Rodríguez, John Sherman, Zoltan 
> Haindrich, Krisztian Kasa, Steve Carlin, and Vineet Garg.
> 
> 
> Bugs: HIVE-23440
> https://issues.apache.org/jira/browse/HIVE-23440
> 
> 
> Repository: hive-git
> 
> 
> Description
> ---
> 
> Move q tests to TestMiniLlapLocal from TestCliDriver where the output is 
> different, batch 4
> 
> 
> Diffs
> -
> 
>   ql/src/test/queries/clientpositive/union.q 3f40a25d49 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition.q.out
>  8b1cfad5ab 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition2.q.out
>  413a3f2a63 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition3.q.out
>  12d5d59fb1 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition4.q.out
>  8ddbb96fca 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition5.q.out
>  7dbf56cf39 
>   ql/src/test/results/clientpositive/llap/temp_table_options1.q.out 
> be31a5a289 
>   
> ql/src/test/results/clientpositive/llap/temp_table_parquet_mixed_partition_formats2.q.out
>  23bb41edfe 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_boolexpr.q.out 
> d4af83b320 
>   
> ql/src/test/results/clientpositive/llap/temp_table_partition_condition_remover.q.out
>  18f5348f0f 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_ctas.q.out 
> bd3574f03f 
>   
> ql/src/test/results/clientpositive/llap/temp_table_partition_multilevels.q.out
>  2ea8bf8631 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_pruning.q.out 
> f6fdd61928 
>   
> ql/src/test/results/clientpositive/llap/temp_table_windowing_expressions.q.out
>  c45f36e988 
>   ql/src/test/results/clientpositive/llap/test_teradatabinaryfile.q.out 
> 75584e9ba2 
>   ql/src/test/results/clientpositive/llap/timestamp.q.out 90a46f58f4 
>   ql/src/test/results/clientpositive/llap/timestamp_comparison3.q.out 
> 3977be77f7 
>   ql/src/test/results/clientpositive/llap/timestamp_ints_casts.q.out 
> 572c49ea72 
>   ql/src/test/results/clientpositive/llap/timestamp_literal.q.out cfcd06f907 
>   ql/src/test/results/clientpositive/llap/timestamptz.q.out 09c50ddf10 
>   ql/src/test/results/clientpositive/llap/truncate_column_list_bucket.q.out 
> c8e40bd447 
>   ql/src/test/results/clientpositive/llap/type_cast_1.q.out 22dad1a0f2 
>   ql/src/test/results/clientpositive/llap/type_widening.q.out f295e66ee9 
>   ql/src/test/results/clientpositive/llap/udaf_binarysetfunctions.q.out 
> 86dbcf6f57 
>   
> ql/src/test/results/clientpositive/llap/udaf_binarysetfunctions_no_cbo.q.out 
> 6857ca9739 
>   ql/src/test/results/clientpositive/llap/udaf_number_format.q.out 822ea784ba 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_approx_23.q.out 
> c200ecf75a 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_cont.q.out 
> 509ae7bfe6 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_disc.q.out 
> e7efcf9302 
>   ql/src/test/results/clientpositive/llap/udf1.q.out 9647770bcd 
>   ql/src/test/results/clientpositive/llap/udf2.q.out bcc2faa16a 
>   ql/src/test/results/clientpositive/llap/udf3.q.out 18abd9560c 
>   ql/src/test/results/clientpositive/llap/udf4.q.out d9b841aab9 
>   ql/src/test/results/clientpositive/llap/udf5.q.out 58a1dab60b 
>   ql/src/test/results/clientpositive/llap/udf6.q.out e6d58324c7 
>   ql/src/test/results/clientpositive/llap/udf7.q.out 44b282f82e 
>   ql/src/test/results/clientpositive/llap/udf8.q.out 8e8ca424b4 
>   ql/src/test/results/clientpositive/llap/udf9.q.out a55b3cdb34 
>   ql/src/test/results/clientpositive/llap/udf_10_trims.q.out 41eefa3e8c 
>   ql/src/test/results/clientpositive/llap/udf_E.q.out 469f396a85 
>   ql/src/test/results/clientpositive/llap/udf_PI.q.out a9ec8c1e06 
>   ql/src/test/results/clientpositive/llap/udf_abs.q.out fee7592ec9 
>

Re: Review Request 72491: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 4

2020-05-11 Thread Krisztian Kasa 

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




ql/src/test/results/clientpositive/llap/union_view.q.out
Line 36 (original)


Statistics missing



ql/src/test/results/clientpositive/llap/uniquejoin.q.out
Lines 98 (patched)


Result set changed: rows added



ql/src/test/results/clientpositive/llap/uniquejoin.q.out
Lines 121 (patched)


Result set changed: rows added



ql/src/test/results/clientpositive/llap/uniquejoin.q.out
Lines 170 (patched)


Result set changed: this row was added



ql/src/test/results/clientpositive/llap/uniquejoin.q.out
Lines 193 (patched)


Result set changed: rows added



ql/src/test/results/clientpositive/llap/varchar_serde.q.out
Lines 81 (patched)


Result set changed: order changed



ql/src/test/results/clientpositive/llap/varchar_serde.q.out
Lines 142 (patched)


Result set changed. Probably because limit without ourder by.



ql/src/test/results/clientpositive/llap/varchar_serde.q.out
Lines 202 (patched)


Result set changed: limit without ourder by.



ql/src/test/results/clientpositive/llap/varchar_serde.q.out
Lines 262 (patched)


Result set changed: limit without ourder by.



ql/src/test/results/clientpositive/llap/varchar_serde.q.out
Lines 322 (patched)


Result set changed: limit without ourder by.



ql/src/test/results/clientpositive/llap/varchar_serde.q.out
Lines 382 (patched)


Result set changed: limit without ourder by.



ql/src/test/results/clientpositive/llap/vectorization_parquet_ppd_decimal.q.out
Lines 38 (patched)


Result set changed: rows added



ql/src/test/results/clientpositive/llap/vectorization_parquet_ppd_decimal.q.out
Lines 56 (patched)


Result set changed: rows added



ql/src/test/results/clientpositive/llap/vectorization_parquet_ppd_decimal.q.out
Lines 74 (patched)


Result set changed: rows added



ql/src/test/results/clientpositive/llap/vectorization_parquet_ppd_decimal.q.out
Lines 92 (patched)


Result set changed: rows added



ql/src/test/results/clientpositive/llap/vectorization_parquet_ppd_decimal.q.out
Lines 110 (patched)


Result set changed: rows added



ql/src/test/results/clientpositive/llap/vectorization_parquet_ppd_decimal.q.out
Lines 128 (patched)


Result set changed: rows added



ql/src/test/results/clientpositive/llap/vectorization_parquet_ppd_decimal.q.out
Line 111 (original)


Result set changed: empty result set



ql/src/test/results/clientpositive/llap/vectorization_parquet_ppd_decimal.q.out
Line 124 (original)


Result set changed: empty result set



ql/src/test/results/clientpositive/llap/vectorization_parquet_ppd_decimal.q.out
Line 137 (original)


Result set changed: empty result set



ql/src/test/results/clientpositive/llap/vectorization_parquet_ppd_decimal.q.out
Line 150 (original)


Result set changed: empty result set



ql/src/test/results/clientpositive/llap/vectorization_parquet_ppd_decimal.q.out
Line 838 (original)


This file contains lots of result set changes



ql/src/test/results/clientpositive/llap/view_alias.q.out
Lines 330 (patched)


Result set changed



ql/src/test/results/clientpositive/llap/windowing_expressions.q.out
Line 596 (original), 596 (patched)


Result set changed



ql/src/test/results/clientpositive/llap/windowing_multipartitioning.q.out
Lines 238 (patched)


Result set changed



ql/src/test/results/clientpositive/llap/windowing_range_multiorder.q.out
Lines 478 (patched)


Result set changed

Re: Review Request 72491: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 4

2020-05-11 Thread Jesús Camacho Rodríguez 

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




ql/src/test/results/clientpositive/llap/temp_table_options1.q.out
Lines 248 (patched)


Different order



ql/src/test/results/clientpositive/llap/temp_table_parquet_mixed_partition_formats2.q.out
Lines 99 (patched)


Different order



ql/src/test/results/clientpositive/llap/temp_table_partition_multilevels.q.out
Lines 482 (patched)


Different order all over this file. Results are the same



ql/src/test/results/clientpositive/llap/temp_table_partition_multilevels.q.out
Lines 523 (patched)


Different order



ql/src/test/results/clientpositive/llap/udaf_percentile_approx_23.q.out
Line 280 (original), 280 (patched)


Different results. This may be due to order of rows since it is an 
approximate UDF. However, before moving the test, we should make sure that is 
the case.


- Jesús Camacho Rodríguez


On May 11, 2020, 7:05 p.m., Miklos Gergely wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72491/
> ---
> 
> (Updated May 11, 2020, 7:05 p.m.)
> 
> 
> Review request for hive, Jesús Camacho Rodríguez, John Sherman, Zoltan 
> Haindrich, Krisztian Kasa, Steve Carlin, and Vineet Garg.
> 
> 
> Bugs: HIVE-23440
> https://issues.apache.org/jira/browse/HIVE-23440
> 
> 
> Repository: hive-git
> 
> 
> Description
> ---
> 
> Move q tests to TestMiniLlapLocal from TestCliDriver where the output is 
> different, batch 4
> 
> 
> Diffs
> -
> 
>   ql/src/test/queries/clientpositive/union.q 3f40a25d49 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition.q.out
>  8b1cfad5ab 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition2.q.out
>  413a3f2a63 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition3.q.out
>  12d5d59fb1 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition4.q.out
>  8ddbb96fca 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition5.q.out
>  7dbf56cf39 
>   ql/src/test/results/clientpositive/llap/temp_table_options1.q.out 
> be31a5a289 
>   
> ql/src/test/results/clientpositive/llap/temp_table_parquet_mixed_partition_formats2.q.out
>  23bb41edfe 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_boolexpr.q.out 
> d4af83b320 
>   
> ql/src/test/results/clientpositive/llap/temp_table_partition_condition_remover.q.out
>  18f5348f0f 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_ctas.q.out 
> bd3574f03f 
>   
> ql/src/test/results/clientpositive/llap/temp_table_partition_multilevels.q.out
>  2ea8bf8631 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_pruning.q.out 
> f6fdd61928 
>   
> ql/src/test/results/clientpositive/llap/temp_table_windowing_expressions.q.out
>  c45f36e988 
>   ql/src/test/results/clientpositive/llap/test_teradatabinaryfile.q.out 
> 75584e9ba2 
>   ql/src/test/results/clientpositive/llap/timestamp.q.out 90a46f58f4 
>   ql/src/test/results/clientpositive/llap/timestamp_comparison3.q.out 
> 3977be77f7 
>   ql/src/test/results/clientpositive/llap/timestamp_ints_casts.q.out 
> 572c49ea72 
>   ql/src/test/results/clientpositive/llap/timestamp_literal.q.out cfcd06f907 
>   ql/src/test/results/clientpositive/llap/timestamptz.q.out 09c50ddf10 
>   ql/src/test/results/clientpositive/llap/truncate_column_list_bucket.q.out 
> c8e40bd447 
>   ql/src/test/results/clientpositive/llap/type_cast_1.q.out 22dad1a0f2 
>   ql/src/test/results/clientpositive/llap/type_widening.q.out f295e66ee9 
>   ql/src/test/results/clientpositive/llap/udaf_binarysetfunctions.q.out 
> 86dbcf6f57 
>   
> ql/src/test/results/clientpositive/llap/udaf_binarysetfunctions_no_cbo.q.out 
> 6857ca9739 
>   ql/src/test/results/clientpositive/llap/udaf_number_format.q.out 822ea784ba 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_approx_23.q.out 
> c200ecf75a 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_cont.q.out 
> 509ae7bfe6 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_disc.q.out 
> e7efcf9302 
>   ql/src/test/results/clientpositive/llap/udf1.q.out 9647770bcd 
>   ql/src/test/results/clientpositive/llap/udf2.q.out bcc2faa16a 
>   ql/src/test/results/clientpositive/llap/udf3.q.out 18abd9560c 
>   ql/src/test/results/clientpositive/llap/udf4.q.out d9b841aab9 
>   ql/src/test/results/clientpositive/llap/udf5.q.out

Re: Review Request 72491: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 4

2020-05-11 Thread Vineet Garg 

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




ql/src/test/results/clientpositive/llap/udf_mask_show_last_n.q.out
Line 80 (original), 78 (patched)


Result mismatch


- Vineet Garg


On May 11, 2020, 7:05 p.m., Miklos Gergely wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72491/
> ---
> 
> (Updated May 11, 2020, 7:05 p.m.)
> 
> 
> Review request for hive, Jesús Camacho Rodríguez, John Sherman, Zoltan 
> Haindrich, Krisztian Kasa, Steve Carlin, and Vineet Garg.
> 
> 
> Bugs: HIVE-23440
> https://issues.apache.org/jira/browse/HIVE-23440
> 
> 
> Repository: hive-git
> 
> 
> Description
> ---
> 
> Move q tests to TestMiniLlapLocal from TestCliDriver where the output is 
> different, batch 4
> 
> 
> Diffs
> -
> 
>   ql/src/test/queries/clientpositive/union.q 3f40a25d49 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition.q.out
>  8b1cfad5ab 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition2.q.out
>  413a3f2a63 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition3.q.out
>  12d5d59fb1 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition4.q.out
>  8ddbb96fca 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition5.q.out
>  7dbf56cf39 
>   ql/src/test/results/clientpositive/llap/temp_table_options1.q.out 
> be31a5a289 
>   
> ql/src/test/results/clientpositive/llap/temp_table_parquet_mixed_partition_formats2.q.out
>  23bb41edfe 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_boolexpr.q.out 
> d4af83b320 
>   
> ql/src/test/results/clientpositive/llap/temp_table_partition_condition_remover.q.out
>  18f5348f0f 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_ctas.q.out 
> bd3574f03f 
>   
> ql/src/test/results/clientpositive/llap/temp_table_partition_multilevels.q.out
>  2ea8bf8631 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_pruning.q.out 
> f6fdd61928 
>   
> ql/src/test/results/clientpositive/llap/temp_table_windowing_expressions.q.out
>  c45f36e988 
>   ql/src/test/results/clientpositive/llap/test_teradatabinaryfile.q.out 
> 75584e9ba2 
>   ql/src/test/results/clientpositive/llap/timestamp.q.out 90a46f58f4 
>   ql/src/test/results/clientpositive/llap/timestamp_comparison3.q.out 
> 3977be77f7 
>   ql/src/test/results/clientpositive/llap/timestamp_ints_casts.q.out 
> 572c49ea72 
>   ql/src/test/results/clientpositive/llap/timestamp_literal.q.out cfcd06f907 
>   ql/src/test/results/clientpositive/llap/timestamptz.q.out 09c50ddf10 
>   ql/src/test/results/clientpositive/llap/truncate_column_list_bucket.q.out 
> c8e40bd447 
>   ql/src/test/results/clientpositive/llap/type_cast_1.q.out 22dad1a0f2 
>   ql/src/test/results/clientpositive/llap/type_widening.q.out f295e66ee9 
>   ql/src/test/results/clientpositive/llap/udaf_binarysetfunctions.q.out 
> 86dbcf6f57 
>   
> ql/src/test/results/clientpositive/llap/udaf_binarysetfunctions_no_cbo.q.out 
> 6857ca9739 
>   ql/src/test/results/clientpositive/llap/udaf_number_format.q.out 822ea784ba 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_approx_23.q.out 
> c200ecf75a 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_cont.q.out 
> 509ae7bfe6 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_disc.q.out 
> e7efcf9302 
>   ql/src/test/results/clientpositive/llap/udf1.q.out 9647770bcd 
>   ql/src/test/results/clientpositive/llap/udf2.q.out bcc2faa16a 
>   ql/src/test/results/clientpositive/llap/udf3.q.out 18abd9560c 
>   ql/src/test/results/clientpositive/llap/udf4.q.out d9b841aab9 
>   ql/src/test/results/clientpositive/llap/udf5.q.out 58a1dab60b 
>   ql/src/test/results/clientpositive/llap/udf6.q.out e6d58324c7 
>   ql/src/test/results/clientpositive/llap/udf7.q.out 44b282f82e 
>   ql/src/test/results/clientpositive/llap/udf8.q.out 8e8ca424b4 
>   ql/src/test/results/clientpositive/llap/udf9.q.out a55b3cdb34 
>   ql/src/test/results/clientpositive/llap/udf_10_trims.q.out 41eefa3e8c 
>   ql/src/test/results/clientpositive/llap/udf_E.q.out 469f396a85 
>   ql/src/test/results/clientpositive/llap/udf_PI.q.out a9ec8c1e06 
>   ql/src/test/results/clientpositive/llap/udf_abs.q.out fee7592ec9 
>   ql/src/test/results/clientpositive/llap/udf_add_months.q.out 7150f59d68 
>   ql/src/test/results/clientpositive/llap/udf_aes_decrypt.q.out 89caf994cf 
>   ql/src/test/results/clientpositive/llap/udf_aes_encrypt.q.out 3a96c4bc44 
>   ql/src/test/results/clientpositive/llap/udf_array.q.out a2ebabaaff 
>

Re: Review Request 72491: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 4

2020-05-11 Thread Steve Carlin via Review Board 

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




ql/src/test/results/clientpositive/llap/udf_concat_insert1.q.out
Lines 56 (patched)


order difference



ql/src/test/results/clientpositive/llap/udf_explode.q.out
Lines 218 (patched)


order difference


- Steve Carlin


On May 11, 2020, 7:05 p.m., Miklos Gergely wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72491/
> ---
> 
> (Updated May 11, 2020, 7:05 p.m.)
> 
> 
> Review request for hive, Jesús Camacho Rodríguez, John Sherman, Zoltan 
> Haindrich, Krisztian Kasa, Steve Carlin, and Vineet Garg.
> 
> 
> Bugs: HIVE-23440
> https://issues.apache.org/jira/browse/HIVE-23440
> 
> 
> Repository: hive-git
> 
> 
> Description
> ---
> 
> Move q tests to TestMiniLlapLocal from TestCliDriver where the output is 
> different, batch 4
> 
> 
> Diffs
> -
> 
>   ql/src/test/queries/clientpositive/union.q 3f40a25d49 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition.q.out
>  8b1cfad5ab 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition2.q.out
>  413a3f2a63 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition3.q.out
>  12d5d59fb1 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition4.q.out
>  8ddbb96fca 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition5.q.out
>  7dbf56cf39 
>   ql/src/test/results/clientpositive/llap/temp_table_options1.q.out 
> be31a5a289 
>   
> ql/src/test/results/clientpositive/llap/temp_table_parquet_mixed_partition_formats2.q.out
>  23bb41edfe 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_boolexpr.q.out 
> d4af83b320 
>   
> ql/src/test/results/clientpositive/llap/temp_table_partition_condition_remover.q.out
>  18f5348f0f 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_ctas.q.out 
> bd3574f03f 
>   
> ql/src/test/results/clientpositive/llap/temp_table_partition_multilevels.q.out
>  2ea8bf8631 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_pruning.q.out 
> f6fdd61928 
>   
> ql/src/test/results/clientpositive/llap/temp_table_windowing_expressions.q.out
>  c45f36e988 
>   ql/src/test/results/clientpositive/llap/test_teradatabinaryfile.q.out 
> 75584e9ba2 
>   ql/src/test/results/clientpositive/llap/timestamp.q.out 90a46f58f4 
>   ql/src/test/results/clientpositive/llap/timestamp_comparison3.q.out 
> 3977be77f7 
>   ql/src/test/results/clientpositive/llap/timestamp_ints_casts.q.out 
> 572c49ea72 
>   ql/src/test/results/clientpositive/llap/timestamp_literal.q.out cfcd06f907 
>   ql/src/test/results/clientpositive/llap/timestamptz.q.out 09c50ddf10 
>   ql/src/test/results/clientpositive/llap/truncate_column_list_bucket.q.out 
> c8e40bd447 
>   ql/src/test/results/clientpositive/llap/type_cast_1.q.out 22dad1a0f2 
>   ql/src/test/results/clientpositive/llap/type_widening.q.out f295e66ee9 
>   ql/src/test/results/clientpositive/llap/udaf_binarysetfunctions.q.out 
> 86dbcf6f57 
>   
> ql/src/test/results/clientpositive/llap/udaf_binarysetfunctions_no_cbo.q.out 
> 6857ca9739 
>   ql/src/test/results/clientpositive/llap/udaf_number_format.q.out 822ea784ba 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_approx_23.q.out 
> c200ecf75a 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_cont.q.out 
> 509ae7bfe6 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_disc.q.out 
> e7efcf9302 
>   ql/src/test/results/clientpositive/llap/udf1.q.out 9647770bcd 
>   ql/src/test/results/clientpositive/llap/udf2.q.out bcc2faa16a 
>   ql/src/test/results/clientpositive/llap/udf3.q.out 18abd9560c 
>   ql/src/test/results/clientpositive/llap/udf4.q.out d9b841aab9 
>   ql/src/test/results/clientpositive/llap/udf5.q.out 58a1dab60b 
>   ql/src/test/results/clientpositive/llap/udf6.q.out e6d58324c7 
>   ql/src/test/results/clientpositive/llap/udf7.q.out 44b282f82e 
>   ql/src/test/results/clientpositive/llap/udf8.q.out 8e8ca424b4 
>   ql/src/test/results/clientpositive/llap/udf9.q.out a55b3cdb34 
>   ql/src/test/results/clientpositive/llap/udf_10_trims.q.out 41eefa3e8c 
>   ql/src/test/results/clientpositive/llap/udf_E.q.out 469f396a85 
>   ql/src/test/results/clientpositive/llap/udf_PI.q.out a9ec8c1e06 
>   ql/src/test/results/clientpositive/llap/udf_abs.q.out fee7592ec9 
>   ql/src/test/results/clientpositive/llap/udf_add_months.q.out 7150f59d68 
>   ql/src/test/results/clientpositive/llap/udf_aes_decrypt.q.out 89caf994cf 
>

Re: Review Request 72491: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 4

2020-05-11 Thread Miklos Gergely 

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




ql/src/test/results/clientpositive/llap/udf_sort_array_by.q.out
Line 322 (original)


Order has changed



ql/src/test/results/clientpositive/llap/udf_width_bucket.q.out
Line 630 (original), 628 (patched)


Order has changed



ql/src/test/results/clientpositive/llap/udtf_explode.q.out
Lines 251 (patched)


Order has changed



ql/src/test/results/clientpositive/llap/udtf_parse_url_tuple.q.out
Line 427 (original)


Order has changed



ql/src/test/results/clientpositive/llap/union.q.out
Line 136 (original)


result set change, most likely only the order



ql/src/test/results/clientpositive/llap/union13.q.out
Line 70 (original), 81 (patched)


Result set change, most likely just the order



ql/src/test/results/clientpositive/llap/union15.q.out
Line 171 (original), 177 (patched)


Result set change, most likely just the order


- Miklos Gergely


On May 11, 2020, 7:05 p.m., Miklos Gergely wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72491/
> ---
> 
> (Updated May 11, 2020, 7:05 p.m.)
> 
> 
> Review request for hive, Jesús Camacho Rodríguez, John Sherman, Zoltan 
> Haindrich, Krisztian Kasa, Steve Carlin, and Vineet Garg.
> 
> 
> Bugs: HIVE-23440
> https://issues.apache.org/jira/browse/HIVE-23440
> 
> 
> Repository: hive-git
> 
> 
> Description
> ---
> 
> Move q tests to TestMiniLlapLocal from TestCliDriver where the output is 
> different, batch 4
> 
> 
> Diffs
> -
> 
>   ql/src/test/queries/clientpositive/union.q 3f40a25d49 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition.q.out
>  8b1cfad5ab 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition2.q.out
>  413a3f2a63 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition3.q.out
>  12d5d59fb1 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition4.q.out
>  8ddbb96fca 
>   
> ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition5.q.out
>  7dbf56cf39 
>   ql/src/test/results/clientpositive/llap/temp_table_options1.q.out 
> be31a5a289 
>   
> ql/src/test/results/clientpositive/llap/temp_table_parquet_mixed_partition_formats2.q.out
>  23bb41edfe 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_boolexpr.q.out 
> d4af83b320 
>   
> ql/src/test/results/clientpositive/llap/temp_table_partition_condition_remover.q.out
>  18f5348f0f 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_ctas.q.out 
> bd3574f03f 
>   
> ql/src/test/results/clientpositive/llap/temp_table_partition_multilevels.q.out
>  2ea8bf8631 
>   ql/src/test/results/clientpositive/llap/temp_table_partition_pruning.q.out 
> f6fdd61928 
>   
> ql/src/test/results/clientpositive/llap/temp_table_windowing_expressions.q.out
>  c45f36e988 
>   ql/src/test/results/clientpositive/llap/test_teradatabinaryfile.q.out 
> 75584e9ba2 
>   ql/src/test/results/clientpositive/llap/timestamp.q.out 90a46f58f4 
>   ql/src/test/results/clientpositive/llap/timestamp_comparison3.q.out 
> 3977be77f7 
>   ql/src/test/results/clientpositive/llap/timestamp_ints_casts.q.out 
> 572c49ea72 
>   ql/src/test/results/clientpositive/llap/timestamp_literal.q.out cfcd06f907 
>   ql/src/test/results/clientpositive/llap/timestamptz.q.out 09c50ddf10 
>   ql/src/test/results/clientpositive/llap/truncate_column_list_bucket.q.out 
> c8e40bd447 
>   ql/src/test/results/clientpositive/llap/type_cast_1.q.out 22dad1a0f2 
>   ql/src/test/results/clientpositive/llap/type_widening.q.out f295e66ee9 
>   ql/src/test/results/clientpositive/llap/udaf_binarysetfunctions.q.out 
> 86dbcf6f57 
>   
> ql/src/test/results/clientpositive/llap/udaf_binarysetfunctions_no_cbo.q.out 
> 6857ca9739 
>   ql/src/test/results/clientpositive/llap/udaf_number_format.q.out 822ea784ba 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_approx_23.q.out 
> c200ecf75a 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_cont.q.out 
> 509ae7bfe6 
>   ql/src/test/results/clientpositive/llap/udaf_percentile_disc.q.out 
> e7efcf9302 
>   ql/src/test/results/clientpositive/llap/udf1.q.out 9647770bcd 
>   ql/src/test/results/clientpositive/llap/udf2.q.out bcc2faa16a 
>

Review Request 72491: Move q tests to TestMiniLlapLocal from TestCliDriver where the output is different, batch 4

2020-05-11 Thread Miklos Gergely 

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

Review request for hive, Jesús Camacho Rodríguez, John Sherman, Zoltan 
Haindrich, Krisztian Kasa, Steve Carlin, and Vineet Garg.


Bugs: HIVE-23440
https://issues.apache.org/jira/browse/HIVE-23440


Repository: hive-git


Description
---

Move q tests to TestMiniLlapLocal from TestCliDriver where the output is 
different, batch 4


Diffs
-

  ql/src/test/queries/clientpositive/union.q 3f40a25d49 
  
ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition.q.out
 8b1cfad5ab 
  
ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition2.q.out
 413a3f2a63 
  
ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition3.q.out
 12d5d59fb1 
  
ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition4.q.out
 8ddbb96fca 
  
ql/src/test/results/clientpositive/llap/temp_table_merge_dynamic_partition5.q.out
 7dbf56cf39 
  ql/src/test/results/clientpositive/llap/temp_table_options1.q.out be31a5a289 
  
ql/src/test/results/clientpositive/llap/temp_table_parquet_mixed_partition_formats2.q.out
 23bb41edfe 
  ql/src/test/results/clientpositive/llap/temp_table_partition_boolexpr.q.out 
d4af83b320 
  
ql/src/test/results/clientpositive/llap/temp_table_partition_condition_remover.q.out
 18f5348f0f 
  ql/src/test/results/clientpositive/llap/temp_table_partition_ctas.q.out 
bd3574f03f 
  
ql/src/test/results/clientpositive/llap/temp_table_partition_multilevels.q.out 
2ea8bf8631 
  ql/src/test/results/clientpositive/llap/temp_table_partition_pruning.q.out 
f6fdd61928 
  
ql/src/test/results/clientpositive/llap/temp_table_windowing_expressions.q.out 
c45f36e988 
  ql/src/test/results/clientpositive/llap/test_teradatabinaryfile.q.out 
75584e9ba2 
  ql/src/test/results/clientpositive/llap/timestamp.q.out 90a46f58f4 
  ql/src/test/results/clientpositive/llap/timestamp_comparison3.q.out 
3977be77f7 
  ql/src/test/results/clientpositive/llap/timestamp_ints_casts.q.out 572c49ea72 
  ql/src/test/results/clientpositive/llap/timestamp_literal.q.out cfcd06f907 
  ql/src/test/results/clientpositive/llap/timestamptz.q.out 09c50ddf10 
  ql/src/test/results/clientpositive/llap/truncate_column_list_bucket.q.out 
c8e40bd447 
  ql/src/test/results/clientpositive/llap/type_cast_1.q.out 22dad1a0f2 
  ql/src/test/results/clientpositive/llap/type_widening.q.out f295e66ee9 
  ql/src/test/results/clientpositive/llap/udaf_binarysetfunctions.q.out 
86dbcf6f57 
  ql/src/test/results/clientpositive/llap/udaf_binarysetfunctions_no_cbo.q.out 
6857ca9739 
  ql/src/test/results/clientpositive/llap/udaf_number_format.q.out 822ea784ba 
  ql/src/test/results/clientpositive/llap/udaf_percentile_approx_23.q.out 
c200ecf75a 
  ql/src/test/results/clientpositive/llap/udaf_percentile_cont.q.out 509ae7bfe6 
  ql/src/test/results/clientpositive/llap/udaf_percentile_disc.q.out e7efcf9302 
  ql/src/test/results/clientpositive/llap/udf1.q.out 9647770bcd 
  ql/src/test/results/clientpositive/llap/udf2.q.out bcc2faa16a 
  ql/src/test/results/clientpositive/llap/udf3.q.out 18abd9560c 
  ql/src/test/results/clientpositive/llap/udf4.q.out d9b841aab9 
  ql/src/test/results/clientpositive/llap/udf5.q.out 58a1dab60b 
  ql/src/test/results/clientpositive/llap/udf6.q.out e6d58324c7 
  ql/src/test/results/clientpositive/llap/udf7.q.out 44b282f82e 
  ql/src/test/results/clientpositive/llap/udf8.q.out 8e8ca424b4 
  ql/src/test/results/clientpositive/llap/udf9.q.out a55b3cdb34 
  ql/src/test/results/clientpositive/llap/udf_10_trims.q.out 41eefa3e8c 
  ql/src/test/results/clientpositive/llap/udf_E.q.out 469f396a85 
  ql/src/test/results/clientpositive/llap/udf_PI.q.out a9ec8c1e06 
  ql/src/test/results/clientpositive/llap/udf_abs.q.out fee7592ec9 
  ql/src/test/results/clientpositive/llap/udf_add_months.q.out 7150f59d68 
  ql/src/test/results/clientpositive/llap/udf_aes_decrypt.q.out 89caf994cf 
  ql/src/test/results/clientpositive/llap/udf_aes_encrypt.q.out 3a96c4bc44 
  ql/src/test/results/clientpositive/llap/udf_array.q.out a2ebabaaff 
  ql/src/test/results/clientpositive/llap/udf_ascii.q.out 42dd231560 
  ql/src/test/results/clientpositive/llap/udf_between.q.out 1a4eb076ef 
  ql/src/test/results/clientpositive/llap/udf_bitwise_shiftleft.q.out 
0dd68b74b7 
  ql/src/test/results/clientpositive/llap/udf_bitwise_shiftright.q.out 
9b3b17d4a9 
  ql/src/test/results/clientpositive/llap/udf_bitwise_shiftrightunsigned.q.out 
be499580ab 
  ql/src/test/results/clientpositive/llap/udf_case.q.out 60a348e530 
  ql/src/test/results/clientpositive/llap/udf_case_column_pruning.q.out 
658cfa8361 
  ql/src/test/results/clientpositive/llap/udf_case_thrift.q.out e540ff9fe9 
  ql/src/test/results/clientpositive/llap/udf_cbrt.q.out fd02051240 
  ql/src/test/results/clientpositive/llap/udf_character_length.q.out