> On March 24, 2014, 10:03 p.m., Alena Prokharchyk wrote:
> > Also if you are hardcoding the state to search by, to Running in your
> > method, there is no need to define it in the method signature:
> >
> > public List<DomainRouterVO> listByRunningStateAndManagementServer(State
> > state, long mgmtSrvrId) { - remove it from the signature
> > 308
> > SearchCriteria<DomainRouterVO> sc =
> > RunningSearchByManagementServerId.create();
> > 309
> > sc.setParameters("state", State.Running); - here is the hard coding
Thank you Alena, I'll make this a generic method to
listbyStateAndManagementServer so that it can be used for any state.
- Harikrishna
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19578/#review38375
-----------------------------------------------------------
On March 24, 2014, 2:17 p.m., Harikrishna Patnala wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19578/
> -----------------------------------------------------------
>
> (Updated March 24, 2014, 2:17 p.m.)
>
>
> Review request for cloudstack, Jayapal Reddy, Murali Reddy, and Sheng Yang.
>
>
> Bugs: CLOUDSTACK-6253
> https://issues.apache.org/jira/browse/CLOUDSTACK-6253
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> CLOUDSTACK-6253: Optimizing VR alerts getting algorithm
> In addition to this a new configuration parameter is added
> router.alerts.check.interval defaulted to 30minutes to check for alerts in
> Virtual Router
>
>
> Diffs
> -----
>
> core/src/com/cloud/agent/api/GetRouterAlertsAnswer.java 06a7a7a
>
> core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java
> e13fbf6
> engine/schema/src/com/cloud/vm/dao/DomainRouterDao.java e3f75fa
> engine/schema/src/com/cloud/vm/dao/DomainRouterDaoImpl.java 6b62f56
> server/src/com/cloud/network/router/VirtualNetworkApplianceManager.java
> e3597ac
> server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java
> c692491
> systemvm/patches/debian/config/opt/cloud/bin/getRouterAlerts.sh e5e8abe
> test/integration/component/test_VirtualRouter_alerts.py PRE-CREATION
> test/integration/smoke/test_VirtualRouter_alerts.py 2333d84
>
> Diff: https://reviews.apache.org/r/19578/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Harikrishna Patnala
>
>
