Skip to content

Make Option[T] in case classes un-boxed by default #528

@lihaoyi

Description

@lihaoyi

Currently we serialize them as k: [v] or k: [], which is generally not what people want. We should instead serialize them as k: v or nothing.

Nested options could probably continue to be serialized boxed in a single-element array. Options in sequences and other things that serialize to arrays can conitnue to be boxed as well.

This would be a breaking change and would need to go into upickle 4.0.0

Metadata

Metadata

Assignees

No one assigned

    Labels

    compat-breakerThis PR is good to merge, but breaks compatibility and needs to wait till we prepare a major release

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions