-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26823/
-----------------------------------------------------------
Review request for Ambari, Nate Cole and Tom Beerbower.
Bugs: AMBARI-7817
https://issues.apache.org/jira/browse/AMBARI-7817
Repository: ambari
Description
-------
Exposed pagination and sorting via the SPI Request so that concrete
ResourceProviders can do their own paging and sorting. The providers are
responsibile for setting the returned ResourceProviderPageResponse so that the
ClusterController knows not to perform in-memory processing of the result set.
Implemented the new framework for alert history and notices.
Currently, the Alert DAOs return 0 for the total count of items since I don't
yet know if the UI will need this information. If we determine that we want to
implement it, there are stub methods in the DAOs for getting the counts. I
didn't want to incur the JPA penalty if it wasn't needed.
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/api/query/JpaPredicateVisitor.java
660bae7
ambari-server/src/main/java/org/apache/ambari/server/api/query/JpaSortBuilder.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/api/query/QueryImpl.java
6dfdb49
ambari-server/src/main/java/org/apache/ambari/server/controller/AlertHistoryRequest.java
9dc6cc4
ambari-server/src/main/java/org/apache/ambari/server/controller/AlertNoticeRequest.java
40c7c67
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertHistoryResourceProvider.java
300f02f
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AlertNoticeResourceProvider.java
c4a96ff
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/ClusterControllerImpl.java
5d1024a
ambari-server/src/main/java/org/apache/ambari/server/controller/internal/RequestImpl.java
93eaf0a
ambari-server/src/main/java/org/apache/ambari/server/controller/spi/Request.java
13a2856
ambari-server/src/main/java/org/apache/ambari/server/controller/spi/ResourceProviderPageResponse.java
PRE-CREATION
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertDispatchDAO.java
a49a04a
ambari-server/src/main/java/org/apache/ambari/server/orm/dao/AlertsDAO.java
8414765
ambari-server/src/test/java/org/apache/ambari/server/controller/internal/ClusterControllerImplTest.java
261c95d
ambari-server/src/test/java/org/apache/ambari/server/orm/AlertDaoHelper.java
e5a5638
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertDispatchDAOTest.java
430bede
ambari-server/src/test/java/org/apache/ambari/server/orm/dao/AlertsDAOTest.java
5886ab4
Diff: https://reviews.apache.org/r/26823/diff/
Testing
-------
New tests written to cover alerts and ResourceProviderPageResponse.
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 24:00 min
[INFO] Finished at: 2014-10-16T17:03:28-04:00
[INFO] Final Memory: 29M/239M
[INFO] ------------------------------------------------------------------------
Thanks,
Jonathan Hurley
