On Fri, Dec 14, 2012 at 4:14 PM, Karl O. Pinc wrote:
> On 12/13/2012 11:02:56 PM, Josh Kupershmidt wrote:
>> On Thu, Dec 13, 2012 at 9:03 PM, Karl O. Pinc wrote:
>
>> > Sorry to be so persnickety, and unhelpful until now.
>> > It seemed like it should be doable, but something
>> > was going wrong
On 12/13/2012 11:02:56 PM, Josh Kupershmidt wrote:
> On Thu, Dec 13, 2012 at 9:03 PM, Karl O. Pinc wrote:
> > Sorry to be so persnickety, and unhelpful until now.
> > It seemed like it should be doable, but something
> > was going wrong between keyboard and chair. I guess
> > I should be doing t
On Thu, Dec 13, 2012 at 9:03 PM, Karl O. Pinc wrote:
> My brain seems to have turned itself on. I went and (re)read
> the docbook manual and was able to come up with this,
> which works:
>
>
>
>
>--table
>-t
>
> table
>
> Yay! (indentation et-al aside)
That
On 12/13/2012 09:24:24 PM, Josh Kupershmidt wrote:
> On Thu, Dec 13, 2012 at 7:24 PM, Karl O. Pinc wrote:
> > As a fallback I'd do to the clusterdb, reindexdb, and vacuumdb
> > syntax summaries what was done to the pg_dump and pg_restore
> > syntax summaries. Remove all the detail. This is suck
On Thu, Dec 13, 2012 at 7:24 PM, Karl O. Pinc wrote:
> The problem is that the pg man pages would then have a
> syntax different from all the other man pages in the system,
> which all have ... outside of square braces.
> See "man cat", e.g.
FWIW, `man cat` on my OS X machine has synopsis:
On 12/13/2012 07:37:27 PM, Josh Kupershmidt wrote:
> On Thu, Dec 13, 2012 at 6:05 AM, Karl O. Pinc wrote:
> > On 12/13/2012 12:35:14 AM, Karl O. Pinc wrote:
> >
> >> On 12/12/2012 11:04:53 PM, Josh Kupershmidt wrote:
> >> > On Wed, Dec 12, 2012 at 8:14 AM, Karl O. Pinc
> wrote:
> >> > > On 12/11/
On Thu, Dec 13, 2012 at 6:05 AM, Karl O. Pinc wrote:
> On 12/13/2012 12:35:14 AM, Karl O. Pinc wrote:
>
>> On 12/12/2012 11:04:53 PM, Josh Kupershmidt wrote:
>> > On Wed, Dec 12, 2012 at 8:14 AM, Karl O. Pinc wrote:
>> > > On 12/11/2012 10:25:43 PM, Josh Kupershmidt wrote:
>> > >> On Tue, Dec 11,
On 12/13/2012 12:35:14 AM, Karl O. Pinc wrote:
> On 12/12/2012 11:04:53 PM, Josh Kupershmidt wrote:
> > On Wed, Dec 12, 2012 at 8:14 AM, Karl O. Pinc wrote:
> > > On 12/11/2012 10:25:43 PM, Josh Kupershmidt wrote:
> > >> On Tue, Dec 11, 2012 at 11:46 AM, Karl O. Pinc
> > wrote:
> > >> >> I belie
Hi Josh,
The good news is that there's only this little bit of
documentation formatting to fix
On 12/12/2012 11:04:53 PM, Josh Kupershmidt wrote:
> On Wed, Dec 12, 2012 at 8:14 AM, Karl O. Pinc wrote:
> > On 12/11/2012 10:25:43 PM, Josh Kupershmidt wrote:
> >> On Tue, Dec 11, 2012 at 11:46 A
On Wed, Dec 12, 2012 at 8:14 AM, Karl O. Pinc wrote:
> On 12/11/2012 10:25:43 PM, Josh Kupershmidt wrote:
>> On Tue, Dec 11, 2012 at 11:46 AM, Karl O. Pinc wrote:
>> >> I believe you need ellipses behind --table in the syntax summaries
>> >> of the command reference docs.
>>
>> Hrm, I was followi
On 12/11/2012 10:25:43 PM, Josh Kupershmidt wrote:
> On Tue, Dec 11, 2012 at 11:46 AM, Karl O. Pinc wrote:
> > Yes, the current pg_restore silently
> > ignores multiple --table arguments, and seems to use the last
> > one. You are introducing a backwards incompatible
> > change here.
> Agreed
On Tue, Dec 11, 2012 at 11:46 AM, Karl O. Pinc wrote:
> Hi Josh,
>
> I've signed up to review this patch.
Thanks!
> I configured with assertions, built, and tested using
> the attached script. It seems to do what it's supposed
> to and the code looks ok to me.
>
> The docs build. The text is r
On Tue, Dec 11, 2012 at 1:46 PM, Karl O. Pinc wrote:
> Yes, the current pg_restore silently
> ignores multiple --table arguments, and seems to use the last
> one. You are introducing a backwards incompatible
> change here. I don't know what to do about it, other
> than perhaps to have the patch
Hi Josh,
I've signed up to review this patch.
I configured with assertions, built, and tested using
the attached script. It seems to do what it's supposed
to and the code looks ok to me.
The docs build. The text is reasonable.
I also diffed the output of the attached script with
the output of
On 10/30/2012 10:14:19 PM, Josh Kupershmidt wrote:
> I went ahead and cooked up a patch to allow pg_restore, clusterdb,
> vacuumdb, and reindexdb to accept multiple --table switches. Hope I
> didn't overlook a similar tool, but these were the only other
> commands
> I found taking a --table argum
On Sun, Oct 28, 2012 at 4:30 PM, Josh Kupershmidt wrote:
> I see there's already a TODO for allowing pg_restore to accept
> multiple --table arguments[1], but would folks support adding this
> capability to various other commands which currently accept only a
> single --table argument, such as cl
Hi all,
I see there's already a TODO for allowing pg_restore to accept
multiple --table arguments[1], but would folks support adding this
capability to various other commands which currently accept only a
single --table argument, such as clusterdb, vacuumdb, and reindexdb?
Looking at pg_dump, it
17 matches
Mail list logo
