ALTER VIEW

From: Dave Page <dpage(at)vale-housing(dot)co(dot)uk>
To: "'jm(dot)poure(at)freesurf(dot)fr'" <jm(dot)poure(at)freesurf(dot)fr>, pgadmin-hackers(at)postgresql(dot)org
Subject: ALTER VIEW
Date: 2002-02-22 22:06:52
Message-ID: FED2B709E3270E4B903EB0175A49BCB10475E9@dogbert.vale-housing.co.uk
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Jean-Michel,

I just took a look at the latest ALTER VIEW code. Sorry, but I have a number
of problems with this version:

1) Don't create the view twice, create the fake one then rename it - it's
less expensive and avoids the potential for the second create to fail.

2) The szOldDefintion stuff can now be removed.

3) The OID and Definition could be retrieved in one operation rather than 2.

4) A class library such as this should _never_ display any user interface.
That is the job of the application.

5) The modifications to the SQL property are messy & mainly unnecessary
(certainly with #1 above - that just needs bDrop & szViewName).

6) The iLogEvent "Entering..." code should be immediately after the On Error
code, not in the middle of the function.

7) The Comment cache is not being invalidated.

Regards, Dave.

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Jean-Michel POURE 2002-02-22 22:11:21 Re: ALTER VIEW
Previous Message Jean-Michel POURE 2002-02-22 22:06:36 Triggers