okumin commented on code in PR #6331:
URL: https://github.com/apache/hive/pull/6331#discussion_r3027836549
##########
ql/src/java/org/apache/hadoop/hive/ql/optimizer/stats/annotation/StatsRulesProcFactory.java:
##########
@@ -3080,6 +3114,65 @@ public Object process(Node nd, Stack<Node> stack,
NodeProcessorCtx procCtx,
return null;
}
+
+ /**
+ * Refines UDTF column statistics using LVJ's context and defensive
identity mapping.
+ */
+ private void refineUdtfColStats(List<ColStatistics> udtfColStats,
Review Comment:
First, I currently assume we don't need to update LateralViewJoinStatsRule
if the upstream stats align with the schema. We may have to take a deep look.
Second, even if we really need this, the current implementation is too
adhoc, as several UDFs are hardcoded. This might not be scalable. Why don't we
need other UDTFs?
--
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]
