-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/45021/
-----------------------------------------------------------

Review request for lens.


Bugs: lens-990
    https://issues.apache.org/jira/browse/lens-990


Repository: lens


Description
-------

Updated isResultAvailableInDriver() to check for query success first 
(isDriverPersistent() is true even if query fails).


Diffs
-----

  
lens-server-api/src/main/java/org/apache/lens/server/api/driver/DriverQueryStatus.java
 79d8bf1e8f9798714d14e5a0336e24a148c4ca27 
  
lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java
 d01e4a41c0cb1f4a604df27092b335e593b11ae4 
  
lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java
 4c95506269058554d5e32cb367deeb8a4b987966 

Diff: https://reviews.apache.org/r/45021/diff/


Testing
-------

mvn clean install -Dtest=org.apache.lens.server.query.TestQueryService
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [2.247s]
[INFO] Lens .............................................. SUCCESS [3.706s]
[INFO] Lens API .......................................... SUCCESS [28.077s]
[INFO] Lens API for server and extensions ................ SUCCESS [16.889s]
[INFO] Lens Cube ......................................... SUCCESS [40.877s]
[INFO] Lens DB storage ................................... SUCCESS [8.629s]
[INFO] Lens Query Library ................................ SUCCESS [11.784s]
[INFO] Lens Hive Driver .................................. SUCCESS [14.949s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [15.763s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [15.070s]
[INFO] Lens Server ....................................... SUCCESS [7:16.786s]
[INFO] Lens client ....................................... SUCCESS [16.436s]
[INFO] Lens CLI .......................................... SUCCESS [14.067s]
[INFO] Lens Examples ..................................... SUCCESS [9.952s]
[INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.800s]
[INFO] Lens Distribution ................................. SUCCESS [10.034s]
[INFO] Lens ML Lib ....................................... SUCCESS [18.355s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [2.143s]
[INFO] Lens Regression ................................... SUCCESS [13.020s]
[INFO] Lens UI ........................................... SUCCESS [27.963s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------


Thanks,

Puneet Gupta

Reply via email to