-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/18015/
-----------------------------------------------------------
(Updated Feb. 13, 2014, 4:46 p.m.)
Review request for Ambari, Dmitro Lisnichenko and Nate Cole.
Changes
-------
Updated patch, because for current trunk it's not actual.
Bugs: AMBARI-4630
https://issues.apache.org/jira/browse/AMBARI-4630
Repository: ambari
Description
-------
UI makes the following call where there are multiple params which should end up
in the target URL
/proxy?url=http://server:8188/ws/v1/apptimeline/HIVE_QUERY_ID?fields=events,primaryfilters&limit=10&primaryFilter=user:hiveuser1
None of the filter params (fields, limit, primaryFilter) are ending up on the
target server. The passthrough API should pass through all extra params (other
than url).
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/proxy/ProxyService.java
0de826a
ambari-server/src/test/java/org/apache/ambari/server/proxy/ProxyServiceTest.java
326eb13
Diff: https://reviews.apache.org/r/18015/diff/
Testing
-------
-------------------------------------------------------
T E S T S
-------------------------------------------------------
Listening for transport dt_socket at address: 5006
Running org.apache.ambari.server.proxy.ProxyServiceTest
Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 8.34 sec
Results :
Tests run: 7, Failures: 0, Errors: 0, Skipped: 0
Thanks,
Vitalyi Brodetskyi
