----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65885/ -----------------------------------------------------------
(Updated March 26, 2018, 8:07 p.m.) Review request for atlas and David Radley. Changes ------- Resolved all indicated issues from review. Of special note, changes were made to the storm bridge/shim pom files to force the dropwizard metric jars to match the version used by other pieces in Atlas. This avoids random test failures due to classpath loading ordering. 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 (updated) ----- 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 Diff: https://reviews.apache.org/r/65885/diff/5/ Changes: https://reviews.apache.org/r/65885/diff/4-5/ 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