Re: [HACKERS] Mail thread references in commits

2016-12-01 Thread Stephen Frost
* Tom Lane (t...@sss.pgh.pa.us) wrote: > Andres Freund writes: > > This: > > >> Discussion: > >> https://postgr.es/m/20161128182113.6527.58...@wrigleys.postgresql.org > >> Discussion: > >>

Re: [HACKERS] Mail thread references in commits

2016-12-01 Thread Tom Lane
Andres Freund writes: > This: >> Discussion: >> https://postgr.es/m/20161128182113.6527.58...@wrigleys.postgresql.org >> Discussion: >> https://postgr.es/m/CA+renyUEE29=X01JXdz8_TQvo6n9=2xoebbrnq8rklyr+kj...@mail.gmail.com > still looks better than: >> Discussion: >>

Re: [HACKERS] Mail thread references in commits

2016-12-01 Thread Andres Freund
On 2016-12-01 18:12:34 -0500, Tom Lane wrote: > Robert Haas writes: > > On Thu, Dec 1, 2016 at 4:33 PM, Tom Lane wrote: > >> When and if somebody tries to game that, we can do something about it, > >> but I'm not very worried. It's not like it's not

Re: [HACKERS] Mail thread references in commits

2016-12-01 Thread Tom Lane
Robert Haas writes: > On Thu, Dec 1, 2016 at 4:33 PM, Tom Lane wrote: >> When and if somebody tries to game that, we can do something about it, >> but I'm not very worried. It's not like it's not trivial to get your >> company's name, or $badword of

Re: [HACKERS] Mail thread references in commits

2016-12-01 Thread Andres Freund
On 2016-12-01 18:05:19 -0500, Tom Lane wrote: > ... the shortener isn't really doing anything for us. You end up with a > line longer than 80 characters with message-IDs generated by either gmail > or the bug report form, for instance these examples from recent commits: Still seems quite useful

Re: [HACKERS] Mail thread references in commits

2016-12-01 Thread Tom Lane
Stephen Frost writes: > Further, we seem agreed that URLs are what we want to have in the > commits rather than just the message-ID. If we're set on doing that, then ... > The question on the table at the moment seems to be if we want to use > https://postgr.es/m/ or

Re: [HACKERS] Mail thread references in commits

2016-12-01 Thread Stephen Frost
All, * Tom Lane (t...@sss.pgh.pa.us) wrote: > I think this is a straw man. We've already decided to use message-IDs > as the basic identity of messages for this purpose; other proposals > were considered before and rejected as too inconvenient. I tend to agree with Tom on this, for better or

Re: [HACKERS] Mail thread references in commits

2016-12-01 Thread Robert Haas
On Thu, Dec 1, 2016 at 4:33 PM, Tom Lane wrote: > Robert Haas writes: >> On Wed, Nov 30, 2016 at 1:50 PM, Greg Stark wrote: >>> I can't say I feel especially strongly either way on this but just to >>> toss out a small thing that might

Re: [HACKERS] Mail thread references in commits

2016-12-01 Thread Tom Lane
Robert Haas writes: > On Wed, Nov 30, 2016 at 1:50 PM, Greg Stark wrote: >> I can't say I feel especially strongly either way on this but just to >> toss out a small thing that might make a small difference >> >> If you happen to know how your

Re: [HACKERS] Mail thread references in commits

2016-12-01 Thread Robert Haas
On Wed, Nov 30, 2016 at 1:50 PM, Greg Stark wrote: > On 30 November 2016 at 16:19, Andrew Dunstan wrote: >> >> https://www.postgresql.org/message-id/cab7npqthydyf-fo+fzvxrhz-7_hptm4rodbcsy9-noqhvet...@mail.gmail.com >> >> I'll be interested to know if it

Re: [HACKERS] Mail thread references in commits

2016-11-30 Thread Greg Stark
On 30 November 2016 at 16:19, Andrew Dunstan wrote: > > https://www.postgresql.org/message-id/cab7npqthydyf-fo+fzvxrhz-7_hptm4rodbcsy9-noqhvet...@mail.gmail.com > > I'll be interested to know if it breaks anyone's MUA. If it doesn't all we > will be arguing about are

Re: [HACKERS] Mail thread references in commits

2016-11-30 Thread Andrew Dunstan
On 11/30/2016 10:46 AM, Heikki Linnakangas wrote: Agreed. I just did my first commit with the shortened URL, and I didn't like it. If we want to use URLs, let's use the canonical https://www.postgresql.org/message-id/ format. Do we know of an actual length limit that is useful to aim

Re: [HACKERS] Mail thread references in commits

2016-11-30 Thread Heikki Linnakangas
On 11/30/2016 04:52 PM, Peter Eisentraut wrote: On 11/27/16 8:37 AM, Magnus Hagander wrote: Ok, we now have it. https://postgr.es/m/messageid will redirect to that messageid in the main archives. I like the idea of recording the location of the discussion, but I'm not fond of the URL

Re: [HACKERS] Mail thread references in commits

2016-11-30 Thread Peter Eisentraut
On 11/27/16 8:37 AM, Magnus Hagander wrote: > Ok, we now have it. https://postgr.es/m/messageid will redirect to that > messageid in the main archives. I like the idea of recording the location of the discussion, but I'm not fond of the URL shortener. Besides the general problem that URL

Re: [HACKERS] Mail thread references in commits

2016-11-28 Thread Andrew Dunstan
On 11/28/2016 09:49 AM, Alvaro Herrera wrote: Magnus Hagander wrote: I don't really read perl enough to take it apart. But http://git.kernel.org/cgit/git/git.git/tree/gitweb/gitweb.perl is the code (we're probably on an older version). I'm guessing it's coming out of format_log_line (

Re: [HACKERS] Mail thread references in commits

2016-11-28 Thread Alvaro Herrera
Magnus Hagander wrote: > I don't really read perl enough to take it apart. But > http://git.kernel.org/cgit/git/git.git/tree/gitweb/gitweb.perl is the code > (we're probably on an older version). I'm guessing it's coming out of > format_log_line ( >

Re: [HACKERS] Mail thread references in commits

2016-11-28 Thread Magnus Hagander
On Mon, Nov 28, 2016 at 3:36 AM, Tom Lane wrote: > Magnus Hagander writes: > > Ok, we now have it. https://postgr.es/m/messageid will redirect to that > > messageid in the main archives. > > I pushed a patch using this new convention: >

Re: [HACKERS] Mail thread references in commits

2016-11-27 Thread Tom Lane
Magnus Hagander writes: > Ok, we now have it. https://postgr.es/m/messageid will redirect to that > messageid in the main archives. I pushed a patch using this new convention:

Re: [HACKERS] Mail thread references in commits

2016-11-27 Thread Magnus Hagander
On Mon, Nov 21, 2016 at 11:04 AM, Magnus Hagander wrote: > On Sun, Nov 20, 2016 at 11:02 PM, Andrew Dunstan > wrote: > >> >> >> On 11/20/2016 11:34 AM, Magnus Hagander wrote: >> >>> >>> >>> >>> I'm prepared to go along with this if there's general

Re: [HACKERS] Mail thread references in commits

2016-11-21 Thread Magnus Hagander
On Sun, Nov 20, 2016 at 11:02 PM, Andrew Dunstan wrote: > > > On 11/20/2016 11:34 AM, Magnus Hagander wrote: > >> >> >> >> I'm prepared to go along with this if there's general agreement. >> However, maybe we should make it actually work first, because I >> get a

Re: [HACKERS] Mail thread references in commits

2016-11-21 Thread Heikki Linnakangas
On 11/19/2016 06:38 PM, Tom Lane wrote: Andrew Dunstan writes: Every proposal along these lines strikes me as making more unobvious hoops for people to jump through. We're just reinventing the wheel here because we don't like the color of the standard wheels that some

Re: [HACKERS] Mail thread references in commits

2016-11-20 Thread Andrew Dunstan
On 11/20/2016 11:34 AM, Magnus Hagander wrote: I'm prepared to go along with this if there's general agreement. However, maybe we should make it actually work first, because I get a 404 from http://postgr.es/m/29df5b73-d823-ade4-cb17-47142742a...@dunslane.net

Re: [HACKERS] Mail thread references in commits

2016-11-20 Thread Vik Fearing
On 11/20/2016 03:59 PM, Andrew Dunstan wrote: > > On 11/20/2016 06:55 AM, Magnus Hagander wrote: >> >> We can replace the website part with a http://postgr.es/m/ >> which will make it a bit shorter and still as easy to generate from >> the client side and to search off. It'd be trivial to do, and

Re: [HACKERS] Mail thread references in commits

2016-11-20 Thread Magnus Hagander
On Sun, Nov 20, 2016 at 5:26 PM, Tom Lane wrote: > Andrew Dunstan writes: > > On 11/20/2016 06:55 AM, Magnus Hagander wrote: > >> We can replace the website part with a http://postgr.es/m/ > >> which will make it a bit shorter and still as easy to

Re: [HACKERS] Mail thread references in commits

2016-11-20 Thread Tom Lane
Andrew Dunstan writes: > On 11/20/2016 06:55 AM, Magnus Hagander wrote: >> We can replace the website part with a http://postgr.es/m/ >> which will make it a bit shorter and still as easy to generate from >> the client side and to search off. > That's 19 extra characters,

Re: [HACKERS] Mail thread references in commits

2016-11-20 Thread Magnus Hagander
On Sun, Nov 20, 2016 at 3:59 PM, Andrew Dunstan wrote: > > > On 11/20/2016 06:55 AM, Magnus Hagander wrote: > >> >> >> We can replace the website part with a http://postgr.es/m/ >> which will make it a bit shorter and still as easy to generate from the >> client side and to

Re: [HACKERS] Mail thread references in commits

2016-11-20 Thread Andrew Dunstan
On 11/20/2016 06:55 AM, Magnus Hagander wrote: We can replace the website part with a http://postgr.es/m/ which will make it a bit shorter and still as easy to generate from the client side and to search off. It'd be trivial to do, and since there is no state held at the forwarder for it,

Re: [HACKERS] Mail thread references in commits

2016-11-20 Thread Magnus Hagander
On Sat, Nov 19, 2016 at 5:38 PM, Tom Lane wrote: > Andrew Dunstan writes: > > On 11/19/2016 10:11 AM, Stephen Frost wrote: > >> That's actually not the case if we use a hash, because the client could > >> figure out what the hash is before sending it. >

Re: [HACKERS] Mail thread references in commits

2016-11-19 Thread Joshua Drake
My solution requires that everything have an issue. E.g., hackers becomes a tracker. Sincerely, Jd On Nov 19, 2016 09:04, "Tom Lane" wrote: > "Joshua D. Drake" writes: > > I wonder if now is the time (again) to consider an issue tracker. > > That

Re: [HACKERS] Mail thread references in commits

2016-11-19 Thread Tom Lane
"Joshua D. Drake" writes: > I wonder if now is the time (again) to consider an issue tracker. That would make the problem at hand worse, not better, because you'd get nothing at all for cases that were too trivial to make an issue tracker entry for, or that the committer

Re: [HACKERS] Mail thread references in commits

2016-11-19 Thread Tom Lane
Andrew Dunstan writes: > On 11/19/2016 10:11 AM, Stephen Frost wrote: >> That's actually not the case if we use a hash, because the client could >> figure out what the hash is before sending it. > The fact that it could possibly be done is not a good reason for doing > it.

Re: [HACKERS] Mail thread references in commits

2016-11-19 Thread Joshua D. Drake
On 11/17/2016 01:02 PM, Andrew Dunstan wrote: I love seeing references to email threads in commit messages. It would make them a lot friendlier though if a full http URL were included instead of just a Message-ID, i.e. instead of put

Re: [HACKERS] Mail thread references in commits

2016-11-19 Thread Andrew Dunstan
On 11/19/2016 10:11 AM, Stephen Frost wrote: * Magnus Hagander (mag...@hagander.net) wrote: On Nov 19, 2016 15:33, "Andrew Dunstan" wrote: I can think of at least one scenario where you might not easily have access to any invented ID - you're the author and you have

Re: [HACKERS] Mail thread references in commits

2016-11-19 Thread Stephen Frost
* Magnus Hagander (mag...@hagander.net) wrote: > On Nov 19, 2016 15:33, "Andrew Dunstan" wrote: > > I can think of at least one scenario where you might not easily have > access to any invented ID - you're the author and you have mailing list > me-too turned off, so you don't

Re: [HACKERS] Mail thread references in commits

2016-11-19 Thread Magnus Hagander
On Nov 19, 2016 15:33, "Andrew Dunstan" wrote: > > > > On 11/19/2016 07:36 AM, Stephen Frost wrote: >> >> Magnus, >> >> * Magnus Hagander (mag...@hagander.net) wrote: >>> >>> On Sat, Nov 19, 2016 at 1:07 AM, Stephen Frost wrote: * Magnus

Re: [HACKERS] Mail thread references in commits

2016-11-19 Thread Andrew Dunstan
On 11/19/2016 07:36 AM, Stephen Frost wrote: Magnus, * Magnus Hagander (mag...@hagander.net) wrote: On Sat, Nov 19, 2016 at 1:07 AM, Stephen Frost wrote: * Magnus Hagander (mag...@hagander.net) wrote: It would make the URLs actually short, but as mentioned upthread,

Re: [HACKERS] Mail thread references in commits

2016-11-19 Thread Stephen Frost
Magnus, * Magnus Hagander (mag...@hagander.net) wrote: > On Sat, Nov 19, 2016 at 1:07 AM, Stephen Frost wrote: > > * Magnus Hagander (mag...@hagander.net) wrote: > > > It would make the URLs actually short, but as mentioned upthread, that > > > wouldn't work at all if

Re: [HACKERS] Mail thread references in commits

2016-11-19 Thread Magnus Hagander
On Sat, Nov 19, 2016 at 1:07 AM, Stephen Frost wrote: > Magnus, > > * Magnus Hagander (mag...@hagander.net) wrote: > > It would make the URLs actually short, but as mentioned upthread, that > > wouldn't work at all if offline. So it'd be a tradeoff between those, but > > so

Re: [HACKERS] Mail thread references in commits

2016-11-18 Thread Andrew Dunstan
On 11/18/2016 09:05 PM, Andres Freund wrote: On November 18, 2016 1:06:18 PM PST, Tom Lane wrote: I think it's important to have the message-ID in cleartext. I do ask of that offline or at least locally. So I won't use something that makes that uncomfortable. I

Re: [HACKERS] Mail thread references in commits

2016-11-18 Thread Michael Paquier
On Fri, Nov 18, 2016 at 6:05 PM, Andres Freund wrote: > I do ask of that offline or at least locally. So I won't use something that > makes that uncomfortable. No committer here. But I do search for mails using the discussion ID present in the log message by querying in my

Re: [HACKERS] Mail thread references in commits

2016-11-18 Thread Andres Freund
On November 18, 2016 1:06:18 PM PST, Tom Lane wrote: >Robert Haas writes: >> On Thu, Nov 17, 2016 at 10:43 PM, Joshua Drake >wrote: >>> Why not hash the URL? Something like: >>> Http://postgresopen.org/archive/743257890976432

Re: [HACKERS] Mail thread references in commits

2016-11-18 Thread Andrew Dunstan
On 11/18/2016 07:07 PM, Stephen Frost wrote: Magnus, * Magnus Hagander (mag...@hagander.net) wrote: It would make the URLs actually short, but as mentioned upthread, that wouldn't work at all if offline. So it'd be a tradeoff between those, but so are pretty much all other options that don't

Re: [HACKERS] Mail thread references in commits

2016-11-18 Thread Jim Nasby
On 11/18/16 4:35 PM, Magnus Hagander wrote: Another option would be to teach the URL shortener about our messages specifically, and just use an internal surrogate key we already have for them. That way it could be something like the current Planet URLs (which can be for example

Re: [HACKERS] Mail thread references in commits

2016-11-18 Thread Stephen Frost
Magnus, * Magnus Hagander (mag...@hagander.net) wrote: > It would make the URLs actually short, but as mentioned upthread, that > wouldn't work at all if offline. So it'd be a tradeoff between those, but > so are pretty much all other options that don't include the full message-id. This is a bit

Re: [HACKERS] Mail thread references in commits

2016-11-18 Thread Magnus Hagander
On Fri, Nov 18, 2016 at 11:08 PM, Alvaro Herrera wrote: > Tom Lane wrote: > > > AFAICS the proposal here is to go from, eg, > > > > Discussion: whht8...@mail.gmail.com> > > > > to > > > > Discussion: https://www.postgresql.org/message-id/CAF4Au4w6rrH_ >

Re: [HACKERS] Mail thread references in commits

2016-11-18 Thread Alvaro Herrera
Tom Lane wrote: > AFAICS the proposal here is to go from, eg, > > Discussion: > > > to > > Discussion: > https://www.postgresql.org/message-id/caf4au4w6rrh_j1bvvhzposrihcog7sgj3lsx0ty8zdwhht8...@mail.gmail.com > > which is

Re: [HACKERS] Mail thread references in commits

2016-11-18 Thread Tom Lane
Robert Haas writes: > On Thu, Nov 17, 2016 at 10:43 PM, Joshua Drake wrote: >> Why not hash the URL? Something like: >> Http://postgresopen.org/archive/743257890976432 > I suggested that upthread... Don't like the hashing idea because that would

Re: [HACKERS] Mail thread references in commits

2016-11-18 Thread Joshua Drake
Sorry didn't see it. On Nov 18, 2016 12:44, "Robert Haas" wrote: > On Thu, Nov 17, 2016 at 10:43 PM, Joshua Drake > wrote: > > Why not hash the URL? Something like: > > > > Http://postgresopen.org/archive/743257890976432 > > I suggested that

Re: [HACKERS] Mail thread references in commits

2016-11-18 Thread Robert Haas
On Thu, Nov 17, 2016 at 10:43 PM, Joshua Drake wrote: > Why not hash the URL? Something like: > > Http://postgresopen.org/archive/743257890976432 I suggested that upthread... -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company --

Re: [HACKERS] Mail thread references in commits

2016-11-18 Thread Dave Page
On Fri, Nov 18, 2016 at 1:38 AM, Alvaro Herrera wrote: > Tom Lane wrote: >> Andrew Dunstan writes: >> > I love seeing references to email threads in commit messages. It would >> > make them a lot friendlier though if a full http URL were included >>

Re: [HACKERS] Mail thread references in commits

2016-11-17 Thread Joshua Drake
Why not hash the URL? Something like: Http://postgresopen.org/archive/743257890976432 Where the hash is derived from the message if? On Nov 17, 2016 17:40, "Alvaro Herrera" wrote: > > Tom Lane wrote: > > Andrew Dunstan writes: > > > I love seeing

Re: [HACKERS] Mail thread references in commits

2016-11-17 Thread Alvaro Herrera
Tom Lane wrote: > Andrew Dunstan writes: > > I love seeing references to email threads in commit messages. It would > > make them a lot friendlier though if a full http URL were included > > instead of just a Message-ID, > > I've intentionally not done that, because it

Re: [HACKERS] Mail thread references in commits

2016-11-17 Thread Andrew Dunstan
On 11/17/2016 04:06 PM, Andres Freund wrote: On November 17, 2016 1:02:38 PM PST, Andrew Dunstan wrote: I love seeing references to email threads in commit messages. It would make them a lot friendlier though if a full http URL were included instead of just a

Re: [HACKERS] Mail thread references in commits

2016-11-17 Thread Tom Lane
Andrew Dunstan writes: > I love seeing references to email threads in commit messages. It would > make them a lot friendlier though if a full http URL were included > instead of just a Message-ID, I've intentionally not done that, because it does not seem very

Re: [HACKERS] Mail thread references in commits

2016-11-17 Thread Alvaro Herrera
Robert Haas wrote: > On Thu, Nov 17, 2016 at 4:06 PM, Andres Freund wrote: > > On November 17, 2016 1:02:38 PM PST, Andrew Dunstan > > wrote: > >>I love seeing references to email threads in commit messages. It would > >>make them a lot friendlier though

Re: [HACKERS] Mail thread references in commits

2016-11-17 Thread Alvaro Herrera
Andrew Dunstan wrote: > I love seeing references to email threads in commit messages. It would make > them a lot friendlier though if a full http URL were included instead of > just a Message-ID, i.e. instead of put > . I

Re: [HACKERS] Mail thread references in commits

2016-11-17 Thread Robert Haas
On Thu, Nov 17, 2016 at 4:06 PM, Andres Freund wrote: > On November 17, 2016 1:02:38 PM PST, Andrew Dunstan > wrote: >>I love seeing references to email threads in commit messages. It would >>make them a lot friendlier though if a full http URL were

Re: [HACKERS] Mail thread references in commits

2016-11-17 Thread Andres Freund
On November 17, 2016 1:02:38 PM PST, Andrew Dunstan wrote: >I love seeing references to email threads in commit messages. It would >make them a lot friendlier though if a full http URL were included >instead of just a Message-ID, i.e. instead of put