Add JacksonCollectors with toArrayNode() implementation

[rikkarth]

See back reference: #4691

cc @cowtowncoder

[cowtowncoder]

Looks good overall, CLA seems to be sent.

One last thing: I do think name toArrayNode() would make sense, given this is what is result of collection. So let's make that one change.

[rikkarth]

Looks good overall, CLA seems to be sent.

One last thing: I do think name toArrayNode() would make sense, given this is what is result of collection. So let's make that one change.

Done as instructed. Thank you.

(force pushes just to clean up the commit messages)

[rikkarth]

Just for future reference future Stream support implementations may include all types of object.

Example

// don't mind the signature name
// notice the return has changed from JsonNode to String
public static Collector<String, ArrayNode, ArrayNode> someNewSignature(JsonNodeCreator nodeCreator) {
    return Collector.of(
            nodeCreator::arrayNode, // supplier
            ArrayNode::add, // accumulator <---- every type supported by add can become a collector implementation now
            ArrayNode::addAll // combiner
        );
}

Use-Case

@Test
public void testToArrayNsode()
{
    final ObjectMapper objectMapper = new ObjectMapper();

    final JsonNode jsonNodeResult = IntStream.range(0, 10)
        .mapToObj(i -> "Hello")
        .collect(JacksonCollectors.toArrayNodeString());

    System.out.println(jsonNodeResult);
}

Output

["Hello", "Hello", "Hello", "Hello", "Hello", "Hello", "Hello", "Hello", "Hello", "Hello"]

[cowtowncoder]

Merged: thank you for contributing this, @rikkarth ! Will be in 2.18.0, to be released within week or so.

[rikkarth]

Merged: thank you for contributing this, @rikkarth ! Will be in 2.18.0, to be released within week or so.

Thank you, you were very supportive and nice, I really appreciate it. I intend to continue supporting you in the future.

[cowtowncoder]

Glad this worked out very well; looking forward to our future collaboration @rikkarth !