[PING] RFR: JDK-8150704 XALAN: ERROR: 'No more DTM IDs are available' when transforming with lots of temporary result trees
huizhe.wang at oracle.com
Mon Mar 7 02:26:56 UTC 2016
Thanks for reporting and providing patch for the issue! Looks like a
nice solution that may potentially reduce memory requirement for some
large templates. Could you also verify that the patch also fixes
JDK-8150699  that was created the same day as yours?
I assume the stylesheet is created to just illustrate the issue. If it's
a real use case, then it should have made the variable global to avoid
creating a lot of RTFs, and therefore avoid the whole "No more DTM IDs"
issue. It would make the process a lot more efficient.
Some classes, such as Sort.java, still contain the old header, please
update them with the new ones such as that in DOM.java.
The $Id section, such as the following, can all be removed, they were
from legacy repository, misleading since it implies the file was last
updated, in this case, in 2005:
21 * $Id: Sort.java,v 184.108.40.206 2005/09/12 11:08:12 pvedula Exp $
For the new test, it's probably better to add some kind of assertion in
the test, e.g. expected result, than failing on a broad Exception. What
if the test passes but the transform operation isn't because of the changes?
The test is also not sufficient. The release methods seem to be okay.
However, they don't seem to have been fully exercised in the test (only
simple RTs were created?). In that sense, the sample attached in
JDK-8150699 provided an opportunity to better verify the changes.
It would be good to add some javadoc or dev notes to the test. While
consolidating tests (into TransformerTest), please make sure
notes/javadoc are copied over or added.
On 3/3/2016 11:50 PM, Langer, Christoph wrote:
> Ping - any comments or reviews for this bugfix?
> From: Langer, Christoph
> Sent: Freitag, 26. Februar 2016 16:02
> To: core-libs-dev at openjdk.java.net
> Subject: RFR: JDK-8150704 XALAN: ERROR: 'No more DTM IDs are available' when transforming with lots of temporary result trees
> I've created a fix proposal for the issue I have reported in this bug:
> The webrev can be found here:
> The Xalan parser would eventually run out of DTM IDs if xsl transformations involve lots of temporary result trees. Those are never released although they could. A testcase is included for this. I've also done some cleanups in the Xalan code and in the tests.
> Thanks in advance for looking at this :)
> Best regards
More information about the core-libs-dev