atlas-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tom Beerbower" <tbeerbo...@hortonworks.com>
Subject Review Request 38341: Provide Entity Change Notification
Date Mon, 14 Sep 2015 04:08:49 GMT

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

Review request for atlas, John Speidel and Shwetha GS.


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


Repository: atlas


Description
-------

Add entity change notification to Atlas based using the existing atlas-notification module.

First cut at a patch for the Atlas entity change notification.  Note that at a minimum additional
unit tests are required.  I'm putting up the review to get some initial feedback.


Diffs
-----

  notification/src/main/java/org/apache/atlas/notification/entity/Entity.java PRE-CREATION

  notification/src/main/java/org/apache/atlas/notification/entity/EntityChangeConsumer.java
PRE-CREATION 
  notification/src/main/java/org/apache/atlas/notification/entity/EntityChangeListener.java
PRE-CREATION 
  notification/src/main/java/org/apache/atlas/notification/entity/EntityImpl.java PRE-CREATION

  notification/src/main/java/org/apache/atlas/notification/entity/EntityNotification.java
PRE-CREATION 
  notification/src/main/java/org/apache/atlas/notification/entity/Trait.java PRE-CREATION

  notification/src/main/java/org/apache/atlas/notification/entity/TraitImpl.java PRE-CREATION

  notification/src/test/java/org/apache/atlas/notification/entity/EntityImplTest.java PRE-CREATION

  notification/src/test/java/org/apache/atlas/notification/entity/TraitImplTest.java PRE-CREATION

  repository/pom.xml 8e4d0f3 
  repository/src/main/java/org/apache/atlas/RepositoryMetadataModule.java fbd01de 
  repository/src/main/java/org/apache/atlas/listener/EntityChangeListener.java f58d6de 
  repository/src/main/java/org/apache/atlas/services/DefaultMetadataService.java 56168db 

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


Testing
-------

mvn clean test

All existing tests pass.

New unit tests added (more required).


Thanks,

Tom Beerbower


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