-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12228/#review33400
-----------------------------------------------------------
client/WEB-INF/web.xml
<https://reviews.apache.org/r/12228/#comment62858>
can you use tabs instead of spaces (and while you are at it remove trailing spaces from
this file)?
client/pom.xml
<https://reviews.apache.org/r/12228/#comment62859>
please remove tab
server/src/com/cloud/servlet/StaticResourceServlet.java
<https://reviews.apache.org/r/12228/#comment62857>
please add a license header
server/test/com/cloud/servlet/StaticResourceServletTest.java
<https://reviews.apache.org/r/12228/#comment62856>
please add a license header
- daan Hoogland
On Feb. 2, 2014, 5:04 p.m., Laszlo Hornyak wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12228/
> -----------------------------------------------------------
>
> (Updated Feb. 2, 2014, 5:04 p.m.)
>
>
> Review request for cloudstack, Brian Federle, Darren Shepherd, and Prasanna Santhanam.
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> CloudStack at first use downloads some 3.5 MB of css and javascript to the client. With
a weak internet connection, this might take a long time. With gzip compression content can
be compressed to 850 KB.
>
> This version of the patch uses a custom plugin to compress static resources, so that
no dynamic compression is needed at runtime. When the static resource servlet notices that
there is gzipped version of the resource and the client accepts gzipped content, then it is
going to send the gziped version, while still respects http caching.
>
>
> Diffs
> -----
>
> client/WEB-INF/web.xml e5c05d3
> client/pom.xml 119c96e
> server/src/com/cloud/servlet/StaticResourceServlet.java PRE-CREATION
> server/test/com/cloud/servlet/StaticResourceServletTest.java PRE-CREATION
>
> Diff: https://reviews.apache.org/r/12228/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Laszlo Hornyak
>
>
|