db-derby-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From d..@apache.org
Subject svn commit: r630150 - /db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/execute/CreateRoleConstantAction.java
Date Fri, 22 Feb 2008 10:58:18 GMT
Author: dag
Date: Fri Feb 22 02:58:15 2008
New Revision: 630150

URL: http://svn.apache.org/viewvc?rev=630150&view=rev
Log:
DERBY-3173 SQL roles: add catalog support

Bug fix: Moved transaction start up to before the first dictionary access.

Modified:
    db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/execute/CreateRoleConstantAction.java

Modified: db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/execute/CreateRoleConstantAction.java
URL: http://svn.apache.org/viewvc/db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/execute/CreateRoleConstantAction.java?rev=630150&r1=630149&r2=630150&view=diff
==============================================================================
--- db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/execute/CreateRoleConstantAction.java
(original)
+++ db/derby/code/trunk/java/engine/org/apache/derby/impl/sql/execute/CreateRoleConstantAction.java
Fri Feb 22 02:58:15 2008
@@ -81,6 +81,8 @@
         //
         final String currentAuthId = lcc.getAuthorizationId();
 
+        dd.startWriting(lcc);
+
         //
         // Check if this role already exists. If it does, throw.
         //
@@ -105,7 +107,6 @@
             true,         // with admin option
             true);        // is definition
 
-        dd.startWriting(lcc);
         dd.addDescriptor(rd,
                          null,  // parent
                          DataDictionary.SYSROLES_CATALOG_NUM,



Mime
View raw message