;;;; -*- Mode: LISP; Syntax: ANSI-Common-Lisp; Base: 10 -*-
;;;; *************************************************************************
;;;;
-;;;; $Id: $
+;;;; $Id$
;;;;
;;;; Generic postgresql layer, used by db-postgresql and db-postgresql-socket
;;;;
(declare (ignore type args database))
"VARCHAR")
-(defmethod database-get-type-specifier ((type (eql 'simple-base-string)) args database
+(defmethod database-get-type-specifier ((type (eql 'string)) args database
(db-type (eql :postgresql)))
(declare (ignore database))
(if args
- (format nil "VARCHAR(~A)" (car args))
- "VARCHAR"))
+ (format nil "CHAR(~A)" (car args))
+ "VARCHAR"))
-(defmethod database-get-type-specifier ((type (eql 'simple-string)) args database
+(defmethod database-get-type-specifier ((type (eql 'tinyint)) args database
(db-type (eql :postgresql)))
- (declare (ignore database))
- (if args
- (format nil "VARCHAR(~A)" (car args))
- "VARCHAR"))
+ (declare (ignore args database))
+ "INT2")
-(defmethod database-get-type-specifier ((type (eql 'string)) args database
+(defmethod database-get-type-specifier ((type (eql 'smallint)) args database
(db-type (eql :postgresql)))
- (declare (ignore database))
- (if args
- (format nil "VARCHAR(~A)" (car args))
- "VARCHAR"))
+ (declare (ignore args database))
+ "INT2")
(defmethod database-get-type-specifier ((type (eql 'wall-time)) args database
(db-type (eql :postgresql)))
(declare (ignore args database))
"TIMESTAMP WITHOUT TIME ZONE")
+(defmethod database-get-type-specifier ((type (eql 'number)) args database
+ (db-type (eql :postgresql)))
+ (declare (ignore database db-type))
+ (cond
+ ((and (consp args) (= (length args) 2))
+ (format nil "NUMERIC(~D,~D)" (first args) (second args)))
+ ((and (consp args) (= (length args) 1))
+ (format nil "NUMERIC(~D)" (first args)))
+ (t
+ "NUMERIC")))
;;; Backend functions
(result
(mapcar #'car
(database-query
- (format nil "SELECT attname FROM pg_class,pg_attribute WHERE pg_class.oid=attrelid AND relname='~A'~A"
+ (format nil "SELECT attname FROM pg_class,pg_attribute WHERE pg_class.oid=attrelid AND attisdropped = FALSE AND relname='~A'~A"
(string-downcase table)
owner-clause)
database nil nil))))
(parse-integer
(caar
(database-query
- (concatenate 'string "SELECT LAST_VALUE ('" sequence-name "')")
+ (concatenate 'string "SELECT LAST_VALUE FROM " sequence-name)
database nil nil)))))
(defun postgresql-database-list (connection-spec type)
(defmethod database-list (connection-spec (type (eql :postgresql-socket)))
(postgresql-database-list connection-spec type))
-
+#+nil
(defmethod database-describe-table ((database generic-postgresql-database) table)
+ ;; MTP: LIST-ATTRIBUTE-TYPES currently executes separate queries for
+ ;; each attribute. It would be more efficient to have a single SQL
+ ;; query return the type data for all attributes. This code is
+ ;; retained as an example of how to do this for PostgreSQL.
(database-query
(format nil "select a.attname, t.typname
from pg_class c, pg_attribute a, pg_type t
(sql-escape (string-downcase table)))
database :auto nil))
-
-;; Capabilities
+;;; Prepared statements
+
+(defvar *next-prepared-id-num* 0)
+(defun next-prepared-id ()
+ (let ((num (incf *next-prepared-id-num*)))
+ (format nil "CLSQL_PS_~D" num)))
+
+(defclass postgresql-stmt ()
+ ((database :initarg :database :reader database)
+ (id :initarg :id :reader id)
+ (bindings :initarg :bindings :reader bindings)
+ (field-names :initarg :field-names :accessor stmt-field-names)
+ (result-types :initarg :result-types :reader result-types)))
+
+(defun clsql-type->postgresql-type (type)
+ (cond
+ ((in type :int :integer) "INT4")
+ ((in type :short) "INT2")
+ ((in type :bigint) "INT8")
+ ((in type :float :double :number) "NUMERIC")
+ ((and (consp type) (in (car type) :char :varchar)) "VARCHAR")
+ (t
+ (error 'sql-user-error
+ :message
+ (format nil "Unknown clsql type ~A." type)))))
+
+(defun prepared-sql-to-postgresql-sql (sql)
+ ;; FIXME: Convert #\? to "$n". Don't convert within strings
+ (declare (simple-string sql))
+ (with-output-to-string (out)
+ (do ((len (length sql))
+ (param 0)
+ (in-str nil)
+ (pos 0 (1+ pos)))
+ ((= len pos))
+ (declare (fixnum len param pos))
+ (let ((c (schar sql pos)))
+ (declare (character c))
+ (cond
+ ((or (char= c #\") (char= c #\'))
+ (setq in-str (not in-str))
+ (write-char c out))
+ ((and (char= c #\?) (not in-str))
+ (write-char #\$ out)
+ (write-string (write-to-string (incf param)) out))
+ (t
+ (write-char c out)))))))
+
+(defmethod database-prepare (sql-stmt types (database generic-postgresql-database) result-types field-names)
+ (let ((id (next-prepared-id)))
+ (database-execute-command
+ (format nil "PREPARE ~A (~{~A~^,~}) AS ~A"
+ id
+ (mapcar #'clsql-type->postgresql-type types)
+ (prepared-sql-to-postgresql-sql sql-stmt))
+ database)
+ (make-instance 'postgresql-stmt
+ :id id
+ :database database
+ :result-types result-types
+ :field-names field-names
+ :bindings (make-list (length types)))))
+
+(defmethod database-bind-parameter ((stmt postgresql-stmt) position value)
+ (setf (nth (1- position) (bindings stmt)) value))
+
+(defun binding-to-param (binding)
+ (typecase binding
+ (string
+ (concatenate 'string "'" (sql-escape-quotes binding) "'"))
+ (t
+ binding)))
+
+(defmethod database-run-prepared ((stmt postgresql-stmt))
+ (with-slots (database id bindings field-names result-types) stmt
+ (let ((query (format nil "EXECUTE ~A (~{~A~^,~})"
+ id (mapcar #'binding-to-param bindings))))
+ (cond
+ ((and field-names (not (consp field-names)))
+ (multiple-value-bind (res names)
+ (database-query query database result-types field-names)
+ (setf field-names names)
+ (values res names)))
+ (field-names
+ (values (nth-value 0 (database-query query database result-types nil))
+ field-names))
+ (t
+ (database-query query database result-types field-names))))))
+
+;;; Capabilities
(defmethod db-type-has-fancy-math? ((db-type (eql :postgresql)))
t)
(defmethod db-type-default-case ((db-type (eql :postgresql)))
:lower)
+(defmethod db-type-has-prepared-stmt? ((db-type (eql :postgresql)))
+ t)
+
+(defmethod db-type-has-prepared-stmt? ((db-type (eql :postgresql-socket)))
+ t)
+