From: | Sergey Muraviov <sergey(dot)k(dot)muraviov(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Noah Misch <noah(at)leadboat(dot)com>, Greg Stark <stark(at)mit(dot)edu>, Peter Eisentraut <peter_e(at)gmx(dot)net>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, Emre Hasegeli <emre(at)hasegeli(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: wrapping in extended mode doesn't work well with default pager |
Date: | 2014-06-12 09:49:05 |
Message-ID: | CAJTaR30koAru2tucYiAZ=JjFFi8TZ0K7dhfSY-y7bsFj347zdg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi.
I tried to take into account all of yours comments.
PS
This patch prints newline indicators in aligned mode, but this
functionality can be easily disabled.
PPS
I'm on vacations now and don't read mail everyday.
2014-06-12 6:16 GMT+04:00 Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>:
> Noah Misch <noah(at)leadboat(dot)com> writes:
> > Based on the commit message and procedural history, I thought commit
> 6513633
> > was changing behavior solely for the combination of "\pset expanded" and
> > "\pset format wrapped". Peter's and my test cases show that it also
> changed
> > behavior for "\pset expanded" alone. That's a bug, unless someone sees
> to
> > argue that the new "\pset expanded" behavior is a desirable improvement
> in
> > spite of its origin as an accident. Altering an entrenched psql output
> format
> > is a big deal.
>
> TBH I'm wondering if we shouldn't just revert that patch (and the
> subsequent fix attempts). It was not a major feature and I'm thinking
> we have better things to do right now than try to fix the multiple
> logic holes it evidently has. The author's certainly welcome to try
> again with a more carefully thought-through patch for 9.5.
>
> regards, tom lane
>
--
Best regards,
Sergey Muraviov
Attachment | Content-Type | Size |
---|---|---|
psql-wrapped-expanded-fix-v5.patch | text/x-patch | 63.8 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Naoya Anzai | 2014-06-12 10:09:41 | Re: "cancelling statement due to user request error" occurs but the transaction has committed. |
Previous Message | Jochem van Dieten | 2014-06-12 09:46:02 | Re: "RETURNING PRIMARY KEY" syntax extension |