Should static resources be cached in production mode ?

2013-09-09 Thread Martin Grigorov
Hi, At the moment when a PackageResource is requested in production mode Wicket always re-reads the file from the disk, then executes any ITextResourceProcessor on it and finally delivers it. This is true for normal resources and for bundles. I just filled WICKET-5344 - bundles does all this twice

Re: Should static resources be cached in production mode ?

2013-09-09 Thread Andrea Del Bene
+1 yes we should... > Hi, > > At the moment when a PackageResource is requested in production mode Wicket > always re-reads the file from the disk, then executes any > ITextResourceProcessor on it and finally delivers it. > This is true for normal resources and for bundles. I just filled > WICKET-5

Re: Add .map files to default allowed files.

2013-09-09 Thread Andrea Del Bene
ok, thank you for your feedbacks. things will remain like they are now. > On Sat, Sep 7, 2013 at 7:12 AM, Martin Grigorov wrote: > >> I suggest to remove the .map files for now and re-add them again when they >> become more popular some day. >> > +1 - and then I'm not even sure that it should reall

Re: Should static resources be cached in production mode ?

2013-09-09 Thread Thomas Matthijs
On Mon, Sep 9, 2013 at 11:18 AM, Martin Grigorov wrote: > Hi, > > At the moment when a PackageResource is requested in production mode Wicket > always re-reads the file from the disk, then executes any > ITextResourceProcessor on it and finally delivers it. > This is true for normal resources and

Re: Add .map files to default allowed files.

2013-09-09 Thread Martin Grigorov
On Mon, Sep 9, 2013 at 12:24 PM, Andrea Del Bene wrote: > ok, thank you for your feedbacks. things will remain like they are now. > No. We have to remove the .map files for the next release. > > On Sat, Sep 7, 2013 at 7:12 AM, Martin Grigorov >wrote: > > > >> I suggest to remove the .map files

Re: Should static resources be cached in production mode ?

2013-09-09 Thread Michael Haitz
+1 because compressing and optimizing them with compilers like google's closure compiler is very expensive. >> Hi, >> >> At the moment when a PackageResource is requested in production mode Wicket >> always re-reads the file from the disk, then executes any >> ITextResourceProcessor on it and fi

Re: Add .map files to default allowed files.

2013-09-09 Thread Andrea Del Bene
Ok, sorry. I' missed the line where you say that you have added them... > On Mon, Sep 9, 2013 at 12:24 PM, Andrea Del Bene wrote: > >> ok, thank you for your feedbacks. things will remain like they are now. >> > No. We have to remove the .map files for the next release. > > >>> On Sat, Sep 7, 2013