[ 
https://issues.apache.org/jira/browse/HCATALOG-527?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13473608#comment-13473608
 ] 

Travis Crawford commented on HCATALOG-527:
------------------------------------------

Totally agree we need a slow deprecation cycle. Maybe we mark this as 
deprecated in 0.5.0, and mark it for removal in 0.6.0 (or perhaps 0.7.0, but 
that gives more time for users to write code against the API we want to remove).
                
> InputJobInfo should not be public
> ---------------------------------
>
>                 Key: HCATALOG-527
>                 URL: https://issues.apache.org/jira/browse/HCATALOG-527
>             Project: HCatalog
>          Issue Type: Bug
>            Reporter: Travis Crawford
>            Assignee: Travis Crawford
>
> Writing a MapReduce job using HCatalog involves the following step (from 
> [Input and Output 
> Interfaces|http://incubator.apache.org/hcatalog/docs/r0.4.0/inputoutput.html):
> {code}
> HCatInputFormat.setInput(job, InputJobInfo.create(dbName,
>                 inputTableName, null));
> {code}
> Notice how we expose 
> [InputJobInfo|https://svn.apache.org/repos/asf/incubator/hcatalog/trunk/src/java/org/apache/hcatalog/mapreduce/InputJobInfo.java]
>  as part of the public API. The purpose of this class is to store the 
> HiveMetaStore response data, and by serializing to the jobconf, passes that 
> info to backend workers.
> Really this class is an implementation detail that should be hidden from 
> users. In HCATALOG-453 where the size of InputJobInfo was addressed we had to 
> be aware of potentially user-affecting changes to this class. In HCATALOG-341 
> we clarified the usage of InputJobInfo because there was some confusion.
> Ideally we could go through a deprecation cycle and change our interface to 
> hide this implementation detail from users, which clarifies things for users, 
> and give us more implementation flexibility.
> {code}
> HCatInputFormat.setInput(job, dbName, inputTableName, null);
> {code}
> Thoughts?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to