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

(Updated Feb. 19, 2018, 11 p.m.)


Review request for atlas, Apoorv Naik, Ashutosh Mestry, and Madhan Neethiraj.


Changes
-------

addressed review comments.


Bugs: ATLAS-2456
    https://issues.apache.org/jira/browse/ATLAS-2456


Repository: atlas


Description
-------

Scalable way to quickly and efficiently propagate tags for efficient searches 
and tag based security. Likewise tags for derivative dataset should be 
inherited from the parent. For example, if an entity is tagged "PII" then 
resulting entity created from a CTAS operation should also be tagged "secret" 
to maintain the classification of the parent. In the case where 2 or more 
datasets are aggregated the derivative dataset should be a union of all parent 
tags.

This changeset includes introduction of v2 data structures for notifications 
and audit events.


Diffs (updated)
-----

  addons/models/0000-Area0/0010-base_model.json 0296e8f9 
  addons/models/1000-Hadoop/1030-hive_model.json 32d91793 
  addons/models/1000-Hadoop/1060-hbase_model.json 9280f599 
  common/src/main/java/org/apache/atlas/repository/Constants.java 265be788 
  
graphdb/api/src/main/java/org/apache/atlas/repository/graphdb/AtlasElement.java 
42837f49 
  
graphdb/api/src/main/java/org/apache/atlas/repository/graphdb/AtlasVertex.java 
a68d8ebe 
  
graphdb/janus/src/main/java/org/apache/atlas/repository/graphdb/janus/AtlasJanusVertex.java
 aef20f03 
  
graphdb/titan0/src/main/java/org/apache/atlas/repository/graphdb/titan0/Titan0Vertex.java
 ca48e3d9 
  intg/src/main/java/org/apache/atlas/AtlasErrorCode.java ff09e6c9 
  intg/src/main/java/org/apache/atlas/listener/EntityChangeListenerV2.java 
PRE-CREATION 
  intg/src/main/java/org/apache/atlas/model/audit/EntityAuditEventV2.java 
PRE-CREATION 
  intg/src/main/java/org/apache/atlas/model/instance/AtlasClassification.java 
f594a814 
  intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java 08d1ce11 
  
intg/src/main/java/org/apache/atlas/model/notification/EntityNotification.java 
b272b733 
  
intg/src/main/java/org/apache/atlas/v1/model/notification/EntityNotificationV2.java
 PRE-CREATION 
  
repository/src/main/java/org/apache/atlas/repository/audit/EntityAuditListener.java
 74d3b913 
  
repository/src/main/java/org/apache/atlas/repository/audit/EntityAuditListenerV2.java
 PRE-CREATION 
  
repository/src/main/java/org/apache/atlas/repository/audit/EntityAuditRepository.java
 9dc78350 
  
repository/src/main/java/org/apache/atlas/repository/audit/HBaseBasedAuditRepository.java
 774934c7 
  
repository/src/main/java/org/apache/atlas/repository/audit/InMemoryEntityAuditRepository.java
 22d2a810 
  
repository/src/main/java/org/apache/atlas/repository/audit/NoopEntityAuditRepository.java
 c3826019 
  
repository/src/main/java/org/apache/atlas/repository/converters/AtlasInstanceConverter.java
 2884f8f0 
  
repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java
 31620b1e 
  repository/src/main/java/org/apache/atlas/repository/graph/GraphHelper.java 
d61bff24 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityChangeNotifier.java
 2b6beade 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java
 ca0eeeb6 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasRelationshipStoreV1.java
 7389f49b 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java
 779bc387 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphRetriever.java
 b05a9a32 
  
repository/src/main/java/org/apache/atlas/util/AtlasGremlin3QueryProvider.java 
1fda2415 
  repository/src/main/java/org/apache/atlas/util/AtlasGremlinQueryProvider.java 
a79abaaf 
  
repository/src/main/java/org/apache/atlas/util/AtlasRepositoryConfiguration.java
 0e1e5b6c 
  repository/src/test/java/org/apache/atlas/TestModules.java 13bdcb08 
  
repository/src/test/java/org/apache/atlas/repository/audit/AuditRepositoryTestBase.java
 47a61ee2 
  
webapp/src/main/java/org/apache/atlas/notification/EntityNotificationListenerV2.java
 PRE-CREATION 
  
webapp/src/main/java/org/apache/atlas/notification/NotificationEntityChangeListener.java
 396a2920 
  webapp/src/main/java/org/apache/atlas/web/resources/EntityResource.java 
715a54db 
  webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java 21402e19 
  webapp/src/test/java/org/apache/atlas/web/adapters/TestEntityREST.java 
ea6fe313 
  webapp/src/test/resources/atlas-application.properties d9009166 


Diff: https://reviews.apache.org/r/65700/diff/7/

Changes: https://reviews.apache.org/r/65700/diff/6-7/


Testing
-------

mvn clean install -> succeeded in 17:11 min

UTs/ITs for tag propagation work in progress


Thanks,

Sarath Subramanian

Reply via email to