[hibernate-commits] Hibernate SVN: r11381 - branches/Branch_3_2/Hibernate3/src/org/hibernate/dialect.

[hibernate-commits at lists.jboss.org]

Author: steve.ebersole at jboss.com
Date: 2007-04-02 11:20:55 -0400 (Mon, 02 Apr 2007)
New Revision: 11381

Modified:
   branches/Branch_3_2/Hibernate3/src/org/hibernate/dialect/Cache71Dialect.java
Log:
HHH-2535 : CacheSQL does not really support sequences; yes it does; no it doesnt; ...

Modified: branches/Branch_3_2/Hibernate3/src/org/hibernate/dialect/Cache71Dialect.java
===================================================================
--- branches/Branch_3_2/Hibernate3/src/org/hibernate/dialect/Cache71Dialect.java	2007-03-30 23:45:23 UTC (rev 11380)
+++ branches/Branch_3_2/Hibernate3/src/org/hibernate/dialect/Cache71Dialect.java	2007-04-02 15:20:55 UTC (rev 11381)
@@ -491,29 +491,32 @@
 	// SEQUENCE support ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
 	public boolean supportsSequences() {
-		return true;
+		return false;
 	}
 
-	public String getSequenceNextValString(String sequenceName) {
-		return "select InterSystems.Sequences_GetNext('" + sequenceName + "') from InterSystems.Sequences where ucase(name)=ucase('" + sequenceName + "')";
-	}
+// It really does support sequences, but InterSystems elects to suggest usage of IDENTITY instead :/
+// Anyway, below are the actual support overrides for users wanting to use this combo...
+//
+//	public String getSequenceNextValString(String sequenceName) {
+//		return "select InterSystems.Sequences_GetNext('" + sequenceName + "') from InterSystems.Sequences where ucase(name)=ucase('" + sequenceName + "')";
+//	}
+//
+//	public String getSelectSequenceNextValString(String sequenceName) {
+//		return "(select InterSystems.Sequences_GetNext('" + sequenceName + "') from InterSystems.Sequences where ucase(name)=ucase('" + sequenceName + "'))";
+//	}
+//
+//	public String getCreateSequenceString(String sequenceName) {
+//		return "insert into InterSystems.Sequences(Name) values (ucase('" + sequenceName + "'))";
+//	}
+//
+//	public String getDropSequenceString(String sequenceName) {
+//		return "delete from InterSystems.Sequences where ucase(name)=ucase('" + sequenceName + "')";
+//	}
+//
+//	public String getQuerySequencesString() {
+//		return "select name from InterSystems.Sequences";
+//	}
 
-	public String getSelectSequenceNextValString(String sequenceName) {
-		return "(select InterSystems.Sequences_GetNext('" + sequenceName + "') from InterSystems.Sequences where ucase(name)=ucase('" + sequenceName + "'))";
-	}
-
-	public String getCreateSequenceString(String sequenceName) {
-		return "insert into InterSystems.Sequences(Name) values (ucase('" + sequenceName + "'))";
-	}
-
-	public String getDropSequenceString(String sequenceName) {
-		return "delete from InterSystems.Sequences where ucase(name)=ucase('" + sequenceName + "')";
-	}
-
-	public String getQuerySequencesString() {
-		return "select name from InterSystems.Sequences";
-	}
-
 	// lock acquisition support ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
 	public boolean supportsForUpdate() {