Skip to content

Improve Parsing for KV Format in tree explain. #15098

Closed

Description

@irenjj

Describe the bug

part of #14914

While adding more complex physical plan explain format, here are some issues met while parsing format line: writeln!(f, "{}={}", key, value) into extra_info in render tree:

  • Every line should only be kv format.
  • If there is a \n inside the key or value, it must be treated as a new line.

To Reproduce

No response

Expected behavior

We need a more flexible parser.

Additional context

No response

Activity

  1. irenjj commented on Mar 8, 2025

    @irenjj
    ContributorAuthor

    take

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions