Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-11-13 Thread Boszormenyi Zoltan
Alvaro Herrera írta: > Boszormenyi Zoltan escribió: > >> Alvaro Herrera írta: >> >>> I have applied this patch after some tinkering. I mainly added support >>> for "fetch_args: FORWARD opt_from_in name" and "BACKWARD opt_from_in >>> name" in ecpg.addons which apparently you forgot. >>>

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-11-13 Thread Robert Haas
2009/11/13 Hans-Jürgen Schönig : > > On Nov 13, 2009, at 8:06 AM, Michael Meskes wrote: > >> On Thu, Nov 12, 2009 at 03:07:27PM -0500, Robert Haas wrote: If you want to submit patches in a series like this one, they need to be considered standalone, I think.  The Linux kernel devs wo

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-11-13 Thread Hans-Jürgen Schönig
On Nov 13, 2009, at 8:06 AM, Michael Meskes wrote: On Thu, Nov 12, 2009 at 03:07:27PM -0500, Robert Haas wrote: If you want to submit patches in a series like this one, they need to be considered standalone, I think. The Linux kernel devs work differently than us here. Zoltan broke them

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-11-12 Thread Michael Meskes
On Thu, Nov 12, 2009 at 03:07:27PM -0500, Robert Haas wrote: > > If you want to submit patches in a series like this one, they need to be > > considered standalone, I think.  The Linux kernel devs work differently > > than us here. > > Zoltan broke them up because Michael asked him to do so. Actu

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-11-12 Thread Robert Haas
On Thu, Nov 12, 2009 at 2:49 PM, Alvaro Herrera wrote: > Boszormenyi Zoltan escribió: >> Alvaro Herrera írta: > >> > I have applied this patch after some tinkering.  I mainly added support >> > for "fetch_args: FORWARD opt_from_in name" and "BACKWARD opt_from_in >> > name" in ecpg.addons which app

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-11-12 Thread Alvaro Herrera
Boszormenyi Zoltan escribió: > Alvaro Herrera írta: > > I have applied this patch after some tinkering. I mainly added support > > for "fetch_args: FORWARD opt_from_in name" and "BACKWARD opt_from_in > > name" in ecpg.addons which apparently you forgot. > > Thanks. Your fix is correct if this pa

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-11-12 Thread Boszormenyi Zoltan
Alvaro Herrera írta: > Boszormenyi Zoltan escribió: > > >> 2) 1b-cursor_name-ctxdiff.patch >> >> "name" -> "cursor_name" transition in core grammar >> and ecpg grammar. Currently it does nothing, it's a >> preparation phase. Depends on patch 1. >> > > Applied this part too. > > I cannot app

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-11-12 Thread Boszormenyi Zoltan
Alvaro Herrera írta: > Boszormenyi Zoltan escribió: > > >> I have split up (and cleaned up a little) the dynamic >> cursorname patch into smaller logical, easier-to-review >> pieces. Descriptions below. >> >> 1) 1a-unified-optfromin-fetch-ctxdiff.patch >> >> ecpg supports optional FROM/IN in FET

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-11-11 Thread Alvaro Herrera
Boszormenyi Zoltan escribió: > 2) 1b-cursor_name-ctxdiff.patch > > "name" -> "cursor_name" transition in core grammar > and ecpg grammar. Currently it does nothing, it's a > preparation phase. Depends on patch 1. Applied this part too. I cannot apply the other ones -- they belong to the ECPG ma

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-11-11 Thread Alvaro Herrera
Tom Lane escribió: > Alvaro Herrera writes: > > I have applied this patch after some tinkering. > > Shouldn't there be a docs change in that commit? True -- fixed. -- Alvaro Herrerahttp://www.CommandPrompt.com/ The PostgreSQL Company - Command Prompt, Inc. --

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-11-11 Thread Tom Lane
Alvaro Herrera writes: > I have applied this patch after some tinkering. Shouldn't there be a docs change in that commit? regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-11-11 Thread Alvaro Herrera
Boszormenyi Zoltan escribió: > I have split up (and cleaned up a little) the dynamic > cursorname patch into smaller logical, easier-to-review > pieces. Descriptions below. > > 1) 1a-unified-optfromin-fetch-ctxdiff.patch > > ecpg supports optional FROM/IN in FETCH and > MOVE statements (mainly b

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-15 Thread Robert Haas
On Thu, Oct 15, 2009 at 2:27 AM, Heikki Linnakangas wrote: > Robert Haas wrote: >> On Wed, Sep 30, 2009 at 12:24 PM, Heikki Linnakangas >> wrote:  Do you have any sense of how soon you'll feel confident to commit either patch? >>> I'm bad at estimating. Not this week for sure, and

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-14 Thread Heikki Linnakangas
Robert Haas wrote: > On Wed, Sep 30, 2009 at 12:24 PM, Heikki Linnakangas > wrote: >>> Do you >>> have any sense of how soon you'll feel confident to commit either >>> patch? >> I'm bad at estimating. Not this week for sure, and next week I'm >> traveling and won't be able to spend as much time o

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-14 Thread Robert Haas
2009/10/14 KaiGai Kohei : > Robert Haas wrote: >> On Wed, Sep 30, 2009 at 12:24 PM, Heikki Linnakangas >> wrote:  Do you have any sense of how soon you'll feel confident to commit either patch? >>> I'm bad at estimating. Not this week for sure, and next week I'm >>> traveling and wo

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-14 Thread KaiGai Kohei
Robert Haas wrote: > On Wed, Sep 30, 2009 at 12:24 PM, Heikki Linnakangas > wrote: >>> Do you >>> have any sense of how soon you'll feel confident to commit either >>> patch? >> I'm bad at estimating. Not this week for sure, and next week I'm >> traveling and won't be able to spend as much time o

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-14 Thread Robert Haas
On Wed, Sep 30, 2009 at 12:24 PM, Heikki Linnakangas wrote: >>  Do you >> have any sense of how soon you'll feel confident to commit either >> patch? > > I'm bad at estimating. Not this week for sure, and next week I'm > traveling and won't be able to spend as much time on it as I am right > now.

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-03 Thread Joe Conway
Itagaki Takahiro wrote: > The point is *memory leak* in dblink when a query is canceled or > become time-out. I think it is a bug, and my patch could fix it. Please see if this works for you. Joe Index: dblink.c === RCS file: /opt/s

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-03 Thread Robert Haas
On Thu, Oct 1, 2009 at 7:48 AM, Magnus Hagander wrote: > On Thu, Oct 1, 2009 at 04:11, Itagaki Takahiro > wrote: >> >> Magnus Hagander wrote: >> >>> I can certainly review the win32 encoding patch, but I was rather >>> hoping for some comments from others on if we're interested in a win32 >>> on

ECPG dynamic cursorname patch revised and split Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-02 Thread Boszormenyi Zoltan
Robert Haas írta: > On Fri, Oct 2, 2009 at 9:01 PM, Boszormenyi Zoltan wrote: > >> Hi, >> >> Michael Meskes írta: >> >>> It is accepted either way. I was just pointing out that it might be easier >>> to >>> review/commit at least parts of your patches if they can be applied >>> seperatel

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-02 Thread Robert Haas
On Fri, Oct 2, 2009 at 9:01 PM, Boszormenyi Zoltan wrote: > Hi, > > Michael Meskes írta: >> It is accepted either way. I was just pointing out that it might be easier to >> review/commit at least parts of your patches if they can be applied >> seperately. >> > > I have split up (and cleaned up a

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-02 Thread Boszormenyi Zoltan
Michael Meskes írta: > On Thu, Oct 01, 2009 at 09:05:55PM +0200, Boszormenyi Zoltan wrote: > >> Yes, but technical problems and solutions do. ECPG claims >> to be ESQL/C compatible, but at places it's only half compatible. >> > > Where does it claim to be fully compatible? > I didn't sa

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-02 Thread Michael Meskes
On Thu, Oct 01, 2009 at 09:05:55PM +0200, Boszormenyi Zoltan wrote: > Yes, but technical problems and solutions do. ECPG claims > to be ESQL/C compatible, but at places it's only half compatible. Where does it claim to be fully compatible? > This comment is misleading and reflects quite a narrow

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-01 Thread Boszormenyi Zoltan
Michael Meskes írta: > On Thu, Oct 01, 2009 at 07:21:54PM +0200, Boszormenyi Zoltan wrote: > >> Please, accept my apologies, I only tried to express my >> frustration, this is not a good situation for either of us. >> > > Apologies accepted, email is a difficult means of communication anywa

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-01 Thread Michael Meskes
On Thu, Oct 01, 2009 at 07:21:54PM +0200, Boszormenyi Zoltan wrote: > Please, accept my apologies, I only tried to express my > frustration, this is not a good situation for either of us. Apologies accepted, email is a difficult means of communication anyway. It leads to misunderstanding IMO. > Y

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-01 Thread Boszormenyi Zoltan
Michael Meskes írta: > On Thu, Oct 01, 2009 at 03:47:07PM +0200, Boszormenyi Zoltan wrote: > >> You're not being fair with me. The dependencies are quite >> technical. >> > > I'm sorry that you interpreted my email this way, it wasn't at all meant to > offend you. > Please, accept my ap

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-01 Thread Michael Meskes
On Thu, Oct 01, 2009 at 03:47:07PM +0200, Boszormenyi Zoltan wrote: > You're not being fair with me. The dependencies are quite > technical. I'm sorry that you interpreted my email this way, it wasn't at all meant to offend you. > First, Tom Lane suggested to unify core and ecpg FETCH > syntaxes

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-01 Thread Alvaro Herrera
Boszormenyi Zoltan escribió: > First, Tom Lane suggested to unify core and ecpg FETCH > syntaxes so both will accept optional FROM/IN, which I did. > SQLDA support adds new FETCH forms (Informix-specific > ones) so naturally these patches clash. There's no simple way > to make they separately appl

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-01 Thread Boszormenyi Zoltan
Michael Meskes írta: > On Wed, Sep 30, 2009 at 12:34:23PM -0400, Tom Lane wrote: > >> qualified to review them. (I don't actually think we have anybody >> except Michael who's really familiar with ecpg.) >> > > I'm afraid I'm simply not able to spend much time on this in the near future >

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-01 Thread Michael Meskes
On Wed, Sep 30, 2009 at 12:34:23PM -0400, Tom Lane wrote: > qualified to review them. (I don't actually think we have anybody > except Michael who's really familiar with ecpg.) I'm afraid I'm simply not able to spend much time on this in the near future as I'm simply too busy atm. I spend some ti

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-10-01 Thread Magnus Hagander
On Thu, Oct 1, 2009 at 04:11, Itagaki Takahiro wrote: > > Magnus Hagander wrote: > >> I can certainly review the win32 encoding patch, but I was rather >> hoping for some comments from others on if we're interested in a win32 >> only solution, or if we want something more generic. Should we just

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Itagaki Takahiro
Magnus Hagander wrote: > I can certainly review the win32 encoding patch, but I was rather > hoping for some comments from others on if we're interested in a win32 > only solution, or if we want something more generic. Should we just go > with the win32-only one for now? Yes, because Windows is

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Itagaki Takahiro
Joe Conway wrote: > The patch basically forces all use of libpq by dblink to be asynchronous > (internally) so that a cancel can be sensed and passed down to the > remote side and everything cleaned up. Possibly the right thing to do, > but dblink already allows the use of async queries, and the

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Tom Lane
Joe Conway writes: > The issue is not so much technical as it is philosophical. > The patch basically forces all use of libpq by dblink to be asynchronous > (internally) so that a cancel can be sensed and passed down to the > remote side and everything cleaned up. Possibly the right thing to do,

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Magnus Hagander
On Wed, Sep 30, 2009 at 21:38, Tom Lane wrote: > Magnus Hagander writes: >> I can certainly review the win32 encoding patch, but I was rather >> hoping for some comments from others on if we're interested in a win32 >> only solution, or if we want something more generic. Should we just go >> with

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Tom Lane
Magnus Hagander writes: > I can certainly review the win32 encoding patch, but I was rather > hoping for some comments from others on if we're interested in a win32 > only solution, or if we want something more generic. Should we just go > with the win32-only one for now? That was actually the on

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Alvaro Herrera
Magnus Hagander escribió: > On Wed, Sep 30, 2009 at 18:34, Tom Lane wrote: > > Robert Haas writes: > >> ... (and many of the more > >> significant remaining patches look like they are right up Tom's alley > >> anyway). > > > > FWIW, if left to my own devices I will eventually get to everything >

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Magnus Hagander
On Wed, Sep 30, 2009 at 18:34, Tom Lane wrote: > Robert Haas writes: >> ... (and many of the more >> significant remaining patches look like they are right up Tom's alley >> anyway). > > FWIW, if left to my own devices I will eventually get to everything > except the dblink, ecpg, and encoding/wi

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Joe Conway
Robert Haas wrote: > On Wed, Sep 30, 2009 at 12:27 PM, Tom Lane wrote: >> Joe Conway writes: >>> Robert Haas wrote: - There is one dblink pach left over from last CommitFest. Joe Conway was going to review it the weekend of July 18th-19th, but that didn't end up happening and so t

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Andrew Dunstan
Robert Haas wrote: On Wed, Sep 30, 2009 at 12:34 PM, Tom Lane wrote: Robert Haas writes: ... (and many of the more significant remaining patches look like they are right up Tom's alley anyway). FWIW, if left to my own devices I will eventually get to everything except the db

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Robert Haas
On Wed, Sep 30, 2009 at 12:34 PM, Tom Lane wrote: > Robert Haas writes: >> ... (and many of the more >> significant remaining patches look like they are right up Tom's alley >> anyway). > > FWIW, if left to my own devices I will eventually get to everything > except the dblink, ecpg, and encoding

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Robert Haas
On Wed, Sep 30, 2009 at 12:27 PM, Tom Lane wrote: > Joe Conway writes: >> Robert Haas wrote: >>> - There is one dblink pach left over from last CommitFest.  Joe Conway >>> was going to review it the weekend of July 18th-19th, but that didn't >>> end up happening and so that patch is still waiting

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Tom Lane
Robert Haas writes: > ... (and many of the more > significant remaining patches look like they are right up Tom's alley > anyway). FWIW, if left to my own devices I will eventually get to everything except the dblink, ecpg, and encoding/win32 patches. I don't intend to touch any of those because

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Tom Lane
Joe Conway writes: > Robert Haas wrote: >> - There is one dblink pach left over from last CommitFest. Joe Conway >> was going to review it the weekend of July 18th-19th, but that didn't >> end up happening and so that patch is still waiting. We might be able >> to find someone else to review it,

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Heikki Linnakangas
Robert Haas wrote: > On Wed, Sep 30, 2009 at 11:36 AM, Tom Lane wrote: >> Heikki Linnakangas writes: >>> OTOH, I'd hate to hold the commitfest hostage for that. Perhaps it >>> should be returned to author at this point, I should move on to other >>> patches to get the commitfest closed ASAP, and

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Joe Conway
Robert Haas wrote: > - There is one dblink pach left over from last CommitFest. Joe Conway > was going to review it the weekend of July 18th-19th, but that didn't > end up happening and so that patch is still waiting. We might be able > to find someone else to review it, but I'm not sure whether

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Robert Haas
On Wed, Sep 30, 2009 at 11:36 AM, Tom Lane wrote: > Heikki Linnakangas writes: >> OTOH, I'd hate to hold the commitfest hostage for that. Perhaps it >> should be returned to author at this point, I should move on to other >> patches to get the commitfest closed ASAP, and continue reviewing and >>

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Tom Lane
Heikki Linnakangas writes: > OTOH, I'd hate to hold the commitfest hostage for that. Perhaps it > should be returned to author at this point, I should move on to other > patches to get the commitfest closed ASAP, and continue reviewing and > polishing that right after the commitfest. FWIW, I'd ra

Re: [HACKERS] CommitFest 2009-09, two weeks on

2009-09-30 Thread Heikki Linnakangas
Robert Haas wrote: > - Hot Standby and Streaming Replication are both huge new features in > this CommitFest, and there seems to be a fair amount of activity > around both patches. Heikki previously expressed optimism about > getting Hot Standby done this CommitFest, but I am not sure whether he >