You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@cloudstack.apache.org by Alena Prokharchyk <al...@citrix.com> on 2012/08/30 19:31:04 UTC

Re: Review Request: override toString() in ExceptionResponse

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

Ship it!


Ship It!

- Alena Prokharchyk


On July 25, 2012, 7:21 p.m., Venkata Siva Vijayendra Bhamidipati wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/5871/
> -----------------------------------------------------------
> 
> (Updated July 25, 2012, 7:21 p.m.)
> 
> 
> Review request for cloudstack, Sheng Yang and Alena Prokharchyk.
> 
> 
> Description
> -------
> 
> Overriding toString() in ExceptionResponse for better logging. Also detected an older Citrix copyright header in AsyncJobManagerImpl.java, so removed that. The whitespace corrections were necessary to not run into errors when using git format-patch.
> 
> 
> This addresses bug CS-15508.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/api/response/ExceptionResponse.java 8d1e3de 
> 
> Diff: https://reviews.apache.org/r/5871/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Venkata Siva Vijayendra Bhamidipati
> 
>