From: | Mark Dilger <hornschnorter(at)gmail(dot)com> |
---|---|
To: | Surafel Temesgen <surafel3000(at)gmail(dot)com> |
Cc: | Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: FETCH FIRST clause PERCENT option |
Date: | 2018-09-25 15:54:48 |
Message-ID: | BAE042C9-6825-403E-AF07-1CFC8294DAFD@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> On Sep 25, 2018, at 8:08 AM, Mark Dilger <hornschnorter(at)gmail(dot)com> wrote:
>
>
>
>> On Sep 25, 2018, at 5:07 AM, Surafel Temesgen <surafel3000(at)gmail(dot)com> wrote:
>>
>> hey
>>
>> On 9/21/18, Mark Dilger <hornschnorter(at)gmail(dot)com> wrote:
>>
>>> Surafel, there are no regression tests that I can see in your patch. It
>>> would help if you added some, as then I could precisely what behavior you
>>> are expecting.
>> thank you for looking at it .the attach patch add regression tests
>> <fetch-wth-percent-v4.patch>
>
> Surafel,
>
> I messed around with your changes to the grammar and it seems you don't
> need to add PERCENT as a reserved keyword. Moving this to the unreserved
> keyword section does not cause any shift/reduce errors, and the regression
> tests still pass. Relative to your patch v4, these changes help:
I spoke too soon. The main regression tests pass, but your change to
src/test/modules/test_ddl_deparse/sql/create_table.sql per Thomas's
suggestion is no longer needed, since PERCENT no longer needs to be
quoted.
I recommend you also apply the following to your v4 patch, which just
rolls back that one change you made, and at least for me, is enough
to get `make check-world` to pass:
diff --git a/src/test/modules/test_ddl_deparse/sql/create_table.sql b/src/test/modules/test_ddl_deparse/sql/create_table.sql
index 4325de2d04..5e78452729 100644
--- a/src/test/modules/test_ddl_deparse/sql/create_table.sql
+++ b/src/test/modules/test_ddl_deparse/sql/create_table.sql
@@ -94,7 +94,7 @@ CREATE TABLE student (
) INHERITS (person);
CREATE TABLE stud_emp (
- "percent" int4
+ percent int4
) INHERITS (emp, student);
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2018-09-25 16:05:42 | Re: Allowing printf("%m") only where it actually works |
Previous Message | Alvaro Herrera | 2018-09-25 15:48:29 | Re: Allowing printf("%m") only where it actually works |