Indhumathi27 commented on code in PR #6202:
URL: https://github.com/apache/hive/pull/6202#discussion_r2668539128
##########
ql/src/java/org/apache/hadoop/hive/ql/parse/TezCompiler.java:
##########
@@ -1322,6 +1330,15 @@ private static void
runTopNKeyOptimization(OptimizeTezProcContext procCtx)
ogw.startWalking(topNodes, null);
}
+ private static boolean hasPTFReduceSink(OptimizeTezProcContext ctx) {
+ for (ReduceSinkOperator rs : ctx.visitedReduceSinks) {
+ if (rs.getConf().isPTFReduceSink()) {
+ return true;
+ }
+ }
+ return false;
+ }
+
Review Comment:
For windowing queries, since there is not much performance issues with
TopNKey enabled, currently making the queries to use TopNkey Path. But to match
the plan, there is no sequence of PTF%RS% patterns for some queries. only RS%
will work for this case.
I chosed this approach, to avoid traversing the tree to check query has PTF
operator.
can you suggest a solution for the windowing queries
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
