From: | Pavan Deolasee <pavan(dot)deolasee(at)gmail(dot)com> |
---|---|
To: | Stephen Frost <sfrost(at)snowman(dot)net> |
Cc: | Kevin Grittner <Kevin(dot)Grittner(at)wicourts(dot)gov>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: pg_dump transaction's read-only mode |
Date: | 2012-12-31 06:38:01 |
Message-ID: | CABOikdPr6RN2M2EY8oAZmk7s3ULMiZ9Ck=ZfVYJgNYeRJ9Nz2A@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sun, Dec 30, 2012 at 12:38 AM, Stephen Frost <sfrost(at)snowman(dot)net> wrote:
> * Pavan Deolasee (pavan(dot)deolasee(at)gmail(dot)com) wrote:
>> On Fri, Sep 7, 2012 at 6:06 PM, Kevin Grittner
>> > That makes sense to me. The reason I didn't make that change when I
>> > added the serializable special case to pg_dump was that it seemed
>> > like a separate question; I didn't want to complicate an already big
>> > patch with unnecessary changes to non-serializable transactions.
>> >
>>
>> If we agree, should we change that now ?
>
> This is on the next commitfest, so I figure it deserves some comment.
> For my part- I tend to agree that we should have it always use a read
> only transaction. Perhaps we should update the pg_dump documentation to
> mention this as well though? Pavan, do you want to put together an
> actual patch?
>
I'd posted actual patch on this thread, but probably linked wrong
message-id in the commitfest page. Will check and correct. Regarding
pg_dump's documentation, I don't have strong views on that. Whether
pg_dump runs as a read-only transaction or not is entirely internal to
its implementation, but then if we make this change, it might be worth
telling users that they can trust that pg_dump will not make any
changes to their database and hence a safe operation to carry out.
Thanks,
Pavan
Pavan Deolasee
http://www.linkedin.com/in/pavandeolasee
From | Date | Subject | |
---|---|---|---|
Next Message | Craig Ringer | 2012-12-31 06:53:38 | [PATCH] Change Windows build docs to point to flex and bison from msys |
Previous Message | Peter Eisentraut | 2012-12-31 05:26:37 | Re: Documentation bug for LDAP authentication |