Hackers,
I'm looking at the grammar modifications for the 2PC patch, and I am
wondering if we should leave PREPARE TRANSACTION in the same parser/
UtilityStmt node TransactionStmt or should use a different parser node,
say PrepTransactionStmt.
Using a different parser node seems to leave a gram.c file that's 1.5 kb
smaller. However it means we need to create new copyfuncs and
equalfuncs functions; and to double a couple of checks:
(IsA(TransactionStmt, node) ||
IsA(PrepTransactionStmt, node))
However it doesn't seem to me this is too big a drawback.
What do people think? Is this important at all?
--
Alvaro Herrera (<alvherre[a]surnet.cl>)
"On the other flipper, one wrong move and we're Fatal Exceptions"
(T.U.X.: Term Unit X - http://www.thelinuxreview.com/TUX/)
