Re: Vote needed: revert beta2 changes or not? - Mailing list pgsql-hackers

From Dave Page
Subject Re: Vote needed: revert beta2 changes or not?
Date
Msg-id E7F85A1B5FF8D44C8A1AF6885BC9A0E4CC3289@ratbert.vale-housing.co.uk
Whole thread Raw
In response to Vote needed: revert beta2 changes or not?  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Vote needed: revert beta2 changes or not?
List pgsql-hackers

> -----Original Message-----
> From: Robert Treat [mailto:xzilla@users.sourceforge.net]
> Sent: 07 October 2005 16:36
> To: pgsql-hackers@postgresql.org
> Cc: Dave Page; Tom Lane
> Subject: Re: [HACKERS] Vote needed: revert beta2 changes or not?
>
> On Friday 07 October 2005 03:50, Dave Page wrote:
> > > -----Original Message-----
> > > From: pgsql-hackers-owner@postgresql.org
> > > [mailto:pgsql-hackers-owner@postgresql.org] On Behalf Of Tom Lane
> > > Sent: 07 October 2005 02:28
> > > To: pgsql-hackers@postgresql.org
> > > Subject: [HACKERS] Vote needed: revert beta2 changes or not?
> > >
> > > 2.  Revert the result type of pg_cancel_backend() to int, but
> > > leave the
> > >     rest as-is (minimum change to avoid a compatibility break
> > > with 8.0).
> >
> > +1 (I do know people who will need to modify scripts because of this
> > change), though I'm obviously not going to win having
> already scanned
> > the entire thread :-)
>
> I'm sympathetic to this, but doesn't it seem worse to have
> this one function
> return int if all the others return boolean?

It's not pretty, but then how many other names might we change these
days because they don't fit in with current thinking?

>  Also they
> don't need to modify
> scripts, can't they just write thier own pg_cacnel_backend to
> return int
> based on the boolean version?

No, because you can't overload based purely on return type. I suppose
they could write it to take an int8 pid or something, but that's a hack.

Regards, Dave.


pgsql-hackers by date:

Previous
From: Robert Treat
Date:
Subject: Re: Vote needed: revert beta2 changes or not?
Next
From: Tom Lane
Date:
Subject: Re: Vote needed: revert beta2 changes or not?