Applied patch from Teofilis Martisius to improve performance.
authorBarry Lind
Sun, 20 Oct 2002 02:55:50 +0000 (02:55 +0000)
committerBarry Lind
Sun, 20 Oct 2002 02:55:50 +0000 (02:55 +0000)
Also removed some unused files and fixed the which needed a small change
after the previous patch to build.xml.

 Modified Files:
  jdbc/Makefile jdbc/org/postgresql/core/Encoding.java
  jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
 Removed Files:
  jdbc/utils/CheckVersion.java jdbc/utils/buildDriver
  jdbc/utils/changelog.pl

src/interfaces/jdbc/Makefile
src/interfaces/jdbc/org/postgresql/core/Encoding.java
src/interfaces/jdbc/org/postgresql/jdbc1/AbstractJdbc1Connection.java
src/interfaces/jdbc/utils/CheckVersion.java [deleted file]
src/interfaces/jdbc/utils/buildDriver [deleted file]
src/interfaces/jdbc/utils/changelog.pl [deleted file]

index fc881a34ba2060eda6f14c69aade7a64831c7659..3fbafe54ce8c30e4c83149b38ff73a58f8f9f70c 100644 (file)
@@ -4,7 +4,7 @@
 #
 # Copyright (c) 2001, PostgreSQL Global Development Group
 #
-# $Header: /cvsroot/pgsql/src/interfaces/jdbc/Attic/Makefile,v 1.35 2002/07/23 03:59:54 barry Exp $
+# $Header: /cvsroot/pgsql/src/interfaces/jdbc/Attic/Makefile,v 1.36 2002/10/20 02:55:50 barry Exp $
 #
 #-------------------------------------------------------------------------
 
@@ -29,7 +29,7 @@ install: installdirs
      -Dinstall.directory=$(javadir) $(properties)
 
 installdirs:
-   $(mkinstalldirs) $(javadir)
+   $(mkinstalldirs) $(javadir) 
 
 uninstall:
    $(ANT) -buildfile $(srcdir)/build.xml uninstall \
@@ -39,4 +39,4 @@ clean distclean maintainer-clean:
    $(ANT) -buildfile $(srcdir)/build.xml clean
 
 check: all
-   $(ANT) -buildfile $(srcdir)/build.xml test
+   $(ANT) -buildfile $(srcdir)/build.xml test $(properties)
index 95d4af5781febc91e2f6e6239db1348a5eac96e9..ea395b844bc4c6e2d33fa5a477da0167e6b4e235 100644 (file)
@@ -8,7 +8,7 @@ import org.postgresql.util.*;
 /*
  * Converts to and from the character encoding used by the backend.
  *
- * $Id: Encoding.java,v 1.6 2002/09/06 21:23:05 momjian Exp $
+ * $Id: Encoding.java,v 1.7 2002/10/20 02:55:50 barry Exp $
  */
 
 public class Encoding
@@ -161,6 +161,9 @@ public class Encoding
            }
            else
            {
+               if (encoding.equals("UTF-8")) {
+                   return decodeUTF8(encodedString, offset, length);
+               }
                return new String(encodedString, offset, length, encoding);
            }
        }
@@ -223,4 +226,44 @@ public class Encoding
            return false;
        }
    }
+
+   /**
+    * custom byte[] -> String conversion routine, 3x-10x faster than
+    * standard new String(byte[])
+    */
+   private static final int pow2_6 = 64;       // 26
+   private static final int pow2_12 = 4096;    // 212
+   private static char[] cdata = new char[50];
+
+   private synchronized String decodeUTF8(byte data[], int offset, int length) {
+       char[] l_cdata = cdata;
+       if (l_cdata.length < (length-offset)) {
+           l_cdata = new char[length-offset];
+       }
+       int i = offset;
+       int j = 0;
+       int z, y, x, val;
+       while (i < length) {
+           z = data[i] & 0xFF;
+           if (z < 0x80) {
+               l_cdata[j++] = (char)data[i];
+               i++;
+           } else if (z >= 0xE0) {     // length == 3
+               y = data[i+1] & 0xFF;
+               x = data[i+2] & 0xFF;
+               val = (z-0xE0)*pow2_12 + (y-0x80)*pow2_6 + (x-0x80);
+               l_cdata[j++] = (char) val;
+               i+= 3;
+           } else {        // length == 2 (maybe add checking for length > 3, throw exception if it is
+               y = data[i+1] & 0xFF;
+               val = (z - 0xC0)* (pow2_6)+(y-0x80);
+               l_cdata[j++] = (char) val;
+               i+=2;
+           } 
+       }
+   
+       String s = new String(l_cdata, 0, j);
+       return s;
+   }
+
 }
index dbf785709b17fabc78d2b4d474d64d762baa36be..f161c6616cd3af3e1bb4fae15ab54d6dbd7da238 100644 (file)
@@ -14,7 +14,7 @@ import org.postgresql.largeobject.LargeObjectManager;
 import org.postgresql.util.*;
 
 
-/* $Header: /cvsroot/pgsql/src/interfaces/jdbc/org/postgresql/jdbc1/Attic/AbstractJdbc1Connection.java,v 1.11 2002/10/17 05:33:52 barry Exp $
+/* $Header: /cvsroot/pgsql/src/interfaces/jdbc/org/postgresql/jdbc1/Attic/AbstractJdbc1Connection.java,v 1.12 2002/10/20 02:55:50 barry Exp $
  * This class defines methods of the jdbc1 specification.  This class is
  * extended by org.postgresql.jdbc2.AbstractJdbc2Connection which adds the jdbc2
  * methods.  The real Connection class (for jdbc1) is org.postgresql.jdbc1.Jdbc1Connection
@@ -367,10 +367,16 @@ public abstract class AbstractJdbc1Connection implements org.postgresql.PGConnec
        //jdbc by default assumes autocommit is on until setAutoCommit(false)
        //is called.  Therefore we need to ensure a new connection is 
        //initialized to autocommit on.
+       //We also set the client encoding so that the driver only needs 
+       //to deal with utf8.  We can only do this in 7.3 because multibyte 
+       //support is now always included
        if (haveMinimumServerVersion("7.3")) 
        {
            java.sql.ResultSet acRset =
-               ExecSQL("show autocommit");
+               ExecSQL("set client_encoding = 'UNICODE'; show autocommit");
+
+           //set encoding to be unicode
+           encoding = Encoding.getEncoding("UNICODE", null);
 
            if (!acRset.next())
            {
diff --git a/src/interfaces/jdbc/utils/CheckVersion.java b/src/interfaces/jdbc/utils/CheckVersion.java
deleted file mode 100644 (file)
index a2438cd..0000000
+++ /dev/null
@@ -1,74 +0,0 @@
-package utils;
-
-/*
- * This little app checks to see what version of JVM is being used.
- * It does this by checking first the java.vm.version property, and
- * if that fails, it looks for certain classes that should be present.
- */
-public class CheckVersion
-{
-   /*
-    * Check for the existence of a class by attempting to load it
-    */
-   public static boolean checkClass(String c)
-   {
-       try
-       {
-           Class.forName(c);
-       }
-       catch (Exception e)
-       {
-           return false;
-       }
-       return true;
-   }
-
-   /*
-    * This first checks java.vm.version for 1.1, 1.2 or 1.3.
-    *
-    * It writes jdbc1 to stdout for the 1.1.x VM.
-    *
-    * For 1.2 or 1.3, it checks for the existence of the javax.sql.DataSource
-    * interface, and if found writes enterprise to stdout. If the interface
-    * is not found, it writes jdbc2 to stdout.
-    *
-    * PS: It also looks for the existence of java.lang.Byte which appeared in
-    * JDK1.1.0 incase java.vm.version is not heeded by some JVM's.
-    *
-    * If it can't work it out, it writes huho to stdout.
-    *
-    * The make file uses the written results to determine which rule to run.
-    *
-    * Bugs: This needs thorough testing.
-    */
-   public static void main(String args[])
-   {
-       String vmversion = System.getProperty("java.vm.version");
-
-       System.out.println("postgresql.jdbc=" + System.getProperty("postgresql.jdbc"));
-
-       // We are running a 1.1 JVM
-       if (vmversion.startsWith("1.1"))
-       {
-           System.out.println("jdbc1");
-           //System.exit(0);
-       }
-       else
-           // We are running a 1.2 or 1.3 JVM
-           if (vmversion.startsWith("1.2") ||
-                   vmversion.startsWith("1.3") ||
-                   checkClass("java.lang.Byte")
-              )
-           {
-
-               // Check to see if we have the standard extensions. If so, then
-               // we want the enterprise edition, otherwise the jdbc2 driver.
-               if (checkClass("javax.sql.DataSource"))
-                   System.out.println("enterprise");
-               else
-                   System.out.println("jdbc2");
-               //System.exit(0);
-           }
-       System.setProperty("postgresql.jdbc", "yoyo");
-   }
-}
diff --git a/src/interfaces/jdbc/utils/buildDriver b/src/interfaces/jdbc/utils/buildDriver
deleted file mode 100755 (executable)
index 8cca1d9..0000000
+++ /dev/null
@@ -1,47 +0,0 @@
-#!/bin/sh
-#
-# $Id: buildDriver,v 1.2 2000/12/20 16:22:49 peter Exp $
-#
-# This script generates the org/postgresql/Driver.java file from the template
-# org/postgresql/Driver.java.in
-#
-# We do this because we need to include the version number from Makefile.global
-# and some other goodies.
-#
-# This used to be in Makefile, but as it's now done three times, it's better
-# to have it as a separate script.
-#
-# If you have any problems, please let us know ;-)
-#
-# Syntax: buildDriver version class
-#
-# Where:
-#  version The version string from Makefile.global
-#  class   The class implementing java.sql.Connection
-#  edition The driver edition being built
-#  source  The file to build. We assume that ${source}.in exists
-#
-
-VERSION=$1
-CLASS=$2
-EDITION=$3
-SOURCE=$4
-
-#---------------------------------------------------------------------------
-# Extract the version. This will work until version x.9 (and assuming we don't
-# have 7.10 etc). We only handle 1 digit for MINORVERSION to handle things like
-# 7.1devel etc
-#
-MAJORVERSION=`echo $VERSION | cut -f1 -d'.'`
-MINORVERSION=`echo $VERSION | cut -f2 -d'.' | cut -c1`
-
-#---------------------------------------------------------------------------
-# Now finally build the driver
-sed \
-   -e "s/@JDBCCONNECTCLASS@/$CLASS/g" \
-   -e "s/@VERSION@/$VERSION $EDITION/g" \
-   -e "s/@MAJORVERSION@/$MAJORVERSION/g" \
-   -e "s/@MINORVERSION@/$MINORVERSION/g" \
-   <${SOURCE}.in \
-   >$SOURCE
-#---------------------------------------------------------------------------
diff --git a/src/interfaces/jdbc/utils/changelog.pl b/src/interfaces/jdbc/utils/changelog.pl
deleted file mode 100644 (file)
index 3cba15a..0000000
+++ /dev/null
@@ -1,23 +0,0 @@
-#!/bin/perl
-
-while(<>) {
-    chomp();
-    s/\t+/ /g;
-    if(substr($_,0,3) eq ' - ') {
-   print "
    " if !$inlist;
-   $inlist=1;
-   print "
  • ".substr($_,3)."\n";
  • -    } else {
    -   if($_ eq "" || $_ eq " ") {
    -       print "" if $inlist;
    -       $inlist=0;
    -       print "
    \n";
    -   } elsif(substr($_,0,1) eq " ") {
    -       print $_;
    -   } else {
    -       print "" if $inlist;
    -       $inlist=0;
    -       print "

    ".$_."

    \n";
    -   }
    -    }
    -}