Modified to not load the entire Sequence into memory during serialization.

[k163377]

Fixes #368

[cowtowncoder]

LGTM wrt older code. Won't work correctly wrt polymorphic types (old version didn't either), would need bigger changes to handle that, so I think this is a good incremental improvement.

[k163377]

@cowtowncoder
Thanks for the review.
Could you please share test cases and any other information that would be helpful?

[cowtowncoder]

@k163377 if we are taking about missing pieces, I am sure polymorphic Collections are tested in jackson-databind in some places. But the idea is that there'd be something like:

// Java version (modify for Kotlin)
@JsonTypeInfo(....)
public class Base { }

public class Impl extends Base { }

public class Wrapper {
   List<Base> values;

   public Sequence<Base> getValues() { ... }

    @JsonCreator(mode = Mode.DELEGATING)
    public Wrapper(List<Base> values) {
      this.values = values;
    }
}

// and then test will first create and write a `Wrapper`, then
// read it back, to verify polymorphic handling works.

so the only (?) complication is that you can only serialize Sequence (or Iterator etc) so reading back requires real container type.

You could also just verify serialization but I like the idea of round-trip testing that one can write and read back. Use of Wrapper object is to avoid Type Erasure problems.

[k163377]

Thank you very much.
I have submitted a new isssue.
#671