[jira] [Commented] (IMPALA-4337) Wrap long lines in explain plans

["Balazs Jeszenszky (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/IMPALA-4337?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16759879#comment-16759879 ] 

Balazs Jeszenszky commented on IMPALA-4337:
-------------------------------------------

I don't think we should do this - text editors can wrap as needed, but if Impala wraps the text, there's no way to easily unwrap it. It'd make it more difficult to eyeball plans with long predicate lists.

> Wrap long lines in explain plans
> --------------------------------
>
>                 Key: IMPALA-4337
>                 URL: https://issues.apache.org/jira/browse/IMPALA-4337
>             Project: IMPALA
>          Issue Type: Improvement
>          Components: Frontend
>    Affects Versions: Impala 2.8.0
>            Reporter: Henry Robinson
>            Priority: Minor
>              Labels: newbie, ramp-up
>
> Explain plans can have very long lines, particularly when printing lists of expressions. It should be possible to wrap, and still correctly indent, those lines.
> This is trickier than it sounds because they have to be wrapped in the context of their place in the plan (i.e. with appropriate prefixes etc). It's a good opportunity to split out explain plan generation from presentation, centralizing the logic so that this kind of change is easy to make.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-all-unsubscribe@impala.apache.org
For additional commands, e-mail: issues-all-help@impala.apache.org