Re: Review Request 52259: Adding API for data completeness checker (LENS-1340)

2016-09-28 Thread Narayan Periwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52259/ --- (Updated Sept. 29, 2016, 6:34 a.m.) Review request for lens. Bugs: LENS-1340

Re: Review Request 52259: Adding API for data completeness checker (LENS-1340)

2016-09-28 Thread Narayan Periwal
> On Sept. 29, 2016, 5:04 a.m., Amareshwari Sriramadasu wrote: > > lens-cube/src/main/java/org/apache/lens/cube/parse/DataCompletenessChecker.java, > > line 35 > > > > > > Will it be measure names here or measure tag

Re: Review Request 52259: Adding API for data completeness checker (LENS-1340)

2016-09-28 Thread Amareshwari Sriramadasu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52259/#review150811 --- lens-cube/src/main/java/org/apache/lens/cube/parse/DataCompletene

Re: Review Request 52040: Handling data delay

2016-09-28 Thread Amareshwari Sriramadasu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52040/#review150810 --- lens-api/src/main/java/org/apache/lens/api/scheduler/SchedulerJob

Re: [VOTE] Release of Apache Lens 2.6.1

2016-09-28 Thread Raju Bairishetti
+1(binding) from my side Verified checksum, signatures and nexus repo. Able to build successfully from source. just a minor note, contrib/*.pom.xml.**swp* file exists in the source release package. I think we can ignore this. On Wed, Sep 28, 2016 at 5:00 PM, Puneet Gupta wrote: > Guys, I wil

Re: Review Request 52088: LENS-743: Query failure retries for transient errors

2016-09-28 Thread Rajat Khandelwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52088/ --- (Updated Sept. 28, 2016, 7:47 p.m.) Review request for lens. Bugs: LENS-743

Re: Review Request 52088: LENS-743: Query failure retries for transient errors

2016-09-28 Thread Rajat Khandelwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52088/ --- (Updated Sept. 28, 2016, 7:55 p.m.) Review request for lens. Bugs: LENS-743

Re: Review Request 52341: LENS-1325: Support encoded urls for query HTTP notifications

2016-09-28 Thread Puneet Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52341/ --- (Updated Sept. 28, 2016, 11:18 a.m.) Review request for lens. Changes ---

Review Request 52341: LENS-1325: Support encoded urls for query HTTP notifications

2016-09-28 Thread Puneet Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52341/ --- Review request for lens. Bugs: lens-1325 https://issues.apache.org/jira/bro

[jira] [Work started] (LENS-1325) Support encoded urls for query HTTP notifications

2016-09-28 Thread Puneet Gupta (JIRA)
[ https://issues.apache.org/jira/browse/LENS-1325?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Work on LENS-1325 started by Puneet Gupta. -- > Support encoded urls for query HTTP notifications >

[jira] [Commented] (LENS-743) Query failure retries for transient errors

2016-09-28 Thread Rajat Khandelwal (JIRA)
[ https://issues.apache.org/jira/browse/LENS-743?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15529044#comment-15529044 ] Rajat Khandelwal commented on LENS-743: --- It will retry only on d3. That is also depend

Re: Review Request 52259: Adding API for data completeness checker (LENS-1340)

2016-09-28 Thread Narayan Periwal
> On Sept. 28, 2016, 8:59 a.m., Amareshwari Sriramadasu wrote: > > lens-cube/src/main/java/org/apache/lens/cube/parse/DataCompletenessChecker.java, > > line 34 > > > > > > Please add java for all method params, descr

Re: Review Request 52259: Adding API for data completeness checker (LENS-1340)

2016-09-28 Thread Narayan Periwal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52259/ --- (Updated Sept. 28, 2016, 9:25 a.m.) Review request for lens. Bugs: LENS-1340

Re: Review Request 52259: Adding API for data completeness checker (LENS-1340)

2016-09-28 Thread Amareshwari Sriramadasu
> On Sept. 26, 2016, 1:19 p.m., Rajat Khandelwal wrote: > > lens-cube/src/main/java/org/apache/lens/cube/parse/DataCompletenessChecker.java, > > line 32 > > > > > > What interval are the keys in the map spaced over?

Re: [VOTE] Release of Apache Lens 2.6.1

2016-09-28 Thread Puneet Gupta
Guys, I will be traveling. Will be back after a week. In my absence Rajat Khandelwal will take care of 2.6.1 release Thanks, Puneet Gupta On Wed, Sep 28, 2016 at 2:24 PM, Amareshwari Sriramdasu < amareshw...@apache.org> wrote: > +1 to release the candidate. But found a couple of issues with rele

Re: Review Request 52259: Adding API for data completeness checker (LENS-1340)

2016-09-28 Thread Amareshwari Sriramadasu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52259/#review150695 --- lens-cube/src/main/java/org/apache/lens/cube/parse/DataCompletene

Fwd: [VOTE] Release of Apache Lens 2.6.1

2016-09-28 Thread Amareshwari Sriramdasu
+1 to release the candidate. But found a couple of issues with release which are noted down. Verified the following : + Jars in Nexus repo : Good. + Signatures : Good + Checksums : Good + No unexpected binaries found in the source release. + NOTICE is good. + Building from source : Successful - L

Re: Review Request 52337: LENS-1324: Support more hooks in DriverQueryHook

2016-09-28 Thread Amareshwari Sriramadasu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52337/#review150694 --- Ship it! Ship It! - Amareshwari Sriramadasu On Sept. 28, 201

[jira] [Created] (LENS-1341) UnmarshalException while reading response of saved query getParameters() call

2016-09-28 Thread Archana H (JIRA)
Archana H created LENS-1341: --- Summary: UnmarshalException while reading response of saved query getParameters() call Key: LENS-1341 URL: https://issues.apache.org/jira/browse/LENS-1341 Project: Apache Lens

Re: Review Request 52337: LENS-1324: Support more hooks in DriverQueryHook

2016-09-28 Thread Puneet Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52337/ --- (Updated Sept. 28, 2016, 7:04 a.m.) Review request for lens. Changes ---