Lists: | pgsql-hackers |
---|
From: | Haribabu kommi <haribabu(dot)kommi(at)huawei(dot)com> |
---|---|
To: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Regress tests to improve the function coverage of schemacmds and user and tablespace files |
Date: | 2013-10-24 08:59:47 |
Message-ID: | 8977CB36860C5843884E0A18D8747B0372BCA170@szxeml558-mbs.china.huawei.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
Here I have added some more regression tests to improve the missing function coverage of
schemacmds.c, user.c and tablespace.c.
The added tests are mainly RENAME TO and OWNER TO support.
patches are attached in the mail.
please check and provide your suggestions.
Regards,
Hari babu.
Attachment | Content-Type | Size |
---|---|---|
schema_user_v1.patch | application/octet-stream | 5.9 KB |
tablespace_v1.patch | application/octet-stream | 2.0 KB |
From: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
---|---|
To: | Haribabu kommi <haribabu(dot)kommi(at)huawei(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Regress tests to improve the function coverage of schemacmds and user and tablespace files |
Date: | 2013-10-25 05:31:39 |
Message-ID: | CAB7nPqQg43g1inYbPgSHZZr0YFhiSGJ-6ppR-Xanh-SEBVTjDQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
Hi,
On Thu, Oct 24, 2013 at 5:59 PM, Haribabu kommi
<haribabu(dot)kommi(at)huawei(dot)com> wrote:
> Here I have added some more regression tests to improve the missing function
> coverage of schemacmds.c, user.c and tablespace.c.
> The added tests are mainly RENAME TO and OWNER TO support.
Could you add those patches to the next commitfest such as they don't
get lost in the flow?
Here is a URL to it:
https://commitfest.postgresql.org/action/commitfest_view?id=20
Note that you will need a community account to register your patches.
Thanks,
--
Michael
From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
---|---|
To: | Haribabu kommi <haribabu(dot)kommi(at)huawei(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Regress tests to improve the function coverage of schemacmds and user and tablespace files |
Date: | 2013-11-23 11:31:42 |
Message-ID: | CAApHDvqMHNU6yB4hpW4nfQ3nTUWuWTEQtm4cAqFPUjr1o9d_QQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Thu, Oct 24, 2013 at 9:59 PM, Haribabu kommi
<haribabu(dot)kommi(at)huawei(dot)com>wrote:
>
>
> Here I have added some more regression tests to improve the missing
> function coverage of
>
> schemacmds.c, user.c and tablespace.c.
>
>
>
> The added tests are mainly RENAME TO and OWNER TO support.
>
>
I've just had a look at both of these patches. All tests that have been
added seem to cover new areas that are not previously tested, they also
seem to cleanup properly after themselves, so I think these should be a
worthwhile addition to the regression tests.
The only thing I can pickup on which is at fault is the the trailing white
space
src/test/regress/sql/privileges.sql:837: trailing whitespace.
+\c -
But I can't imagine it's worth submitting a new patch to fix it.
I've marked the patch as ready for commiter
Regards
David Rowley
>
>
> patches are attached in the mail.
>
> please check and provide your suggestions.
>
>
>
> Regards,
>
> Hari babu.
>
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
>
>
From: | Kevin Grittner <kgrittn(at)ymail(dot)com> |
---|---|
To: | David Rowley <dgrowleyml(at)gmail(dot)com>, Haribabu kommi <haribabu(dot)kommi(at)huawei(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Regress tests to improve the function coverage of schemacmds and user and tablespace files |
Date: | 2013-11-23 17:37:12 |
Message-ID: | 1385228232.54991.YahooMailNeo@web162901.mail.bf1.yahoo.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
David Rowley <dgrowleyml(at)gmail(dot)com> wrote:
> I've just had a look at both of these patches. All tests that
> have been added seem to cover new areas that are not previously
> tested, they also seem to cleanup properly after themselves, so I
> think these should be a worthwhile addition to the regression
> tests.
Thanks for reviewing! Did you happen to note the impact on `make
check` runtime? There are many people who run that many times per
day while working on development, so we try to keep new tests that
significantly extend that separate. We haven't quite worked out
the best way to exercise such longer-running tests, but I suspect
we soon will. At any rate, this is a piece of information the
committer will want, so you will be helping whoever that is if you
can supply it.
--
Kevin Grittner
EDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
---|---|
To: | Kevin Grittner <kgrittn(at)ymail(dot)com> |
Cc: | Haribabu kommi <haribabu(dot)kommi(at)huawei(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Regress tests to improve the function coverage of schemacmds and user and tablespace files |
Date: | 2013-11-23 21:34:05 |
Message-ID: | CAApHDvqhxwPyvf5nbkt4zAxPSETWLTf=b4oq3t5bvKL5COgweg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On Sun, Nov 24, 2013 at 6:37 AM, Kevin Grittner <kgrittn(at)ymail(dot)com> wrote:
> David Rowley <dgrowleyml(at)gmail(dot)com> wrote:
>
> > I've just had a look at both of these patches. All tests that
> > have been added seem to cover new areas that are not previously
> > tested, they also seem to cleanup properly after themselves, so I
> > think these should be a worthwhile addition to the regression
> > tests.
>
> Thanks for reviewing! Did you happen to note the impact on `make
> check` runtime? There are many people who run that many times per
> day while working on development, so we try to keep new tests that
> significantly extend that separate. We haven't quite worked out
> the best way to exercise such longer-running tests, but I suspect
> we soon will. At any rate, this is a piece of information the
> committer will want, so you will be helping whoever that is if you
> can supply it.
>
>
I've done a quick benchmark on this this morning.
Note that I'm using windows here and I used powershell to time the
regression run with the following command:
PS D:\Postgres\b\src\tools\msvc> Measure-Command { .\vcregress.bat check }
I ran the tests 10 times each.
I ran the patched version first, then just did git reset --hard to revert
the patched changes then I ran the tests again.
The average and median results over the 10 runs are as follows:
Patched Unpatched Time increased by
Average 48.23265888 47.70979854 101.10%
Median 47.8993686 47.51177815 100.82%
The slowdown is not too bad. It just around 1% increase of time.
I've attached the results in spreadsheet format.
Regards
David Rowley
--
> Kevin Grittner
> EDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
Attachment | Content-Type | Size |
---|---|---|
regression_test_benchmark.xlsx | application/vnd.openxmlformats-officedocument.spreadsheetml.sheet | 10.7 KB |
From: | Haribabu kommi <haribabu(dot)kommi(at)huawei(dot)com> |
---|---|
To: | David Rowley <dgrowleyml(at)gmail(dot)com> |
Cc: | Kevin Grittner <kgrittn(at)ymail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Regress tests to improve the function coverage of schemacmds and user and tablespace files |
Date: | 2013-11-26 13:27:08 |
Message-ID: | 8977CB36860C5843884E0A18D8747B0372BEFF86@szxeml558-mbs.china.huawei.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Lists: | pgsql-hackers |
On 24 November 2013 03:04 David Rowley wrote:
>I've done a quick benchmark on this this morning.
>Note that I'm using windows here and I used powershell to time the regression run with the following command:
>
>PS D:\Postgres\b\src\tools\msvc> Measure-Command { .\vcregress.bat check }
>
>I ran the tests 10 times each.
>I ran the patched version first, then just did git reset --hard to revert the patched changes then I ran the tests again.
>
>The average and median results over the 10 runs are as follows:
>
> Patched Unpatched Time increased by
>Average 48.23265888 47.70979854 101.10%
>Median 47.8993686 47.51177815 100.82%
>
>
>The slowdown is not too bad. It just around 1% increase of time.
>
>I've attached the results in spreadsheet format.
Thanks for the review and benchmark test.
Regards,
Hari babu.