Unix Technical Forum

Re: [PATCHES] Proposed patch: synchronized_scanningGUC variable

This is a discussion on Re: [PATCHES] Proposed patch: synchronized_scanningGUC variable within the pgsql Hackers forums, part of the PostgreSQL category; --> Tom Lane wrote: > Bruce Momjian <bruce@momjian.us> writes: > > OK, but keep in mind if we use synchronized_seqscans ...


Go Back   Unix Technical Forum > Database Server Software > PostgreSQL > pgsql Hackers

FAQ Members List Calendar Search Today's Posts Mark Forums Read
  #1 (permalink)  
Old 04-15-2008, 11:42 PM
Bruce Momjian
 
Posts: n/a
Default Re: [PATCHES] Proposed patch: synchronized_scanningGUC variable

Tom Lane wrote:
> Bruce Momjian <bruce@momjian.us> writes:
> > OK, but keep in mind if we use synchronized_seqscans in pg_dump we will
> > have to recognize that GUC forever.

>
> No, because it's being used on the query side, not in the emitted dump.
> We have *never* promised that pg_dump version N could dump from server
> version N+1 .., in fact, personally I'd like to make that case be a hard
> error, rather than something people could override with -i.


Oh, yea, interesting, it is part of the connection. That will help.

--
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. +

---------------------------(end of broadcast)---------------------------
TIP 1: if posting/reading through Usenet, please send an appropriate
subscribe-nomail command to majordomo@postgresql.org so that your
message can get through to the mailing list cleanly

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #2 (permalink)  
Old 04-15-2008, 11:42 PM
Alvaro Herrera
 
Posts: n/a
Default Re: [PATCHES] Proposed patch: synchronized_scanning GUCvariable

> Tom Lane wrote:

> > in fact, personally I'd like to make that case be a hard error,
> > rather than something people could override with -i.


+1 to this idea. TODO for 8.4?

--
Alvaro Herrera http://www.CommandPrompt.com/
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #3 (permalink)  
Old 04-15-2008, 11:42 PM
Simon Riggs
 
Posts: n/a
Default Re: [PATCHES] Proposed patch: synchronized_scanning GUCvariable

On Thu, 2008-01-31 at 11:20 -0300, Alvaro Herrera wrote:
> > Tom Lane wrote:

>
> > > in fact, personally I'd like to make that case be a hard error,
> > > rather than something people could override with -i.

>
> +1 to this idea. TODO for 8.4?


-1 without some more planning about the effects and implications.

--
Simon Riggs
2ndQuadrant http://www.2ndQuadrant.com


---------------------------(end of broadcast)---------------------------
TIP 9: In versions below 8.0, the planner will ignore your desire to
choose an index scan if your joining column's datatypes do not
match

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #4 (permalink)  
Old 04-15-2008, 11:42 PM
Alvaro Herrera
 
Posts: n/a
Default Remove pg_dump -i option (was Re: Proposed patch:synchronized_scanning GUC variable)

Simon Riggs escribió:
> On Thu, 2008-01-31 at 11:20 -0300, Alvaro Herrera wrote:
> > > Tom Lane wrote:

> >
> > > > in fact, personally I'd like to make that case be a hard error,
> > > > rather than something people could override with -i.

> >
> > +1 to this idea. TODO for 8.4?

>
> -1 without some more planning about the effects and implications.


Effect: we would stop receiving complaints that an old pg_dump can talk
to a server that most likely is incompatible with it. People would
learn on the spot that they must install the newer pg_dump.

Implication: you cannot dump a newer database and expect it to load on
an older server (it would work for certain versions, but not all). So
if people have a bleeding-edge test server, they cannot migrate stuff
from it to their older production server.

--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.

---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #5 (permalink)  
Old 04-15-2008, 11:42 PM
Tom Lane
 
Posts: n/a
Default Re: Remove pg_dump -i option (was Re: Proposed patch: synchronized_scanning GUC variable)

Peter Eisentraut <peter_e@gmx.net> writes:
> Am Donnerstag, 31. Januar 2008 schrieb Alvaro Herrera:
>> Effect: we would stop receiving complaints that an old pg_dump can talk
>> to a server that most likely is incompatible with it. People would
>> learn on the spot that they must install the newer pg_dump.


> I think a more moderate measure might be to clarify the error message
> "aborting because of version mismatch (Use the -i option to proceed
> anyway.)\n"


I would be satisfied with that if I thought people would actually read
the message. My complaint is really directed at certain admin packages
(and they know who they are) that invoke pg_dump *by default*, behind
the user's back, with -i.

regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #6 (permalink)  
Old 04-15-2008, 11:42 PM
Peter Eisentraut
 
Posts: n/a
Default Re: Remove pg_dump -i option (was Re: Proposed patch: synchronized_scanning GUC variable)

Am Donnerstag, 31. Januar 2008 schrieb Alvaro Herrera:
> Effect: we would stop receiving complaints that an old pg_dump can talk
> to a server that most likely is incompatible with it. People would
> learn on the spot that they must install the newer pg_dump.


I think a more moderate measure might be to clarify the error message

"aborting because of version mismatch (Use the -i option to proceed
anyway.)\n"

I'm not sure how many of the mentioned complaints we are getting, but the
error message might make people think that it's complaining because the
versions are not equal rather than that the versions are known not
compatible.

--
Peter Eisentraut
http://developer.postgresql.org/~petere/

---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #7 (permalink)  
Old 04-15-2008, 11:42 PM
Bruce Momjian
 
Posts: n/a
Default Re: Remove pg_dump -i option (was Re: Proposed

Peter Eisentraut wrote:
> Am Donnerstag, 31. Januar 2008 schrieb Alvaro Herrera:
> > Effect: we would stop receiving complaints that an old pg_dump can talk
> > to a server that most likely is incompatible with it. People would
> > learn on the spot that they must install the newer pg_dump.

>
> I think a more moderate measure might be to clarify the error message
>
> "aborting because of version mismatch (Use the -i option to proceed
> anyway.)\n"
>
> I'm not sure how many of the mentioned complaints we are getting, but the
> error message might make people think that it's complaining because the
> versions are not equal rather than that the versions are known not
> compatible.


Agreed --- that error message is just prompting people to try something
they shouldn't and not explaining why.

--
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. +

---------------------------(end of broadcast)---------------------------
TIP 3: Have you checked our extensive FAQ?

http://www.postgresql.org/docs/faq

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #8 (permalink)  
Old 04-15-2008, 11:42 PM
Bruce Momjian
 
Posts: n/a
Default Re: Remove pg_dump -i option (was Re: Proposed

Tom Lane wrote:
> Peter Eisentraut <peter_e@gmx.net> writes:
> > Am Donnerstag, 31. Januar 2008 schrieb Alvaro Herrera:
> >> Effect: we would stop receiving complaints that an old pg_dump can talk
> >> to a server that most likely is incompatible with it. People would
> >> learn on the spot that they must install the newer pg_dump.

>
> > I think a more moderate measure might be to clarify the error message
> > "aborting because of version mismatch (Use the -i option to proceed
> > anyway.)\n"

>
> I would be satisfied with that if I thought people would actually read
> the message. My complaint is really directed at certain admin packages
> (and they know who they are) that invoke pg_dump *by default*, behind
> the user's back, with -i.


Oh? That isn't good.

--
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. +

---------------------------(end of broadcast)---------------------------
TIP 9: In versions below 8.0, the planner will ignore your desire to
choose an index scan if your joining column's datatypes do not
match

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
Reply


Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Trackbacks are On
Pingbacks are On
Refbacks are On
Forum Jump


All times are GMT. The time now is 11:35 PM.


Powered by vBulletin® Version 3.6.5
Copyright ©2000 - 2008, Jelsoft Enterprises Ltd.
SEO by vBSEO 3.2.0
www.UnixAdminTalk.com