kgyrtkirk commented on a change in pull request #1553:
URL: https://github.com/apache/hive/pull/1553#discussion_r503826556
##########
File path:
ql/src/test/results/clientpositive/llap/cbo_SortUnionTransposeRule.q.out
##########
@@ -1006,6 +1027,22 @@ STAGE PLANS:
output format:
org.apache.hadoop.hive.ql.io.HiveSequenceFileOutputFormat
serde:
org.apache.hadoop.hive.serde2.lazy.LazySimpleSerDe
Reducer 5
+ Execution mode: vectorized, llap
Review comment:
This is caused by enabling the schema merge for all the optimizations.
Apparently the greedy operator chain matching logic worked slightly better when
it was run w/o ts-merge first and only then executed to also consider merging
the schema.
In this patch I will only introduce the new optimization and do the
generalization in either in the "downstream merge" patch or completely
separately; it will worth because it will enable the `RemoveSemiJoin` mode to
consider merging the ts schema
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
