Java: Simplify interpretOutput

[alexet]

I am pretty sure that this is equivalent and avoids weird join ordering issues as both disjuncts now use the same variables. Otherwise we need n and ast bound to do the disjunction but that is a CP externally. Currently it is Ok as it seems these cases are always empty before the CP by the c = "Parameter" or c = "" and we know the input data is static but the CP is there if someone ever writes one of those cases (or in my case we change the optimiser to be less aggressive about pushing equalities after join ordering)

I feel it might be possible to write QL4QL query for this case noting that the fact that n and ast are unused in the disjunct is what makes the substitution valid.

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Tip: Copilot code review supports C#, Go, Java, JavaScript, Markdown, Python, Ruby and TypeScript, with more languages coming soon. Learn more

[owen-mc]

Fwiw I made the node.asNode() change in the go version of this function in this commit, at the same time as adding some binding pragmas. It hasn't caused any problems that I'm aware of. I just checked the same function in other languages and didn't find the same node.asNode() issue - we must have copied it from the java library to the go library at some point.