Patch applied. Thanks.
---
Gavin Sherry wrote:
> Here is a change to the alter_table regression test to use ALTER INDEX ...
> RENAME instead of ALTER TABLE ... RENAME in the case of an index renaming.
> Its probably not th
Shouldn't the regression tests reflect that?
Robert Treat
On Tue, 2004-08-24 at 22:31, Gavin Sherry wrote:
> Yes
>
> On Tue, 24 Aug 2004, Robert Treat wrote:
>
> > I think I missed this in the earlier patches, but does the alter table syntax
> > still work for indexes in addition to the new syn
Yes
On Tue, 24 Aug 2004, Robert Treat wrote:
> I think I missed this in the earlier patches, but does the alter table syntax
> still work for indexes in addition to the new syntax?
>
> On Saturday 21 August 2004 22:47, Gavin Sherry wrote:
> > Here is a change to the alter_table regression test to
I think I missed this in the earlier patches, but does the alter table syntax
still work for indexes in addition to the new syntax?
On Saturday 21 August 2004 22:47, Gavin Sherry wrote:
> Here is a change to the alter_table regression test to use ALTER INDEX ...
> RENAME instead of ALTER TABLE .
I have a whole separate CVS checkout for every path I'm working on -
makes life a lot easier :)
Dammit - I meant 'patch'.
Chris
---(end of broadcast)---
TIP 9: the planner will ignore your desire to choose an index scan if your
joining column's
Thanks for spotting this sloppy work. Basically, I cropped my diff because
there was lots of unrelated code in the work space. But I was a little too
hard and culled allfiles.sgml, alter.c, equal/copyfuncs.c and analyze.c.
I was wondering if something like that might have happened --- I know
you k
Here is a change to the alter_table regression test to use ALTER INDEX ...
RENAME instead of ALTER TABLE ... RENAME in the case of an index renaming.
Its probably not the right place for it, but I'm not sure where is the
right place for it.
Thanks,
Gavin
On Sat, 21 Aug 2004, Tom Lane wrote:
> G
Gavin Sherry <[EMAIL PROTECTED]> writes:
> Thanks for spotting this sloppy work. Basically, I cropped my diff because
> there was lots of unrelated code in the work space. But I was a little too
> hard and culled allfiles.sgml, alter.c, equal/copyfuncs.c and analyze.c.
I was wondering if something
Umm... oops.
On Sat, 21 Aug 2004, Tom Lane wrote:
> [EMAIL PROTECTED] (Bruce Momjian) writes:
> > Add ALTER INDEX, particularly for moving tablespaces.
>
> This patch is a perfect example of why unreviewed patches should not
> go in during beta.
>
> So far I have noticed the following problems wi
"Marc G. Fournier" <[EMAIL PROTECTED]> writes:
> Considering point 3 seems a wee bit critical, at least to me, should the
> patch be removed and postponed for a non-Beta period?
No, the fixes are all trivial. I'm just chewing out Bruce and Gavin
for sloppiness ;-)
regard
On Sat, 21 Aug 2004, Bruce Momjian wrote:
Tom Lane wrote:
[EMAIL PROTECTED] (Bruce Momjian) writes:
Add ALTER INDEX, particularly for moving tablespaces.
This patch is a perfect example of why unreviewed patches should not
go in during beta.
So far I have noticed the following problems with it:
* A
Tom Lane wrote:
> [EMAIL PROTECTED] (Bruce Momjian) writes:
> > Add ALTER INDEX, particularly for moving tablespaces.
>
> This patch is a perfect example of why unreviewed patches should not
> go in during beta.
>
> So far I have noticed the following problems with it:
>
> * Added reference page
[EMAIL PROTECTED] (Bruce Momjian) writes:
> Add ALTER INDEX, particularly for moving tablespaces.
This patch is a perfect example of why unreviewed patches should not
go in during beta.
So far I have noticed the following problems with it:
* Added reference page wasn't linked into the docs build
Log Message:
---
Add ALTER INDEX, particularly for moving tablespaces.
Gavin Sherry
Modified Files:
--
pgsql-server/src/backend/parser:
gram.y (r2.471 -> r2.472)
(http://developer.postgresql.org/cvsweb.cgi/pgsql-server/src/backend/parser/gram.y.diff?r1=2.4
14 matches
Mail list logo
