[ 
https://issues.apache.org/jira/browse/HIVE-1108?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12805608#action_12805608
 ] 

Namit Jain commented on HIVE-1108:
----------------------------------

65               private ArrayList<Task<? extends Serializable>> rootTasks;     
                                66              private FetchTask fetchTask;    
                                67              private HashSet<ReadEntity> 
inputs;     
                                68              private HashSet<WriteEntity> 
outputs;   
                                69                      
                                70              private HashMap<String, String> 
idToTableNameMap;



Do you think instead of adding the following fields in QueryPlan, you might 
create a new serializable structure and share it between SemanticAnalyzer and 
QueryPlan ?

> QueryPlan to be independent from BaseSemanticAnalyzer
> -----------------------------------------------------
>
>                 Key: HIVE-1108
>                 URL: https://issues.apache.org/jira/browse/HIVE-1108
>             Project: Hadoop Hive
>          Issue Type: Improvement
>          Components: Query Processor
>    Affects Versions: 0.6.0
>            Reporter: Zheng Shao
>            Assignee: Zheng Shao
>         Attachments: HIVE-1108.1.patch
>
>
> QueryPlan is the class to store compiled plan. However currently it contains 
> references to BaseSemanticAnalyzer.
> We should remove the reference. Otherwise QueryPlan won't be serializable.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to