-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43953/
-----------------------------------------------------------
Review request for Ambari, Alejandro Fernandez, Nahappan Somasundaram, and
Sumit Mohanty.
Bugs: AMBARI-15167
https://issues.apache.org/jira/browse/AMBARI-15167
Repository: ambari
Description
-------
hosts filtering by multiple state results in server error but filtering by
single state works fine.
GET api/v1/clusters/clustername/hosts -d
'{"RequestInfo":{"query":"stack_versions/HostStackVersions/state.in(INSTALLING,INSTALL_FAILED,OUT_OF_SYNC)"}}'
{
"status" : 500,
"message" : "org.apache.ambari.server.controller.spi.SystemException: Stack
version should have numerical id"
}
GET api/v1/clusters/clustername/hosts -d '
{"RequestInfo":{"query":"stack_versions/HostStackVersions/state.in(INSTALLING)"}}'
Request successful.
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/HostStackVersionResourceProvider.java
c7e3ca9
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/HostStackVersionResourceProviderTest.java
0824ae1
Diff: https://reviews.apache.org/r/43953/diff/
Testing
-------
Unit test and manually testing API.
trunk-test-patch build is successful.
+1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12789885/rb43953.patch
against trunk revision .
+1 @author. The patch does not contain any @author tags.
+1 tests included. The patch appears to include 1 new or modified test files.
+1 javac. The applied patch does not increase the total number of javac
compiler warnings.
+1 release audit. The applied patch does not increase the total number of
release audit warnings.
+1 core tests. The patch passed unit tests in ambari-server.
Test results:
https://builds.apache.org/job/Ambari-trunk-test-patch/5562//testReport/
Console output:
https://builds.apache.org/job/Ambari-trunk-test-patch/5562//console
This message is automatically generated.
Thanks,
Ajit Kumar
