Re: pgsql: psql: fix \connect with URIs and conninfo strings

From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
To: Robert Haas <robertmhaas(at)gmail(dot)com>
Cc: Bruce Momjian <bruce(at)momjian(dot)us>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, pgsql-committers <pgsql-committers(at)postgresql(dot)org>
Subject: Re: pgsql: psql: fix \connect with URIs and conninfo strings
Date: 2015-04-02 16:05:44
Message-ID: 20150402160544.GA30341@eldon.alvh.no-ip.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Robert Haas wrote:
> On Thu, Apr 2, 2015 at 10:57 AM, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> > Uh, I didn't see a huge amount of discussion, but I guess it is
> > sufficient:
> >
> > http://www.postgresql.org/message-id/flat/20150109191551(dot)GA32660(at)fetter(dot)org#20150109191551(dot)GA32660@fetter.org
>
> There are at least a dozen messages on that thread that are entirely
> dedicated to whether this should be back-patched. How much discussion
> did you want?

Yeah, this patch did get more discussion than many other patches do.
Sometimes you just have to go against what looks like apathy, and move
forward. That's what commitfests are supposed to be good for, after
all: personally, I care not one bit about this functionality, but
somebody went huge lengths to get it to work, which must mean that there
definitely is value in it.

> Now, mind you, my view did not prevail, so I can't be completely happy
> with the outcome, but I was in the minority. At least the version
> that broke essentially the entire build-farm only went into master, so
> that's something.

Yeah, I realized that backpatching the refactoring was the wrong
approach, though I didn't notice the breakage in master until it was
much too late. I pushed the 9.4 version to master now. Apologies for
not fixing it sooner -- had a funeral to attend, a drive 5 hours away.

--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2015-04-02 17:33:12 pgsql: autovacuum: Fix polarity of "wraparound" variable
Previous Message Robert Haas 2015-04-02 15:59:57 pgsql: Add missing calls to DatumGetUInt32.