> On Dec. 17, 2019, 10:01 p.m., Ramesh Kumar Thangarajan wrote:
> > ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/Vectorizer.java
> > Lines 4341 (patched)
> > <https://reviews.apache.org/r/71820/diff/14/?file=2185755#file2185755line4341>
> >
> >     What is the bug in Decimal64 to Decimal conversion? 
> >     
> >     Do we need a call to the function 
> > fixDecimalDataTypePhysicalVariations() ? Because I see the function 
> > getVectorExpressionsUpConvertDecimal64() is calling 
> > wrapWithDecimal64ToDecimalConversion() on all the child expressions.
> 
> Krisztian Kasa wrote:
>     Please consider the following query: 
>     ```
>     CREATE TABLE decimal_test_small_n0 STORED AS ORC AS SELECT cdouble, CAST 
> (((cdouble*22.1)/37) AS DECIMAL(10,3)) AS cdecimal1, CAST (((cdouble*9.3)/13) 
> AS DECIMAL(7,2)) AS cdecimal2 FROM alltypesorc;
>     
>     SELECT cdecimal1 - (2*cdecimal2) as c2 FROM decimal_test_small_n0
>     ORDER BY c2
>     LIMIT 10;
>     ```
>     
>     With given keyColumns: GenericUDFOPMinus(Column[cdecimal1], 
> GenericUDFOPMultiply(Const decimal(1,0) 2, Column[cdecimal2])) 
>     ```
>     keyExpressions = 
> vContext.getVectorExpressionsUpConvertDecimal64(keyColumns);
>     ```
>     will produce keyExpressions: 
>     DecimalColSubtractDecimalColumn(col 4:decimal(10,3), col 
> 5:decimal(9,2)/DECIMAL_64)
>     (children: 
>         ConvertDecimal64ToDecimal(col 1:decimal(10,3)/DECIMAL_64) -> 
> 4:decimal(10,3), 
>         Decimal64ScalarMultiplyDecimal64ColumnUnscaled(decimal64Val 2, 
> decimalVal 2, col 2:decimal(7,2)/DECIMAL_64) -> 5:decimal(9,2)/DECIMAL_64
>     ) -> 6:decimal(11,3)
>     
>     So the 2nd child of DecimalColSubtractDecimalColumn is not converted from 
> decimal64 to decimal and I got a 
>     ```
>      java.lang.ClassCastException: 
> org.apache.hadoop.hive.ql.exec.vector.Decimal64ColumnVector cannot be cast to 
> org.apache.hadoop.hive.ql.exec.vector.DecimalColumnVector
>       at 
> org.apache.hadoop.hive.ql.exec.vector.expressions.gen.DecimalColSubtractDecimalColumn.evaluate(DecimalColSubtractDecimalColumn.java:69)
>       at 
> org.apache.hadoop.hive.ql.exec.vector.VectorTopNKeyOperator.process(VectorTopNKeyOperator.java:101)
>     ```
>     
>     Checking the code of `getVectorExpressionsUpConvertDecimal64` i found 
> that it calls `getVectorExpressions` and for each result expression if that 
> expression's output type is decimal64 then it wraps it with conversion. But 
> it does not checks the children expressions.
>     
>     Probably we don't need both function calls, please let me check in a 
> follow up patch if `fixDecimalDataTypePhysicalVariations` is enough.

Makes sense! Thanks for the explanation


- Ramesh Kumar


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


On Dec. 14, 2019, 10:31 a.m., Krisztian Kasa wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/71820/
> -----------------------------------------------------------
> 
> (Updated Dec. 14, 2019, 10:31 a.m.)
> 
> 
> Review request for hive, Jesús Camacho Rodríguez and Zoltan Haindrich.
> 
> 
> Bugs: HIVE-20150
>     https://issues.apache.org/jira/browse/HIVE-20150
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> TopNKey pushdown
> ================
> 1. Apply patch: 
> https://issues.apache.org/jira/secure/attachment/12941630/HIVE-20150.11.patch
> 2. TopNKey introduction depends only from Reduce Sink with topn property >= 0
> 3. Implement TopNKey operator pushdown through: projection, group by, redeuce 
> sink, left outer join, other topnkey
> 4. Add sort order and null sort order direction check when determining if the 
> topnkey op can be pushed
> 5. Implement handling cases when topnkey op and the parent op has a common 
> key prefix only.
> 6. fix Key object inspectors non-vectorized mode 
> 7. fix decimal64 to decimal cast issues when creating VectorExpressions of 
> keyExpression during TopnKey vectorization
> 
> 
> Diffs
> -----
> 
>   kudu-handler/src/test/results/positive/kudu_complex_queries.q.out 
> 1324b27f8e 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/TopNKeyOperator.java bbbde7978b 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/TopNKeyProcessor.java 
> 0d6cf3c755 
>   ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/Vectorizer.java 
> 6876787e11 
>   
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/topnkey/CommonKeyPrefix.java 
> PRE-CREATION 
>   
> ql/src/java/org/apache/hadoop/hive/ql/optimizer/topnkey/TopNKeyPushdownProcessor.java
>  PRE-CREATION 
>   ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java 5c7a64c950 
>   
> ql/src/test/org/apache/hadoop/hive/ql/optimizer/topnkey/TestCommonKeyPrefix.java
>  PRE-CREATION 
>   ql/src/test/queries/clientpositive/topnkey.q 057b6a45ba 
>   ql/src/test/queries/clientpositive/vector_topnkey.q 85c5880cd6 
>   ql/src/test/results/clientpositive/druid/druidmini_expressions.q.out 
> 31862546b0 
>   ql/src/test/results/clientpositive/druid/druidmini_test1.q.out fe75ddd894 
>   ql/src/test/results/clientpositive/llap/bucket_groupby.q.out 0c051c926b 
>   ql/src/test/results/clientpositive/llap/bucketmapjoin7.q.out 53e57b892f 
>   ql/src/test/results/clientpositive/llap/check_constraint.q.out 9f2c9a1cd0 
>   ql/src/test/results/clientpositive/llap/constraints_optimization.q.out 
> b6d210becf 
>   ql/src/test/results/clientpositive/llap/ctas.q.out c10d75d081 
>   
> ql/src/test/results/clientpositive/llap/dynpart_sort_opt_vectorization.q.out 
> 4e62779446 
>   ql/src/test/results/clientpositive/llap/dynpart_sort_optimization.q.out 
> 6688944b78 
>   ql/src/test/results/clientpositive/llap/enforce_constraint_notnull.q.out 
> 9343e078b7 
>   ql/src/test/results/clientpositive/llap/explainuser_1.q.out 64fdc481eb 
>   ql/src/test/results/clientpositive/llap/explainuser_2.q.out 0219af8833 
>   ql/src/test/results/clientpositive/llap/external_jdbc_table_perf.q.out 
> 545cce75a9 
>   ql/src/test/results/clientpositive/llap/filter_union.q.out 0df77762a0 
>   ql/src/test/results/clientpositive/llap/groupby_groupingset_bug.q.out 
> 63d56172b7 
>   ql/src/test/results/clientpositive/llap/insert1_overwrite_partitions.q.out 
> d51654bc5b 
>   ql/src/test/results/clientpositive/llap/insert_into1.q.out dd9140d3b6 
>   ql/src/test/results/clientpositive/llap/insert_into2.q.out 161318b720 
>   ql/src/test/results/clientpositive/llap/lateral_view.q.out b742230a71 
>   ql/src/test/results/clientpositive/llap/limit_join_transpose.q.out 
> 6258f4fe1e 
>   ql/src/test/results/clientpositive/llap/limit_pushdown.q.out 3fdd77d802 
>   ql/src/test/results/clientpositive/llap/limit_pushdown3.q.out efa8c38d7c 
>   ql/src/test/results/clientpositive/llap/materialized_view_rewrite_8.q.out 
> 303111a7d4 
>   ql/src/test/results/clientpositive/llap/offset_limit.q.out 23f2de46e5 
>   ql/src/test/results/clientpositive/llap/offset_limit_ppd_optimizer.q.out 
> 4ecb7bc46d 
>   ql/src/test/results/clientpositive/llap/orc_predicate_pushdown.q.out 
> 2fd8faa7b5 
>   ql/src/test/results/clientpositive/llap/orc_struct_type_vectorization.q.out 
> 0eac389eb7 
>   
> ql/src/test/results/clientpositive/llap/parquet_complex_types_vectorization.q.out
>  4362fb6f2e 
>   
> ql/src/test/results/clientpositive/llap/parquet_map_type_vectorization.q.out 
> 24468c9a1b 
>   ql/src/test/results/clientpositive/llap/parquet_predicate_pushdown.q.out 
> 8c900157c5 
>   
> ql/src/test/results/clientpositive/llap/parquet_struct_type_vectorization.q.out
>  45890a1890 
>   ql/src/test/results/clientpositive/llap/q93_with_constraints.q.out 
> 5bba57633b 
>   ql/src/test/results/clientpositive/llap/smb_mapjoin_15.q.out 8ebc139faa 
>   ql/src/test/results/clientpositive/llap/subquery_ALL.q.out d910c1a79d 
>   ql/src/test/results/clientpositive/llap/subquery_ANY.q.out 91472d631e 
>   ql/src/test/results/clientpositive/llap/subquery_in.q.out 479e60fda9 
>   ql/src/test/results/clientpositive/llap/subquery_notin.q.out 3dc247591c 
>   ql/src/test/results/clientpositive/llap/subquery_scalar.q.out fd65f872c6 
>   ql/src/test/results/clientpositive/llap/subquery_select.q.out 9f40436116 
>   ql/src/test/results/clientpositive/llap/temp_table.q.out c972b10601 
>   ql/src/test/results/clientpositive/llap/tez_fixed_bucket_pruning.q.out 
> 5367d33e0d 
>   ql/src/test/results/clientpositive/llap/topnkey.q.out 1e77587f82 
>   ql/src/test/results/clientpositive/llap/union_top_level.q.out ac39cc2622 
>   ql/src/test/results/clientpositive/llap/vector_case_when_2.q.out 3501884ce5 
>   ql/src/test/results/clientpositive/llap/vector_case_when_conversion.q.out 
> 4da58e0c2b 
>   ql/src/test/results/clientpositive/llap/vector_cast_constant.q.out 
> cc2dc47280 
>   ql/src/test/results/clientpositive/llap/vector_char_2.q.out f7e76e5a8b 
>   ql/src/test/results/clientpositive/llap/vector_coalesce.q.out 9573b3db77 
>   ql/src/test/results/clientpositive/llap/vector_data_types.q.out df9c3499bf 
>   ql/src/test/results/clientpositive/llap/vector_decimal_expressions.q.out 
> 8500562893 
>   
> ql/src/test/results/clientpositive/llap/vector_groupby_grouping_sets_limit.q.out
>  6fd15e7101 
>   ql/src/test/results/clientpositive/llap/vector_groupby_reduce.q.out 
> d6325982e3 
>   ql/src/test/results/clientpositive/llap/vector_identity_reuse.q.out 
> 00aa4fbf7d 
>   ql/src/test/results/clientpositive/llap/vector_llap_text_1.q.out 1f7ed5a129 
>   ql/src/test/results/clientpositive/llap/vector_partitioned_date_time.q.out 
> 48e840584e 
>   ql/src/test/results/clientpositive/llap/vector_string_concat.q.out 
> a8019be7aa 
>   ql/src/test/results/clientpositive/llap/vector_topnkey.q.out c140bdfd37 
>   ql/src/test/results/clientpositive/llap/vector_varchar_simple.q.out 
> 6bb2fbde0c 
>   ql/src/test/results/clientpositive/llap/vectorization_13.q.out 3f9f1b7770 
>   ql/src/test/results/clientpositive/llap/vectorization_7.q.out bfcd6a3568 
>   ql/src/test/results/clientpositive/llap/vectorization_8.q.out 1253c2bc83 
>   ql/src/test/results/clientpositive/llap/vectorization_div0.q.out 73ca096e22 
>   ql/src/test/results/clientpositive/llap/vectorization_limit.q.out 
> 7326adf522 
>   ql/src/test/results/clientpositive/llap/vectorization_part_project.q.out 
> 1c22230f2d 
>   ql/src/test/results/clientpositive/llap/vectorization_short_regress.q.out 
> cdeb909b02 
>   ql/src/test/results/clientpositive/perf/tez/cbo_query14.q.out e9308cd709 
>   ql/src/test/results/clientpositive/perf/tez/cbo_query54.q.out e96d4c60a6 
>   ql/src/test/results/clientpositive/perf/tez/cbo_query58.q.out 523f02e7d7 
>   ql/src/test/results/clientpositive/perf/tez/cbo_query6.q.out 944b113e7b 
>   ql/src/test/results/clientpositive/perf/tez/cbo_query77.q.out 02caf99f7d 
>   ql/src/test/results/clientpositive/perf/tez/constraints/cbo_query14.q.out 
> 43e1b2b5c2 
>   ql/src/test/results/clientpositive/perf/tez/constraints/cbo_query54.q.out 
> ece76c3e4e 
>   ql/src/test/results/clientpositive/perf/tez/constraints/cbo_query58.q.out 
> 954bdc4dd1 
>   ql/src/test/results/clientpositive/perf/tez/constraints/cbo_query6.q.out 
> eb872d7e6d 
>   ql/src/test/results/clientpositive/perf/tez/constraints/cbo_query77.q.out 
> 2f75361df1 
>   ql/src/test/results/clientpositive/perf/tez/constraints/mv_query44.q.out 
> a546472908 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query1.q.out 
> 3aea12dda6 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query10.q.out 
> bb3b1b6660 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query11.q.out 
> cc47c7e7a1 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query12.q.out 
> 57c16347e5 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query14.q.out 
> 228b20a8d7 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query15.q.out 
> 5268ed3ecf 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query17.q.out 
> d96222d9e1 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query18.q.out 
> c3c17a2efc 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query19.q.out 
> fff6c4951d 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query1b.q.out 
> a6276715b4 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query20.q.out 
> b71831c7c6 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query21.q.out 
> af2e33e785 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query22.q.out 
> cd3c0cc431 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query25.q.out 
> adabb76e04 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query26.q.out 
> 824bbe6769 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query27.q.out 
> abbd02d6c9 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query29.q.out 
> c308771dfb 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query3.q.out 
> 4c1f168168 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query30.q.out 
> ea1622883b 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query33.q.out 
> f00567eeb1 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query35.q.out 
> 23b3399123 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query36.q.out 
> f7bccf94f7 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query37.q.out 
> 187ad5c5b5 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query4.q.out 
> 68796c2274 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query40.q.out 
> 070b5cb1f5 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query42.q.out 
> b2dc7b3a8a 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query43.q.out 
> b5a6c746d1 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query44.q.out 
> 7644f5ec61 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query45.q.out 
> 3f5dbf4beb 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query46.q.out 
> 804013fb80 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query47.q.out 
> 701861a888 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query49.q.out 
> b384aea779 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query5.q.out 
> d3f79820f2 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query50.q.out 
> 8c9754967f 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query51.q.out 
> 4255d5cc33 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query52.q.out 
> 56d73b8d46 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query53.q.out 
> f8f495d3a8 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query54.q.out 
> d85597af02 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query55.q.out 
> fb52de2aba 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query56.q.out 
> cd438d4ae3 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query57.q.out 
> 7896f97fed 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query58.q.out 
> 5fa126bf6e 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query59.q.out 
> 82148531bc 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query6.q.out 
> d6031153b1 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query60.q.out 
> 06a5689938 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query63.q.out 
> 8d45d8260d 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query65.q.out 
> 8b8a4146a6 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query66.q.out 
> be612609cf 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query67.q.out 
> ee1bf12068 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query68.q.out 
> 97b97ce19d 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query69.q.out 
> d7469ae5a9 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query7.q.out 
> b2eccdbe90 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query70.q.out 
> d5efae9aeb 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query72.q.out 
> 9d6a3f6a00 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query74.q.out 
> 68ff9ebf26 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query75.q.out 
> b1341a2fdb 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query76.q.out 
> ce4f7cb061 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query77.q.out 
> 95ab61bed2 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query78.q.out 
> ecb434d3cd 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query79.q.out 
> 9e9bbec9e2 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query8.q.out 
> 170bccf406 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query80.q.out 
> b18f89373c 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query81.q.out 
> 2b859c230c 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query82.q.out 
> 8dd6ae9f0f 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query83.q.out 
> a2163802ad 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query84.q.out 
> 89b11e66e2 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query85.q.out 
> 6bd833fd12 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query86.q.out 
> 0b6ee2c156 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query89.q.out 
> 28d604a433 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query93.q.out 
> 193a542b53 
>   ql/src/test/results/clientpositive/perf/tez/constraints/query99.q.out 
> c77a73f4d5 
>   ql/src/test/results/clientpositive/perf/tez/query1.q.out f7affa4f14 
>   ql/src/test/results/clientpositive/perf/tez/query10.q.out b346a5c5fb 
>   ql/src/test/results/clientpositive/perf/tez/query11.q.out 96bdd7b504 
>   ql/src/test/results/clientpositive/perf/tez/query12.q.out 4183059b2d 
>   ql/src/test/results/clientpositive/perf/tez/query14.q.out 069fad2b4a 
>   ql/src/test/results/clientpositive/perf/tez/query15.q.out 3670a718b3 
>   ql/src/test/results/clientpositive/perf/tez/query17.q.out df70fbc46e 
>   ql/src/test/results/clientpositive/perf/tez/query18.q.out d61a57d469 
>   ql/src/test/results/clientpositive/perf/tez/query19.q.out 5cdeab2bf0 
>   ql/src/test/results/clientpositive/perf/tez/query1b.q.out b7975918c3 
>   ql/src/test/results/clientpositive/perf/tez/query20.q.out 2553c1ed76 
>   ql/src/test/results/clientpositive/perf/tez/query21.q.out 5e158834d7 
>   ql/src/test/results/clientpositive/perf/tez/query22.q.out e721f09f82 
>   ql/src/test/results/clientpositive/perf/tez/query25.q.out d006795c79 
>   ql/src/test/results/clientpositive/perf/tez/query26.q.out a1bf3b099b 
>   ql/src/test/results/clientpositive/perf/tez/query27.q.out 6f49de2344 
>   ql/src/test/results/clientpositive/perf/tez/query29.q.out 5066893829 
>   ql/src/test/results/clientpositive/perf/tez/query3.q.out fe97d402c4 
>   ql/src/test/results/clientpositive/perf/tez/query30.q.out ac683c2515 
>   ql/src/test/results/clientpositive/perf/tez/query33.q.out dc4f6b17c1 
>   ql/src/test/results/clientpositive/perf/tez/query35.q.out 265c51bb72 
>   ql/src/test/results/clientpositive/perf/tez/query36.q.out 544d883846 
>   ql/src/test/results/clientpositive/perf/tez/query37.q.out 2724fd44dc 
>   ql/src/test/results/clientpositive/perf/tez/query4.q.out 99552295f0 
>   ql/src/test/results/clientpositive/perf/tez/query40.q.out 4b65c82e00 
>   ql/src/test/results/clientpositive/perf/tez/query42.q.out 3c059f85f6 
>   ql/src/test/results/clientpositive/perf/tez/query43.q.out eb19d41926 
>   ql/src/test/results/clientpositive/perf/tez/query44.q.out adaea8c46e 
>   ql/src/test/results/clientpositive/perf/tez/query45.q.out 4538a6540d 
>   ql/src/test/results/clientpositive/perf/tez/query46.q.out 3e6358d907 
>   ql/src/test/results/clientpositive/perf/tez/query47.q.out e54c366848 
>   ql/src/test/results/clientpositive/perf/tez/query49.q.out 9c34eccceb 
>   ql/src/test/results/clientpositive/perf/tez/query5.q.out 38fba27a8e 
>   ql/src/test/results/clientpositive/perf/tez/query50.q.out 6e34831de6 
>   ql/src/test/results/clientpositive/perf/tez/query51.q.out 906d702207 
>   ql/src/test/results/clientpositive/perf/tez/query52.q.out b579d5fbff 
>   ql/src/test/results/clientpositive/perf/tez/query53.q.out 1bee5e099c 
>   ql/src/test/results/clientpositive/perf/tez/query54.q.out a19bd171e5 
>   ql/src/test/results/clientpositive/perf/tez/query55.q.out 5cee50ed50 
>   ql/src/test/results/clientpositive/perf/tez/query56.q.out 2be886a3eb 
>   ql/src/test/results/clientpositive/perf/tez/query57.q.out b732cc873f 
>   ql/src/test/results/clientpositive/perf/tez/query58.q.out 57f3477e48 
>   ql/src/test/results/clientpositive/perf/tez/query59.q.out 566f888d40 
>   ql/src/test/results/clientpositive/perf/tez/query6.q.out 5c49ab4517 
>   ql/src/test/results/clientpositive/perf/tez/query60.q.out e77c89ba69 
>   ql/src/test/results/clientpositive/perf/tez/query63.q.out 3cdf37b734 
>   ql/src/test/results/clientpositive/perf/tez/query65.q.out dbc5c79a87 
>   ql/src/test/results/clientpositive/perf/tez/query66.q.out 7ddcc21f92 
>   ql/src/test/results/clientpositive/perf/tez/query67.q.out cbacaae150 
>   ql/src/test/results/clientpositive/perf/tez/query68.q.out b9321679a9 
>   ql/src/test/results/clientpositive/perf/tez/query69.q.out d11b5494e0 
>   ql/src/test/results/clientpositive/perf/tez/query7.q.out c17ec8aeb9 
>   ql/src/test/results/clientpositive/perf/tez/query70.q.out 7fe3a62399 
>   ql/src/test/results/clientpositive/perf/tez/query72.q.out bebc3852b7 
>   ql/src/test/results/clientpositive/perf/tez/query74.q.out f98b9c8815 
>   ql/src/test/results/clientpositive/perf/tez/query75.q.out b426bf2d1a 
>   ql/src/test/results/clientpositive/perf/tez/query76.q.out c0d60e88cc 
>   ql/src/test/results/clientpositive/perf/tez/query77.q.out ab2b3dc570 
>   ql/src/test/results/clientpositive/perf/tez/query78.q.out 4d090af463 
>   ql/src/test/results/clientpositive/perf/tez/query79.q.out e1ef37ad38 
>   ql/src/test/results/clientpositive/perf/tez/query8.q.out 0af8fdf3df 
>   ql/src/test/results/clientpositive/perf/tez/query80.q.out 47844158fa 
>   ql/src/test/results/clientpositive/perf/tez/query81.q.out 7feb6175fe 
>   ql/src/test/results/clientpositive/perf/tez/query82.q.out c7721acffe 
>   ql/src/test/results/clientpositive/perf/tez/query83.q.out 54c1ba6161 
>   ql/src/test/results/clientpositive/perf/tez/query84.q.out 4fd546bc07 
>   ql/src/test/results/clientpositive/perf/tez/query85.q.out c411a92145 
>   ql/src/test/results/clientpositive/perf/tez/query86.q.out 37414f697f 
>   ql/src/test/results/clientpositive/perf/tez/query89.q.out 927da252da 
>   ql/src/test/results/clientpositive/perf/tez/query93.q.out 0df41d120b 
>   ql/src/test/results/clientpositive/perf/tez/query99.q.out c01122f435 
>   ql/src/test/results/clientpositive/tez/explainanalyze_3.q.out 7d303849d5 
>   ql/src/test/results/clientpositive/tez/explainuser_3.q.out 2377a6674e 
>   ql/src/test/results/clientpositive/tez/topnkey.q.out cf2ecf7133 
>   ql/src/test/results/clientpositive/tez/vector_non_string_partition.q.out 
> d449fd3754 
>   ql/src/test/results/clientpositive/tez/vector_topnkey.q.out d179013e28 
>   ql/src/test/results/clientpositive/topnkey.q.out cecbe89b1c 
> 
> 
> Diff: https://reviews.apache.org/r/71820/diff/14/
> 
> 
> Testing
> -------
> 
> Run q tests using TestMiniLlapLocalCliDriver
> topnkey.q
> vector_topnkey.q
> 
> 
> Thanks,
> 
> Krisztian Kasa
> 
>

Reply via email to