Neil Conway <[EMAIL PROTECTED]> writes: > I've got most of this finished; I'll post a patch soon. One issue I ran > into is how to handle query_tree_mutator() and query_tree_walker(): they > both expect to be able to traverse a Query's in_info_list, which my > patch moves into the QueryState struct. If maintaining this property is > essential, it seems that we'll need a way to get the QueryState > associated with a given Query.
whereupon the entire premise of a read-only tree collapses ... That's a bit nasty. I'm fairly sure that I added in_info_list to the walker recursion because I had to; I don't recall the exact scenario, but I think it needs to be possible to reassign relation numbers within that data structure if we are doing it elsewhere in a query tree. regards, tom lane ---------------------------(end of broadcast)--------------------------- TIP 3: if posting/reading through Usenet, please send an appropriate subscribe-nomail command to [EMAIL PROTECTED] so that your message can get through to the mailing list cleanly