Hi, I'm reviewing OPEN FOR EXECUTE USING patch and have a couple of
Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> wrote:
> this small patch add missing USING clause to OPEN FOR EXECUTE statement
> + cleaning part of exec_stmt_open function
- Can we use read_sql_expression2() instead of read_sql_construct()
in gram.y? It could simplify the code a bit.
- I'd prefer to change the new argument for exec_dynquery_with_params()
from "char *portalname" to "const char *curname".
Other than the above minor issues, this patch is ready to commit.
NTT Open Source Software Center
In response to
pgsql-hackers by date
|Next:||From: Takahiro Itagaki||Date: 2010-01-12 10:10:09|
|Subject: Re: NOT NULL violation and error-message|
|Previous:||From: Pavel Stehule||Date: 2010-01-12 09:34:54|
|Subject: Re: Typed tables|