Refactor Iterator Materialization about rumble HOT 3 CLOSED

@ghislainfourny . This issue was created as per your request to refactor iterator materialization in the top level iterator class. The comment above gives full details. In the current implementation, this function is implemented in RuntimeIterator:

https://github.com/Sparksoniq/sparksoniq/blob/12210e01129cfd6e3fad03abd71f5c18e11ac1b9/src/main/java/sparksoniq/jsoniq/runtime/iterator/RuntimeIterator.java#L117

What kind of a change did you have in mind? Could you please elaborate.

from rumble.

I think what I had in mind was adding a public method to the Iterator class that materializes an iterator:

List materialize(DynamicContext context)

This would simply make this functionality official and clean.

This function should ideally return an error if the iterator is already open (for this we may need to add an isOpen() method and a private field that keeps track of the "openness" of the iterator).

from rumble.

I think this is all addressed with various functions materializeAtMostOneItem, materializeOneItemOrNull, etc.

from rumble.