Add spliterator support in JsonNode
#4865
Conversation
| * | ||
| * @since 3.0 | ||
| */ | ||
| public Spliterator<JsonNode> elementsSpliterator() { |
There was a problem hiding this comment.
Based on valueStream() method, should probably be value(s?)Spliterator()
| * for object nodes; empty spliterator (no contents) for other types | ||
| * @since 3.0 | ||
| */ | ||
| public Spliterator<Map.Entry<String, JsonNode>> fieldsSpliterator() { |
There was a problem hiding this comment.
3.0 uses term "property" everywhere, all references to "field" are changed elsewhere, although looks like JsonNode missed one such change.
I will need to remove method fields()
For the combination of property name and value, "entry" is used in JsonNode, see entryStream().
So should be entrySpliterator()?
There was a problem hiding this comment.
Actually, let me think about this one: I may have chosen wrong term: instead "entry" should really use "property"/"properties, as per:
https://github.com/FasterXML/jackson-future-ideas/wiki/JSTEP-6
will change entryStream to propertyStream(); and then we should have propertySpliterator.
Or, if we are decide to go with plurals, propertiesSpliterator().
There was a problem hiding this comment.
Singular it is: will change to propertySpliterator(). Similar for value case.
cowtowncoder
changed the title
cowtowncoder
changed the title
JsonNode
relates to FasterXML/jackson-core#1369