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

2016-10-19 Thread Amareshwari Sriramadasu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52341/#review153263 --- Ship it! Ship It! - Amareshwari Sriramadasu On Oct. 19,

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

2016-10-19 Thread Puneet Gupta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52341/ --- (Updated Oct. 19, 2016, 9:50 a.m.) Review request for lens. Changes ---

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

2016-10-03 Thread Amareshwari Sriramadasu
> On Sept. 29, 2016, 9:59 a.m., Amareshwari Sriramadasu wrote: > > lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java, > > line 1910 > > > > > > Should we encode the full url, instead of

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

2016-10-02 Thread Puneet Gupta
> On Sept. 29, 2016, 9:59 a.m., Amareshwari Sriramadasu wrote: > > lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java, > > line 1910 > > > > > > Should we encode the full url, instead of

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

2016-09-29 Thread Amareshwari Sriramadasu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52341/#review150831 ---

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