----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45811/#review127353 -----------------------------------------------------------
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CompatibleRepositoryVersionResourceProvider.java (line 70) <https://reviews.apache.org/r/45811/#comment190632> Does this also need "CompatibleRepositoryVersions/"? If not, ship it. - Alejandro Fernandez On April 6, 2016, 2:03 p.m., Nate Cole wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45811/ > ----------------------------------------------------------- > > (Updated April 6, 2016, 2:03 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Jayush > Luniya. > > > Bugs: AMBARI-15717 > https://issues.apache.org/jira/browse/AMBARI-15717 > > > Repository: ambari > > > Description > ------- > > A previous bug was allowing compatible stacks to come across correctly. > After that fix, that made the compatible_repository_version API filter > responses that it shouldn't have. This led to an interesting problem whereby > we needed a mechanism to alter the incoming predicate to allow those > resources to be included. > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java > 36ed189 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterControllerImpl.java > 9f864f8 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CompatibleRepositoryVersion.java > 1c5fe89 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/CompatibleRepositoryVersionResourceProvider.java > 26b9645 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ReadOnlyResourceProvider.java > a72527d > > ambari-server/src/main/java/org/apache/ambari/server/controller/spi/ClusterController.java > aa1f09f > > ambari-server/src/test/java/org/apache/ambari/server/api/query/QueryImplTest.java > f869ba3 > > ambari-server/src/test/java/org/apache/ambari/server/controller/internal/CompatibleRepositoryVersionResourceProviderTest.java > faef1c9 > > Diff: https://reviews.apache.org/r/45811/diff/ > > > Testing > ------- > > Manual. Automated pending. > > > Thanks, > > Nate Cole > >
