Re: [Wicket-develop] [ wicket-Bugs-1359899 ] Out of heapspace error while reusing component instance

Sat, 19 Nov 2005 01:10:04 -0800

ah so all component.markupstreamposistions are kept outside of component but somewhere in the Markup(component) itself
So WicketComponentTag->position

that seems possible. But how do we use that position if we say: RequestCycle.renderComponent(comp) ?
Do we know then what wicket tag is rendered so that we know the position?


johan


On 11/18/05, Igor Vaynberg <[EMAIL PROTECTED]> wrote:
but the markup remains constant. we know where all the components are so we create fragments for everything that has a wicket:id tags and resolver tags.

havent thought it all the way through yet but seems plausible no?


-Igor


On 11/18/05, Johan Compagner < [EMAIL PROTECTED]> wrote:
How do you preparse it with our dynamic things?
Like a ListView and its row if you don;t have data how can you build up the tree completely?

Also the component structure isn;t fixed. You can replace them at runtime with other components.

johan



On 11/18/05, Igor Vaynberg < [EMAIL PROTECTED]> wrote:
is it possible to pre process this when the the template is loaded instead of on-first-render? this is how tapestry works and i recall jonathan was going to propose this but i guess he never got around to it. tapestry preprocesses the template and assigns to each component a "fragment" of markup they are responsible for rendering. jonathan proposed building a tree structure of these fragments as opposed to having a flat stream, but i dont think we need to go that far yet.

i think this will fix all these weird problems we are having. sure, components need to be 1:1 but i thought we were going to have that anyways.

with this approach we can also probably do some nifty things if we allow pragmatic assigned for the fragment to a component.

-Igor



On 11/18/05, Juergen Donnerstag < [EMAIL PROTECTED]> wrote:
Not sure, but I guess it was last week. Please check the revision date
for Component.java for details.

Juergen

On 11/18/05, Martijn Dashorst < [EMAIL PROTECTED] > wrote:
> Juergen,
>
>  When did you try to fix it? We are not working on the /real/ head, here,
> but have created our own branch. Last update on our side was from last
> monday, 10:22am CET
>
>  Martijn
>
>
>
> On 11/18/05, Juergen Donnerstag <[EMAIL PROTECTED] > wrote:
> >
> > At least I tried to fixed it. There should be a proper exception and a
> > test for it now.
> >
> > Though isFullRender can not replace markupStreamPos and though IMO it
> > introduces some redundancy (because though not completely true it
> > basically is markupStreamPos < 0), it offers additional flexibility
> > and the code is more readable. This change is not yet committed.
> >
> > Juergen
> >
> > On 11/18/05, Martijn Dashorst < [EMAIL PROTECTED] > wrote:
> > > The markupStreamPos was causing the panel to re-iterate the render
> process
> > > until the heapspace error occurred. The problem is, that when the second
> > > time the wicket:id="foo" is processed, the component points to earlier
> in
> > > the markupstream. Thus never ending the loop.
> > >
> > > Martijn
> > >
> > >
> > >
> > > On 11/18/05, Johan Compagner < [EMAIL PROTECTED]> wrote:
> > > > i thought juergen did fix this ?
> > > > There is a test now if it was rendered twice Or is that test to late?
> > > >
> > > > I still think that a isFullRender() from the request cycle instead of
> that
> > > markupstreamPos test is a better thing to do.
> > > >
> > > > johan
> > > >
> > > >
> > > >
> > > >
> > > > On 11/18/05, SourceForge.net < [EMAIL PROTECTED]> wrote:
> > > > > Bugs item #1359899, was opened at 2005-11-18 12:52
> > > > > Message generated for change (Tracker Item Submitted) made by Item
> > > Submitter
> > > > > You can respond by visiting:
> > > > >
> > >
> https://sourceforge.net/tracker/?func=detail&atid=684975&aid=1359899&group_id=119783
> > > > >
> > > > > Please note that this message will contain a full copy of the
> comment
> > > thread,
> > > > > including the initial issue submission, for this request,
> > > > > not just the latest update.
> > > > > Category: core
> > > > > Group: 1.2
> > > > > Status: Open
> > > > > Resolution: None
> > > > > Priority: 9
> > > > > Submitted By: Martijn Dashorst (dashorst)
> > > > > Assigned to: Nobody/Anonymous (nobody)
> > > > > Summary: Out of heapspace error while reusing component instance
> > > > >
> > > > > Initial Comment:
> > > > > This code example should give a wicket error, but
> > > > > instead gives a HTTP ERROR: 500 Java+heap+space.
> > > > >
> > > > > <html>
> > > > >     <body>
> > > > >         <span wicket:id="foo">foo</span>
> > > > >         <span wicket:id="foo">bar</span>
> > > > >     </body>
> > > > > </html>
> > > > >
> > > > >
> > > > > import wicket.markup.html.basic.Label ;
> > > > >
> > > > > public class Index extends QuickStartPage
> > > > > {
> > > > >     public Index()
> > > > >     {
> > > > >         add(new Label("foo", "die, Die, DIE!"));
> > > > >     }
> > > > > }
> > > > >
> > > > >
> > > > >
> > >
> ----------------------------------------------------------------------
> > > > >
> > > > > You can respond by visiting:
> > > > >
> > >
> https://sourceforge.net/tracker/?func=detail&atid=684975&aid=1359899&group_id=119783
> > > > >
> > > > >
> > > > >
> -------------------------------------------------------
> > > > > This SF.Net email is sponsored by the JBoss Inc.  Get Certified
> Today
> > > > > Register for a JBoss Training Course.  Free Certification Exam
> > > > > for All Training Attendees Through End of 2005. For more info visit:
> > > > >
> http://ads.osdn.com/?ad_id=7628&alloc_id=16845&op=click
> > > > > _______________________________________________
> > > > > Wicket-develop mailing list
> > > > > [email protected]
> > > > >
> > >
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
> > > > >
> > > >
> > > >
> > >
> > >
> > >
> > > --
> > > Living a wicket life...
> > >
> > > Martijn Dashorst - http://www.jroller.com/page/dashorst
> > >
> > > Wicket 1.1 is out:
> http://wicket.sourceforge.net/wicket-1.1
> >
> >
> > -------------------------------------------------------
> > This SF.Net email is sponsored by the JBoss Inc.  Get Certified Today
> > Register for a JBoss Training Course.  Free Certification Exam
> > for All Training Attendees Through End of 2005. For more info visit:
> > http://ads.osdn.com/?ad_idv28&alloc_id845&opclick
> > _______________________________________________
> > Wicket-develop mailing list
> > [email protected]
> >
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
> >
>
>
>
> --
> Living a wicket life...
>
> Martijn Dashorst - http://www.jroller.com/page/dashorst
>
> Wicket 1.1 is out: http://wicket.sourceforge.net/wicket-1.1


-------------------------------------------------------
This SF.Net email is sponsored by the JBoss Inc.  Get Certified Today
Register for a JBoss Training Course.  Free Certification Exam
for All Training Attendees Through End of 2005. For more info visit:
http://ads.osdn.com/?ad_idv28&alloc_id845&opclick
_______________________________________________
Wicket-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/wicket-develop




Reply via email to