Re: Review Request 65885: ATLAS-2470 - Add JanusGraph Cassandra support to Atlas

[David Radley <da...@apache.org>]

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


Ship it!




Ship It!

- David Radley


On March 28, 2018, 3:28 p.m., Pierre Padovani wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65885/
> -----------------------------------------------------------
> 
> (Updated March 28, 2018, 3:28 p.m.)
> 
> 
> Review request for atlas and David Radley.
> 
> 
> Bugs: ATLAS-2470
>     https://issues.apache.org/jira/browse/ATLAS-2470
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> This updates pom's to add the needed cassandra jars, and adds a dist profile embedded-cassandra-solr.
> 
> 
> Diffs
> -----
> 
>   addons/storm-bridge-shim/pom.xml c25fbfe5 
>   addons/storm-bridge/pom.xml 2627d92e 
>   distro/pom.xml 0103bef6 
>   distro/src/bin/atlas_config.py e6415cf4 
>   distro/src/bin/atlas_start.py 39be6b7c 
>   distro/src/bin/atlas_stop.py 66edd904 
>   distro/src/conf/atlas-application.properties 10fb7b86 
>   distro/src/conf/atlas-env.sh 68b24e93 
>   distro/src/conf/cassandra.yml.template PRE-CREATION 
>   distro/src/conf/zookeeper/log4j.properties PRE-CREATION 
>   distro/src/conf/zookeeper/zoo.cfg.template PRE-CREATION 
>   distro/src/main/assemblies/standalone-package.xml 1881082e 
>   docs/src/site/twiki/InstallationSteps.twiki 6b9f0313 
>   graphdb/janus/pom.xml 143b775f 
>   pom.xml 9ddc8e18 
>   repository/pom.xml b1d6b1f9 
>   repository/src/main/java/org/apache/atlas/repository/audit/AbstractStorageBasedAuditRepository.java PRE-CREATION 
>   repository/src/main/java/org/apache/atlas/repository/audit/CassandraBasedAuditRepository.java PRE-CREATION 
>   repository/src/main/java/org/apache/atlas/repository/audit/HBaseBasedAuditRepository.java cee7374e 
>   repository/src/test/java/org/apache/atlas/repository/audit/AuditRepositoryTestBase.java 4d70b7f7 
>   repository/src/test/java/org/apache/atlas/repository/audit/CassandraAuditRepositoryTest.java PRE-CREATION 
>   repository/src/test/resources/cassandra_test.yml PRE-CREATION 
>   server-api/src/main/java/org/apache/atlas/listener/ActiveStateChangeHandler.java 7d282b30 
>   test-tools/src/main/resources/solr/solr.xml 041f0058 
>   webapp/pom.xml 284f538f 
> 
> 
> Diff: https://reviews.apache.org/r/65885/diff/6/
> 
> 
> Testing
> -------
> 
> Full build with the new embedded-cassandra-solr, and testing to make sure Atlas comes up and is functional.
> 
> Be aware that we have been running Cassandra backed JanusGraph for months with no issues.
> 
> This patch now includes support for Cassandra as an audit store. I've also refactored the audit unit tests, and added cassandra specific tests.
> 
> 
> Thanks,
> 
> Pierre Padovani
> 
>