Martin Holmes

Results 189 comments of Martin Holmes

@joeytakeda Interestingly, my testing suggests that the problem is actually worse in dev. I added the new context (in branch issue-345-dev): `` Now I would expect that searching for "Amending"...

@joeytakeda The problem could be worked around by doing tokenization around line 433 of `json.xsl`, or by changing line 87 from this: `` to this: `` (This causes failures in...

Did you test in my branch from dev, issue-345-dev? I was seeing test fails.

Tests passing for me too. I'll add a specific test addressing this issue.

I believe this is only open because I was intending to add a specific test for this. Given that this is not urgent, I'm going to push this to 2.1...

Fix committed -- do we need to document this?

Jenkins is currently using Java 21, incidentally.

I'm bringing this issue back, because I'm now seeing OOM errors in the json target for the BBTI project (which ends up needing to create 177,000+ JSON files for 135,000+...

I've now tested removing fork="true" and also adding up to 4096m of memory to the java call, and I'm still getting the OOM error. This is a project on which...

Our first theory is that switching from indented JSON output to non-indented might be causing the problem because of the way Saxon serializes output.