RFR 7147666 : High lock time for com.sun.org.apache.xerces.internal.impl.dv.DTDDVFactory.getInstance()

Alan Bateman Alan.Bateman at oracle.com
Mon Mar 5 19:52:12 UTC 2012


On 05/03/2012 18:46, Joe Wang wrote:
> Hi all,
>
> This is a low risk, proven patch made in Xerces back in 2007 [1], 
> removing unnecessary "synchronization" in DTDDVFactory.  As reported 
> in 7147666, this patch improves performance in one of the xml test.
>
> Webrev : http://cr.openjdk.java.net/~joehw/7u4/cr7147666/webrev/
>
> [1] http://svn.apache.org/viewvc?view=revision&revision=558581
>
> Please review.
Looks okay to me, I wonder what they had it synchronized in the first place.

-Alan.



More information about the core-libs-dev mailing list