incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rajesh Battala" <rajesh.batt...@citrix.com>
Subject Re: Review Request: CS-15778:-Fetching-thumbnail
Date Sat, 18 Aug 2012 04:58:24 GMT


> On Aug. 17, 2012, 11:08 p.m., Kelven Yang wrote:
> > I can't apply the patch as git throws errors, could you double check if your patch
is properly generated (for example, no unresolved conflicts, use space in place of tabs and
don't mix space and tabs)

Hi Kelven, 
I had uses spaces instead of tabs (eclipse setting). This patch is two diff's right. there
is a parent patch and later I had updated diff's with your review comments?


- Rajesh


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


On Aug. 17, 2012, 1:52 p.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/6608/
> -----------------------------------------------------------
> 
> (Updated Aug. 17, 2012, 1:52 p.m.)
> 
> 
> Review request for cloudstack and Kelven Yang.
> 
> 
> Description
> -------
> 
> When a viewer is launched, the images are getting stored in AjaxImageFifoCache class.
> When request for thumbnail, we need to pass "key" param to retreive the image and its
not getting passed so thumbnail is requested its failing with NPE. 
> Issue is fixed by passing key param with default value 0 and Image is retrieved with
the current key value.
> 
> 
> This addresses bug CS-15778.
> 
> 
> Diffs
> -----
> 
>   console-proxy/src/com/cloud/consoleproxy/ConsoleProxyAjaxImageHandler.java 1fdda2e

> 
> Diff: https://reviews.apache.org/r/6608/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message