Hi, Leonardo, can you please review : https://issues.apache.org/jira/browse/MYFACES-2922 https://issues.apache.org/jira/browse/MYFACES-2862
and apply them if they are suitable? Thanks, Kočičák Martin Koci píše v Út 14. 09. 2010 v 23:05 +0200: > Hi, > > > please review https://issues.apache.org/jira/browse/MYFACES-2922 and > apply if possible. > > Thanks, > > Kočičák > > > Leonardo Uribe píše v Pá 06. 08. 2010 v 13:07 -0500: > > Hi > > > > 2010/8/6 Martin Koci <[email protected]> > > Hi, > > > > > > is it possible to cache inspected classes in > > RequestViewContext? I did > > something like that: > > > > if (isProduction && ! > > requestViewContext.isAlreadyInspected(inspectedClass)) { > > > > _handleListenerForAnnotations(context, inspected, > > inspectedClass, component, isProduction); > > > > _handleResourceDependencyAnnotations(context, > > inspectedClass, component, isProduction); > > > > requestViewContext.setAsProcessed(inspectedClass); > > } > > > > in _handleAnnotations and it reduces restore view time to > > 30-40 ms. > > > > > > It is necessary to apply @ListenerFor annotations on every component > > that has registered > > it in the view. The reason why we can cache @ResourceDependency is > > this annotation > > cause a component resource to be added, and that one will be always > > the "same". > > > > regards, > > > > Leonardo > > > > > > regards, > > > > Martin Kočí > > > > Leonardo Uribe píše v Čt 05. 08. 2010 v 15:56 -0500: > > > > > Hi > > > > > > Ok, good to know that. I closed MYFACES-2854. Maybe on > > MYFACES-2862 we > > > can use FacesContext.isProjectStage(ProjectStage). > > > > > > regards, > > > > > > Leonardo > > > > > > 2010/8/5 Martin Koci <[email protected]> > > > Hi, > > > > > > success! > > > > > > myfaces + MYFACES-2854-2.patch + MYFACES-2862 = ~ 70 > > ms in > > > restore view > > > phase. It was *750 ms* before. > > > > > > Thanks, > > > > > > Martin Kočí > > > > > > Leonardo Uribe píše v St 04. 08. 2010 v 22:09 -0500: > > > > > > > Hi > > > > > > > > I implemented a proposal for this one on > > > MYFACES-2854-2.patch using > > > > the suggestion > > > > proposed (do not apply ResourceDependency if it > > was already > > > > processed). I hope that > > > > patch solve the problem. > > > > > > > > regards, > > > > > > > > Leonardo > > > > > > > > > > > > > > > > > > > > > > > > > >
