> On March 19, 2018, 7:28 p.m., David Radley wrote: > > docs/src/site/twiki/InstallationSteps.twiki > > Lines 45 (patched) > > <https://reviews.apache.org/r/65885/diff/3/?file=1982164#file1982164line45> > > > > I built Atlas as per these instructions. I then started atlas. Atlas > > started and worked - but when I looked at the application.log there was > > only 3 log entries. > > > > 2018-03-19 12:22:22,447 WARN - [main:] ~ No meta.properties file under > > dir > > /Users/david/atlasreview2/distro/target/apache-atlas-1.0.0-SNAPSHOT-bin/apache-atlas-1.0.0-SNAPSHOT/data/kafka/kafka/meta.properties > > (Logging$class:87) > > 2018-03-19 12:22:22,922 WARN - [main:] ~ No meta.properties file under > > dir > > /Users/david/atlasreview2/distro/target/apache-atlas-1.0.0-SNAPSHOT-bin/apache-atlas-1.0.0-SNAPSHOT/data/kafka/kafka/meta.properties > > (Logging$class:87) > > 2018-03-19 12:22:22,986 INFO - [NotificationHookConsumer thread-0:] ~ > > [atlas-hook-consumer-thread]: Starting (Logging$class:72) > > > > > > I would have expected much more logging. > > Pierre Padovani wrote: > This confused me as well. All of the logging is in the the newest > atlas.<date stamp>.out file. Which makes sense when log4j is configured to > log to a rolling date/time stamped file. In previous builds of Atlas the > logging all showed up in the application.log file. This change in behavior > happened to me within the last few weeks. Do we know if a change was made to > induce this? or Was this induced by my changes? > > I'll do some testing on my side.
Just retested off of master... something in the cassandra jars for Janus must be overriding the Atlas logging configuration. I'll investigate. - Pierre ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65885/#review199456 ----------------------------------------------------------- On March 19, 2018, 11:51 a.m., Pierre Padovani wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65885/ > ----------------------------------------------------------- > > (Updated March 19, 2018, 11:51 a.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 > ----- > > 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 > 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 > > > Diff: https://reviews.apache.org/r/65885/diff/3/ > > > 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 > >
