[OpenJDK 2D-Dev] Resurrecting Lauren'ts work on speeding up Pisces

[Andrea Aime]

Hi all,
it's been a few months since my last mail.
If you remember, I've tried participating to Laurent webrev discussions by
providing
a real world benchmark using GeoServer running the public OSGeo WMS
shootout.
If you want to read the last discussion on the topic, this was the thread:
http://comments.gmane.org/gmane.comp.java.openjdk.2d.devel/3467

Well, I'm back since I've managed to build a pure java PNG encoder that's
significantly
faster than both the JDK one, and the ImageIO native one (aka
CLibPNGImageWriter),
especially when run on OpenJDK, so I'm back trying to make the renderer
itself faster
for server side, headless workflows :-)
(if you want more info on the PNG encoder work I'm at your disposal too,
let's start a
separate thread).

Now, Laurent's work pretty much stopped at the fourth webrev, and I'm
interested
in trying to take over, provided there is a chance for the changes to be
merged (either
in JDK 8 or JDK 9).

>From my existing experience with open source projects I believe the patches
can
have a better chance of being merged by avoiding code style changes, and
minimizing
the changes to the bare minimum that provides the most significant speedup,
and
then make a choice on how the arrays should be cached (the CQL approach
seems
to be more promising) to further simplify the code.

So, if I sign the OCA, can I look over Laurent's code and build a new webrev
oriented to providing the minimum of changes to the existing code, possibly
trying to avoid any change that would affect the ductus pipeline?

Would this be a good approach to get the improvements merged?
In general, what do you suggest would be the best course of action?

Cheers
Andrea

-- 
==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for 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

-------------------------------------------------------
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20131019/e96d29ba/attachment.html>