Lists: | pgsql-docspgsql-patches |
---|
From: | Mark Stosberg <mark(at)summersault(dot)com> |
---|---|
To: | pgsql-docs(at)postgresql(dot)org |
Subject: | suggestion for improving TMPDIR and "--format" docs for pg_dump |
Date: | 2007-03-09 02:30:22 |
Message-ID: | esqgrv$50o$1@news.hub.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-docs pgsql-patches |
A suggested addition to the pg_dump docs:
This would be an extension to the documentation for the "tar" format:
The tar format needs some space to create temporary files. By default
"/tmp" is used. On systems with small "/tmp" partitions, set the
"TMPDIR" environment variable to a location with more space, such as
"/usr/tmp".
Also, I would like it if the pg_dump docs more assertively recommended
one of the "tar" or "custom" formats. They seem very similiar.
It woud also be nice to document that the full names "custom" and "tar" are
supported. Longer names can be nice for clarity.
( Unfortunately, wrong formats like "txx" also work instead of throwing
an error. )
Mark
From: | Peter Eisentraut <peter_e(at)gmx(dot)net> |
---|---|
To: | pgsql-docs(at)postgresql(dot)org |
Cc: | Mark Stosberg <mark(at)summersault(dot)com> |
Subject: | Re: suggestion for improving TMPDIR and "--format" docs for pg_dump |
Date: | 2007-03-09 10:21:00 |
Message-ID: | 200703091121.01155.peter_e@gmx.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-docs pgsql-patches |
Am Freitag, 9. März 2007 03:30 schrieb Mark Stosberg:
> The tar format needs some space to create temporary files. By default
> "/tmp" is used. On systems with small "/tmp" partitions, set the
> "TMPDIR" environment variable to a location with more space, such as
> "/usr/tmp".
That is really standard operating system behavior. We don't need to duplicate
that information.
> It woud also be nice to document that the full names "custom" and "tar" are
> supported. Longer names can be nice for clarity.
They are. You must be looking at an older version.
> ( Unfortunately, wrong formats like "txx" also work instead of throwing
> an error. )
Send a patch.
--
Peter Eisentraut
http://developer.postgresql.org/~petere/
From: | Mark Stosberg <mark(at)summersault(dot)com> |
---|---|
To: | pgsql-docs(at)postgresql(dot)org |
Subject: | Re: suggestion for improving TMPDIR and "--format" docs for pg_dump |
Date: | 2007-03-09 14:31:27 |
Message-ID: | esrr51$2n2n$1@news.hub.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-docs pgsql-patches |
Peter Eisentraut wrote:
> Am Freitag, 9. März 2007 03:30 schrieb Mark Stosberg:
>> The tar format needs some space to create temporary files. By default
>> "/tmp" is used. On systems with small "/tmp" partitions, set the
>> "TMPDIR" environment variable to a location with more space, such as
>> "/usr/tmp".
>
> That is really standard operating system behavior. We don't need to duplicate
> that information.
So is
./configure
make
make install
These are all external, standard programs, but they are common and
useful to document with applications. I think a small amount of
redundancy is documentation is a good thing.
When I wanted to use a different tmp dir, I searched the PostgreSQL docs
for how to do that, because I was using a PostgreSQL tool. I had no why
of knowing whether PostgreSQL was even using standard conventions of
tmpdir handling, of if it did it's own thing. Also, after several years
of being professional involved with Unix system maintenance, I was not
aware that this was a standard feature. Surely then, less advanced users
would benefit from the documentation.
>> It woud also be nice to document that the full names "custom" and "tar" are
>> supported. Longer names can be nice for clarity.
>
> They are. You must be looking at an older version.
You are correct. Sorry about that.
>> ( Unfortunately, wrong formats like "txx" also work instead of throwing
>> an error. )
>
> Send a patch.
I appreciate the sentiment, by my C skills are weak.
Mark
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Mark Stosberg <mark(at)summersault(dot)com> |
Cc: | pgsql-docs(at)postgresql(dot)org |
Subject: | Re: suggestion for improving TMPDIR and "--format" docs for pg_dump |
Date: | 2007-03-19 17:43:28 |
Message-ID: | 200703191743.l2JHhSS04401@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-docs pgsql-patches |
Mark Stosberg wrote:
> It woud also be nice to document that the full names "custom" and "tar" are
> supported. Longer names can be nice for clarity.
>
> ( Unfortunately, wrong formats like "txx" also work instead of throwing
> an error. )
I don't see that with current CVS:
$ pg_dump --format=x test
pg_dump: invalid output format "x" specified
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From: | Mark Stosberg <mark(at)summersault(dot)com> |
---|---|
To: | |
Cc: | pgsql-docs(at)postgresql(dot)org |
Subject: | Re: suggestion for improving TMPDIR and "--format" docs for pg_dump |
Date: | 2007-03-19 17:47:20 |
Message-ID: | 45FECCA8.8040204@summersault.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-docs pgsql-patches |
Bruce Momjian wrote:
> Mark Stosberg wrote:
>> It woud also be nice to document that the full names "custom" and "tar" are
>> supported. Longer names can be nice for clarity.
>>
>> ( Unfortunately, wrong formats like "txx" also work instead of throwing
>> an error. )
>
> I don't see that with current CVS:
>
> $ pg_dump --format=x test
> pg_dump: invalid output format "x" specified
Bruce,
I think the specific test case would have to start with a valid letter,
like "t", and then include invalid characters. Try this instead:
pg_dump --format=tx test
Mark
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Mark Stosberg <mark(at)summersault(dot)com> |
Cc: | PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [DOCS] suggestion for improving TMPDIR and "--format" docs for pg_dump |
Date: | 2007-03-22 19:43:24 |
Message-ID: | 200703221943.l2MJhOj17367@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-docs pgsql-patches |
Mark Stosberg wrote:
> Bruce Momjian wrote:
> > Mark Stosberg wrote:
> >> It woud also be nice to document that the full names "custom" and "tar" are
> >> supported. Longer names can be nice for clarity.
> >>
> >> ( Unfortunately, wrong formats like "txx" also work instead of throwing
> >> an error. )
> >
> > I don't see that with current CVS:
> >
> > $ pg_dump --format=x test
> > pg_dump: invalid output format "x" specified
>
> Bruce,
>
> I think the specific test case would have to start with a valid letter,
> like "t", and then include invalid characters. Try this instead:
>
> pg_dump --format=tx test
Thanks for the report. I have corrected this and the fix will be in
PostgreSQL 8.3. Interestingly, we support non-documented option
"append" and "file" too.
Patch attached and applied.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
Attachment | Content-Type | Size |
---|---|---|
/rtmp/diff | text/x-diff | 2.3 KB |
From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Mark Stosberg <mark(at)summersault(dot)com>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Re: [DOCS] suggestion for improving TMPDIR and "--format" docs for pg_dump |
Date: | 2007-03-22 20:17:42 |
Message-ID: | 26390.1174594662@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-docs pgsql-patches |
Bruce Momjian <bruce(at)momjian(dot)us> writes:
> Thanks for the report. I have corrected this and the fix will be in
> PostgreSQL 8.3. Interestingly, we support non-documented option
> "append" and "file" too.
Surely these should all be pg_strcasecmp?
regards, tom lane
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Mark Stosberg <mark(at)summersault(dot)com>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Re: [DOCS] suggestion for improving TMPDIR and "--format" docs for pg_dump |
Date: | 2007-03-22 20:18:52 |
Message-ID: | 200703222018.l2MKIqG22990@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-docs pgsql-patches |
Tom Lane wrote:
> Bruce Momjian <bruce(at)momjian(dot)us> writes:
> > Thanks for the report. I have corrected this and the fix will be in
> > PostgreSQL 8.3. Interestingly, we support non-documented option
> > "append" and "file" too.
>
> Surely these should all be pg_strcasecmp?
Yep, fixed.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From: | Dave Page <dpage(at)postgresql(dot)org> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Mark Stosberg <mark(at)summersault(dot)com>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Re: [DOCS] suggestion for improving TMPDIR and "--format" docs for pg_dump |
Date: | 2007-03-22 20:41:42 |
Message-ID: | 4602EA06.90204@postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-docs pgsql-patches |
Bruce Momjian wrote:
> Thanks for the report. I have corrected this and the fix will be in
> PostgreSQL 8.3. Interestingly, we support non-documented option
> "append" and "file" too.
Append is undocumented as it's intended to be used by pg_dumpall, not
directly by users.
Regards, Dave.
From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Dave Page <dpage(at)postgresql(dot)org> |
Cc: | Mark Stosberg <mark(at)summersault(dot)com>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Re: [DOCS] suggestion for improving TMPDIR and "--format" docs for pg_dump |
Date: | 2007-03-22 20:44:34 |
Message-ID: | 200703222044.l2MKiYx06471@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-docs pgsql-patches |
Dave Page wrote:
> Bruce Momjian wrote:
>
> > Thanks for the report. I have corrected this and the fix will be in
> > PostgreSQL 8.3. Interestingly, we support non-documented option
> > "append" and "file" too.
>
> Append is undocumented as it's intended to be used by pg_dumpall, not
> directly by users.
Ah, OK, but there should be a clear comment somewhere about that. I
will add it.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com
+ If your life is a hard drive, Christ can be your backup. +
From: | Dave Page <dpage(at)postgresql(dot)org> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | Mark Stosberg <mark(at)summersault(dot)com>, PostgreSQL-patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: Re: [DOCS] suggestion for improving TMPDIR and "--format" docs for pg_dump |
Date: | 2007-03-22 20:47:38 |
Message-ID: | 4602EB6A.9030307@postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-docs pgsql-patches |
Bruce Momjian wrote:
> Dave Page wrote:
>> Bruce Momjian wrote:
>>
>>> Thanks for the report. I have corrected this and the fix will be in
>>> PostgreSQL 8.3. Interestingly, we support non-documented option
>>> "append" and "file" too.
>> Append is undocumented as it's intended to be used by pg_dumpall, not
>> directly by users.
>
> Ah, OK, but there should be a clear comment somewhere about that. I
> will add it.
>
Thanks.
Regards Dave