> 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.

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.


- 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
> 
>

Reply via email to