On Thu, 23 Jan 2020 at 18:10, Alex Herbert <alex.d.herb...@gmail.com> wrote:
>
>
> On 23/01/2020 13:55, sebb wrote:
> > I think we don't want temporary serialisation fixes to encourage the
> > use of serialisation.
> >
> > So I suggest that the Release Notes and Javadoc should point out that
> > although serialisation is possible, it is not fully supported, and
> > that there are plans to drop all serialisation support.
>
> The javadoc for the new field that is not serialized have been
> documented. This current code is able to deserialize a record created
> using version 1.0 and 1.6. I did not test the in between releases.
> Serialisation broke in 1.7.
>
> Should a note be added to the header for CSVRecord stating that the
> class is serialization compatible with version 1.0 - 1.6. Fields added
> after version 1.6 are not serialized and the intension is to remove
> serialisation support in version 2.0.
>
> WDYT?

LGTM (apart from some spelling issues!)

However, I think it's worth noting in the Release Notes as well.

> Alex
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to