Re: TABLESAMPLE patch

Hi, I took a look on this and found nice.

By the way, the parameter for REPEATABLE seems allowing to be a
expression in ParseTableSample but the grammer rejects it.

The following change seems enough.

diff --git a/src/backend/parser/gram.y b/src/backend/parser/gram.y
index 4578b5e..8cf09d5 100644
--- a/src/backend/parser/gram.y
+++ b/src/backend/parser/gram.y
@@ -10590,7 +10590,7 @@ tablesample_clause:        ;opt_repeatable_clause:
-            REPEATABLE '(' AexprConst ')'    { $$ = (Node *) $3; }
+            REPEATABLE '(' a_expr ')'    { $$ = (Node *) $3; }            | /*EMPTY*/                        { $$ =
NULL;}        ;
 


regards,

-- 
Kyotaro Horiguchi
NTT Open Source Software Center