From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Stephen Frost <sfrost(at)snowman(dot)net> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] Regression tests vs existing users in an installation |
Date: | 2019-06-28 18:42:49 |
Message-ID: | 3606.1561747369@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
OK, here's a completed patch to add checking for naming-rule violations.
I updated regress.sgml to clarify the naming rules (and failed to
resist the temptation to update a lot of other somewhat-obsolete
statements there, too).
Also worth noting is that I added an IsReservedName check to
pg_replication_origin_create(), ie it will no longer allow
user-selected origin names starting with "pg_". This seems
like a good idea considering that we generate internal origin
names that look like that. I have not done anything equivalent
for subscription names, but should we consider doing so?
I touched the TAP tests only to the extent necessary to make them pass
cleanly --- mostly I had to fool with pg_dump/t/010_dump_connstr.pl
because it whines if the restore step emits any warnings. Perhaps
there's another way to address that with less invasive changes to that
test script; but I couldn't think of one other than ignoring warnings,
which didn't seem like a great idea.
This patch doesn't address the issue that we think rolenames.sql is
unsafe to run in "make installcheck" mode. That seems like a separable
problem, and we'd have to adjust that script as shown here anyway.
Barring objections, I'll push this shortly (to HEAD only) and turn on
the enabling switch on one or two of my buildfarm critters.
regards, tom lane
Attachment | Content-Type | Size |
---|---|---|
0001-code-and-doc-fixes.patch | text/x-diff | 15.8 KB |
0002-clean-up-test-cases.patch | text/x-diff | 48.2 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Geoghegan | 2019-06-28 18:46:14 | Re: Feature improvement: can we add queryId for pg_catalog.pg_stat_activity view? |
Previous Message | Luis Carril | 2019-06-28 17:55:52 | Re: Option to dump foreign data in pg_dump |