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