Good stuff.
Gabriel may have further insights, but here's my perspective on what might
be worth considering:
1) No serious profiling work has been done on this build, and it wouldnt
suprise me if significant gains can be made
2) Fixes should target the trunk, so we'll need to re-run the profiling
against the trunk when we test the modules against the new plumbing
3) The database seems to be hit twice per query, which may be a significant
overhead if the db side is slow
4) The Converter factory seems to be used inefficiently - although the
architecture is flexible, in practice the required converter per attribute
will be the same for each retrieved databse row, but we dont seem to be
caching the choice and end up hitting all the factories in a loop each
value.
5) There is a "mutable attribute" for handling multi-valued properties
without instantiating objects for every vale, but it still caches the result
set from the JDBC database, leading to a scalability limitation. It would be
better if this was streamed, but this is a significant rewrite worth
considering only after getting stuff to trunk. It can handle quite a few
records as is, but does not max out gracefully yet.
Rob
On Fri, May 16, 2008 at 10:10 AM, Cameron Shorter <
[EMAIL PROTECTED]> wrote:
> As part of our robustness testing of AWDIP deployments of Geoserver
> (which includes Community Schemas), we have signed up to do "White box
> testing". Ie, work out what is happening inside Geoserver for the AWDIP
> use cases.
> The end goal is to identify areas in the code where performance can be
> improved.
>
> To this end, Stefan has started setting up profiling for Community
> Schema deploys of Geoserver. Unfortunately we have a limited budget for
> this and so we are looking for the 80% wins from 20% effort.
>
> So what I'm interested to hear are suggestions as to what parts of the
> code we should focus on.
>
> We will be collating the results of our tests here:
>
> https://extranet.lisasoft.com/wiki/index.php/AWDIP_Software_Test_Results#Observations_from_White_Box_Testing
>
> --
> Cameron Shorter
> Geospatial Systems Architect
> Tel: +61 (0)2 8570 5050
> Mob: +61 (0)419 142 254
>
> Think Globally, Fix Locally
> Commercial Support for Geospatial Open Source Solutions
> http://www.lisasoft.com/LISAsoft/SupportedProducts.html
>
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2008.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> _______________________________________________
> Geoserver-devel mailing list
> Geoserver-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geoserver-devel
>
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
