Hi all,

I have sent you the following pull requests to review. They should be
merged in the following order:

Geotools --> https://github.com/geotools/geotools/pull/384
GeoWebCache --> https://github.com/GeoWebCache/geowebcache/pull/229
GeoServer --> https://github.com/geoserver/geoserver/pull/524

I'm pleased to answer to your questions.

Cheers,
Nicola.


2014-03-03 9:29 GMT+01:00 Nicola Lagomarsini <
nicola.lagomars...@geo-solutions.it>:

> Hi Jody,
>
> I will apply my changes only when the 11.0 will go out.
>
> Cheers,
> Nicola.
>
>
> 2014-03-01 13:32 GMT+01:00 Jody Garnett <jody.garn...@gmail.com>:
>
> Question: are you wishing to apply this to 11.x before 11.0 goes out? That
>> is a bit risk, if that is your intention you will need to move promptly to
>> perform the change prior to RC2 going out.
>>
>> Jody
>>
>> Jody Garnett
>>
>>
>> On Sat, Mar 1, 2014 at 2:06 AM, Nicola Lagomarsini <
>> nicola.lagomars...@geo-solutions.it> wrote:
>>
>>> Hi all,
>>>
>>> I would recall the proposal for modifying the WMSTileFuser class in
>>> order to reach better performances on image reading and writing.
>>>
>>> The modifications on the WMSTileFuser class can be divided in 3 groups:
>>>
>>>    - improving the reading of every tile and the final image writing by
>>>    adding few classes and avoiding to call directly the ImageIO class, which
>>>    requires a lookup of the streams and readers/writers;
>>>    - improving the PNG8/GIF images by applying the Quantizer on them;
>>>    - improving the scaling of the final image by adding a better
>>>    RenderingHints configuration.
>>>
>>>
>>> These proposal requires to move some parts of the code from GeoServer to
>>> GeoTools and ImageIO-Ext for using the Quantizer class and the PNGWriter
>>> class located in the gs-wms module.
>>>
>>> With these modifications it could be possible to use the following
>>> classes on GeoWebCache without cyclic dependencies.
>>>
>>> I opened the following JIRA in order to trace the operation:
>>> https://jira.codehaus.org/browse/GEOS-6376.
>>>
>>> The exact order of the pull requests that will be sent will be:
>>>
>>>    - pull request for GeoTools;
>>>    - pull request for GeoWebCache;
>>>    - pull request for GeoServer.
>>>
>>>
>>> These modifications will be on the master branch for GeoServer,
>>> GeoTools, GeoWebCache, but a backport will be needed:
>>>
>>>    - for Geoserver 10.x. and 11.x
>>>    - for GeoWebCache on 1.5.0;
>>>    - for Geoserver on 2.4.x and 2.5.x;
>>>
>>> If there is any question I would be pleased to answer.
>>> Cheers,
>>> Nicola.
>>>
>>>
>>> 2014-02-10 22:10 GMT+01:00 Jody Garnett <jody.garn...@gmail.com>:
>>>
>>> Not from me, technically what you are doing is very clear cut.
>>>>
>>>> It would be good to hear back from GWC list as that appears to be the
>>>> most effected (i.e. new functionality).
>>>>
>>>> Jody Garnett
>>>>
>>>>
>>>> On Tue, Feb 11, 2014 at 2:12 AM, Nicola Lagomarsini <
>>>> nicola.lagomars...@geo-solutions.it> wrote:
>>>>
>>>>> Ok I will send an email to the ML.
>>>>>
>>>>> Is there another feedback?
>>>>>
>>>>> Cheers,
>>>>> Nicola.
>>>>>
>>>>>
>>>>> 2014-02-10 12:37 GMT+01:00 Jody Garnett <jody.garn...@gmail.com>:
>>>>>
>>>>> I am always troubled by such questions, let me think.
>>>>>>
>>>>>> As the original contributor you can pick up the code and contribute
>>>>>> it to another project.
>>>>>>
>>>>>> Since it is Nicola is doing the work, your +1 on the GeoServer list
>>>>>> as a PSC will work out fine. That way the GeoServer list has a record 
>>>>>> that
>>>>>> the request was made, and that a PSC member approved.
>>>>>>
>>>>>>
>>>>>> Jody Garnett
>>>>>>
>>>>>>
>>>>>> On Mon, Feb 10, 2014 at 7:44 PM, Andrea Aime <
>>>>>> andrea.a...@geo-solutions.it> wrote:
>>>>>>
>>>>>>> On Sun, Feb 9, 2014 at 1:29 PM, Jody Garnett <jody.garn...@gmail.com
>>>>>>> > wrote:
>>>>>>>
>>>>>>>> So I understand what is being proposed. I suppose my questions
>>>>>>>> twofold a) what is your time frame for this work and b) why 10.x ( 
>>>>>>>> since we
>>>>>>>> are nearing the 11.0)?
>>>>>>>>
>>>>>>>
>>>>>>>> As for moving code between projects, everything looks okay. For the
>>>>>>>> GeoServer code subject to a license change we will need a email 
>>>>>>>> request w/
>>>>>>>> response from GeoServer PSC (Quantizer / PNGEncoder).
>>>>>>>>
>>>>>>>
>>>>>>> No problem with it, but do we? I'm the sole author of both bits of
>>>>>>> code, so my agreement with the backport should be enough. Or not?
>>>>>>>
>>>>>>> Cheers
>>>>>>> Andrea
>>>>>>>
>>>>>>> --
>>>>>>> == Our support, Your Success! Visit http://opensdi.geo-solutions.itfor 
>>>>>>> more information ==
>>>>>>>
>>>>>>> Ing. Andrea Aime
>>>>>>> @geowolf
>>>>>>> Technical Lead
>>>>>>>
>>>>>>> GeoSolutions S.A.S.
>>>>>>> Via Poggio alle Viti 1187
>>>>>>> 55054  Massarosa (LU)
>>>>>>> Italy
>>>>>>> phone: +39 0584 962313
>>>>>>> fax: +39 0584 1660272
>>>>>>> mob: +39  339 8844549
>>>>>>>
>>>>>>> http://www.geo-solutions.it
>>>>>>> http://twitter.com/geosolutions_it
>>>>>>>
>>>>>>> -------------------------------------------------------
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> ==
>>>>> Our support, Your Success! Visit http://opensdi.geo-solutions.it for
>>>>> more information.
>>>>> ==
>>>>>
>>>>> Ing. Nicola Lagormarsini
>>>>> Junior Software Engineer
>>>>>
>>>>> GeoSolutions S.A.S.
>>>>> Via Poggio alle Viti 1187
>>>>> 55054  Massarosa (LU)
>>>>> Italy
>>>>> phone: +39 0584 962313
>>>>> fax:   +39 0584 1660272
>>>>>
>>>>> http://www.geo-solutions.it
>>>>> http://twitter.com/geosolutions_it
>>>>>
>>>>> -------------------------------------------------------
>>>>>
>>>>
>>>>
>>>
>>>
>>> --
>>> ==
>>> Our support, Your Success! Visit http://opensdi.geo-solutions.it for
>>> more information.
>>> ==
>>>
>>> Ing. Nicola Lagormarsini
>>> Junior Software Engineer
>>>
>>> GeoSolutions S.A.S.
>>> Via Poggio alle Viti 1187
>>> 55054  Massarosa (LU)
>>> Italy
>>> phone: +39 0584 962313
>>> fax:   +39 0584 1660272
>>>
>>> http://www.geo-solutions.it
>>> http://twitter.com/geosolutions_it
>>>
>>> -------------------------------------------------------
>>>
>>
>>
>
>
> --
> ==
> Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
> information.
> ==
>
> Ing. Nicola Lagormarsini
> Junior Software Engineer
>
> GeoSolutions S.A.S.
> Via Poggio alle Viti 1187
> 55054  Massarosa (LU)
> Italy
> phone: +39 0584 962313
> fax:   +39 0584 1660272
>
> http://www.geo-solutions.it
> http://twitter.com/geosolutions_it
>
> -------------------------------------------------------
>



-- 
==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
information.
==

Ing. Nicola Lagormarsini
Junior Software Engineer

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054  Massarosa (LU)
Italy
phone: +39 0584 962313
fax:   +39 0584 1660272

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

-------------------------------------------------------
------------------------------------------------------------------------------
Subversion Kills Productivity. Get off Subversion & Make the Move to Perforce.
With Perforce, you get hassle-free workflows. Merge that actually works. 
Faster operations. Version large binaries.  Built-in WAN optimization and the
freedom to use Git, Perforce or both. Make the move to Perforce.
http://pubads.g.doubleclick.net/gampad/clk?id=122218951&iu=/4140/ostg.clktrk
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to