I don't think this is the rowset problem although I have to admit I have not looked at the XNodeSet code very closely.
The problem with the rowset code is that the SortedWalker thinks that the
DTM Document is not in Document Order, so it walks the document trying
to put it back in document order. Once it does this though, the JDBC recordset
is exhausted, in streaming mode, and the only row left is the last one. Now if you
select back on the rowset, there is only one element so the sorting happens but has
no effect. Subsequent templates i.e. "row" don't try to re-order the document because
the re-order already occurred. This bug has appeared before and Scot Boag has
fixed it but then it comes back. I think we are battling another bug that needs
preordered documents. Last I heard there was an effort to support DTM pruning
which would operate on preorderd documents.
-JG
I will need to do a little research on this. It was quite a while ago that I submitted this and I have long since forgotten the details. I just remember that it had something to do with results coming back (or not) or something like that. I am also fairly certain that slightly different versions were needed for Xalan 2.4 and 2.5. I thought that I had described the details at the time in an e-mail. I will see if I can find that somewhere. Maybe I will also do some DIFFs to figure out what I changed. Unfortunately, unless I was particularly diligent in an e-mail that I can find (or it is obvious from the code change) the rational may be lost.
It could be that this fixes the rowset problem described earlier... at least the version of Xalan that I am running does not exhibit that problem - but that could be because all my XConnections have multiple-results enabled. At least I think so, personally I have not used the Xalan SQL extension in a long time, other members of my team have been doing all of that.
I will try to get to this in the next few days... maybe tonight if I finish the stuff I am working on early - or if curiosity gets the better of me...
Art
