[
https://issues.apache.org/jira/browse/HADOOP-4084?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12631948#action_12631948
]
Ashish Thusoo commented on HADOOP-4084:
---------------------------------------
I don;t know why this is not compiling. I checked the submitted patch and the
references to UDAFRegistry have actually been removed from the sources. Is this
a merge problem?
The part of the patch file that does this is as follows:
Index:
src/contrib/hive/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
===================================================================
---
src/contrib/hive/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
(revision 695984)
+++
src/contrib/hive/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
(working copy)
@@ -134,7 +134,7 @@
assert (expressionTree.getChildCount() != 0);
assert (expressionTree.getChild(0).getType() == HiveParser.Identifier);
String functionName = expressionTree.getChild(0).getText();
- if (UDAFRegistry.getUDAF(functionName) != null) {
+ if (FunctionRegistry.getUDAF(functionName) != null) {
aggregations.put(expressionTree.toStringTree(), expressionTree);
return;
}
@@ -987,7 +987,7 @@
for (Map.Entry<String, CommonTree> entry : aggregationTrees.entrySet()) {
CommonTree value = entry.getValue();
String aggName = value.getChild(0).getText();
- Class<? extends UDAF> aggClass = UDAFRegistry.getUDAF(aggName);
+ Class<? extends UDAF> aggClass = FunctionRegistry.getUDAF(aggName);
assert (aggClass != null);
ArrayList<exprNodeDesc> aggParameters = new ArrayList<exprNodeDesc>();
ArrayList<Class<?>> aggClasses = new ArrayList<Class<?>>();
@@ -1006,7 +1006,7 @@
aggClasses.add(paraExprInfo.getType().getPrimitiveClass());
}
- if (null == UDAFRegistry.getUDAFMethod(aggName, aggClasses)) {
+ if (null == FunctionRegistry.getUDAFMethod(aggName, aggClasses)) {
String reason = "Looking for UDAF \"" + aggName + "\" with parame
> Add explain plan capabilities to Hive QL
> ----------------------------------------
>
> Key: HADOOP-4084
> URL: https://issues.apache.org/jira/browse/HADOOP-4084
> Project: Hadoop Core
> Issue Type: New Feature
> Components: contrib/hive
> Reporter: Ashish Thusoo
> Assignee: Ashish Thusoo
> Fix For: 0.19.0
>
> Attachments: patch-4084, patch-4084
>
>
> Adding explain plan for queries in hive.
> The current proposal is to support something like:
> EXPLAIN [EXTENDED]
> SELECT ....
> This will output the following:
> Abstract Syntax Tree:
> Number of Stages:
> Dependencies between Stages:
> Plan for each stage:
> If EXTENDED keyword is used then much more information will be emitted where
> as without that keyword only logical information will be emitted.
> e.g. In case of a group by query
> EXPLAIN
> SELECT T.c1, count(1) FROM T GROUP BY T.c1;
> The explain plan itself has two stages
> Stage1 and Stage2
> Stage1 will have the plan for generating the partial aggregates
> and Stage2 will have the plan for generating the complete aggregates.
> I also plan to convert the parse and semantic analysis tests so that they use
> this for finding differences in the plan instead of the programmatic plan
> dumps that we are using today (tests/queries/positive).
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
