----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38271/#review98487 -----------------------------------------------------------
Ship it! Ship It! - Sumit Mohanty On Sept. 10, 2015, 9:55 p.m., Jonathan Maron wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38271/ > ----------------------------------------------------------- > > (Updated Sept. 10, 2015, 9:55 p.m.) > > > Review request for Ambari, Erik Bergenholtz, John Speidel, and Sumit Mohanty. > > > Bugs: AMBARI-12964 > https://issues.apache.org/jira/browse/AMBARI-12964 > > > Repository: ambari > > > Description > ------- > > To avoid name collisions with other services the log4j configuration for > atlas is now prefixed with "atlas-". This patch updates the service > definition to align with that change. > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/configuration/atlas-env.xml > 98b1a47 > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/files/atlas-log4j.xml > PRE-CREATION > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/files/log4j.xml > 42d3c47 > > ambari-server/src/main/resources/common-services/ATLAS/0.1.0.2.3/package/scripts/metadata.py > a1627b9 > ambari-server/src/test/python/stacks/2.3/ATLAS/test_metadata_server.py > fd4c1ed > > Diff: https://reviews.apache.org/r/38271/diff/ > > > Testing > ------- > > Unit tests. > Installation of ambari with updates service definition to validate the log4j > configuration is correct. > > > Thanks, > > Jonathan Maron > >
