> On June 15, 2015, 6:25 a.m., Shwetha GS wrote: > > core/src/main/java/org/apache/oozie/util/MetricsInstrumentation.java, line > > 108 > > <https://reviews.apache.org/r/34854/diff/7/?file=983898#file983898line108> > > > > change log level to debug
Since this is logged only once at the startup of the oozie service, it would be ok to leave this at Info level, as it is useful to know what the metrics sync is. - Srikanth ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34854/#review87894 ----------------------------------------------------------- On June 15, 2015, 12:49 p.m., Narayan Periwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/34854/ > ----------------------------------------------------------- > > (Updated June 15, 2015, 12:49 p.m.) > > > Review request for oozie. > > > Bugs: OOZIE-2251 > https://issues.apache.org/jira/browse/OOZIE-2251 > > > Repository: oozie-git > > > Description > ------- > > We have been logging so many important matrices in oozie-instrumentation.log > . These information is very useful for oozie functional monitoring. But it is > always difficult to get the meaning from flat file. If we expose this > information on some graphing tool, We can get the lot of meaning out of it > and can take some actions based on it. > > > Diffs > ----- > > core/pom.xml 7877773 > > core/src/main/java/org/apache/oozie/service/MetricsInstrumentationService.java > 2a00a73 > core/src/main/java/org/apache/oozie/util/Instrumentation.java 4eb6386 > core/src/main/java/org/apache/oozie/util/MetricsInstrumentation.java > 185b67e > core/src/main/resources/oozie-default.xml 4dc127e > docs/src/site/twiki/AG_Install.twiki 0ce2609 > webapp/pom.xml e42e219 > > Diff: https://reviews.apache.org/r/34854/diff/ > > > Testing > ------- > > Done > > > Thanks, > > Narayan Periwal > >
