Bug #3313
closed
Streaming for-each-group with test="not(current-group()[2])"
Fix Committed on Branch:
9.8
Fixed in Maintenance Release:
Completely empirically, if we remove the test
&& !forGrouping
at TransmissionAdjunct#33
then the test works (all varieties). Moreover, the change seems to have no adverse effects.
I've no idea what the purpose of this condition is or was.
- Status changed from New to Resolved
- Applies to branch 9.8 added
- Fix Committed on Branch 9.8 added
The actual condition is
posture.isIncremental() && !forGrouping
and I have established that there are no other tests where posture.isIncremental() is true and forGrouping is also true, so the change seems safe enough. But why the condition was there in the first place eludes me. The code was present in 9.6, and streamed grouping has come on a long way since then, so I think it's just a left-over that's no longer appropriate.
I tried making the same change on 9.7 and it doesn't help us pass more tests, so I won't attempt it there.
- % Done changed from 0 to 100
- Fixed in Maintenance Release 9.8.0.3 added
Bug fix applied in the Saxon 9.8.0.3 maintenance release.
- Status changed from Resolved to Closed
Please register to edit this issue
Also available in: Atom
PDF
.