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

2015-10-13 Thread Amareshwari Sriramadasu
--- 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: [DISCUSS] Sushil Mohanty as Lens committer

2015-10-13 Thread Jean-Baptiste Onofré
+1 Regards JB On 10/13/2015 09:18 AM, Rajat Khandelwal wrote: Hi all I'd like to propose Sushil be made committer for Apache Lens. His contributions on GitHub He's worked on 32 issues

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

2015-10-13 Thread Rajat Khandelwal
> On Oct. 13, 2015, 1:37 p.m., Amareshwari Sriramadasu wrote: > > lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java, > > line 207 > > > > > > Can we avoid opening up the constructor?

[jira] [Commented] (LENS-807) Priority is not getting set in QueryContext. It's always NORMAL

2015-10-13 Thread Rajat Khandelwal (JIRA)
[ https://issues.apache.org/jira/browse/LENS-807?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14954603#comment-14954603 ] Rajat Khandelwal commented on LENS-807: --- Taking patch from reviewboard and attaching > Priority is

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

2015-10-13 Thread Amareshwari Sriramadasu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39127/#review102408 ---

[jira] [Commented] (LENS-816) ThreadSafeEstimatedQueryCollection is ignoring driver argument in getQueries and getQueryCount

2015-10-13 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/LENS-816?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14954618#comment-14954618 ] Hadoop QA commented on LENS-816: Applied patch:

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

2015-10-13 Thread Amareshwari Sriramadasu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39191/#review102403 ---

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

2015-10-13 Thread Raju Bairishetti
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38521/ --- (Updated Oct. 13, 2015, 10:54 a.m.) Review request for lens, Amareshwari

[jira] [Commented] (LENS-607) Test Failures on java8

2015-10-13 Thread Raju Bairishetti (JIRA)
[ https://issues.apache.org/jira/browse/LENS-607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14954990#comment-14954990 ] Raju Bairishetti commented on LENS-607: --- Thanks [~prongs] for detailed review :) . > Test Failures on

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

2015-10-13 Thread Deepak Barr
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39161/ --- (Updated Oct. 13, 2015, 11:48 a.m.) Review request for lens. Changes ---

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

2015-10-13 Thread Rajat Khandelwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38521/#review102442 --- Ship it! Ship It! - Rajat Khandelwal On Oct. 13, 2015, 6:16

[jira] [Commented] (LENS-607) Test Failures on java8

2015-10-13 Thread Hadoop QA (JIRA)
[ https://issues.apache.org/jira/browse/LENS-607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14955278#comment-14955278 ] Hadoop QA commented on LENS-607: Applied patch:

[jira] [Updated] (LENS-607) Test Failures on java8

2015-10-13 Thread Raju Bairishetti (JIRA)
[ https://issues.apache.org/jira/browse/LENS-607?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Raju Bairishetti updated LENS-607: -- Status: Patch Available (was: In Progress) > Test Failures on java8 > -- > >

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

2015-10-13 Thread Deepak Barr
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39286/ --- Review request for lens. Repository: lens Description --- Removed the

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

2015-10-13 Thread Gabor Liptak
> On Oct. 13, 2015, 8:08 a.m., Rajat Khandelwal wrote: > > lens-cli/src/test/java/org/apache/lens/cli/TestLensQueryCommands.java, > > lines 281-291 > > > > > > Data providers. You can look at > >