Re: Performance issue -> next release
ok, no problem. -- View this message in context: http://struts.1045723.n5.nabble.com/Performance-issue-next-release-tp5711393p5711432.html Sent from the Struts - Dev mailing list archive at Nabble.com. - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
Re: Performance issue -> next release
2012/12/17 mgreau : > Hi, > > It's done : https://issues.apache.org/jira/browse/WW-3947 A bit to late ... but if we handle migration to new website publishing mechanism I'm going prepare a new release asap :-) Regards -- Łukasz + 48 606 323 122 http://www.lenart.org.pl/ - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
Re: Performance issue -> next release
Hi, It's done : https://issues.apache.org/jira/browse/WW-3947 Regards. Maxime -- View this message in context: http://struts.1045723.n5.nabble.com/Performance-issue-next-release-tp5711393p5711429.html Sent from the Struts - Dev mailing list archive at Nabble.com. - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
Re: Performance issue -> next release
2012/12/13 mgreau : > Hi, > > I have to prepare a patch for the portlet archetype which doesn't work > (${packageName} is not replace in Java classes and struts.xml). > > It's small changes which doesn't affect core code or plugins. Can it will be > included in 2.3.8 release ? Sure :-) Create an issue and attach the patch. Regards -- Łukasz + 48 606 323 122 http://www.lenart.org.pl/ - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
Re: Performance issue -> next release
Hi, I have to prepare a patch for the portlet archetype which doesn't work (${packageName} is not replace in Java classes and struts.xml). It's small changes which doesn't affect core code or plugins. Can it will be included in 2.3.8 release ? Regards. Maxime -- View this message in context: http://struts.1045723.n5.nabble.com/Performance-issue-next-release-tp5711393p5711402.html Sent from the Struts - Dev mailing list archive at Nabble.com. - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
Re: Performance issue -> next release
2012/12/13 Johannes Geppert : > +1 for an Xmas Release. > > @Łukasz > Can we implement your suggested doStartTag Fix in 2.3.8? No as this can be a breaking change and can introduce new issues - just small improvements. Regards -- Łukasz + 48 606 323 122 http://www.lenart.org.pl/ - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
Re: Performance issue -> next release
+1 for an Xmas Release. @Łukasz Can we implement your suggested doStartTag Fix in 2.3.8? Johannes - web: http://www.jgeppert.com twitter: http://twitter.com/jogep -- View this message in context: http://struts.1045723.n5.nabble.com/Performance-issue-next-release-tp5711393p5711400.html Sent from the Struts - Dev mailing list archive at Nabble.com. - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
Re: Performance issue -> next release
The plan is as follow: 1. I'm going to solve few more issues till weekend (NPE, some small changes) 2. Release a test build during weekend 3. Give 3-4 days for testing 4. Call for a vote on wednesday 4. Finish release before Xmas Regards -- Łukasz + 48 606 323 122 http://www.lenart.org.pl/ - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
Re: Performance issue -> next release
On Thu, Dec 13, 2012 at 7:43 AM, Lukasz Lenart wrote: > 2012/12/12 Philip Luppens : > > I'd say to go for it, but please do explain what the issue was for > archival > > reasons, since the original ticket[1] did not receive any update yet. > > Ok, you're right. The problem is related to my FileManager > refactorings and introduced FileManagerFactory with default > implementation > > ConfigurationManager#conditionalReload() from version 2.3.7 looks like > this: > > public synchronized void conditionalReload(Container container) { > FileManager fileManager = > container.getInstance(FileManagerFactory.class).getFileManager(); > if (fileManager.isReloadingConfigs() || providersChanged) { > > } > > so each time FileManager instance was looked up just to check if > reloadConfigs flag is set to true, but right now it looks like this > and the flag is checked directly > > public synchronized void conditionalReload(Container container) { > boolean reloadConfigs = > Boolean.parseBoolean(container.getInstance(String.class, > XWorkConstants.RELOAD_XML_CONFIGURATION)); > if (reloadConfigs || providersChanged) { > > } > Great, thank you very much for the writeup. I'm also in favor of a quick release of 2.3.8 (I've just seen that AppFuse 2.2.1 was released with the affected Struts 2.3.7). Thanks again, Phil > > > Regards > -- > Łukasz > + 48 606 323 122 http://www.lenart.org.pl/ > > - > To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org > For additional commands, e-mail: dev-h...@struts.apache.org > > -- "We cannot change the cards we are dealt, just how we play the hand." - Randy Pausch
Re: Performance issue -> next release
And there is still place for improvement, take a look at the ComponentTagSupport#doStartTag, each time instance of Container is taken from Dispatcher, which means each time configuration is checked if needs to be reloaded - obviously it doesn't make sense to check that or reload configuration during each tag execution. Configuration should checked/reloaded on the beginning of request and stay constant till the end of request. So I think it is better to put Configuration into HttpServletRequest (or wrapped one) and used it inside tags instead of using Dispatcher.getInstance().getContainer(). Regards -- Łukasz + 48 606 323 122 http://www.lenart.org.pl/ - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
Re: Performance issue -> next release
2012/12/12 Philip Luppens : > I'd say to go for it, but please do explain what the issue was for archival > reasons, since the original ticket[1] did not receive any update yet. Ok, you're right. The problem is related to my FileManager refactorings and introduced FileManagerFactory with default implementation ConfigurationManager#conditionalReload() from version 2.3.7 looks like this: public synchronized void conditionalReload(Container container) { FileManager fileManager = container.getInstance(FileManagerFactory.class).getFileManager(); if (fileManager.isReloadingConfigs() || providersChanged) { } so each time FileManager instance was looked up just to check if reloadConfigs flag is set to true, but right now it looks like this and the flag is checked directly public synchronized void conditionalReload(Container container) { boolean reloadConfigs = Boolean.parseBoolean(container.getInstance(String.class, XWorkConstants.RELOAD_XML_CONFIGURATION)); if (reloadConfigs || providersChanged) { } Regards -- Łukasz + 48 606 323 122 http://www.lenart.org.pl/ - To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org For additional commands, e-mail: dev-h...@struts.apache.org
Re: Performance issue -> next release
On Wed, Dec 12, 2012 at 6:30 PM, Łukasz Lenart wrote: > Johannes discovered a huge performance issue which is already solved in > trunk (2.3.8) so I'm going to prepare a new release ASAP. Is threre any > other important outstanding issue that should be solved before the release? > > (on mobile) > I'd say to go for it, but please do explain what the issue was for archival reasons, since the original ticket[1] did not receive any update yet. - Phil [1] https://issues.apache.org/jira/browse/WW-3941 -- "We cannot change the cards we are dealt, just how we play the hand." - Randy Pausch
Performance issue -> next release
Johannes discovered a huge performance issue which is already solved in trunk (2.3.8) so I'm going to prepare a new release ASAP. Is threre any other important outstanding issue that should be solved before the release? (on mobile)
Re: Performance issue
> From: "Gupta, Vikas" <[EMAIL PROTECTED]> > as i am using datepicker in one of the page (the number of datepicker > based on the records fetched from databse), so what i can do to improve > performance. Please ask questions on the struts-user list; the struts-dev list is for the development of Struts itself. Thanks, Dave - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Performance issue
Hello everybody, i am facing related to performance issue while using struts2 as i am using datepicker in one of the page (the number of datepicker based on the records fetched from databse), so what i can do to improve performance. pls do reply Thanks & Regards, Vikas Gupta - This message and any attachments are intended only for the use of the addressee and may contain information that is privileged and confidential. If the reader of the message is not the intended recipient or an authorized representative of the intended recipient, you are hereby notified that any dissemination of this communication is strictly prohibited. If you have received this communication in error, notify the sender immediately by return email and delete the message and any attachments from your system.