Alias to rollback keyword
Hey,
After dealing with a production fault and having to rollback all the time, I
kept typing a different word instead of rollback. So I created a patch to
accept this word as an alias for rollback. Obviously it's not part of the sql
standard, but could be a nice extension for postgresql. See the patch for
more details.
Cheers
Matt
Attachments:
pgsql_rollback.patchtext/x-patch; charset=ISO-8859-1; name=pgsql_rollback.patchDownload
diff -U5 -r postgresql-8.4.2.orig/src/include/parser/kwlist.h postgresql-8.4.2/src/include/parser/kwlist.h
--- postgresql-8.4.2.orig/src/include/parser/kwlist.h 2009-04-06 18:12:53.000000000 +0930
+++ postgresql-8.4.2/src/include/parser/kwlist.h 2010-04-01 10:55:46.000000000 +1030
@@ -55,10 +55,11 @@
PG_KEYWORD("begin", BEGIN_P, UNRESERVED_KEYWORD)
PG_KEYWORD("between", BETWEEN, TYPE_FUNC_NAME_KEYWORD)
PG_KEYWORD("bigint", BIGINT, COL_NAME_KEYWORD)
PG_KEYWORD("binary", BINARY, TYPE_FUNC_NAME_KEYWORD)
PG_KEYWORD("bit", BIT, COL_NAME_KEYWORD)
+PG_KEYWORD("bollocks", ROLLBACK, UNRESERVED_KEYWORD)
PG_KEYWORD("boolean", BOOLEAN_P, COL_NAME_KEYWORD)
PG_KEYWORD("both", BOTH, RESERVED_KEYWORD)
PG_KEYWORD("by", BY, UNRESERVED_KEYWORD)
PG_KEYWORD("cache", CACHE, UNRESERVED_KEYWORD)
PG_KEYWORD("called", CALLED, UNRESERVED_KEYWORD)
On Mar 31, 2010, at 5:42 PM, Matthew Altus wrote:
After dealing with a production fault and having to rollback all the time, I
kept typing a different word instead of rollback. So I created a patch to
accept this word as an alias for rollback. Obviously it's not part of the sql
standard, but could be a nice extension for postgresql. See the patch for
more details.
+100
Best,
David
Matthew Altus wrote:
Hey,
After dealing with a production fault and having to rollback all the time, I
kept typing a different word instead of rollback. So I created a patch to
accept this word as an alias for rollback. Obviously it's not part of the sql
standard, but could be a nice extension for postgresql. See the patch for
more details.
Applied.
--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com
On Wed, Mar 31, 2010 at 10:34 PM, Bruce Momjian <bruce@momjian.us> wrote:
Matthew Altus wrote:
Hey,
After dealing with a production fault and having to rollback all the time, I
kept typing a different word instead of rollback. So I created a patch to
accept this word as an alias for rollback. Obviously it's not part of the sql
standard, but could be a nice extension for postgresql. See the patch for
more details.Applied.
You know, I was about to write a rant about this, until I realized
what day it will be soon (or already is, UTC).
...Robert
Robert Haas wrote:
On Wed, Mar 31, 2010 at 10:34 PM, Bruce Momjian <bruce@momjian.us> wrote:
Matthew Altus wrote:
Hey,
After dealing with a production fault and having to rollback all the time, I
kept typing a different word instead of rollback. ?So I created a patch to
accept this word as an alias for rollback. ?Obviously it's not part of the sql
standard, but could be a nice extension for postgresql. ?See the patch for
more details.Applied.
You know, I was about to write a rant about this, until I realized
what day it will be soon (or already is, UTC).
CVS is GMT-based.
--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com
On Wed, Mar 31, 2010 at 10:34:41PM -0400, Bruce Momjian wrote:
Matthew Altus wrote:
Hey,
After dealing with a production fault and having to rollback all the time, I
kept typing a different word instead of rollback. So I created a patch to
accept this word as an alias for rollback. Obviously it's not part of the sql
standard, but could be a nice extension for postgresql. See the patch for
more details.Applied.
Hmm, a careful review of the patch leads me to believe that this is one
of the few times that a keyword might benefit from localization. Seems
l10n efforts in this area have mostly focused on the filtering case, but
I'm sure we can repurose such lists. Licensing might be an issue.
Clearly this needs to be controlled by the client locale, not the
server. Any need for a guc?
Ross
--
Ross Reedstrom, Ph.D. reedstrm@rice.edu
Systems Engineer & Admin, Research Scientist phone: 713-348-6166
The Connexions Project http://cnx.org fax: 713-348-3665
Rice University MS-375, Houston, TX 77005
GPG Key fingerprint = F023 82C8 9B0E 2CC6 0D8E F888 D3AE 810E 88F0 BEDE