ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Madhan Neethiraj <mad...@apache.org>
Subject Re: Review Request 52230: RANGER-1177: updated tagsync to support Atlas notification for hdfs_path entity-type
Date Mon, 26 Sep 2016 19:04:57 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52230/
-----------------------------------------------------------

(Updated Sept. 26, 2016, 7:04 p.m.)


Review request for ranger, Alok Lal, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, Pradeep
Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy.


Changes
-------

- added support for 'default-cluster-name' (via property ranger.tagsync.atlas.default.cluster.name)
for HDFS, given Atlas hdfs_path doesn't require clusterName or has clusterName in qualifiedName
- addressed review comment


Bugs: RANGER-1177
    https://issues.apache.org/jira/browse/RANGER-1177


Repository: ranger


Description
-------

Added AtlasHdfsResourceMapper class and registred it in AtlasResourceMapperUtil


Diffs (updated)
-----

  tagsync/pom.xml 0e3fb39 
  tagsync/src/main/java/org/apache/ranger/tagsync/source/atlas/AtlasHdfsResourceMapper.java
PRE-CREATION 
  tagsync/src/main/java/org/apache/ranger/tagsync/source/atlas/AtlasHiveResourceMapper.java
ca7b063 
  tagsync/src/main/java/org/apache/ranger/tagsync/source/atlas/AtlasResourceMapper.java d47d052

  tagsync/src/main/java/org/apache/ranger/tagsync/source/atlas/AtlasResourceMapperUtil.java
f05d814 
  tagsync/src/test/java/org/apache/ranger/tagsync/process/TestHdfsResourceMapper.java PRE-CREATION

  tagsync/src/test/java/org/apache/ranger/tagsync/process/TestHiveResourceMapper.java PRE-CREATION

  tagsync/src/test/java/org/apache/ranger/tagsync/process/TestTagSynchronizer.java c644e0d


Diff: https://reviews.apache.org/r/52230/diff/


Testing
-------

- added unit tests
- verified with Atlas notifications for hdfs_path entities


Thanks,

Madhan Neethiraj


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message