Re: [Wicket-user] CompoundPropertyModel without OGNL in 1.1
Okay, I understand. Actually, I don't really want OGNL CompoundPropertyModel to be replaced. I just want to have the possibility of non-ognl compound property model in 1.1. So maybe It could be added to wicket-extensions? -Matej Martijn Dashorst wrote: I'd rather finalize 1.2 sooner, cutting off new features, than backport the OGNL replacement into 1.1. It is a breaking change, and by its nature, a 1.1.x release should not break stuff. Martijn On 11/16/05, Johan Compagner [EMAIL PROTECTED] wrote: are you really seeing big performance improvements when comparing the same app with 1.1 and 1.2? Thats nice to hear :) Backporting to 1.1.1 ... we could do it but it is a big change.. that could affect running installs (you can't just swap 1.1 to 1.1.1 then if you depend a bit more on ognl) This should be an issue where we should count votes! johan On 11/16/05, Matej Knopp [EMAIL PROTECTED] wrote: Hi. My current benchmark shows that CompoundPropertyModel is a serious performance issue when having big dataview. Is there any chance that the new CompoundPropertyModel from 1.2 will be backported to 1.1? -Matej --- 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=7628alloc_id=16845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user -- 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_idv28alloc_id845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user --- 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=7628alloc_id=16845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user
Re: [Wicket-user] CompoundPropertyModel without OGNL in 1.1
Very very nice! then what i did see in the profiler also is very real in the real world! And there is also another factor that you will only really see if the load is high.. and that is that there is also the same factor how many memory is gc'ed! johan On 11/17/05, Matej Knopp [EMAIL PROTECTED] wrote: Okay, I took CompoundPropertyModel from 1.2 and put it to my project, sothat I can directly compare CompoundPropertyModel from 1.1 and 1.2. AndJohan, you are going to like the result :)It's pretty impressive. On one page (with DataView - 7 columns and 10 rows), the rendering withold CompoundPropertyModels takes 330-400 ms and with new 40-80ms. Nowthat's a difference you can really feel.-MatejMatej Knopp wrote: Okay, I understand. Actually, I don't really want OGNL CompoundPropertyModel to be replaced. I just want to have the possibility of non-ognl compound property model in 1.1. So maybe It could be added to wicket-extensions? -Matej Martijn Dashorst wrote: I'd rather finalize 1.2 sooner, cutting off new features, than backport the OGNL replacement into 1.1. It is a breaking change, and by its nature, a 1.1.x release should not break stuff. Martijn On 11/16/05, Johan Compagner [EMAIL PROTECTED] wrote: are you really seeing big performance improvements when comparing the same app with 1.1 and 1.2? Thats nice to hear :) Backporting to 1.1.1 ... we could do it but it is a big change.. that could affect running installs (you can't just swap 1.1 to 1.1.1 then if you depend a bit more on ognl) This should be an issue where we should count votes! johan On 11/16/05, Matej Knopp [EMAIL PROTECTED] wrote: Hi. My current benchmark shows that CompoundPropertyModel is a serious performance issue when having big dataview. Is there any chance that the new CompoundPropertyModel from 1.2 will be backported to 1.1? -Matej --- 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=7628alloc_id=16845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user -- 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_idv28alloc_id845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user --- 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=7628alloc_id=16845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user __ NOD32 1.1288 (20051115) Information __ This message was checked by NOD32 antivirus system. http://www.eset.com ---This SF.Net email is sponsored by the JBoss Inc.Get Certified TodayRegister 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=7628alloc_id=16845op=click___ Wicket-user mailing listWicket-user@lists.sourceforge.nethttps://lists.sourceforge.net/lists/listinfo/wicket-user
Re: [Wicket-user] CompoundPropertyModel without OGNL in 1.1
How about the percentage it takes up in comparison of the total computing for one request? Any other bottlenecks you can identify? Or was - like Johan said - OGNL one of the biggest hurdles? Eelco On 11/17/05, Matej Knopp [EMAIL PROTECTED] wrote: Okay, I took CompoundPropertyModel from 1.2 and put it to my project, so that I can directly compare CompoundPropertyModel from 1.1 and 1.2. And Johan, you are going to like the result :) It's pretty impressive. On one page (with DataView - 7 columns and 10 rows), the rendering with old CompoundPropertyModels takes 330-400 ms and with new 40-80ms. Now that's a difference you can really feel. -Matej Matej Knopp wrote: Okay, I understand. Actually, I don't really want OGNL CompoundPropertyModel to be replaced. I just want to have the possibility of non-ognl compound property model in 1.1. So maybe It could be added to wicket-extensions? -Matej Martijn Dashorst wrote: I'd rather finalize 1.2 sooner, cutting off new features, than backport the OGNL replacement into 1.1. It is a breaking change, and by its nature, a 1.1.x release should not break stuff. Martijn On 11/16/05, Johan Compagner [EMAIL PROTECTED] wrote: are you really seeing big performance improvements when comparing the same app with 1.1 and 1.2? Thats nice to hear :) Backporting to 1.1.1 ... we could do it but it is a big change.. that could affect running installs (you can't just swap 1.1 to 1.1.1 then if you depend a bit more on ognl) This should be an issue where we should count votes! johan On 11/16/05, Matej Knopp [EMAIL PROTECTED] wrote: Hi. My current benchmark shows that CompoundPropertyModel is a serious performance issue when having big dataview. Is there any chance that the new CompoundPropertyModel from 1.2 will be backported to 1.1? -Matej --- 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=7628alloc_id=16845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user -- 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_idv28alloc_id845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user --- 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=7628alloc_id=16845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user __ NOD32 1.1288 (20051115) Information __ This message was checked by NOD32 antivirus system. http://www.eset.com --- 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=7628alloc_id=16845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user --- 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_idv28alloc_id845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user
Re: [Wicket-user] CompoundPropertyModel without OGNL in 1.1
Well, the numbers I've sent are rought, I know, I didn't profile it. It's just a filter that checks how much time does it take to process a request. The only change between the two numbers was that I switch old CompoundPropertyModel to new model (called FastCompoundPropertyModel to distinguish easier :)) I might count the time, how much does it take for the model to access bean data, shouldn't be difficult. I'm not aware of any other bottleneck, the current numbers are quite satisfying for me now. -Matej Eelco Hillenius wrote: How about the percentage it takes up in comparison of the total computing for one request? Any other bottlenecks you can identify? Or was - like Johan said - OGNL one of the biggest hurdles? Eelco On 11/17/05, Matej Knopp [EMAIL PROTECTED] wrote: Okay, I took CompoundPropertyModel from 1.2 and put it to my project, so that I can directly compare CompoundPropertyModel from 1.1 and 1.2. And Johan, you are going to like the result :) It's pretty impressive. On one page (with DataView - 7 columns and 10 rows), the rendering with old CompoundPropertyModels takes 330-400 ms and with new 40-80ms. Now that's a difference you can really feel. -Matej Matej Knopp wrote: Okay, I understand. Actually, I don't really want OGNL CompoundPropertyModel to be replaced. I just want to have the possibility of non-ognl compound property model in 1.1. So maybe It could be added to wicket-extensions? -Matej Martijn Dashorst wrote: I'd rather finalize 1.2 sooner, cutting off new features, than backport the OGNL replacement into 1.1. It is a breaking change, and by its nature, a 1.1.x release should not break stuff. Martijn On 11/16/05, Johan Compagner [EMAIL PROTECTED] wrote: are you really seeing big performance improvements when comparing the same app with 1.1 and 1.2? Thats nice to hear :) Backporting to 1.1.1 ... we could do it but it is a big change.. that could affect running installs (you can't just swap 1.1 to 1.1.1 then if you depend a bit more on ognl) This should be an issue where we should count votes! johan On 11/16/05, Matej Knopp [EMAIL PROTECTED] wrote: Hi. My current benchmark shows that CompoundPropertyModel is a serious performance issue when having big dataview. Is there any chance that the new CompoundPropertyModel from 1.2 will be backported to 1.1? -Matej --- 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=7628alloc_id=16845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user -- 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_idv28alloc_id845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user --- 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=7628alloc_id=16845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user __ NOD32 1.1288 (20051115) Information __ This message was checked by NOD32 antivirus system. http://www.eset.com --- 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=7628alloc_id=16845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user --- 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_idv28alloc_id845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net
Re: [Wicket-user] CompoundPropertyModel without OGNL in 1.1
are you really seeing big performance improvements when comparing the same app with 1.1 and 1.2? Thats nice to hear :) Backporting to 1.1.1 ... we could do it but it is a big change.. that could affect running installs (you can't just swap 1.1 to 1.1.1 then if you depend a bit more on ognl) This should be an issue where we should count votes! johan On 11/16/05, Matej Knopp [EMAIL PROTECTED] wrote: Hi.My current benchmark shows that CompoundPropertyModel is a seriousperformance issue when having big dataview. Is there any chance that thenew CompoundPropertyModel from 1.2 will be backported to 1.1?-Matej---This SF.Net email is sponsored by the JBoss Inc.Get Certified TodayRegister 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=7628alloc_id=16845op=click ___Wicket-user mailing listWicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user
Re: [Wicket-user] CompoundPropertyModel without OGNL in 1.1
I'd rather finalize 1.2 sooner, cutting off new features, than backport the OGNL replacement into 1.1. It is a breaking change, and by its nature, a 1.1.x release should not break stuff. Martijn On 11/16/05, Johan Compagner [EMAIL PROTECTED] wrote: are you really seeing big performance improvements when comparing the same app with 1.1 and 1.2? Thats nice to hear :) Backporting to 1.1.1 ... we could do it but it is a big change.. that could affect running installs (you can't just swap 1.1 to 1.1.1 then if you depend a bit more on ognl) This should be an issue where we should count votes! johan On 11/16/05, Matej Knopp [EMAIL PROTECTED] wrote: Hi. My current benchmark shows that CompoundPropertyModel is a serious performance issue when having big dataview. Is there any chance that the new CompoundPropertyModel from 1.2 will be backported to 1.1? -Matej --- 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=7628alloc_id=16845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user -- 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_idv28alloc_id845op=click ___ Wicket-user mailing list Wicket-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-user
