You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by "Namit Jain (JIRA)" <ji...@apache.org> on 2010/01/27 19:41:34 UTC

[jira] Commented: (HIVE-1108) QueryPlan to be independent from BaseSemanticAnalyzer

    [ 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.