Joe Conway <m...@joeconway.com> writes: > On 06/14/2010 10:58 AM, Tom Lane wrote: >> The current effective behavior of the code is that the column numbers >> are physical numbers. Should we document it that way, or change it?
> Probably it should be changed to deal with dropped columns correctly, > but I won't have time to look at this closely until the end of the month > -- is that soon enough? Actually, I was working on it myself. On further reflection I think that logical numbers are clearly the right thing --- if we define it as being physical numbers then we will have headaches in the future when/if we support rearranging columns. However, there is some small chance of breaking things in existing DBs if we back-patch that change. Thoughts? It strikes me also that the code is not nearly careful enough about defending itself against garbage input in the primary_key_attnums argument ... regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers