Index: cpa/src/main/java/org/castor/cpa/persistence/sql/keygen/SequenceBeforeKeyGenerator.java
===================================================================
--- cpa/src/main/java/org/castor/cpa/persistence/sql/keygen/SequenceBeforeKeyGenerator.java	(revision 8360)
+++ cpa/src/main/java/org/castor/cpa/persistence/sql/keygen/SequenceBeforeKeyGenerator.java	(working copy)
@@ -28,6 +28,7 @@
 import org.exolab.castor.jdo.PersistenceException;
 import org.castor.core.util.Messages;
 import org.castor.cpa.persistence.sql.driver.DB2Factory;
+import org.castor.cpa.persistence.sql.driver.SapDbFactory;
 import org.castor.cpa.persistence.sql.driver.InterbaseFactory;
 import org.castor.cpa.persistence.sql.driver.PostgreSQLFactory;
 import org.castor.cpa.persistence.sql.driver.OracleFactory;
@@ -248,6 +249,31 @@
         }
     }
     
+    /**
+     * Implements SequenceKeyGenValueHandler that generates sql query for fetching
+     * identity from sap/max database.  
+     */
+    private class SapDbType extends SequenceKeyGenValueHandler {
+        
+        /**
+         * Generates sql select query for fetching identity and then calss the
+         * base class getValue method of query execution.
+         * 
+         * @param conn An open JDBC connection.
+         * @param tableName Name of the table from which identity will be fetched.
+         * @param primKeyName Primary key of the table.
+         * @param props database engine specific properties. 
+         * @return ResutlSet containing identity.
+         * @throws Exception If database error occurs. 
+         */
+        protected Object getValue(final Connection conn, final String tableName,
+                final String primKeyName, final Properties props) throws Exception {
+            return getValue("SELECT "
+                    + _factory.quoteName(getSeqName(tableName, primKeyName) + ".nextval")
+                    + " FROM DUAL", conn);
+        }
+    }
+    
     //-----------------------------------------------------------------------------------
     
     /** The <a href="http://jakarta.apache.org/commons/logging/">Jakarta
@@ -341,6 +367,8 @@
             _type = new PostgresqlType();
         } else if (OracleFactory.FACTORY_NAME.equals(factoryName)) {
             _type = new OracleType();
+        } else if (SapDbFactory.FACTORY_NAME.equals(factoryName)) {
+            _type = new SapDbType();
         } else {
             _type = new DefaultType();
         }
Index: src/doc/release-notes.xml
===================================================================
--- src/doc/release-notes.xml	(revision 8362)
+++ src/doc/release-notes.xml	(working copy)
@@ -91,6 +91,26 @@
 </class>]]></code-panel>
             </summary>
             <bugs>
+                <bug id="2724" tool="jira">
+                    <description>
+                        Sequence key generator on MaxDB generates wrong command.
+                    </description>
+                    <contributor>
+                        <name>Ahmad Hassan</name>
+                        <email>ahmad.hassan@gmail.com</email>
+                    </contributor>
+                    <submitter>
+                        <name>Ralf Joachim</name>
+                        <email>ralf.joachim@syscon.eu</email>
+                    </submitter>
+                    <reporter>
+                        <name>Ahmad Hassan</name>
+                        <email>ahmad.hassan@gmail.com</email>
+                    </reporter>
+                    <type>Bug</type>
+                    <module>JDO</module>
+                    <date>20090815</date>
+                </bug>
                 <bug id="2799" tool="jira">
                     <description>
                         Improve SQLStatementInsertCheck to incorporate all the exception handling