Re: Auto Partitioning Patch - WIP version 1 - Mailing list pgsql-patches

From Bruce Momjian
Subject Re: Auto Partitioning Patch - WIP version 1
Date
Msg-id 200803261348.m2QDme903600@momjian.us
Whole thread Raw
In response to Re: Auto Partitioning Patch - WIP version 1  (Alvaro Herrera <alvherre@commandprompt.com>)
Responses Re: Auto Partitioning Patch - WIP version 1
List pgsql-patches
Alvaro Herrera wrote:
> Bruce Momjian escribi?:
> > Alvaro Herrera wrote:
>
> > > I noticed you broke an URL that previously worked: what was
> > > http://momjian.us/mhonarc/patches/d3c4af540703292358s8ed731el7771ab14083aa610@mail.gmail.com.html
> > > is now
> > > http://momjian.us/mhonarc/patches_hold/d3c4af540703292358s8ed731el7771ab14083aa610@mail.gmail.com.html
> > >
> > > May I suggest that the URLs with Message-Ids are stored outside the
> > > particular patch queue directory?  The script I showed you yesterday
> > > could be used to do that.
> >
> > My email are added/removed so I am unsure how to do that easily becuase
> > the lists would share the same directory.  We don't have that problem
> > with the archives.
>
> My point is that you should only _add_ Message-Ids, not remove them.
> You can move the messages from one queue to the other to your heart's
> content, but the Message-Id URL should continue to work with no changes.

Well, when an email is applied, it is deleted.  How do I update the
message-id for that in an automated manner.  Right now a rebuild deleted
all the links and recreates them.

> > > Also I noticed that by moving it to the hold queue, the comments that
> > > may have existed on the patch queue are now gone :-(  I'm not sure if
> > > there were any in this case, but it's better if we're aware of that
> > > fact.  I think this could be solved if the "namespace" of the comment
> > > does not contain the patch queue name.
> >
> > I specifically set things up so the comments should move with the email.
>
> Yeah, I noticed that after sending the email -- the js-kit name seems to
> be only "msgid-<foo>".  I thought the "permalink=" attribute was part of
> that, but perhaps not?  In case you added that permalink attribute
> because of my request the other day, let me clarify that what I was
> actually thinking was having something like
>
> <a href="http://momjian.us/msgid/123foobar@mylaptop">permalink</a>

The permalink is for people who get email --- it tells them which
message got the comment.  (People were complaining before I fixed that.)

> after the name of the poster, so that it would be visible on the index
> page and the user didn't have to open the page to get it.

We can do that if people want.  I used to show the message id on the
thread page but several felt it was too cluttered-looking.

--
  Bruce Momjian  <bruce@momjian.us>        http://momjian.us
  EnterpriseDB                             http://postgres.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

pgsql-patches by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: \password in psql help
Next
From: Alvaro Herrera
Date:
Subject: Re: Auto Partitioning Patch - WIP version 1