> On Sept. 25, 2015, 12:56 a.m., Amareshwari Sriramadasu wrote: > > lens-query-lib/src/test/java/org/apache/lens/lib/query/MockLensResultSetMetadata.java, > > line 63 > > <https://reviews.apache.org/r/38617/diff/3/?file=1083971#file1083971line63> > > > > I dont see any difference between above method and this. Can you see if > > anything needs change or method can be removed? > > Deepak Barr wrote: > The name of the second column name is different.
Hard to find. Thanks! Shall we rename methods to reflect what they are, instead of current names? - Amareshwari ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38617/#review100508 ----------------------------------------------------------- On Sept. 24, 2015, 12:10 p.m., Deepak Barr wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/38617/ > ----------------------------------------------------------- > > (Updated Sept. 24, 2015, 12:10 p.m.) > > > Review request for lens. > > > Repository: lens > > > Description > ------- > > The fix involves serializing QueryOutputFormatter > > > Diffs > ----- > > > lens-query-lib/src/main/java/org/apache/lens/lib/query/AbstractFileFormatter.java > ae5af03 > > lens-query-lib/src/main/java/org/apache/lens/lib/query/WrappedFileFormatter.java > e28c17b > > lens-query-lib/src/test/java/org/apache/lens/lib/query/MockLensResultSetMetadata.java > PRE-CREATION > > lens-query-lib/src/test/java/org/apache/lens/lib/query/TestAbstractFileFormatter.java > 35d7a00 > > lens-server-api/src/main/java/org/apache/lens/server/api/driver/LensResultSetMetadata.java > ef8aeed > > lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java > bed79ac > > lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryOutputFormatter.java > 0a6cc6b > > lens-server/src/main/java/org/apache/lens/server/query/LensPersistentResult.java > 1e9a182 > > lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java > 3bc9a4a > > Diff: https://reviews.apache.org/r/38617/diff/ > > > Testing > ------- > > Yes. Tested with all cases of lens persisted,driver persisted and inmemory > result sets. > > > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules .............................. SUCCESS [ 3.019 > s] > [INFO] Lens ............................................... SUCCESS [ 4.462 > s] > [INFO] Lens API ........................................... SUCCESS [ 29.054 > s] > [INFO] Lens API for server and extensions ................. SUCCESS [ 25.465 > s] > [INFO] Lens Cube .......................................... SUCCESS [07:00 > min] > [INFO] Lens DB storage .................................... SUCCESS [ 30.185 > s] > [INFO] Lens Query Library ................................. SUCCESS [ 21.199 > s] > [INFO] Lens Hive Driver ................................... SUCCESS [04:09 > min] > [INFO] Lens Driver for JDBC ............................... SUCCESS [ 51.156 > s] > [INFO] Lens Elastic Search Driver ......................... SUCCESS [ 22.259 > s] > [INFO] Lens Server ........................................ SUCCESS [08:06 > min] > [INFO] Lens client ........................................ SUCCESS [ 47.492 > s] > [INFO] Lens CLI ........................................... SUCCESS [03:46 > min] > [INFO] Lens Examples ...................................... SUCCESS [ 12.575 > s] > [INFO] Lens Distribution .................................. SUCCESS [ 12.408 > s] > [INFO] Lens ML Lib ........................................ SUCCESS [02:06 > min] > [INFO] Lens ML Ext Distribution ........................... SUCCESS [ 3.021 > s] > [INFO] Lens Regression .................................... SUCCESS [ 15.657 > s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 29:48 min > [INFO] Finished at: 2015-09-22T14:45:48+05:30 > [INFO] Final Memory: 228M/3007M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Deepak Barr > >
