From a620a760ed258a2004645ee87c9b98da71cf5266 Mon Sep 17 00:00:00 2001
From: Bruce Momjian <bruce@momjian.us>
Date: Sat, 10 Jan 2004 02:21:08 +0000
Subject: [PATCH] Allow BEGIN WORK to specify transaction isolation level, like
 START TRANSACTION.

---
 doc/src/sgml/ref/begin.sgml             | 17 ++++++++++++++++-
 doc/src/sgml/ref/start_transaction.sgml |  7 +++----
 src/backend/parser/gram.y               |  6 +++---
 src/bin/psql/tab-complete.c             | 22 +++++++++++++++++++---
 4 files changed, 41 insertions(+), 11 deletions(-)

diff --git a/doc/src/sgml/ref/begin.sgml b/doc/src/sgml/ref/begin.sgml
index 66fcda1776..03fcbf97f2 100644
--- a/doc/src/sgml/ref/begin.sgml
+++ b/doc/src/sgml/ref/begin.sgml
@@ -1,5 +1,5 @@
 <!--
-$PostgreSQL: pgsql/doc/src/sgml/ref/begin.sgml,v 1.27 2003/11/29 19:51:38 pgsql Exp $
+$PostgreSQL: pgsql/doc/src/sgml/ref/begin.sgml,v 1.28 2004/01/10 02:21:08 momjian Exp $
 PostgreSQL documentation
 -->
 
@@ -21,6 +21,8 @@ PostgreSQL documentation
  <refsynopsisdiv>
 <synopsis>
 BEGIN [ WORK | TRANSACTION ]
+    [ ISOLATION LEVEL { READ UNCOMMITTED | READ COMMITTED | REPEATABLE READ | SERIALIZABLE } ]
+    [ READ WRITE | READ ONLY ]
 </synopsis>
  </refsynopsisdiv>
  
@@ -49,6 +51,13 @@ BEGIN [ WORK | TRANSACTION ]
    other sessions will be unable to see the intermediate states
    wherein not all the related updates have been done.
   </para>
+
+  <para>
+   If the isolation level or read/write mode is specified, the new
+   transaction has those characteristics, as if
+   <xref linkend="sql-set-transaction" endterm="sql-set-transaction-title"> 
+   was executed.
+  </para>
  </refsect1>
   
  <refsect1>
@@ -65,6 +74,12 @@ BEGIN [ WORK | TRANSACTION ]
     </listitem>
    </varlistentry>
   </variablelist>
+
+  <para>
+   See under <xref linkend="sql-set-transaction"
+   endterm="sql-set-transaction-title"> about the meaning of the
+   other parameters.
+  </para>
  </refsect1>
   
  <refsect1>
diff --git a/doc/src/sgml/ref/start_transaction.sgml b/doc/src/sgml/ref/start_transaction.sgml
index 3ac66c883a..98881920ba 100644
--- a/doc/src/sgml/ref/start_transaction.sgml
+++ b/doc/src/sgml/ref/start_transaction.sgml
@@ -1,5 +1,5 @@
 <!--
-$PostgreSQL: pgsql/doc/src/sgml/ref/start_transaction.sgml,v 1.9 2003/11/29 19:51:39 pgsql Exp $
+$PostgreSQL: pgsql/doc/src/sgml/ref/start_transaction.sgml,v 1.10 2004/01/10 02:21:08 momjian Exp $
 PostgreSQL documentation
 -->
 
@@ -33,9 +33,8 @@ START TRANSACTION
    This command begins a new transaction. If the isolation level or
    read/write mode is specified, the new transaction has those
    characteristics, as if <xref linkend="sql-set-transaction"
-   endterm="sql-set-transaction-title"> was executed. In all other
-   respects, the behavior of this command is identical to the <xref
-   linkend="sql-begin" endterm="sql-begin-title"> command.
+   endterm="sql-set-transaction-title"> was executed. It is the same
+   as the <xref linkend="sql-begin" endterm="sql-begin-title"> command.
   </para>
  </refsect1>
 
diff --git a/src/backend/parser/gram.y b/src/backend/parser/gram.y
index fa7ba0ecde..cd8c092a45 100644
--- a/src/backend/parser/gram.y
+++ b/src/backend/parser/gram.y
@@ -11,7 +11,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/parser/gram.y,v 2.443 2004/01/07 18:56:27 neilc Exp $
+ *	  $PostgreSQL: pgsql/src/backend/parser/gram.y,v 2.444 2004/01/10 02:21:08 momjian Exp $
  *
  * HISTORY
  *	  AUTHOR			DATE			MAJOR EVENT
@@ -3674,11 +3674,11 @@ TransactionStmt:
 					n->options = NIL;
 					$$ = (Node *)n;
 				}
-			| BEGIN_P opt_transaction
+			| BEGIN_P opt_transaction transaction_mode_list_or_empty
 				{
 					TransactionStmt *n = makeNode(TransactionStmt);
 					n->kind = TRANS_STMT_BEGIN;
-					n->options = NIL;
+					n->options = $3;
 					$$ = (Node *)n;
 				}
 			| START TRANSACTION transaction_mode_list_or_empty
diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.c
index 955e70deb8..58ef84f9a5 100644
--- a/src/bin/psql/tab-complete.c
+++ b/src/bin/psql/tab-complete.c
@@ -3,7 +3,7 @@
  *
  * Copyright (c) 2000-2003, PostgreSQL Global Development Group
  *
- * $PostgreSQL: pgsql/src/bin/psql/tab-complete.c,v 1.97 2003/12/01 22:21:54 momjian Exp $
+ * $PostgreSQL: pgsql/src/bin/psql/tab-complete.c,v 1.98 2004/01/10 02:21:08 momjian Exp $
  */
 
 /*----------------------------------------------------------------------
@@ -723,6 +723,18 @@ psql_completion(char *text, int start, int end)
 	else if (strcasecmp(prev2_wd, "ANALYZE") == 0)
 		COMPLETE_WITH_CONST(";");
 
+/* BEGIN, COMMIT, ROLLBACK, ABORT, */
+	else if (strcasecmp(prev_wd, "BEGIN") == 0 ||
+	         strcasecmp(prev_wd, "END") == 0 ||
+	         strcasecmp(prev_wd, "COMMIT") == 0 ||
+	         strcasecmp(prev_wd, "ROLLBACK") == 0 ||
+	         strcasecmp(prev_wd, "ABORT") == 0)
+	{
+		static const char * const list_TRANS[] =
+		{"WORK", "TRANSACTION", NULL};
+
+		COMPLETE_WITH_LIST(list_TRANS);
+	}
 /* CLUSTER */
 	/* If the previous word is CLUSTER, produce list of indexes. */
 	else if (strcasecmp(prev_wd, "CLUSTER") == 0)
@@ -1099,10 +1111,14 @@ psql_completion(char *text, int start, int end)
 			 strcasecmp(prev_wd, "SHOW") == 0)
 		COMPLETE_WITH_LIST(pgsql_variables);
 	/* Complete "SET TRANSACTION" */
-	else if ((strcasecmp(prev2_wd, "SET") == 0
-			  && strcasecmp(prev_wd, "TRANSACTION") == 0)
+	else if ((strcasecmp(prev2_wd, "SET") == 0 &&
+			  strcasecmp(prev_wd, "TRANSACTION") == 0)
 			 || (strcasecmp(prev2_wd, "START") == 0
 				 && strcasecmp(prev_wd, "TRANSACTION") == 0)
+			 || (strcasecmp(prev2_wd, "BEGIN") == 0
+				 && strcasecmp(prev_wd, "WORK") == 0)
+			 || (strcasecmp(prev2_wd, "BEGIN") == 0
+				 && strcasecmp(prev_wd, "TRANSACTION") == 0)
 			 || (strcasecmp(prev4_wd, "SESSION") == 0
 				 && strcasecmp(prev3_wd, "CHARACTERISTICS") == 0
 				 && strcasecmp(prev2_wd, "AS") == 0
-- 
2.11.0