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

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: https://issues.apache.org/jira/browse/HIVE-15766
>             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
(v6.3.15#6346)

Reply via email to