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

["Timo Walther (Jira)" <ji...@apache.org>]

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

Timo Walther updated FLINK-18090:
---------------------------------
    Description: 
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 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.

  was:FLINK-16998 added a `RowKind` to the `Row` type. We have not updated the toString method yet.


> 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
>
> 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 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.



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