On Sun, Feb 22, 2015 at 12:13 PM, Magnus Hagander mag...@hagander.net
wrote:
On Sun, Feb 15, 2015 at 9:46 PM, Peter Geoghegan p...@heroku.com wrote:
On Sun, Feb 15, 2015 at 4:59 PM, Peter Eisentraut pete...@gmx.net
wrote:
I think the old system where the patch submitter declared, this
On 3/3/15 11:11 AM, Bruce Momjian wrote:
On Tue, Mar 3, 2015 at 10:58:28AM -0500, Bruce Momjian wrote:
Would you suggest removing the automated system completely, or keep it
around
and just make it possible to override it (either by removing the note that
something is a patch, or by making
On Sun, Feb 22, 2015 at 03:12:12PM -0500, Magnus Hagander wrote:
# Attempt to identify the file using magic information
mtype = mag.buffer(contents)
if mtype.startswith('text/x-diff'):
a.ispatch = True
On Tue, Mar 3, 2015 at 10:58:28AM -0500, Bruce Momjian wrote:
Would you suggest removing the automated system completely, or keep it
around
and just make it possible to override it (either by removing the note that
something is a patch, or by making something that's not listed as a patch
On Fri, Feb 27, 2015 at 11:32 AM, Stefan Kaltenbrunner
ste...@kaltenbrunner.cc wrote:
On 02/26/2015 01:59 PM, Michael Paquier wrote:
On Thu, Feb 26, 2015 at 9:15 PM, Asif Naeem anaeem...@gmail.com wrote:
This thread seems relevant, Please guide me to how can access older CF
pages
The
Thank you Michael, It works.
On Thu, Feb 26, 2015 at 5:59 PM, Michael Paquier michael.paqu...@gmail.com
wrote:
On Thu, Feb 26, 2015 at 9:15 PM, Asif Naeem anaeem...@gmail.com wrote:
This thread seems relevant, Please guide me to how can access older CF
pages
The MSVC portion of this fix
On 02/26/2015 01:59 PM, Michael Paquier wrote:
On Thu, Feb 26, 2015 at 9:15 PM, Asif Naeem anaeem...@gmail.com wrote:
This thread seems relevant, Please guide me to how can access older CF pages
The MSVC portion of this fix got completely lost in the void:
On Thu, Feb 26, 2015 at 9:15 PM, Asif Naeem anaeem...@gmail.com wrote:
This thread seems relevant, Please guide me to how can access older CF pages
The MSVC portion of this fix got completely lost in the void:
https://commitfest.postgresql.org/action/patch_view?id=1330
Above link result in
Hi,
This thread seems relevant, Please guide me to how can access older CF
pages e.g.
Thread
http://www.postgresql.org/message-id/flat/51f19059.7050...@pgexperts.com#51f19059.7050...@pgexperts.com
mentions
following link i.e.
The MSVC portion of this fix got completely lost in the void:
On 2/22/15 3:12 PM, Magnus Hagander wrote:
Would you suggest removing the automated system completely, or keep it
around and just make it possible to override it (either by removing the
note that something is a patch, or by making something that's not listed
as a patch become marked as such)?
Magnus Hagander mag...@hagander.net writes:
On Sun, Feb 15, 2015 at 9:46 PM, Peter Geoghegan p...@heroku.com wrote:
also think that it's a waste of screen space to show who within the
annotation view. Granted, the old app supported this, but I tend to
think that if I actually cared who added a
On Sun, Feb 15, 2015 at 7:59 PM, Peter Eisentraut pete...@gmx.net wrote:
On 2/14/15 7:30 AM, Magnus Hagander wrote:
On Mon, Feb 9, 2015 at 4:56 PM, Robert Haas robertmh...@gmail.com
Can we make it smarter, so that the kinds of things people produce
intending for them to be patches
On Sun, Feb 15, 2015 at 9:46 PM, Peter Geoghegan p...@heroku.com wrote:
On Sun, Feb 15, 2015 at 4:59 PM, Peter Eisentraut pete...@gmx.net wrote:
I think the old system where the patch submitter declared, this message
contains my patch, is the only one that will work.
I tend to agree. That
On Sat, Feb 14, 2015 at 6:42 PM, Robert Haas robertmh...@gmail.com wrote:
On Sat, Feb 14, 2015 at 7:29 AM, Magnus Hagander mag...@hagander.net
wrote:
Ok, I've pushed an attempt at doing this.
For each mailthread, you can now create annotations. Each annotation is
connected to a mail in
On 2/14/15 7:30 AM, Magnus Hagander wrote:
On Mon, Feb 9, 2015 at 4:56 PM, Robert Haas robertmh...@gmail.com
Can we make it smarter, so that the kinds of things people produce
intending for them to be patches are thought by the CF app to be
patches?
Doing it wouldn't be too
On Sun, Feb 15, 2015 at 8:36 AM, Magnus Hagander mag...@hagander.net wrote:
Hmm. This kind of looks like the right idea, but it's hard to use,
because all you've got to work with is the subject of the message
(which is the same for all), the time it was sent, and the author. In
the old
On Sun, Feb 15, 2015 at 4:59 PM, Peter Eisentraut pete...@gmx.net wrote:
I think the old system where the patch submitter declared, this message
contains my patch, is the only one that will work.
I tend to agree. That being said, calling out latest attachments is
also useful (or highlighting
On Mon, Feb 9, 2015 at 4:56 PM, Robert Haas robertmh...@gmail.com wrote:
On Mon, Feb 9, 2015 at 5:38 AM, Magnus Hagander mag...@hagander.net
wrote:
On Mon, Feb 9, 2015 at 11:09 AM, Marco Nenciarini
marco.nenciar...@2ndquadrant.it wrote:
Il 08/02/15 17:04, Magnus Hagander ha scritto:
On Sat, Feb 7, 2015 at 1:59 PM, Magnus Hagander mag...@hagander.net wrote:
On Sat, Feb 7, 2015 at 1:53 PM, Tom Lane t...@sss.pgh.pa.us wrote:
Magnus Hagander mag...@hagander.net writes:
On Sat, Feb 7, 2015 at 1:02 AM, Jeff Janes jeff.ja...@gmail.com
wrote:
I'd like the ability to add a
On Sat, Feb 14, 2015 at 7:29 AM, Magnus Hagander mag...@hagander.net wrote:
Ok, I've pushed an attempt at doing this.
For each mailthread, you can now create annotations. Each annotation is
connected to a mail in the thread, and has a free text comment field. The
message will then
On 2/14/15 11:42 AM, Robert Haas wrote:
On Sat, Feb 14, 2015 at 7:29 AM, Magnus Hagander mag...@hagander.net wrote:
Ok, I've pushed an attempt at doing this.
For each mailthread, you can now create annotations. Each annotation is
connected to a mail in the thread, and has a free text comment
On Mon, Feb 9, 2015 at 5:38 AM, Magnus Hagander mag...@hagander.net wrote:
On Mon, Feb 9, 2015 at 11:09 AM, Marco Nenciarini
marco.nenciar...@2ndquadrant.it wrote:
Il 08/02/15 17:04, Magnus Hagander ha scritto:
Filenames are now shown for attachments, including a direct link to the
On Mon, Feb 9, 2015 at 11:09 AM, Marco Nenciarini
marco.nenciar...@2ndquadrant.it wrote:
Il 08/02/15 17:04, Magnus Hagander ha scritto:
Filenames are now shown for attachments, including a direct link to the
attachment itself. I've also run a job to populate all old threads.
I wonder
Il 08/02/15 17:04, Magnus Hagander ha scritto:
Filenames are now shown for attachments, including a direct link to the
attachment itself. I've also run a job to populate all old threads.
I wonder what is the algorithm to detect when an attachment is a patch.
If you look at
On Sat, Feb 7, 2015 at 1:46 PM, Magnus Hagander mag...@hagander.net wrote:
On Sat, Feb 7, 2015 at 1:02 AM, Jeff Janes jeff.ja...@gmail.com wrote:
One thing that would probably *help* is if the list of attachments
mentioned the names of the files that were attached to each message
rather
On Sun, Feb 8, 2015 at 11:04 AM, Magnus Hagander mag...@hagander.net wrote:
Filenames are now shown for attachments, including a direct link to the
attachment itself. I've also run a job to populate all old threads.
Thanks!
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The
Magnus Hagander mag...@hagander.net writes:
On Sat, Feb 7, 2015 at 1:02 AM, Jeff Janes jeff.ja...@gmail.com wrote:
I'd like the ability to add a comment which does not get turned into an
email.
I really don't ;)
The reason I really don't like that is that this now makes it impossible to
On Sat, Feb 7, 2015 at 1:53 PM, Tom Lane t...@sss.pgh.pa.us wrote:
Magnus Hagander mag...@hagander.net writes:
On Sat, Feb 7, 2015 at 1:02 AM, Jeff Janes jeff.ja...@gmail.com wrote:
I'd like the ability to add a comment which does not get turned into an
email.
I really don't ;)
The
On Sat, Feb 7, 2015 at 1:02 AM, Jeff Janes jeff.ja...@gmail.com wrote:
On Fri, Feb 6, 2015 at 5:20 AM, Robert Haas robertmh...@gmail.com wrote:
On Fri, Feb 6, 2015 at 5:51 AM, Magnus Hagander mag...@hagander.net
wrote:
So in an attempt to actually move this forward in a constructive way I'm
On Mon, Jan 19, 2015 at 10:53 PM, Magnus Hagander mag...@hagander.net
wrote:
On Mon, Jan 19, 2015 at 10:05 PM, Robert Haas robertmh...@gmail.com
wrote:
On Mon, Jan 19, 2015 at 3:57 PM, Magnus Hagander mag...@hagander.net
wrote:
The new site has been deployed and should now be usable.
On Mon, Jan 26, 2015 at 10:16 PM, Robert Haas robertmh...@gmail.com wrote:
On Mon, Jan 26, 2015 at 4:07 PM, Magnus Hagander mag...@hagander.net
wrote:
I assume what was referred to was that the old cf app would show the
last 3
(I think it was) comments/patches/whatnot on a patch on the
On Mon, Jan 26, 2015 at 9:46 PM, Robert Haas robertmh...@gmail.com wrote:
On Mon, Jan 26, 2015 at 3:24 PM, Magnus Hagander mag...@hagander.net
wrote:
Yes, and the agreement after that feedback was to try it out and then
figure
out what changes were needed? As about half the feedback said
On Mon, Jan 26, 2015 at 10:29 PM, Robert Haas robertmh...@gmail.com wrote:
(While I'm complaining, the links only go to the flat version of the
thread, while I happen to prefer the version that shows one message at
a time with a message-ID selector to switch messages.)
Then you're clicking
On 2015-02-06 11:51:50 +0100, Magnus Hagander wrote:
So in an attempt to actually move this forward in a constructive way I'm
going to ignore a bunch of what happened after this email, and fork the
discussion at this point.
Sounds good.
First of all - assuming we'lI fix this particular
On Fri, Feb 6, 2015 at 5:51 AM, Magnus Hagander mag...@hagander.net wrote:
So in an attempt to actually move this forward in a constructive way I'm
going to ignore a bunch of what happened after this email, and fork the
discussion at this point.
Thanks, and I probably owe you an apology for
On Fri, Feb 6, 2015 at 5:55 AM, Magnus Hagander mag...@hagander.net wrote:
On Mon, Jan 26, 2015 at 10:29 PM, Robert Haas robertmh...@gmail.com wrote:
(While I'm complaining, the links only go to the flat version of the
thread, while I happen to prefer the version that shows one message at
a
On Mon, Jan 26, 2015 at 3:13 PM, Peter Geoghegan p...@heroku.com wrote:
On Sun, Jan 25, 2015 at 3:22 AM, Andrew Gierth
and...@tao11.riddles.org.uk wrote:
There's a fairly serious readability problem when someone has posted a
patch as a subthread of some more general discussion. For example,
On Mon, Jan 26, 2015 at 9:20 PM, Robert Haas robertmh...@gmail.com wrote:
On Mon, Jan 26, 2015 at 3:13 PM, Peter Geoghegan p...@heroku.com wrote:
On Sun, Jan 25, 2015 at 3:22 AM, Andrew Gierth
and...@tao11.riddles.org.uk wrote:
There's a fairly serious readability problem when someone has
On Sun, Jan 25, 2015 at 3:22 AM, Andrew Gierth
and...@tao11.riddles.org.uk wrote:
There's a fairly serious readability problem when someone has posted a
patch as a subthread of some more general discussion. For example, look
at the adaptive ndistinct estimator patch: it's not obvious which
On Mon, Jan 26, 2015 at 4:16 PM, Josh Berkus j...@agliodbs.com wrote:
Well, if it's essentially unusable, then we've reached parity with the
old app (yes, you deserved that).
No, I didn't. What we had before I wrote that tool was a bunch of
wiki pages you put together which were forever having
On 01/26/2015 01:29 PM, Robert Haas wrote:
Furthermore,
if it sucked so bad, why did it take anyone 5 years to get around to
rewriting it? It took me less than a year to get around to replacing
what you wrote.
Because whoever replaced it knew they'd be facing a shitstorm of criticism?
--
On Mon, Jan 26, 2015 at 9:46 PM, Robert Haas robertmh...@gmail.com wrote:
On Mon, Jan 26, 2015 at 3:24 PM, Magnus Hagander mag...@hagander.net
wrote:
Yes, and the agreement after that feedback was to try it out and then
figure
out what changes were needed? As about half the feedback said
On 2015-01-26 13:32:51 -0800, Josh Berkus wrote:
On 01/26/2015 01:29 PM, Robert Haas wrote:
Furthermore,
if it sucked so bad, why did it take anyone 5 years to get around to
rewriting it? It took me less than a year to get around to replacing
what you wrote.
Because whoever replaced
On Mon, Jan 26, 2015 at 12:46 PM, Robert Haas robertmh...@gmail.com wrote:
Well, I can't speak to anyone else's opinion, but I'm quite sure I
raised the issue that we need a way to call out which messages in the
thread are important, and I think that's pretty much what Peter is
saying, too.
On Mon, Jan 26, 2015 at 4:07 PM, Magnus Hagander mag...@hagander.net wrote:
I assume what was referred to was that the old cf app would show the last 3
(I think it was) comments/patches/whatnot on a patch on the summary page
(and then clickthrough for more details).
Yep.
--
Robert Haas
On 01/26/2015 12:46 PM, Robert Haas wrote:
I find the new tool essentially unusable - having one
link to the whole thread instead of individual links to just the
important messages in that thread is a huge regression for me, as is
the lack of the most recent activity on the summary page.
On Mon, Jan 26, 2015 at 4:01 PM, Magnus Hagander mag...@hagander.net wrote:
According to my mailbox, you didn't even respond on that thread. But it may
well be that your email ended up on some other thread and therefor was not
included when I went back and looked over all the responses I got on
On 2015-01-26 12:54:04 -0800, Peter Geoghegan wrote:
On Mon, Jan 26, 2015 at 12:46 PM, Robert Haas robertmh...@gmail.com wrote:
Well, I can't speak to anyone else's opinion, but I'm quite sure I
raised the issue that we need a way to call out which messages in the
thread are important, and
On Mon, Jan 26, 2015 at 10:05 PM, Andres Freund and...@2ndquadrant.com
wrote:
On 2015-01-26 12:54:04 -0800, Peter Geoghegan wrote:
On Mon, Jan 26, 2015 at 12:46 PM, Robert Haas robertmh...@gmail.com
wrote:
Well, I can't speak to anyone else's opinion, but I'm quite sure I
raised the
On Mon, Jan 26, 2015 at 4:32 PM, Josh Berkus j...@agliodbs.com wrote:
On 01/26/2015 01:29 PM, Robert Haas wrote:
Furthermore,
if it sucked so bad, why did it take anyone 5 years to get around to
rewriting it? It took me less than a year to get around to replacing
what you wrote.
Because
On Mon, Jan 26, 2015 at 5:36 PM, Josh Berkus j...@agliodbs.com wrote:
In order to get a consensus on moving to a new app I had to explain what
was wrong with the old app. Eventually I had to use strong language to
do so, because nobody was paying attention otherwise. While Magnus's
app isn't
Robert,
Didn't stop me. And actually, I didn't face a shitstorm of criticism.
The way I remember it, I got a pretty much unqualified positive
reaction at the time.
Including from me, because it was a huge improvement on what we had
before. As the new app is.
Only later, when you had
On Mon, Jan 26, 2015 at 3:24 PM, Magnus Hagander mag...@hagander.net wrote:
Yes, and the agreement after that feedback was to try it out and then figure
out what changes were needed? As about half the feedback said it was better
without and half said it was better with.
Well, I can't speak to
On Sun, Jan 25, 2015 at 1:28 AM, Tomas Vondra tomas.von...@2ndquadrant.com
wrote:
Hi,
seems the CF app uses an invalid e-mail address when sending messages to
pgsql-hackers - I've added a comment to one of the patches and got this:
pgsql-hackers-testing@localhost
Unrouteable address
Is there any possibility of making it possible to annotate
particular messages (in particular, patch-related messages) with brief
comments? I would like to be able to highlight particular messages as
particular versions of the patch, and have it be apparent what
properties that version has at a
Peter == Peter Geoghegan p...@heroku.com writes:
Peter The mailing list integration is good, but seems like it could
Peter often be overkill. I just want to tag an existing message for
Peter readability here, like with the old commitfest app. I like to
Peter make things easy to find from the
Hi,
seems the CF app uses an invalid e-mail address when sending messages to
pgsql-hackers - I've added a comment to one of the patches and got this:
pgsql-hackers-testing@localhost
Unrouteable address
Maybe that's expected as the CF app is new, but I haven't seen it
mentioned in this
Hi,
Thanks for the excellent work on the new commitfest app. It looks
awesome so far, though I'm betting the commitfest manager is the one who
reaps the most benefits.
Wanted to highlight this request:
Andres Freund wrote:
What I'm also missing from the old app is that previously 'reviews'
Magnus,
Now that I'm back on this side of the Pacific, is there any additional
data entry/cleanup which needs doing?
--
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
On Thu, Jan 22, 2015 at 10:47 AM, Josh Berkus j...@agliodbs.com wrote:
Now that I'm back on this side of the Pacific, is there any additional
data entry/cleanup which needs doing?
An extra look would be worth it. Magnus or I may have missed patch
entries between the old and new apps.
My2c.
--
Michael Paquier michael.paqu...@gmail.com writes:
On Tue, Jan 20, 2015 at 4:31 PM, Andres Freund and...@2ndquadrant.com wrote:
I think you misunderstood me ;). I was talking about the old CF
application providing a RSS feed of all changes to all entries.
On Tue, Jan 20, 2015 at 3:43 PM, Tom Lane t...@sss.pgh.pa.us wrote:
Michael Paquier michael.paqu...@gmail.com writes:
On Tue, Jan 20, 2015 at 4:31 PM, Andres Freund and...@2ndquadrant.com
wrote:
I think you misunderstood me ;). I was talking about the old CF
application providing a RSS
2015-01-20 19:16 GMT+01:00 Pavel Stehule pavel.steh...@gmail.com:
Hi
I cannot to set my name as author for patch:
https://commitfest.postgresql.org/4/112/
It is solved now - I don't understand a autocomplete in first moment
All works well
Regards
Pavel
Regards
Pavel
2015-01-13
Hi
I cannot to set my name as author for patch:
https://commitfest.postgresql.org/4/112/
Regards
Pavel
2015-01-13 6:35 GMT+01:00 Magnus Hagander mag...@hagander.net:
Hi!
Last I said something about the new CF app I said I was planning to deploy
it over the holidays, and that clearly did
On Tue, Jan 20, 2015 at 3:43 PM, Tom Lane t...@sss.pgh.pa.us wrote:
Michael Paquier michael.paqu...@gmail.com writes:
On Tue, Jan 20, 2015 at 4:31 PM, Andres Freund and...@2ndquadrant.com
wrote:
I think you misunderstood me ;). I was talking about the old CF
application providing a RSS
Hi,
On 2015-01-19 21:57:20 +0100, Magnus Hagander wrote:
The new site has been deployed and should now be usable.
I think this unfortunately lost the RSS feature? I found that quite
useful to see who changed what recently (it's forwared to an imap
mailbox for me...).
What I'm also missing from
On Tue, Jan 20, 2015 at 6:54 AM, Magnus Hagander mag...@hagander.net wrote:
On Mon, Jan 19, 2015 at 10:10 PM, Robert Haas robertmh...@gmail.com wrote:
On Mon, Jan 19, 2015 at 4:05 PM, Robert Haas robertmh...@gmail.com
wrote:
On Mon, Jan 19, 2015 at 3:57 PM, Magnus Hagander
On Tue, Jan 20, 2015 at 10:09 AM, Andres Freund and...@2ndquadrant.com wrote:
Hi,
On 2015-01-19 21:57:20 +0100, Magnus Hagander wrote:
The new site has been deployed and should now be usable.
I think this unfortunately lost the RSS feature? I found that quite
useful to see who changed what
On Tue, Jan 20, 2015 at 4:31 PM, Andres Freund and...@2ndquadrant.com wrote:
On 2015-01-20 11:05:43 +0900, Michael Paquier wrote:
On Tue, Jan 20, 2015 at 10:09 AM, Andres Freund and...@2ndquadrant.com
wrote:
Hi,
On 2015-01-19 21:57:20 +0100, Magnus Hagander wrote:
The new site has been
On 2015-01-20 11:05:43 +0900, Michael Paquier wrote:
On Tue, Jan 20, 2015 at 10:09 AM, Andres Freund and...@2ndquadrant.com
wrote:
Hi,
On 2015-01-19 21:57:20 +0100, Magnus Hagander wrote:
The new site has been deployed and should now be usable.
I think this unfortunately lost the
On Mon, Jan 19, 2015 at 10:10 PM, Robert Haas robertmh...@gmail.com wrote:
On Mon, Jan 19, 2015 at 4:05 PM, Robert Haas robertmh...@gmail.com
wrote:
On Mon, Jan 19, 2015 at 3:57 PM, Magnus Hagander mag...@hagander.net
wrote:
The new site has been deployed and should now be usable.
On Mon, Jan 19, 2015 at 10:01 PM, Robert Haas robertmh...@gmail.com wrote:
On Mon, Jan 19, 2015 at 3:57 PM, Magnus Hagander mag...@hagander.net
wrote:
The new site has been deployed and should now be usable.
The old site is still available in readonly mode at
On Mon, Jan 19, 2015 at 4:05 PM, Robert Haas robertmh...@gmail.com wrote:
On Mon, Jan 19, 2015 at 3:57 PM, Magnus Hagander mag...@hagander.net wrote:
The new site has been deployed and should now be usable.
There are, for some reason, three copies of Clarify need for memory
barriers in
On Mon, Jan 19, 2015 at 3:57 PM, Magnus Hagander mag...@hagander.net wrote:
The new site has been deployed and should now be usable.
The old site is still available in readonly mode at
https://commitfest-old.postgresql.org/.
These links, which were originally requested by Tom and which I have
On Tue, Jan 13, 2015 at 2:35 PM, Magnus Hagander mag...@hagander.net wrote:
Further status updates to come as we start working on it...
Things are looking good so far. All the information has been synced up
between both apps for the current CF and the next one. One the switch
is done, I would
75 matches
Mail list logo