Re: pg_dump --pretty-print-views - Mailing list pgsql-hackers

From David Fetter
Subject Re: pg_dump --pretty-print-views
Date
Msg-id 20130110132533.GF2474@fetter.org
Whole thread Raw
In response to pg_dump --pretty-print-views  (Marko Tiikkaja <pgmail@joh.to>)
Responses Re: pg_dump --pretty-print-views
List pgsql-hackers
On Thu, Jan 10, 2013 at 01:23:10PM +0100, Marko Tiikkaja wrote:
> Hi,
> 
> At the company I work for, we've been splitting dumps into separate
> files and diffing them for a while now.  By far the biggest problem
> we had was with views: pg_dump by default dumps views on one line,
> in a format which maximizes compatibility.  Now this has several
> problems for our use case:
> 
>   1) The one-line equivalent of a 200-line view is completely impossible
>      to read.
>   2) If there's a difference between the two dumped view definitions,
>      it takes a long time to find where and what exactly it is.
>   3) For some reason some expressions are dumped differently depending
>      on how exactly they are written, cluttering the diff with false
>      positives.
> 
> While we can do the actual splitting of objects from a -Fc dump
> relatively easily, we can't fix the view definitions after they've
> been dumped.  So I'm proposing a --pretty-print-views setting to
> pg_dump (patch attached).
> 
> Any feedback is welcome.

Why not make this the new default?  That way, new users will have the
benefit, and people with tools or processes that depend on the old
behavior can still have it.

Cheers,
David.
-- 
David Fetter <david@fetter.org> http://fetter.org/
Phone: +1 415 235 3778  AIM: dfetter666  Yahoo!: dfetter
Skype: davidfetter      XMPP: david.fetter@gmail.com
iCal: webcal://www.tripit.com/feed/ical/people/david74/tripit.ics

Remember to vote!
Consider donating to Postgres: http://www.postgresql.org/about/donate



pgsql-hackers by date:

Previous
From: Amit kapila
Date:
Subject: Re: Extra XLOG in Checkpoint for StandbySnapshot
Next
From: Stephen Frost
Date:
Subject: Re: Index build temp files