> On Oct. 4, 2013, 6:52 p.m., Darren Shepherd wrote: > > I would prefer not to do this. From a technical perspective the code uses > > getRealPath and Files which are not compatible across all serlvet > > containers and this implementation forces a lot of file system stat() > > calls, but that isn't my real complaint. This change adds yet another step > > to the already long ACS build and I don't think it address the root issue. > > The root issue, if one was to really care about performance is that the js > > is not cachable or consolidated. ts=$now is added to every js file, > > meaning it must be downloaded every time. Additionally to load the login > > screen there are 66 requests for JS files. Even if you compress 2mb to > > 800k, on a slow connection it won't matter that much because you are going > > to get killed regardless by the round trip latency of making 66 requests.
Hi Darren, I share your concerns about the build process and I do not want to further increase that. What I had in my mind is an optional step (possibly in a profile) acivated only for making releases and by default not in a developement environment. Not that much to save 2 mb of bandwidth, but to save the time the user have to wait. The difficulties with the non cacheable js files is indeed a problem, and I believe that should be addressed separately. (I do not quite understand yet what is that for) Also, the dynamic informations can be compressed if it makes sense, e.g. over a given size. I added the wip prefix to this to make it clear that this patch is "preview" version and more discussion is needed. - Laszlo ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/#review26690 ----------------------------------------------------------- On Oct. 4, 2013, 8:19 p.m., Laszlo Hornyak wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12228/ > ----------------------------------------------------------- > > (Updated Oct. 4, 2013, 8:19 p.m.) > > > Review request for cloudstack, Brian Federle 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 > >
