-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36981/
-----------------------------------------------------------
(Updated Aug. 3, 2015, 10:59 a.m.)
Review request for lens.
Bugs: LENS-666
https://issues.apache.org/jira/browse/LENS-666
Repository: lens
Description
-------
Changes include :
- Remove log4j dependency
- Move existing log4j.propertise to logback.xml
- CLI verbose/debug logger moved to logback
- Move QueryExecutionStat loggers to logback
- Remove transitive log4j dependencies ( couldnt remove provided ones as some
tests are failing)
- update licenses wrt logback
Diffs
-----
bin-dist-files/LICENSE 5d593ad
bin-dist-files/NOTICE 1fbb09c
lens-api/pom.xml e1b763d
lens-cli/src/main/java/org/apache/lens/cli/commands/LensConnectionCommands.java
c059f30
lens-client/pom.xml d83f830
lens-cube/pom.xml 36b508a
lens-cube/src/test/resources/log4j.properties deaf1e1
lens-cube/src/test/resources/logback.xml PRE-CREATION
lens-driver-hive/src/test/resources/log4j.properties 2c2dc45
lens-driver-hive/src/test/resources/logback.xml PRE-CREATION
lens-driver-jdbc/pom.xml 2cb7f58
lens-query-lib/src/test/resources/log4j.properties a69d13a
lens-query-lib/src/test/resources/logback.xml PRE-CREATION
lens-server-api/pom.xml 6822976
lens-server/pom.xml 5174607
lens-server/src/main/java/org/apache/lens/server/LensServletContextListener.java
0945a35
lens-server/src/main/java/org/apache/lens/server/stats/event/LoggableLensStatistics.java
e8a7b0c
lens-server/src/main/java/org/apache/lens/server/stats/store/log/StatisticsLogFileScannerTask.java
4144416
lens-server/src/main/java/org/apache/lens/server/stats/store/log/StatisticsLogLayout.java
67a163c
lens-server/src/main/java/org/apache/lens/server/stats/store/log/StatisticsLogRollupHandler.java
9ff878c
lens-server/src/main/resources/lensserver-default.xml 6af8d10
lens-server/src/main/webapp/WEB-INF/log4j.properties f5fdffa
lens-server/src/main/webapp/WEB-INF/web.xml 4ee54a6
lens-server/src/test/java/org/apache/lens/server/healthcheck/TestHealthChecks.java
3573a1e
lens-server/src/test/java/org/apache/lens/server/stats/TestLogStatisticsStore.java
c8a0ea5
lens-server/src/test/java/org/apache/lens/server/stats/TestStatisticsLogFileScannerTask.java
819c62a
lens-server/src/test/resources/log4j.properties 9cc1e36
lens-server/src/test/resources/logback.xml PRE-CREATION
pom.xml 3c98077
src/site/apt/admin/config.apt 2e1dbe7
src/site/apt/admin/monitoring.apt 3ad2f92
src/site/apt/lenshome/install-and-run.apt 8016491
tools/conf-pseudo-distr/client/log4j.properties 3756cf7
tools/conf-pseudo-distr/client/logback.xml PRE-CREATION
tools/conf-pseudo-distr/server/log4j.properties 2b04ea5
tools/conf-pseudo-distr/server/logback.xml PRE-CREATION
tools/conf/client/log4j.properties 3756cf7
tools/conf/client/logback.xml PRE-CREATION
tools/conf/server/log4j.properties 2b04ea5
tools/conf/server/logback.xml PRE-CREATION
Diff: https://reviews.apache.org/r/36981/diff/
Testing (updated)
-------
Tested query log fetching
Tested query execution statistics
Tested debug/verbose logging
Found a couple of issues while testing:
1. Some issues in lens-server logs:
$ ls logs/
c6c970ca-5034-4266-9cfc-8a9b8efe30ee.log lensserver-requests.log
lensserver.out.2015080315161438595208 unknown.log
lensserver-query-status.log lensserver.log
query-stats.log
> unknown.log is getting created with all the logs that does not belong to
> query logs.
> All the logs that are coming from dependent libraries (hive and hadoop) are
> landing in lensserver.out.2015080315161438595208, as they are not coming from
> slf4j.
2. Bringing up lens-cli gives following warnings :
$ bin/lens-cli.sh
log4j:WARN No appenders could be found for logger
(org.springframework.core.env.StandardEnvironment).
log4j:WARN Please initialize the log4j system properly.
log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more
info.
----
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Lens Checkstyle Rules ............................. SUCCESS [2.122s]
[INFO] Lens .............................................. SUCCESS [4.099s]
[INFO] Lens API .......................................... SUCCESS [21.945s]
[INFO] Lens API for server and extensions ................ SUCCESS [18.553s]
[INFO] Lens Cube ......................................... SUCCESS [8:39.988s]
[INFO] Lens DB storage ................................... SUCCESS [19.586s]
[INFO] Lens Query Library ................................ SUCCESS [15.894s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:55.023s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [39.812s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [15.639s]
[INFO] Lens Server ....................................... SUCCESS [5:49.343s]
[INFO] Lens client ....................................... SUCCESS [37.553s]
[INFO] Lens CLI .......................................... SUCCESS [2:39.779s]
[INFO] Lens Examples ..................................... SUCCESS [11.432s]
[INFO] Lens Distribution ................................. SUCCESS [10.234s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:26.963s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [1.982s]
[INFO] Lens Regression ................................... SUCCESS [11.181s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 25:02.067s
[INFO] Finished at: Mon Aug 03 10:22:32 UTC 2015
[INFO] Final Memory: 191M/1698M
[INFO] ------------------------------------------------------------------------
Thanks,
Amareshwari Sriramadasu
