Euler Taveira de Oliveira wrote: > Alvaro Herrera escreveu: > > Okay, so I've changed things so that the transformRelOptions' caller is > > now in charge of passing an array of valid option namespaces. This is > > working A-OK. I'm now going to figure out appropriate pg_dump support > > and commit as soon as possible. > > > I don't like the spreading validnsps' approach. Isn't there a way to > centralize those variables in one place, i.e., reloption.h ? Also, remove an > obsolete comment about toast tables at reloption.h.
No, that doesn't work, because we don't know centrally what's the allowed list of namespaces. In fact that's precisely the problem: for example, there's no point in having a "toast" namespace for index reloptions. And for a user-defined access method, we don't know what the valid namespaces are. Of course, the easiest way is to just state that there are no valid namespaces other than NULL, and only allow "toast" for heap, but I think that's not thinking far enough ahead. The other option I considered was to have another AM entry point that returns the list of valid namespaces, but that seems to be way overkill, particularly considering that the current arrays are all NULL. -- Alvaro Herrera http://www.amazon.com/gp/registry/CTMLCN8V17R4 "La gente vulgar solo piensa en pasar el tiempo; el que tiene talento, en aprovecharlo" -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
