Author: arminw
Date: Tue Nov  7 05:33:23 2006
New Revision: 472103

URL: http://svn.apache.org/viewvc?view=rev&rev=472103
Log:
change result set count behavior. Seems latest driver support JDBC 2 correct

Modified:
    
db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/platforms/PlatformSapdbImpl.java

Modified: 
db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/platforms/PlatformSapdbImpl.java
URL: 
http://svn.apache.org/viewvc/db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/platforms/PlatformSapdbImpl.java?view=diff&rev=472103&r1=472102&r2=472103
==============================================================================
--- 
db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/platforms/PlatformSapdbImpl.java
 (original)
+++ 
db/ojb/branches/OJB_1_0_RELEASE/src/java/org/apache/ojb/broker/platforms/PlatformSapdbImpl.java
 Tue Nov  7 05:33:23 2006
@@ -172,12 +172,17 @@
 
     /**
      * Override default ResultSet size determination (rs.last();rs.getRow())
-     * with select count(*) operation
-     * SAP db doesn't let you use the .last, .getRow() mechanism (.getRow() 
will return -1)
+     * with select count(*) operation. Older maxDB/sapDB driver doesn't let you
+     * use the .last, .getRow() mechanism (.getRow() will return -1). But the 
latest
+     * ones seem to work correctly. So make sure to use the latest drivers to
+     * get the correct results.
      */
     public boolean useCountForResultsetSize()
     {
-        return true;
+        /*
+        arminw: current driver versions seems to support getRow correctly
+        */
+        return false;
     }
 
     public String createSequenceQuery(String sequenceName)



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to