[
https://issues.apache.org/jira/browse/ATLAS-415?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15115589#comment-15115589
]
Hemanth Yamijala commented on ATLAS-415:
----------------------------------------
I attached the link to the review board request for this JIRA here:
https://reviews.apache.org/r/42724/
Let's take the conversation there. To quickly set context, the changes are:
* Made mockito-all scope to be 'test'
* Added Javadocs for public methods of {{HiveMetaStoreBridge}}. I went over the
current list of public methods in the class and made a few of them restricted
based on usage in the project. We don't have any annotations today to say the
level of visibility we have promised for classes in Atlas to external users. So
hoping tightening up is not an issue. I can revert these to their original
visibility levels if we think this isn't a good idea.
* Mainly, added updates for DB and partitions, which, in discussion with
Shwetha, came out to be important to do.
> Hive import fails when importing a table that is already imported without
> StorageDescriptor information
> -------------------------------------------------------------------------------------------------------
>
> Key: ATLAS-415
> URL: https://issues.apache.org/jira/browse/ATLAS-415
> Project: Atlas
> Issue Type: Bug
> Reporter: Hemanth Yamijala
> Assignee: Hemanth Yamijala
> Attachments: ATLAS-415.patch
>
>
> I found this when testing patches that integrate Storm with Atlas, but guess
> this may occur in other scenarios as well.
> To reproduce:
> * Run a storm topology with Atlas Hook enabled that has a HiveBolt (requires
> patches for ATLAS-181 and friends).
> * Run hive-import following the above.
> The first step creates a Hive DB and table setting just the required
> attributes. Note that the StorageDescriptor is an optional attribute as per
> the Hive DataModel now.
> The second step fails with this exception:
> {code}
> Exception in thread "main" java.lang.NullPointerException
> at
> org.apache.atlas.hive.bridge.HiveMetaStoreBridge.getSDForTable(HiveMetaStoreBridge.java:345)
> at
> org.apache.atlas.hive.bridge.HiveMetaStoreBridge.importTables(HiveMetaStoreBridge.java:219)
> at
> org.apache.atlas.hive.bridge.HiveMetaStoreBridge.importDatabases(HiveMetaStoreBridge.java:104)
> at
> org.apache.atlas.hive.bridge.HiveMetaStoreBridge.importHiveMetadata(HiveMetaStoreBridge.java:96)
> at
> org.apache.atlas.hive.bridge.HiveMetaStoreBridge.main(HiveMetaStoreBridge.java:503)
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
