-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/7017/#review11342
-----------------------------------------------------------


What is this change for?

- Alex Huang


On Sept. 11, 2012, 1:45 p.m., Rohit Yadav wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/7017/
> -----------------------------------------------------------
> 
> (Updated Sept. 11, 2012, 1:45 p.m.)
> 
> 
> Review request for cloudstack, Murali Reddy, Alena Prokharchyk, and Alex 
> Huang.
> 
> 
> Description
> -------
> 
> They will be used by https://reviews.apache.org/r/6781/
> 
>     Dao: Add helpers for listing based on podId
>     
>     - added listByPodId
>     - fixed listByTypeAndState to accept type and then state as arguments
>     - added listByPodIdTypeAndState in VMInstanceDao
>     - added countByPodIdVMTypeAndState in VMInstanceDao
> 
> 
> Download original patch: http://patchbin.baagi.org/p?id=ue9o0z
> 
> 
> Diffs
> -----
> 
>   server/src/com/cloud/vm/dao/DomainRouterDao.java 01e3258 
>   server/src/com/cloud/vm/dao/DomainRouterDaoImpl.java 175d3f2 
>   server/src/com/cloud/vm/dao/VMInstanceDao.java 2cf3d75 
>   server/src/com/cloud/vm/dao/VMInstanceDaoImpl.java 571b5d1 
> 
> Diff: https://reviews.apache.org/r/7017/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Rohit Yadav
> 
>

Reply via email to