-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46968/
-----------------------------------------------------------
(Updated May 9, 2016, 6:36 a.m.)
Review request for lens.
Changes
-------
**Reverted last patch** (Fiexd review comment - added back err counter on
failing to close file)
**Ran Build**
[INFO] Lens Checkstyle Rules ............................. SUCCESS [1.554s]
[INFO] Lens .............................................. SUCCESS [4.660s]
[INFO] Lens API .......................................... SUCCESS [22.497s]
[INFO] Lens API for server and extensions ................ SUCCESS [14.501s]
[INFO] Lens Cube ......................................... SUCCESS [7:03.831s]
[INFO] Lens DB storage ................................... SUCCESS [15.612s]
[INFO] Lens Query Library ................................ SUCCESS [11.233s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:26.034s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [29.249s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [14.076s]
[INFO] Lens Server ....................................... SUCCESS [13:24.649s]
[INFO] Lens client ....................................... SUCCESS [1:43.343s]
[INFO] Lens CLI .......................................... SUCCESS [2:10.200s]
[INFO] Lens Examples ..................................... SUCCESS [7.444s]
[INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.634s]
[INFO] Lens Distribution ................................. SUCCESS [8.993s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:07.881s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [1.595s]
[INFO] Lens Regression ................................... SUCCESS [11.479s]
[INFO] Lens UI ........................................... SUCCESS [49.357s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 31:09.528s
[INFO] Finished at: Sat May 07 12:23:59 UTC 2016
Bugs: lens-1029
https://issues.apache.org/jira/browse/lens-1029
Repository: lens
Description
-------
- Service level persistence isolation. If persisting one service fails, other
services should still be persisted.
- Persistence thread to run only in case SERVER_RESTART_ENABLED = true
- Moved form Timer to ScheduledExecutorService. Graceful shutdown of
ScheduledExecutorService enabled to allow a running persistence task, if any,
to finish
- Catching Exception instead of IOException to prevent the persistence task
from dying.
- using writeObject() instead of writeInt() for automatic null handling.
Diffs (updated)
-----
lens-query-lib/src/main/java/org/apache/lens/lib/query/AbstractFileFormatter.java
8c06621
lens-server-api/src/main/java/org/apache/lens/server/api/LensConfConstants.java
23537cb
lens-server/src/main/java/org/apache/lens/server/LensServices.java 48b3e00
lens-server/src/main/resources/lensserver-default.xml 1a15658
lens-server/src/test/java/org/apache/lens/server/TestServerRestart.java
1fa61ef
src/site/apt/admin/config.apt 5466e7a
src/site/apt/admin/deployment.apt b4f4d0a
Diff: https://reviews.apache.org/r/46968/diff/
Testing
-------
Relying on exiting test case to check persistence TestServerRestart.
**mvn test -Dtest=org.apache.lens.server.TestServerRestart#testQueryService**
Running org.apache.lens.server.TestServerRestart
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 206.408 sec -
in org.apache.lens.server.TestServerRestart
Results :
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
**mvn test -Dtest=org.apache.lens.server.TestServerRestart#testSessionRestart**
Running org.apache.lens.server.TestServerRestart
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 21.711 sec - in
org.apache.lens.server.TestServerRestart
Results :
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
**Apart from this**
- Did some local testing for making sure null Integers are persisted using
writeObject and can be read back as well.
- Did some local testing to check graceful shutdown of ScheduledExecutorService
Don't think we need a test cases for above two cince its supported out of box
by java
Thanks,
Puneet Gupta
