Re: DROP COLUMN Progress

> OK, my guess is that it is checks in parser/.  I would issue each of
> these queries with a non-existant column name, find the error message in
> the code, and add an isdropped check in those places.

OK, I think I've narrowed it down to this block of code in scanRTEForColumn
in parse_relation.c:
       /*        * Scan the user column names (or aliases) for a match. Complain if        * multiple matches.        */       foreach(c, rte->eref->colnames)       {               /* @@ SKIP DROPPED HERE? @@ */               attnum++;               if (strcmp(strVal(lfirst(c)), colname) == 0)               {                       if (result)                               elog(ERROR, "Column reference \"%s\" is
ambiguous", colname);                       result = (Node *) make_var(pstate, rte, attnum);                       rte->checkForRead = true;               }       }


I'm thinking that I should put a 'SearchSysCacheCopy' where my @@ comment is
to retrieve the attribute by name, and then do a check to see if it's
dropped.  Is that the best/fastest way of doing things?  Seems unfortunate
to add a another cache lookup in every parsed query...

Comments?

Chris