Re: Review Request 12228: wip: static resource compression

2014-02-02 Thread daan Hoogland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/#review33400 --- client/WEB-INF/web.xml

Re: Review Request 12228: wip: static resource compression

2014-02-02 Thread Laszlo Hornyak
--- 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,

Re: Review Request 12228: wip: static resource compression

2013-10-30 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12228/ --- (Updated Oct. 31, 2013, 1:36 a.m.) Review request for cloudstack, Brian Federle

Re: Review Request 12228: wip: static resource compression

2013-10-04 Thread Laszlo Hornyak
> 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 tha

Re: Review Request 12228: wip: static resource compression

2013-10-04 Thread Brian Federle
> 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 tha

Re: Review Request 12228: wip: static resource compression

2013-10-04 Thread Laszlo Hornyak
> 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 tha

Re: Review Request 12228: wip: static resource compression

2013-10-04 Thread Laszlo Hornyak
--- 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