hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mohit Sabharwal (JIRA)" <>
Subject [jira] [Commented] (HIVE-15766) DBNotificationlistener leaks JDOPersistenceManager
Date Wed, 01 Mar 2017 12:58:45 GMT


Mohit Sabharwal commented on HIVE-15766:

Wondering if it's better to create a new static version of getMS that takes hiveConf as argument.
That way, both existing getMS() and DbNotificationHandler can call this method. It will allow
us to use the same code path as current getMS() (including ms.verifySchema() call etc.)  and
newRawStore() can be static as well. 

Also wondering if there is any point storing rawStoreClassName separately in HMSHandler since
it can be looked up via hiveConf and it's only used once when RawStore gets created. 

> DBNotificationlistener leaks JDOPersistenceManager
> --------------------------------------------------
>                 Key: HIVE-15766
>                 URL:
>             Project: Hive
>          Issue Type: Bug
>          Components: Metastore
>            Reporter: Vaibhav Gumashta
>            Assignee: Vaibhav Gumashta
>         Attachments: HIVE-15766.1.patch, HIVE-15766.2.patch, HIVE-15766.3.patch

This message was sent by Atlassian JIRA

View raw message