Re: [PATCHES] refactor preprocess_targetlist

2005-03-17 Thread Neil Conway
Neil Conway wrote: This patch moves some code for preprocessing FOR UPDATE from grouping_planner() to preprocess_targetlist(), according to a comment in grouping_planner(). I think the refactoring makes sense, and moves some extraneous details out of grouping_planner(). Applied. -Neil

[PATCHES] refactor preprocess_targetlist

2005-03-16 Thread Neil Conway
This patch moves some code for preprocessing FOR UPDATE from grouping_planner() to preprocess_targetlist(), according to a comment in grouping_planner(). I think the refactoring makes sense, and moves some extraneous details out of grouping_planner(). Barring any objections, I'll apply this to