atlas-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Apoorv Naik <naik.apo...@gmail.com>
Subject Re: Review Request 56184: ATLAS-1499: Notification processing using V2 Store
Date Thu, 16 Feb 2017 20:41:20 GMT

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

(Updated Feb. 16, 2017, 8:41 p.m.)


Review request for atlas, Madhan Neethiraj, Sarath Subramanian, Suma Shivaprasad, and Vimal
Sharma.


Changes
-------

Rebased and fixed one minor issue with the StructFormatConverter where any missing attribute
was being set to null whereas it should've been ignored.


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


Repository: atlas


Description
-------

ATLAS-1499: Notification processing using V2 Store


Diffs (updated)
-----

  intg/src/main/java/org/apache/atlas/AtlasErrorCode.java ae6be843 
  intg/src/main/java/org/apache/atlas/model/instance/AtlasEntity.java edaede0c 
  intg/src/main/java/org/apache/atlas/model/instance/EntityMutationResponse.java 5e8ce351

  repository/src/main/java/org/apache/atlas/repository/audit/EntityAuditListener.java 1ef803c0

  repository/src/test/java/org/apache/atlas/services/MetricsServiceTest.java cf85b2fc 
  server-api/src/main/java/org/apache/atlas/RequestContextV1.java bf731746 
  webapp/src/main/java/org/apache/atlas/notification/NotificationHookConsumer.java f2416810

  webapp/src/main/java/org/apache/atlas/web/adapters/AtlasAbstractFormatConverter.java f1f3d18a

  webapp/src/main/java/org/apache/atlas/web/adapters/AtlasArrayFormatConverter.java aa14aff4

  webapp/src/main/java/org/apache/atlas/web/adapters/AtlasClassificationFormatConverter.java
dc740f55 
  webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEntityFormatConverter.java cb1390d1

  webapp/src/main/java/org/apache/atlas/web/adapters/AtlasEnumFormatConverter.java 6d8e3aee

  webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverter.java a7157a36 
  webapp/src/main/java/org/apache/atlas/web/adapters/AtlasFormatConverters.java 968d74fe 
  webapp/src/main/java/org/apache/atlas/web/adapters/AtlasInstanceRestAdapters.java b1dae56d

  webapp/src/main/java/org/apache/atlas/web/adapters/AtlasMapFormatConverter.java 6967c4f5

  webapp/src/main/java/org/apache/atlas/web/adapters/AtlasObjectIdConverter.java 11a020d8

  webapp/src/main/java/org/apache/atlas/web/adapters/AtlasPrimitiveFormatConverter.java 2b70c5e1

  webapp/src/main/java/org/apache/atlas/web/adapters/AtlasStructFormatConverter.java 920b48b0

  webapp/src/main/java/org/apache/atlas/web/resources/AdminResource.java 2c2c16d7 
  webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java 9518f540 
  webapp/src/test/java/org/apache/atlas/LocalAtlasClientTest.java c5616dfe 
  webapp/src/test/java/org/apache/atlas/notification/NotificationHookConsumerKafkaTest.java
873e5625 
  webapp/src/test/java/org/apache/atlas/notification/NotificationHookConsumerTest.java f06f7912


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


Testing (updated)
-------

mvn clean install -DskipITs executed successfully.

UI/Manual testing details

**Changes seen on UI **
1. Manually created table in hive, showed up in graph and as an entity from the search UI
2. Created a view from the above table and the lineage showed up in atlas dashboard
3. added a column to the above table
4. altered table columns


Will be running HiveHookIT to verify more cases.


Thanks,

Apoorv Naik


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