> Hi! I started reviewing it and noticed that your code repeated this > cycle maybe it would be better to put it in a separate function, for > example in the form of a name like "analyze_stmts"? > > or is it possible to create a macro for them?
Perhaps it may be better to stick this all in a macro, but I did it that way because we have similar pattern for queryId, with some repeating patterns for looping through a list of Query or PlannedStmt. As this patch already got committed, feel free to propose a patch for this improvement. -- Sami Imseih Amazon Web Services (AWS)