Re: Review Request 12228: static resource compression

2014-04-22 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/#review41058 --- ping? - Laszlo Hornyak On April 18, 2014, 11:08 p.m., Laszlo Horn

Re: Review Request 12228: static resource compression

2014-04-18 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/ --- (Updated April 18, 2014, 11:08 p.m.) Review request for cloudstack, Brian Feder

Re: Review Request 12228: static resource compression

2014-04-02 Thread Brian Federle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/#review39322 --- Lazlo, I just tested your changes and everything looks good to me.

Re: Review Request 12228: static resource compression

2014-03-30 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/ --- (Updated March 30, 2014, 8:08 p.m.) Review request for cloudstack, Brian Federl

Re: Review Request 12228: static resource compression

2014-03-25 Thread Laszlo Hornyak
> On March 21, 2014, 8:07 p.m., Laszlo Hornyak wrote: > > https://docs.google.com/spreadsheet/ccc?key=0ApMkIX1Ygx8zdHhqV2RETy05SDU1WER3Z2JEN3ktZHc&usp=sharing > > > > Comparison of 3 configurations, the dynamic compression solution is the > > last one. This test was conducted in a VM with 2 vCP

Re: Review Request 12228: static resource compression

2014-03-21 Thread Laszlo Hornyak
Hi John, I used simple apache benchmark with a random selected css. ab -n 2000 -H "Accept-Encoding: gzip, deflate" http://172.17.10.10/client/css/cloudstack3.css This looks big enough, it is about 200 K uncompressed. I used the configuration you published in your blog and I disabled the static r

Re: Review Request 12228: static resource compression

2014-03-21 Thread John Kinsella
Canya tell us a little more about the test you’re doing? What URL are you fetching, how many times etc. Just curious to tinker myself this weekend if I have some time. :) On Mar 21, 2014, at 1:07 PM, Laszlo Hornyak mailto:laszlo.horn...@gmail.com>> wrote: -

Re: Review Request 12228: static resource compression

2014-03-21 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/#review38181 --- https://docs.google.com/spreadsheet/ccc?key=0ApMkIX1Ygx8zdHhqV2RETy0

Re: Review Request 12228: static resource compression

2014-03-20 Thread Laszlo Hornyak
url: https://wiki.eclipse.org/Jetty/Feature/GZIP_Compression#Static_Content On Thu, Mar 20, 2014 at 11:19 PM, John Kinsella wrote: > Laszlo, can you reference any other open source projects that have > similar solutions to this issue? Anything I've read states dynamic > compression in tomcat/h

Re: Review Request 12228: static resource compression

2014-03-20 Thread Laszlo Hornyak
Hi John, Jetty gives this functionality by default without a filter/servlet, simply it is not available in tomcat. I used this functionality a lot in jetty and that plus the massive size of the UI gave me the idea. My approach to the problem was motivated by that I don't like to see anything waste

Re: Review Request 12228: static resource compression

2014-03-20 Thread John Kinsella
Laszlo, can you reference any other open source projects that have similar solutions to this issue? Anything I’ve read states dynamic compression in tomcat/httpd/nginx does not add significant CPU overhead. On Mar 20, 2014, at 12:53 PM, Laszlo Hornyak mailto:laszlo.horn...@gmail.com>> wrote:

Re: Review Request 12228: static resource compression

2014-03-20 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/ --- (Updated March 20, 2014, 7:53 p.m.) Review request for cloudstack, Brian Federl

Re: Review Request 12228: static resource compression

2014-03-20 Thread daan Hoogland
> On Feb. 2, 2014, 6:03 p.m., daan Hoogland wrote: > > client/WEB-INF/web.xml, line 63 > > > > > > can you use tabs instead of spaces (and while you are at it remove > > trailing spaces from this file)? > > Laszlo Hor

Re: Review Request 12228: static resource compression

2014-03-19 Thread Laszlo Hornyak
> On Feb. 2, 2014, 6:03 p.m., daan Hoogland wrote: > > client/WEB-INF/web.xml, line 63 > > > > > > can you use tabs instead of spaces (and while you are at it remove > > trailing spaces from this file)? trailing white

Re: Review Request 12228: static resource compression

2014-03-19 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/ --- (Updated March 19, 2014, 8:13 p.m.) Review request for cloudstack, Brian Federl

Re: Review Request 12228: static resource compression

2013-10-04 Thread Darren Shepherd
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/#review26690 --- I would prefer not to do this. From a technical perspective the cod

Re: Review Request 12228: static resource compression

2013-09-27 Thread Laszlo Hornyak
> On Sept. 27, 2013, 6:40 p.m., Brian Federle wrote: > > For the next release I'm planning to refactor the stylesheets to use the > > SASS compiler. Will this effect integration of a SASS compiler in any way? > > > > FYI The one I'm considering is https://github.com/Jasig/sass-maven-plugin. >

Re: Review Request 12228: static resource compression

2013-09-27 Thread Brian Federle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/#review26435 --- For the next release I'm planning to refactor the stylesheets to use

Re: Review Request 12228: static resource compression

2013-09-23 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/ --- (Updated Sept. 24, 2013, 6:57 a.m.) Review request for cloudstack, Brian Federl

Re: Review Request 12228: static resource compression

2013-09-23 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/ --- (Updated Sept. 24, 2013, 6:51 a.m.) Review request for cloudstack, Brian Federl

Re: Review Request 12228: static resource compression

2013-09-20 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/ --- (Updated Sept. 20, 2013, 10:01 a.m.) Review request for cloudstack. Changes -

Re: Review Request 12228: static resource compression

2013-07-02 Thread Prasanna Santhanam
> On July 2, 2013, 10:15 a.m., Prasanna Santhanam wrote: > > Over VNC/RDP I've seen the login page take somewhere around 2minutes to > > load. This would definitely be useful. > > Prasanna Santhanam wrote: > Here's the screen grab over RDP demonstrating the issue > http://www.youtube.co

Re: Review Request 12228: static resource compression

2013-07-02 Thread Laszlo Hornyak
> On July 2, 2013, 10:15 a.m., Prasanna Santhanam wrote: > > Over VNC/RDP I've seen the login page take somewhere around 2minutes to > > load. This would definitely be useful. > > Prasanna Santhanam wrote: > Here's the screen grab over RDP demonstrating the issue > http://www.youtube.co

Re: Review Request 12228: static resource compression

2013-07-02 Thread Prasanna Santhanam
> On July 2, 2013, 10:15 a.m., Prasanna Santhanam wrote: > > Over VNC/RDP I've seen the login page take somewhere around 2minutes to > > load. This would definitely be useful. Here's the screen grab over RDP demonstrating the issue http://www.youtube.com/watch?v=lVCMSlBPX0E Because we use tomc

Re: Review Request 12228: static resource compression

2013-07-02 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/#review22657 --- Over VNC/RDP I've seen the login page take somewhere around 2minutes

Review Request 12228: static resource compression

2013-07-02 Thread Laszlo Hornyak
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/ --- Review request for cloudstack. Repository: cloudstack-git Description ---