pgAdmin III commit: Fix CREATE INDEX/CONSTRAINT syntax

From: Guillaume Lelarge <guillaume(at)lelarge(dot)info>
To: pgadmin-hackers(at)postgresql(dot)org
Subject: pgAdmin III commit: Fix CREATE INDEX/CONSTRAINT syntax
Date: 2010-10-06 19:14:38
Message-ID: 201010061914.o96JEc89084146@gothos.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Fix CREATE INDEX/CONSTRAINT syntax

Reverse the order of TABLESPACE and WITH() clauses for indexes. Previously
pgAdminIII generated CREATE scripts like:

CREATE INDEX foo_foo_id ON foo USING btree (foo_id)
TABLESPACE bar WITH (FILLFACTOR=50);

However this is illegal PostgreSQL syntax, the WITH() clause must come before
TABLESPACE.

The statement generated by "new index" dialog is already correct.

Branch
------
REL-1_12_0_PATCHES

Details
-------
http://git.postgresql.org/gitweb?p=pgadmin3.git;a=commitdiff;h=c102486cfdc5976e786526583f45208f20ec5cbb

Modified Files
--------------
pgadmin/schema/pgIndex.cpp | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)

Browse pgadmin-hackers by date

  From Date Subject
Next Message Guillaume Lelarge 2010-10-06 19:18:39 Re: [PATCH] Fix CREATE INDEX/CONSTRAINT syntax when tablespace and fillfactor are specified
Previous Message Guillaume Lelarge 2010-10-06 18:10:49 pgAdmin website commit: Automatic stringmerge using merge script.