Re: TABLESAMPLE patch

Поиск
Список
Период
Сортировка
От Kyotaro HORIGUCHI
Тема Re: TABLESAMPLE patch
Дата
Msg-id 20150128.162331.13401375.horiguchi.kyotaro@lab.ntt.co.jp
обсуждение исходный текст
Ответ на Re: TABLESAMPLE patch  (Petr Jelinek <petr@2ndquadrant.com>)
Ответы Re: TABLESAMPLE patch  (Petr Jelinek <petr@2ndquadrant.com>)
Список pgsql-hackers
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




В списке pgsql-hackers по дате отправления:

Предыдущее
От: Pavel Stehule
Дата:
Сообщение: Re: proposal: row_to_array function
Следующее
От: Pavel Stehule
Дата:
Сообщение: Re: proposal: row_to_array function