You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Timo Walther (Jira)" <ji...@apache.org> on 2021/01/07 17:08:00 UTC

[jira] [Closed] (FLINK-18090) Update the Row.toString method

     [ https://issues.apache.org/jira/browse/FLINK-18090?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Timo Walther closed FLINK-18090.
--------------------------------
    Fix Version/s: 1.13.0
     Release Note: Row.toSting method has been reworked. This is an incompatible change. If the legacy representation is still required for tests, the old behavior can be restored via the flag RowUtils.USE_LEGACY_TO_STRING for the local JVM. However, relying on the row's string representation for tests is not a good idea in general as field data types are not verified.
       Resolution: Fixed

Fixed in 1.13.0:

[core][table] Update Row#toString and provide legacy util
commit 9903ea973f60ae14cc4baddd7ec7d10ad54fe7ad

Update tests for new Row.toString
commit 73cdd3d0d9f6a807b3e47c09eef7983c9aa180c7

> Update the Row.toString method
> ------------------------------
>
>                 Key: FLINK-18090
>                 URL: https://issues.apache.org/jira/browse/FLINK-18090
>             Project: Flink
>          Issue Type: Sub-task
>          Components: API / Core
>            Reporter: Timo Walther
>            Assignee: Timo Walther
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.13.0
>
>
> This updates the Row.toString method to provide a good summary string.
> In particular it fixes the following issues:
> Changeflag: According to FLIP-95, a row describes an entry in a
> changelog. Therefore, it should visible whether a row is an insert,
> delete, or update change. Now indicated with +I, -D, +U, -U.
> Nested rows: In the old implementation it was not visible whether nested
> rows exist or not due to missing start/end boundaries. Now indicated with
> [...] or {...}.
> Positioned rows vs. named rows: According to FLIP-136, it should be visible
> whether a row operates in name-based or position-based field mode. Now
> indicated with [...] or {...}.
> Nested arrays in maps and lists: In the old implementation arrays in maps
> or lists could not be represented.
> Wrong formatting: Most programming languages use a space after a comma.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)