hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Phabricator (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HIVE-2716) Move retry logic in HiveMetaStore to a separe class
Date Fri, 17 Feb 2012 01:28:00 GMT

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

Phabricator commented on HIVE-2716:
-----------------------------------

ashutoshc has commented on the revision "HIVE-2716 [jira] Move retry logic in HiveMetaStore
to a separe class".

  Enis will know more, but AFAIK HCatalog won't get anything out of this change. I like the
idea of the patch for refactoring HiveMetaStore.java which is unwieldy in its current form,
thus making Hive codebase easier to read and follow. Do you have any specific concerns?

REVISION DETAIL
  https://reviews.facebook.net/D1227

                
> Move retry logic in HiveMetaStore to a separe class
> ---------------------------------------------------
>
>                 Key: HIVE-2716
>                 URL: https://issues.apache.org/jira/browse/HIVE-2716
>             Project: Hive
>          Issue Type: Sub-task
>          Components: Metastore
>    Affects Versions: 0.9.0
>            Reporter: Enis Soztutar
>            Assignee: Enis Soztutar
>         Attachments: HIVE-2716.D1227.1.patch, HIVE-2716.D1227.2.patch
>
>
> In HIVE-1219, method retrying for raw store operation are introduced to handle jdo operations
more robustly. However, the abstraction for the RawStore operations can be moved to a separate
class implementing RawStore, which should clean up the code base for HiveMetaStore. 

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