-----------------------------------------------------------
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, 2016, 9:50 a.m., Puneet Gupta wrote:
> 
> -----------------------------------------------------------
> 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.
> 
> 
> Bugs: lens-1325
>     https://issues.apache.org/jira/browse/lens-1325
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> 1. Added test cases to test multiple query parsms 
> 2. Test with complex param value that requires encoding at the time of 
> submission
> 
> 
> Diffs
> -----
> 
>   
> lens-server/src/main/java/org/apache/lens/server/query/QueryEventHttpNotifier.java
>  1760bec 
>   lens-server/src/main/resources/lenssession-default.xml b6bfa2e 
>   
> lens-server/src/test/java/org/apache/lens/server/query/TestQueryNotifictaionResource.java
>  6a57b20 
>   
> lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java 
> 415f56f 
>   src/site/apt/admin/session-config.apt c4b3c04 
> 
> Diff: https://reviews.apache.org/r/52341/diff/
> 
> 
> Testing
> -------
> 
> mvn  test 
> -Dtest="org.apache.lens.server.query.TestQueryService#testFinishedNotifictaion"
>  -pl lens-server
> 
> --------------------------------------------------------
>  T E S T S
> -------------------------------------------------------
> Running org.apache.lens.server.query.TestQueryService
> Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 55.449 sec - 
> in org.apache.lens.server.query.TestQueryService
> 
> Results :
> 
> Tests run: 2, Failures: 0, Errors: 0, Skipped: 0
> 
> 
> **Logs form TestQueryNotifictaionResource show params access_token and data 
> are recived correctly** 
> 
> lens-server/target/surefire-reports/org.apache.lens.server.query.TestQueryService-output.txt:@@@@
>  Received Finished Event for queryid: e16f40de-57cc-4d53-86f6-54dac397b04a 
> queryname:testHttpNotifictaionQuery user:foo status:Progress: 1.0
> lens-server/target/surefire-reports/org.apache.lens.server.query.TestQueryService-output.txt-Status:
>  SUCCESSFUL
> lens-server/target/surefire-reports/org.apache.lens.server.query.TestQueryService-output.txt-Status
>  Message: Query is successful!
> lens-server/target/surefire-reports/org.apache.lens.server.query.TestQueryService-output.txt-Is
>  Result Set Available: true
> lens-server/target/surefire-reports/org.apache.lens.server.query.TestQueryService-output.txt-
>  eventtype:FINISHED access_token:ABC123 data:null
> lens-server/target/surefire-reports/org.apache.lens.server.query.TestQueryService-output.txt-Sep
>  28, 2016 4:35:05 PM org.glassfish.jersey.filter.LoggingFilter log
> --
> lens-server/target/surefire-reports/org.apache.lens.server.query.TestQueryService-output.txt:@@@@
>  Received Finished Event for queryid: e16f40de-57cc-4d53-86f6-54dac397b04a 
> queryname:testHttpNotifictaionQuery user:foo status:Progress: 1.0
> lens-server/target/surefire-reports/org.apache.lens.server.query.TestQueryService-output.txt-Status:
>  SUCCESSFUL
> lens-server/target/surefire-reports/org.apache.lens.server.query.TestQueryService-output.txt-Status
>  Message: Query is successful!
> lens-server/target/surefire-reports/org.apache.lens.server.query.TestQueryService-output.txt-Is
>  Result Set Available: true
> lens-server/target/surefire-reports/org.apache.lens.server.query.TestQueryService-output.txt-
>  eventtype:FINISHED access_token:ABC123 data:x<>yz,"abc
> lens-server/target/surefire-reports/org.apache.lens.server.query.TestQueryService-output.txt-Sep
>  28, 2016 4:35:05 PM org.glassfish.jersey.filter.LoggingFilter log
> 
> 
> **Logs from lens server show succes for both the urls**
> 
> puneet.gupta$ grep "QueryEventHttpNotifier - FINISHED HTTP Notification sent 
> successfully" lens-server/target/test.log
> 28 Sep 2016 11:05:05 e16f40de-57cc-4d53-86f6-54dac397b04a 
> [QueryEndHttpNotifier_AsyncThread-2] INFO  
> org.apache.lens.server.query.QueryEventHttpNotifier - FINISHED HTTP 
> Notification sent successfully for query e16f40de-57cc-4d53-86f6-54dac397b04a 
> to 
> http://localhost:49847/lens-server/queryapi/notifictaion/finished?access_token=ABC123.
>  Response code 204
> 28 Sep 2016 11:05:05 e16f40de-57cc-4d53-86f6-54dac397b04a 
> [QueryEndHttpNotifier_AsyncThread-2] INFO  
> org.apache.lens.server.query.QueryEventHttpNotifier - FINISHED HTTP 
> Notification sent successfully for query e16f40de-57cc-4d53-86f6-54dac397b04a 
> to 
> http://localhost:49847/lens-server/queryapi/notifictaion/finished?access_token=ABC123&data=x%3C%3Eyz%2C%22abc.
>  Response code 204
> 
> 
> Thanks,
> 
> Puneet Gupta
> 
>

Reply via email to