hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-2838) cleanup readentity/writeentity
Date Fri, 09 Mar 2012 11:07:02 GMT

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

Hudson commented on HIVE-2838:
------------------------------

Integrated in Hive-trunk-h0.21 #1300 (See [https://builds.apache.org/job/Hive-trunk-h0.21/1300/])
    HIVE-2838. cleanup readentity/writeentity. (namit via kevinwilfong) (Revision 1298699)

     Result = FAILURE
kevinwilfong : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1298699
Files : 
* /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/hooks/Entity.java
* /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/hooks/ReadEntity.java
* /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/hooks/WriteEntity.java

                
> cleanup readentity/writeentity
> ------------------------------
>
>                 Key: HIVE-2838
>                 URL: https://issues.apache.org/jira/browse/HIVE-2838
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Namit Jain
>            Assignee: Namit Jain
>             Fix For: 0.9.0
>
>         Attachments: HIVE-2838.D2193.1.patch, HIVE-2838.D2193.2.patch
>
>
> Ideally, there should be one common entity instead of readentity/writeentity.
> Unfortunately, that would be a backward incompatible change since users os hive might
have written
> there own hooks, where they are using readentity/writeentity.
> We should atleast create a common class, and then we can deprecate read/write entity
later, for a new release.
> For now, I propose to make a backward compatible change.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message