Re: Review Request 53149: Adding Change listeners to react on changes in TypesDef

2016-10-26 Thread Apoorv Naik

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

(Updated Oct. 26, 2016, 6:57 p.m.)


Review request for atlas, Madhan Neethiraj, Sarath Subramanian, Shwetha GS, and 
Suma Shivaprasad.


Changes
---

Updated test info and description.


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


Repository: atlas


Description (updated)
---

Adding Change listeners to react on changes in TypesDef
1. Restore the typesystem in DefaultMetadataService when any changes occur in 
the new type registry
2. init() and load all graph data in HA configuration when the server becomes 
active


Diffs
-

  intg/src/main/java/org/apache/atlas/listener/ChangeOperation.java 
PRE-CREATION 
  intg/src/main/java/org/apache/atlas/listener/ChangedTypeDefs.java 
PRE-CREATION 
  intg/src/main/java/org/apache/atlas/listener/TypeDefChangeListener.java 
PRE-CREATION 
  repository/src/main/java/org/apache/atlas/RepositoryMetadataModule.java 
bbf75aec5b330929cbfd5bc24dfda5a3054aa910 
  
repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java
 8038815383ebe1a613de7e314bdd5185dab8a629 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasTypeDefGraphStore.java
 12047bfd80c6fb37b61dfd401b35f9e3dfb41043 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasTypeDefGraphStoreV1.java
 663470790dda87ea982ab63f22b94d3d576fd4ee 
  typesystem/pom.xml c564442d2b34dba818b3a2230b340cf1b8cc34db 
  typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 
52637e6f904cbae6122e46a1728237aefb3799d9 
  webapp/src/main/java/org/apache/atlas/web/rest/TypesREST.java 
065836143b3f2604ea6e3754069afea77e87f9c1 
  
webapp/src/main/java/org/apache/atlas/web/service/ActiveInstanceElectorModule.java
 ee6035da62005accea354117423e5f249e39930b 

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


Testing (updated)
---

HA activation tested by UnitTests

Restore not yet working on TypeSystem (getting TypeUpdateException)


Thanks,

Apoorv Naik



Re: Review Request 53149: Adding Change listeners to react on changes in TypesDef

2016-10-26 Thread Apoorv Naik

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

(Updated Oct. 26, 2016, 5:43 p.m.)


Review request for atlas, Madhan Neethiraj, Sarath Subramanian, Shwetha GS, and 
Suma Shivaprasad.


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


Repository: atlas


Description (updated)
---

Adding Change listeners to react on changes in TypesDef
1. Reset/Clear typesystem when any changes occur in the new type registry
2. init() and load all graph data in HA configuration when the server becomes 
active


Diffs
-

  intg/src/main/java/org/apache/atlas/listener/ChangeOperation.java 
PRE-CREATION 
  intg/src/main/java/org/apache/atlas/listener/ChangedTypeDefs.java 
PRE-CREATION 
  intg/src/main/java/org/apache/atlas/listener/TypeDefChangeListener.java 
PRE-CREATION 
  repository/src/main/java/org/apache/atlas/RepositoryMetadataModule.java 
bbf75aec5b330929cbfd5bc24dfda5a3054aa910 
  
repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java
 8038815383ebe1a613de7e314bdd5185dab8a629 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasTypeDefGraphStore.java
 12047bfd80c6fb37b61dfd401b35f9e3dfb41043 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasTypeDefGraphStoreV1.java
 663470790dda87ea982ab63f22b94d3d576fd4ee 
  typesystem/pom.xml c564442d2b34dba818b3a2230b340cf1b8cc34db 
  typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 
52637e6f904cbae6122e46a1728237aefb3799d9 
  webapp/src/main/java/org/apache/atlas/web/rest/TypesREST.java 
065836143b3f2604ea6e3754069afea77e87f9c1 
  
webapp/src/main/java/org/apache/atlas/web/service/ActiveInstanceElectorModule.java
 ee6035da62005accea354117423e5f249e39930b 

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


Testing (updated)
---

HA activation tested by UnitTests


Thanks,

Apoorv Naik



Review Request 53149: Adding Change listeners to react on changes in TypesDef

2016-10-24 Thread Apoorv Naik

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

Review request for atlas, Madhan Neethiraj, Sarath Subramanian, Shwetha GS, and 
Suma Shivaprasad.


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


Repository: atlas


Description
---

[Initial Draft changes]
Adding Change listeners to react on changes in TypesDef


Diffs
-

  intg/src/main/java/org/apache/atlas/listener/ChangeOperation.java 
PRE-CREATION 
  intg/src/main/java/org/apache/atlas/listener/ChangedTypeDefs.java 
PRE-CREATION 
  intg/src/main/java/org/apache/atlas/listener/TypeDefChangeListener.java 
PRE-CREATION 
  repository/src/main/java/org/apache/atlas/RepositoryMetadataModule.java 
bbf75aec5b330929cbfd5bc24dfda5a3054aa910 
  
repository/src/main/java/org/apache/atlas/repository/graph/GraphBackedSearchIndexer.java
 8038815383ebe1a613de7e314bdd5185dab8a629 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/AtlasTypeDefGraphStore.java
 12047bfd80c6fb37b61dfd401b35f9e3dfb41043 
  
repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasTypeDefGraphStoreV1.java
 663470790dda87ea982ab63f22b94d3d576fd4ee 
  typesystem/pom.xml c564442d2b34dba818b3a2230b340cf1b8cc34db 
  typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 
52637e6f904cbae6122e46a1728237aefb3799d9 
  webapp/src/main/java/org/apache/atlas/web/rest/TypesREST.java 
065836143b3f2604ea6e3754069afea77e87f9c1 
  
webapp/src/main/java/org/apache/atlas/web/service/ActiveInstanceElectorModule.java
 ee6035da62005accea354117423e5f249e39930b 

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


Testing
---

Working on HA environment setup to verify the ActiveInstanceListener action


Thanks,

Apoorv Naik