Re: Review Request 39394: LENS-830: Checkstyle version should be picked from project version

394/#comment162025> Can you verify mvn versions:set works without issues? Also verify mvn release:prepare --dry-run works without issues? - Amareshwari Sriramadasu On Oct. 16, 2015, 12:16 p.m., Rajat Khandelwal

Re: Review Request 39576: LENS-833 : Limit number of open sessions per user on session service

> On Oct. 25, 2015, 1:54 a.m., Amareshwari Sriramadasu wrote: > > lens-server/src/main/java/org/apache/lens/server/BaseLensService.java, line > > 89 > > <https://reviews.apache.org/r/39576/diff/2/?file=1103857#file1103857line89> > > > > Why is th

Re: Review Request 39439: LENS-835 : Optimizing fact subquery by removing non aggregated expression columns from group by clause

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39439/#review103915 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 19, 2015

Re: Review Request 39419: LENS-824 Fix all javadoc warnings/errors

, reopened. Gabor, Do you still see errors on generated doc? Raju/Gabor, Is that resulting in build failure in java-8? - Amareshwari Sriramadasu On Oct. 19, 2015, 1:33 a.m., Gabor Liptak wrote: > > --- > This is an automatically ge

Re: Review Request 39483: LENS-838 : Literals with escaped quotes are not honoured in columnarSQLRewriter

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39483/#review103914 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 20, 2015

Re: Review Request 39606: LENS-843 : Columns in having clause not getting push down

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39606/#review103912 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 23, 2015

Re: Review Request 39650: LENS-846: Not all filters on fact is getting push down

e ) <https://reviews.apache.org/r/39650/#comment162185> Can you add test with only filter being present on fact.time and it is not selected - Amareshwari Sriramadasu On Oct. 26, 2015, 11:22 a.m., Sushil M

Re: Review Request 39651: LENS-845: Take regex rename arguments in columnar sql rewriter

g/r/39651/#comment162183> Is this required? - Amareshwari Sriramadasu On Oct. 26, 2015, 11:16 a.m., Rajat Khandelwal wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://re

Re: Review Request 39680: LENS-848: Allow JDBC driver to take connection properties in configuration

include unit test? - Amareshwari Sriramadasu On Oct. 27, 2015, 8:08 a.m., Rajat Khandelwal wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http

Re: Review Request 39680: LENS-848: Allow JDBC driver to take connection properties in configuration

/JDBCDriver.java (line 705) <https://reviews.apache.org/r/39680/#comment162442> JDBC_DB_URI is added twice and JDBC_DRIVER_CLASS is removed - Amareshwari Sriramadasu On Oct. 27, 2015, 12:38 p.m., Rajat Khandelwal

Review Request 39686: LENS-849 : Fix NPE in CandidateTableResolver with expression dimension

Thanks, Amareshwari Sriramadasu

Re: Review Request 39680: LENS-848: Allow JDBC driver to take connection properties in configuration

/DataSourceConnectionProvider.java (line 195) <https://reviews.apache.org/r/39680/#comment162452> Seems a wrong property setting even earlier - Amareshwari Sriramadasu On Oct. 27, 2015, 2:53 p.m., Rajat Khandelwal

Re: Review Request 39651: LENS-845: Take regex rename arguments in columnar sql rewriter

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39651/#review104001 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 26, 2015

Re: Review Request 39686: LENS-849 : Fix NPE in CandidateTableResolver with expression dimension

/ Testing --- All lens-cube tests succeeded Thanks, Amareshwari Sriramadasu

Re: Review Request 39680: LENS-848: Allow JDBC driver to take connection properties in configuration

<https://reviews.apache.org/r/39680/#comment162444> Remove this - Amareshwari Sriramadasu On Oct. 27, 2015, 1:37 p.m., Rajat Khandelwal wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http

Re: Review Request 39911: LENS-760 : Session close should not result in running query failures

> On Nov. 4, 2015, 3:51 p.m., Puneet Gupta wrote: > > lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java, > > line 1249 > > > > > > What happens to skipped(open) sessions ? do we need to

Re: Review Request 39390: LENS-828: Add derived cube validation

tps://reviews.apache.org/r/39390/#comment165653> Let us start the numbers from 3100 or 3500, sothat query errors can be separated from metastore errors? - Amareshwari Sriramadasu On Nov. 10, 2015, 6:54 a.m., Rajat Khandelwal

Re: Review Request 39419: LENS-824 Fix all javadoc warnings/errors

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39419/#review104675 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 19, 2015

Re: Review Request 39809: LENS-719: Allow fact start time to be specified for a storage and a update period

hese changes for? - Amareshwari Sriramadasu On Oct. 30, 2015, 11:24 a.m., Rajat Khandelwal wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://re

Re: Review Request 39897: LENS-860 : Updating committer documentation

l. To reply, visit: > https://reviews.apache.org/r/39897/ > --- > > (Updated Nov. 3, 2015, 12:28 p.m.) > > > Review request for lens, Amareshwari Sriramadasu and Rajat Khandelwal. > > > Repository: lens > > > Desc

Re: Review Request 39911: LENS-760 : Session close should not result in running query failures

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39911/#review105035 --- Can you add unit test? - Amareshwari Sriramadasu On Nov. 3

Re: Review Request 39653: LENS-844: querying a measure with dim attributes not queryable with it gives wrong error

eviews.apache.org/r/39653/#comment162800> returning null ? lens-cube/src/main/java/org/apache/lens/cube/parse/CandidateTableResolver.java (line 272) <https://reviews.apache.org/r/39653/#comment162802> system.out ? Can we remove? - Amareshwari Sriramadasu On Oct. 26, 2015,

Re: Review Request 39419: LENS-824 Fix all javadoc warnings/errors

> On Oct. 25, 2015, 3:40 a.m., Amareshwari Sriramadasu wrote: > > I see the issue is still open, but review request is closed. So, reopened. > > > > Gabor, Do you still see errors on generated doc? > > Raju/Gabor, Is that resulting in build failure in java-8?

Re: Review Request 39390: LENS-828: Add derived cube validation

/DerivedCube.java (line 88) <https://reviews.apache.org/r/39390/#comment163929> Can we map the error with 403 error code and put error message in lens-errors.conf ? - Amareshwari Sriramadasu On Oct. 16, 2015, 11:51 a.m., Rajat Khandelwal

Re: Review Request 39897: LENS-860 : Updating committer documentation

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39897/#review105360 --- Ship it! Ship It! - Amareshwari Sriramadasu On Nov. 4, 2015, 8

Re: Review Request 39576: LENS-833 : Limit number of open sessions per user on session service

o lens-errors.conf and add error to LensCommonErrorCode - Amareshwari Sriramadasu On Oct. 31, 2015, 12:24 a.m., Raju Bairishetti wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 39809: LENS-719: Allow fact start time to be specified for a storage and a update period

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39809/#review105366 --- Ship it! Ship It! - Amareshwari Sriramadasu On Nov. 2, 2015, 9

Re: Review Request 39894: LENS-858: Fixes no cubes available even when there are.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39894/#review105435 --- Ship it! Ship It! - Amareshwari Sriramadasu On Nov. 3, 2015

Re: Review Request 39842: Ability to load different instances of same driver class

trings or enum? - Amareshwari Sriramadasu On Nov. 2, 2015, 4:08 a.m., Puneet Gupta wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 39895: LENS-851: queries where results of two storage tables of same fact are unioned, the rows should be aggregated

similar code in other methods also. Can be separated into another method ? - Amareshwari Sriramadasu On Nov. 3, 2015, 2 p.m., Rajat Khandelwal wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http

Re: Review Request 39895: LENS-851: queries where results of two storage tables of same fact are unioned, the rows should be aggregated

, functions around selected fields (for dimattributes) ? - Amareshwari Sriramadasu On Nov. 3, 2015, 2 p.m., Rajat Khandelwal wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 39430: Create a single jar for shipping with hive queries

g/r/39430/#comment160983> Can we include open csv aswell, which is right now coming as transitive dependency from lens-query-lib? - Amareshwari Sriramadasu On Oct. 19, 2015, 4:46 a.m., Srikanth Sundarrajan

Re: Review Request 39394: LENS-830: Checkstyle version should be picked from project version

mvn release:prepare --dry-run works without issues? - Amareshwari Sriramadasu On Oct. 16, 2015, 12:16 p.m., Rajat Khandelwal wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 39161: LENS-292 : Query result retention policy

/QueryResultPurger.java (line 74) <https://reviews.apache.org/r/39161/#comment161064> Can we rename variable to QUERY_RESULT_PURGER_ERROR_COUNTER - Amareshwari Sriramadasu On Oct. 19, 2015, 11:33 a.m., Deepak Barr

Review Request 39434: LENS-832 : Fix recovery for Queued queries

/server/api/query/TestAbstractQueryContext.java 44978f2 lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java 7b15a3c Diff: https://reviews.apache.org/r/39434/diff/ Testing --- Thanks, Amareshwari Sriramadasu

Re: Review Request 39434: LENS-832 : Fix recovery for Queued queries

UTC 2015 [INFO] Final Memory: 187M/1611M [INFO] Thanks, Amareshwari Sriramadasu

Re: Review Request 39127: LENS-807: Priority is not getting set in QueryContext. It's always NORMAL

/QueryContext.java (line 429) <https://reviews.apache.org/r/39127/#comment159406> I feel instead of adding this public method, setting priority after deciding would have been better. Can we add a unit test ? - Amareshwari Sriramadasu On Oct. 8, 2015, 1:18 p.m., Rajat Khandelwal

Re: Review Request 38758: UI for Saved/Parametrized queries + Bug fixes/Enhancements

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38758/#review101900 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 8, 2015

Re: Review Request 39373: LENS-341: Update docs with java8 compatibility

373/#comment160682> This need not be specified in readme. as end users wont generate docs. Specifying this in contribute doc is enough. - Amareshwari Sriramadasu On Oct. 16, 2015, 2:49 a.m., Raju Bairishetti

Re: Review Request 39344: LENS-827:Expression pushdown for query optimisation in JDBC

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39344/#review102862 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 15, 2015

Re: Review Request 39373: LENS-341: Update docs with java8 compatibility

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39373/#review102873 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 16, 2015

Re: Review Request 39434: LENS-832 : Fix recovery for Queued queries

utomatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39434/#review103221 --- On Oct. 19, 2015, 10:26 a.m., Amareshwari Sriramadasu wrote: > > --- > This is an

Re: Review Request 39161: LENS-292 : Query result retention policy

test to finish, can we change purge interval to 1 second for test and wait for 2 seconds? - Amareshwari Sriramadasu On Oct. 16, 2015, 7:43 p.m., Deepak Barr wrote: > > --- > This is an automatically generated e-mail. To reply, vi

Re: Review Request 39286: LENS-817 : No need to persist metadata class information in DB after LENS-576 fix

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39286/#review102729 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 13, 2015

Re: Review Request 39191: LENS-615 Refactoring of testQueryCommands test case

> On Oct. 14, 2015, 10:04 a.m., Rajat Khandelwal wrote: > > lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java, line > > 451 > > > > > > can you take care of the TODO? > > Gabor Liptak wrote: >

Re: Review Request 39191: LENS-615 Refactoring of testQueryCommands test case

/TestLensQueryCommands.java (line 183) <https://reviews.apache.org/r/39191/#comment160485> Can you reset hive.exec.driver.run.hooks to be empty string? - Amareshwari Sriramadasu On Oct. 14, 2015, 1:30 a.m., Gabor Liptak

Re: Review Request 39311: LENS-820: Add relative end time for facts

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39311/#review102736 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 14, 2015

Re: Review Request 39315: LENS-821:Filters with comma in IN filters not replacing table alias properly

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39315/#review102611 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 14, 2015

Re: Review Request 39161: LENS-292 : Query result retention policy

nal results. lens-server/src/main/resources/lensserver-default.xml (line 288) <https://reviews.apache.org/r/39161/#comment160483> Can you update apt files as well? - Amareshwari Sriramadasu On Oc

Re: Review Request 39191: LENS-615 Refactoring of testQueryCommands test case

re of the TODO? > > Gabor Liptak wrote: > Rajat, what is the recommended way to reset this value? This key might or > might not exists, the getConnectionParam() returns a List and there > doesn't seem to be a method to delete a key ... Thanks > > Amareshwari Srirama

Review Request 39221: LENS-533 : Update critical metrics on monitoring doc

/browse/LENS-533 Repository: lens Description --- Added critical metrics section Diffs - src/site/apt/admin/monitoring.apt 9429394 Diff: https://reviews.apache.org/r/39221/diff/ Testing --- Verified by running mvn site:run Thanks, Amareshwari Sriramadasu

Review Request 39081: LENS-446 : Remove server params from session and driver configurations

at: Wed Oct 07 11:07:20 UTC 2015 [INFO] Final Memory: 195M/2046M [INFO] Thanks, Amareshwari Sriramadasu

Re: Review Request 39081: LENS-446 : Remove server params from session and driver configurations

drivers should not require any of them. - Amareshwari --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39081/#review101757 ----------- O

Re: Review Request 39127: LENS-807: Priority is not getting set in QueryContext. It's always NORMAL

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39127/#review102412 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 12, 2015

Re: Review Request 39127: LENS-807: Priority is not getting set in QueryContext. It's always NORMAL

/QueryContext.java (line 207) <https://reviews.apache.org/r/39127/#comment160043> Can we avoid opening up the constructor? - Amareshwari Sriramadasu On Oct. 12, 2015, 8:22 a.m., Rajat Khandelwal wrote: > > --- > This is a

Re: Review Request 39191: LENS-615 Refactoring of testQueryCommands test case

/TestLensQueryCommands.java (line 69) <https://reviews.apache.org/r/39191/#comment160035> I think it should be @BeforeTest instead of @BeforeClass - Amareshwari Sriramadasu On Oct. 12, 2015, 11:27 p.m., Gabor Liptak wrote: > > --- > This is a

Re: Review Request 38758: UI for Saved/Parametrized queries + Bug fixes/Enhancements

source filter and update version through variable. - Amareshwari Sriramadasu On Oct. 6, 2015, 6:06 a.m., Ankeet Maini wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 39039: LENS-792: StatusPoller logs are useless

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39039/#review101618 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 6, 2015, 7

Re: Review Request 38758: UI for Saved/Parametrized queries + Bug fixes/Enhancements

> On Oct. 6, 2015, 6:06 a.m., Amareshwari Sriramadasu wrote: > > lens-ui/package.json, line 3 > > <https://reviews.apache.org/r/38758/diff/2/?file=1084763#file1084763line3> > > > > If the version is hardcoded this way, and not reading through

Re: Review Request 39127: LENS-807: Priority is not getting set in QueryContext. It's always NORMAL

/QueryContext.java (line 430) <https://reviews.apache.org/r/39127/#comment159578> Can you update unit test to assert on query context's priority? - Amareshwari Sriramadasu On Oct. 8, 2015, 1:18 p.m., Rajat Khandelwal

Re: Review Request 39131: LENS -750 : Handle SemanticException from HiveDriver instead of throwing 500 Internal Server Error

/HiveDriver.java (line 517) <https://reviews.apache.org/r/39131/#comment159579> Can you do the same changes for explain, estimate, executeAsync? Shall we have handleHiveSQLException method throws proper LensException? - Amareshwari Sriramadasu On Oct. 8, 2015, 1:45 p.m., Deepa

Re: Review Request 37346: LENS-693: Query Result Mail is not working when finishedQueries is set to zero

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37346/#review96493 --- Ship it! Ship It! - Amareshwari Sriramadasu On Aug. 21, 2015

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

what is queue number in driver status? lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java (line 618) https://reviews.apache.org/r/37716/#comment152310 why would a launched query have queue number? - Amareshwari Sriramadasu On Aug. 27, 2015, 1:34

Re: Review Request 37346: LENS-693: Query Result Mail is not working when finishedQueries is set to zero

/QueryExecutionServiceImpl.java (line 487) <https://reviews.apache.org/r/37346/#comment153095> One in-memory result that is not fetched should not block other queries to be purged. Can you check if that can happen? - Amareshwari Sriramadasu On Aug. 27, 2015, 12:47 p.m., Rajat Khandelwal

Re: Review Request 37530: LENS-629 - A new improved web client

> On Aug. 19, 2015, 7:27 a.m., Amareshwari Sriramadasu wrote: > > lens-ui/package.json, line 12 > > <https://reviews.apache.org/r/37530/diff/2/?file=1041996#file1041996line12> > > > > How are these dependencies downloaded? > > >

Re: Review Request 37530: LENS-629 - A new improved web client

> On Aug. 19, 2015, 5:38 a.m., Amareshwari Sriramadasu wrote: > > lens-ui/app/adapters/AdhocQueryAdapter.js, line 9 > > <https://reviews.apache.org/r/37530/diff/2/?file=1041958#file1041958line9> > > > > Not accepting adhoc queries on dimensions? >

Re: Review Request 37648: LENS-737: Throw single error out with LensMultiException

of that? - Amareshwari Sriramadasu On Sept. 4, 2015, 2:27 p.m., Sushil Mohanty wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 37648: LENS-737: Throw single error out with LensMultiException

> On Sept. 7, 2015, 8:27 a.m., Amareshwari Sriramadasu wrote: > > I dont see the failing test commented. Can you please take care of that? Failing test is not related to the patch, so suggesting to comment testExplainFailure and should be fixed as part of LENS-769 - Am

Re: Review Request 37934: LENS-742 Saved query and parameterization

lens-server/src/test/java/org/apache/lens/server/query/save/TestSavedQueryService.java (line 105) <https://reviews.apache.org/r/37934/#comment154002> Please go ahead and remove @localhost, if it needs to be removed. lens-server/src/test/resources/lens-site.xml (lines 141 - 152) <http

Re: Review Request 37934: LENS-742 Saved query and parameterization

> On Sept. 7, 2015, 9:44 a.m., Amareshwari Sriramadasu wrote: > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/ParameterCollectionType.java, > > line 31 > > <https://reviews.apache.org/r/37934/diff/3/?file=1064152#file1064152line31>

Re: Review Request 37934: LENS-742 Saved query and parameterization

> On Sept. 7, 2015, 9:44 a.m., Amareshwari Sriramadasu wrote: > > lens-server-api/src/main/java/org/apache/lens/server/api/query/save/ParameterCollectionType.java, > > line 31 > > <https://reviews.apache.org/r/37934/diff/3/?file=1064152#file1064152line31>

Re: Review Request 37648: LENS-737: Throw single error out with LensMultiException

/LensStatement.java (lines 148 - 154) <https://reviews.apache.org/r/37648/#comment154039> Remove commented code? - Amareshwari Sriramadasu On Sept. 7, 2015, 4:32 p.m., Sushil Mohanty wrote: > > --- > This is an automatically g

Re: Review Request 37434: LENS-710 : Allow column name mapping for few/all columns in underlying storage tables

at: Thu Aug 13 10:37:33 UTC 2015 [INFO] Final Memory: 193M/1835M [INFO] Thanks, Amareshwari Sriramadasu

Re: Review Request 37648: LENS-737: Throw single error out with LensMultiException

g/r/37648/#comment153750> Error message construction is not changed to read from LensAPIException here. - Amareshwari Sriramadasu On Sept. 3, 2015, 6:26 a.m., Sushil Mohanty wrote: > > --- > This is an automatically gener

Re: Review Request 37934: LENS-742 Saved query and parameterization

Amruth Sampath wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/37934/ > --- > > (Updated Sept. 8, 2015, 6:11 p.m.) > > > Review request for l

Re: Review Request 37934: LENS-742 Saved query and parameterization

--- > > (Updated Sept. 8, 2015, 6:11 p.m.) > > > Review request for lens, Amareshwari Sriramadasu, Pranav Agarwal, Rajat > Khandelwal, and sharad agarwal. > > > Repository: lens > > > Description > --- > > Refer

Re: Review Request 38402: LENS-752 : Support flattening of columns selected through bridge-tables(many-to-many relationships)

/JoinResolver.java (line 582) <https://reviews.apache.org/r/38402/#comment156075> We should make this function configurable and can pass different function names from drivers. - Amareshwari Sriramadasu On Sept. 15, 2015, 12:48 p.m., Amareshwari Sriramadasu

Re: Review Request 38402: LENS-752 : Support flattening of columns selected through bridge-tables(many-to-many relationships)

or join to alist. If join needs to be converted to select from a bridge table - it becomes a full fledged query with select, from, groupby, filters and optionally more joins. - Amareshwari --- This is an automatically generated e-

Re: Review Request 37934: LENS-742 Saved query and parameterization

ove it? > > Amruth Sampath wrote: > There is an INVALID_XML_ERROR in LensCommonErrorCode which is best suited > to this. > > Rajat Khandelwal wrote: > syntax error was supposed to be for syntax errors in queries. But now > invalid queries are not accepted so I guess removing should be

Re: Review Request 38212: LENS-392: Look ahead not working when not all process time partitions on lower granularity are present

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38212/#review99009 --- Ship it! Ship It! - Amareshwari Sriramadasu On Sept. 14, 2015

Re: Review Request 37716: LENS-753: Queue number for queries in submitted queue

/QueryExecutionServiceImpl.java (line 618) <https://reviews.apache.org/r/37716/#comment155424> Set queue number as null for launched queries? - Amareshwari Sriramadasu On Sept. 14, 2015, 7:42 a.m., Akshay Goyal

Re: Review Request 38920: LENS-797: Improve CLI documentation to mention the helpful role of pressing tab while writing commands

rametters, mentioining of -- will be required. - Amareshwari Sriramadasu On Oct. 1, 2015, 9:11 a.m., Rajat Khandelwal wrote: > > --- > This is an automatically generated e-mail. To reply, visit: >

Re: Review Request 38857: LENS-796 : Slow response times for /metastore/nativetables API

/lens/server/metastore/CubeMetastoreServiceImpl.java (line 1224) <https://reviews.apache.org/r/38857/#comment158841> We might have to again split this up when we take up LENS-750. But I'm fine taking a decision as part of LENS-750 - Amareshwari Sriramadasu On Oct. 1, 2015, 6:

Re: Review Request 38666: LENS-772 : Fix time and size based rolling for logs

.apache.org/r/38666/#review100179 --- On Sept. 23, 2015, 9:02 a.m., Amareshwari Sriramadasu wrote: > > --- > This is an automatically generated e-mail. To reply, v

Re: Review Request 37346: LENS-693: Query Result Mail is not working when finishedQueries is set to zero

> On Sept. 22, 2015, 10:23 a.m., Amareshwari Sriramadasu wrote: > > lens-server-api/src/main/java/org/apache/lens/server/api/LensConfConstants.java, > > line 779 > > <https://reviews.apache.org/r/37346/diff/9/?file=1080331#file1080331line779> > &g

Re: Review Request 37346: LENS-693: Query Result Mail is not working when finishedQueries is set to zero

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37346/#review100149 --- Ship it! Ship It! - Amareshwari Sriramadasu On Sept. 23, 2015

Re: Review Request 38666: LENS-772 : Fix time and size based rolling for logs

g/r/38666/#review100161 --- On Sept. 23, 2015, 9:02 a.m., Amareshwari Sriramadasu wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revie

Re: Review Request 38666: LENS-772 : Fix time and size based rolling for logs

/server/logback.xml c27bea2 Diff: https://reviews.apache.org/r/38666/diff/ Testing --- Verified rolling is fine with the changes. Thanks, Amareshwari Sriramadasu

Re: Review Request 37530: LENS-629 - A new improved web client

530/#comment156932> I think we are already using 1.8, this would be conflicting. - Amareshwari Sriramadasu On Sept. 21, 2015, 12:14 p.m., Ankeet Maini wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 37346: LENS-693: Query Result Mail is not working when finishedQueries is set to zero

onServiceImpl.java (line 903) <https://reviews.apache.org/r/37346/#comment157025> If we are purging immediately, please take care of StatsUpdater also. - Amareshwari Sriramadasu On Sept. 22, 2015, 9:25 a.m.

Re: Review Request 37530: LENS-629 - A new improved web client

> On Aug. 19, 2015, 5:38 a.m., Amareshwari Sriramadasu wrote: > > lens-ui/app/adapters/AdhocQueryAdapter.js, line 9 > > <https://reviews.apache.org/r/37530/diff/2/?file=1041958#file1041958line9> > > > > Not accepting adhoc queries on dimensions? >

Re: Review Request 37530: LENS-629 - A new improved web client

625> Can these folders be moved to target/ ? Otherwise, how will they get cleaned up in next build? - Amareshwari Sriramadasu On Sept. 19, 2015, 8:44 a.m., Ankeet Maini wrote: > > --- > This is an automatically gen

Re: Review Request 38521: LENS-607: Fix Test failures in java8

/ColumnarSQLRewriter.java (line 95) <https://reviews.apache.org/r/38521/#comment156631> Isn't ArrayList ordered anymore? - Amareshwari Sriramadasu On Sept. 19, 2015, 6:48 a.m., Raju Bairishetti wrote: > > --- > This is a

Review Request 38524: LENS-113 : Document how to add feature documentation

://reviews.apache.org/r/38524/diff/ Testing --- Tested by running mvn site:run and verified doc. Thanks, Amareshwari Sriramadasu

Re: Review Request 38617: LENS-576 : Cannot retrieve query results on lens server restart

> On Sept. 24, 2015, 2:18 a.m., Puneet Gupta wrote: > > lens-query-lib/src/test/java/org/apache/lens/lib/query/TestAbstractFileFormatter.java, > > line 210 > > > > > > we can use writeObject() and readObject()

Re: Review Request 38617: LENS-576 : Cannot retrieve query results on lens server restart

> 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> > > > >

Re: Review Request 38617: LENS-576 : Cannot retrieve query results on lens server restart

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38617/#review100538 --- Ship it! Ship It! - Amareshwari Sriramadasu On Sept. 24, 2015

Re: Review Request 38617: LENS-576 : Cannot retrieve query results on lens server restart

/MockLensResultSetMetadata.java (line 63) <https://reviews.apache.org/r/38617/#comment157732> I dont see any difference between above method and this. Can you see if anything needs change or method can be removed? - Amareshwari Sriramadasu On Sept. 24, 2015, 12:10 p.m., Deepak Barr

Re: Review Request 40899: LENS-851: Replace columns with aliases in where clause of the inner query

> On Dec. 3, 2015, 11:34 p.m., Amareshwari Sriramadasu wrote: > > lens-cube/src/main/java/org/apache/lens/cube/parse/HQLParser.java, line 179 > > <https://reviews.apache.org/r/40899/diff/2/?file=1152902#file1152902line179> > > > > I think we already have e

Re: Review Request 40751: LENS - 882: provide option from CLI as well to list queries by driver

/LensQueryCommands.java (line 221) <https://reviews.apache.org/r/40751/#comment168809> Javadoc needs update lens-client/src/main/java/org/apache/lens/client/LensStatement.java (line 385) <https://reviews.apache.org/r/40751/#comment168811> javadoc needs update - Amareshwari Sriramadasu

  1   2   3   4   5   6   7   8   9   10   >