Re: CREATE IF NOT EXISTS INDEX - Mailing list pgsql-hackers

From Fabrízio de Royes Mello
Subject Re: CREATE IF NOT EXISTS INDEX
Date
Msg-id CAFcNs+pnAc4MwGapbBVQ1bqJhVn17gjfCd2sPH2OoWmbYqrWHA@mail.gmail.com
Whole thread Raw
In response to Re: CREATE IF NOT EXISTS INDEX  (Josh Berkus <josh@agliodbs.com>)
List pgsql-hackers


On Tue, Sep 30, 2014 at 8:47 PM, Josh Berkus <josh@agliodbs.com> wrote:
>
> On 09/30/2014 04:16 PM, Andres Freund wrote:
> > On 2014-09-30 16:03:01 -0700, Josh Berkus wrote:
> >> On 09/30/2014 03:53 PM, Andres Freund wrote:
> >>> Good point. I think it's fair enough to only allow CINE on named
> >>> indexes.
> >>
> >> On the other hand, the way we form system-generated names is predicable,
> >> so I think it would be perfectly OK to include them.  Desirable, in fact.
> >
> > It's not really that predicable. Think about expression indexes. They
> > also don't contain information about opclasses et all.
> >
> > Seems like pit of hairy semantics.
> >
> >> Not that I'm a fan of omitting the name ...
> >
> > Me neither.
>
> I'd be OK with a CINE which required you to name the index.
>
> How does your patch work at present, Fabrizio?
>

My patch will work just if you name the index, because postgres generates a index name that doesn't exists.

I don't check to a name if we use IF NOT EXISTS, but I can add this check.

Regards,

--
Fabrízio de Royes Mello
Consultoria/Coaching PostgreSQL
>> Timbira: http://www.timbira.com.br
>> Blog: http://fabriziomello.github.io
>> Linkedin: http://br.linkedin.com/in/fabriziomello
>> Twitter: http://twitter.com/fabriziomello
>> Github: http://github.com/fabriziomello

pgsql-hackers by date:

Previous
From: Josh Berkus
Date:
Subject: Re: CREATE IF NOT EXISTS INDEX
Next
From: Josh Berkus
Date:
Subject: Re: CREATE IF NOT EXISTS INDEX