Re: [HACKERS] unfixed bugs with extensions

2012-11-02 Thread Dimitri Fontaine
Dimitri Fontaine  writes:
> Do you want me to polish it? I'd just need some review or hints to fix
> it the way you want.

Oh, catching-up on -hackers before doing the same thing in -commiters is
not the best approach, sorry about that.

Regards,
-- 
Dimitri Fontaine
http://2ndQuadrant.fr PostgreSQL : Expertise, Formation et Support


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] unfixed bugs with extensions

2012-11-02 Thread Dimitri Fontaine
Alvaro Herrera  writes:
> We have a couple of unfixed bugs regarding extensions, for which patches
> have been proposed but remain unfixed in git.
>
> The oldest is bug #6704, for which a proposed fix for the master branch
> was posted here: 
> http://archives.postgresql.org/message-id/m2zk4e6p7m@2ndquadrant.fr
> There was some disagreement about the proper way forward, so I hadn't
> looked at this patch.  I just did, and find that even if the approach
> taken by the patch is the correct one, it needs some polish before it
> can be committed.

Do you want me to polish it? I'd just need some review or hints to fix
it the way you want.

> The other one was reported by Marko Kreen in

Has been fixed by Tom.

Regards,
-- 
Dimitri Fontaine
http://2ndQuadrant.fr PostgreSQL : Expertise, Formation et Support


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] unfixed bugs with extensions

2012-10-25 Thread Tom Lane
Alvaro Herrera  writes:
> The other one was reported by Marko Kreen in
> http://archives.postgresql.org/message-id/cacmqxcjjauc9jpa64vxskrn67byjuymodz-mgy-_aoz6erg...@mail.gmail.com
> (the thread talks about 2 bugs, but one of them is just pilot error).
> Dimitri posted patches to the real bug for the three affected branches;
> the last one (for master) was in message
> http://archives.postgresql.org/message-id/m2391yy6ub@2ndquadrant.fr
> I eyeballed this patch and it seemed to me to fix the problem, but I am
> unfamiliar enough with this dark corner of pg_dump that I can't be sure
> that there aren't bad behaviors being introduced inadvertently.

Yeah, I think that one's on my plate.

regards, tom lane


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


[HACKERS] unfixed bugs with extensions

2012-10-25 Thread Alvaro Herrera
We have a couple of unfixed bugs regarding extensions, for which patches
have been proposed but remain unfixed in git.

The oldest is bug #6704, for which a proposed fix for the master branch
was posted here: 
http://archives.postgresql.org/message-id/m2zk4e6p7m@2ndquadrant.fr
There was some disagreement about the proper way forward, so I hadn't
looked at this patch.  I just did, and find that even if the approach
taken by the patch is the correct one, it needs some polish before it
can be committed.

The other one was reported by Marko Kreen in
http://archives.postgresql.org/message-id/cacmqxcjjauc9jpa64vxskrn67byjuymodz-mgy-_aoz6erg...@mail.gmail.com
(the thread talks about 2 bugs, but one of them is just pilot error).
Dimitri posted patches to the real bug for the three affected branches;
the last one (for master) was in message
http://archives.postgresql.org/message-id/m2391yy6ub@2ndquadrant.fr
I eyeballed this patch and it seemed to me to fix the problem, but I am
unfamiliar enough with this dark corner of pg_dump that I can't be sure
that there aren't bad behaviors being introduced inadvertently.

-- 
Álvaro Herrerahttp://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers