Initial information for Linux/Unix installation.

cleanups of indexing.c

cleanup

cleanup

OK, folks, here is the pgindent output.

Renaming cleanup, no pgindent yet.

indexing.c cleanups.

Fix for regproc

Fix for regprocout, use underscore, not parens.

OR clause index fix

Two spaces after a colon.

Preserver pg_log and pg_variable in upgrade.  Cleanups.

Fix for possible releasebuffer bug.

From: Massimo Dal Zotto <dz@cs.unitn.it>

Types removed from Postgres: oidint2, oidint4, oidname.
Fix example columns in alter_table.out to use datetime and timespan
 as a substitute for oidint4 and oidname.

From: Tom Lane <tgl@sss.pgh.pa.us>

We're carrying around a copy of install-sh in case the local system
has no install script.  It's wasted baggage, because configure doesn't
know it's there :-(.  (Apparently everyone who's used postgres lately
already had an install script somewhere in their path.  I happened to
try to run configure with a minimal PATH tonight, and it promptly
gave up for lack of an install program.)  Here's the patch.

sequence contrib directory removed, as already integrated into the main
backend...

From: Massimo Dal Zotto <dz@cs.unitn.it>

Here is a tar file the new directories, which substitute the old ones
in contrib. Please remove the old directories array, datetime, miscutil,
string and userlock before unpacking the tar file in contrib.

Note that as the modules are now installed in lib/modules I install all
my sql code in lib/sql. In my opinion also the other contributors should
follow these rules.

From: Massimo Dal Zotto <dz@cs.unitn.it>

After some playing with gdb I found that in printtup() there is a non null
attribute with typeinfo->attrs[i]->atttypid = 0 (invalid oid). Unfortunately
attibutes with invalid type are neither printed nor marked as null, and this
explains why psql doesn't get all the expected data.

So I made this patch to printtup():

Make old db directory configurable.

Update man pages.

Update INSTALL to mention pg_upgrade.

New pg_upgrade command.

Update FAQ.

Improve distclean.

Add missing pgaccess file.

autoconf

--without fix from Brook

Add missing file.

Update INSTALL, etc. for release 6.4.  Update pgaccess to 0.88.

Update of changes

Prevent file descriptor leak from failed COPY.

Ok. BTW Mr. Kataoka who is maintaing Japanese version of PostgreSQL
ODBC driver have found a bug in 6.3.2 pg_dump and have made patches.
I confirmed that the same bug still exists in the current source
tree. So I made up patches based on Kataoka's. Here are some
explanations.

o fmtId() returns pointer to a static memory in it. In the meantime
there is a line where is fmtId() called twice without saving the
first value returned by fmtId(). So second call to fmtId() will
break the first one.

o findTableByName() looks up a table by its name. if a table name
contanins upper letters or non ascii chars, fmtId() will returns
a name quoted in double quotes, which will not what findTableByName()
wants. The result is SEG fault.  -- Tatsuo Ishii t-ishii@sra.co.jp

Fix scanner name length trimming.

Better error message if table exists.

This patch resolves some regression test failures caused by platform
dependencies.

David Hartwig

This is the first (of hopefully few) AIX port patches.   This patch
was tested with Linux/GCC.  I still have some issues with with the
snprintf() function.

David Hartwig

update libpq.rc version.

Hello!

Here is a new patch for libpq, to make it work on Win32 again (since
the latest modifications broke it a little).

Please also add the file "libpq.rc" to the interfaces/libpq directory.
This will allow version-stamping of the generated DLL file, so that
automatic install programs (and interested users) can determine
the version of the file.  The file is currently set as "prerelease".
Before the release, somebody should change the line "FILEFLAGS
VS_FF_PRERELEASE" to "FILEFLAGS 0".  That information should probably
go into toos\RELEASE_CHANGES.

The patch is against the cvs as of ~ 1998-08-26 14:30 CEST.

//Magnus

Truncate identifiers at NAMEDATALEN length.

libpq's pqGetnchar() should not return a null-terminated value
anymore.  Fix for large objects.

From: Michael Meskes <meskes@online-club.de>

This one is against the current archive (so it contains the one I send the
other day). It should fix the AIX problems. Andreas, could you please try
it? Thanks.

+ Wed Aug 26 16:17:39 CEST 1998
+
+       - Sync preproc.y with gram.y
+
+ Thu Aug 27 15:32:23 CEST 1998
+
+       - Fix some minor glitches that the AIX compiler complains about
+       - Added patchlevel to library
+
+ Fri Aug 28 15:36:58 CEST 1998
+
+       - Removed one line of code that AIX complains about since it was not
+         needed anyway
+       - Set library version to 2.6.1

Ignore spaces when reporting regression failures.

From: Massimo Dal Zotto <dz@cs.unitn.it>

I have this small patch which fixes a typo in my previous lock patches.
Fortunately the bug affects only user locks.

-

vacuum fix.

Hi all,

    I  don't know if this is really related to the initdb problem
    discussion (haven't followed it enough). But seems so because
    it  fixes  a  damn  problem  during  index tuple insertion on
    CREATE TABLE into pg_attribute_relid_attnum_index.

    Anyway - this bug was really hard to find. During startup the
    relcache  reads  in  some  prepared  information  about index
    strategies from a file and then  reinitializes  the  function
    pointers  inside  the  scanKey data.  But for sake it assumed
    single attribute index tuples (hasn't that changed recently).
    Thus not all the strategies scanKey entries where initialized
    properly,  resulting  in  invalid  addresses  for  the  btree
    comparision functions.

    With  the  patch  at  the  end  the  regression  tests passed
    excellent except for the sanity_check that crashed at  vacuum
    and the misc test where the select unique1 from onek2 outputs
    the two rows in different order.

Jan

From: Magnus Hagander <mha@edu.sollentuna.se>

Ok. Here is a patch to make psql work on Win32 (as a console mode
application, of course).
It requires getopt.c to be in src/utils - works fine with the FreeBSD
version of it.

Also, the file win32.mak should go into src/bin/psql.

Make 'name' type int aligned, like char/varchar.

Fix 'd ' should be 'd'.

Fix problem where attbyval was wrong for xid types.

Fix atttypmod alignment again, and re-enable ecpg.

Fix atttypmod to be int32 in system catalogs.  Oops.

Make attalign match type alignment.

Fix up crashing symptoms for new serial type by making sure constraint
 and index name fields are pstrdup'd (copied) rather than reused.

Fix for select bug.

cvs add'd two files for the tprintf() patch...

From: Massimo Dal Zotto <dz@cs.unitn.it>

>       these patches define the UNLISTEN sql command. The code already
>       existed but it was unknown to the parser. Now it can be used
>       like the listen command.
>       You must make clean and delete gram.c and parser.h before make.

From: Massimo Dal Zotto <dz@cs.unitn.it>

> tprintf.patch
>
>       tprintf.patch
>
>       adds functions and macros which implement a conditional trace package
>       with the ability to change flags and numeric options of running
>       backends at runtime.
>       Options/flags can be specified in the command line and/or read from
>       the file pg_options in the data directory.

Massimo Dal Zotto <dz@cs.unitn.it>

> socket-flock.patch
>
>       use advisory locks to check if the unix socket can be deleted.
>       A running postmaster keeps a lock on that file. A starting
>       postmaster exits if the file exists and is locked, otherwise
>       it deletes the sockets and proceeds.
>       This avoid the need to remove manually the file after a postmaster
>       or system crash.
>       I don't know if flock is available on any system. If not we could
>       define a HAVE_FLOCK set by configure.

From: Massimo Dal Zotto <dz@cs.unitn.it>

> sinval.patch
>
>       fixes a problem in SI cache which causes table overflow if some
>       backend is idle for a long time while other backends keep adding
>       entries.
>       It uses the new signal handling implemented in tprintf.patch.
>       I have also increacasesed the max number of backends from 32 to 64
>       and the table size from 1000 to 5000.
>       I don't know if anybody is working on SI, but until another
>       solution is found this patch fixes the problem. I have received
>       messages from other people reporting the same problem which I
>       fixed many months ago.

missed adding a new include file

From: Massimo Dal Zotto <dz@cs.unitn.it>

> sequence.patch
>
>       adds the missing setval command to sequences. Owner of sequences
>       can now set the last value to any value between min and max
>       without recreating the sequence. This is useful after loading
>       data from external files.

From: Massimo Dal Zotto <dz@cs.unitn.it>

> ps-status.patch
>
>       macros for ps status, used by postgres.c and utility.c.
>       Unfortunately ps status is system dependent and the current
>       code doesn't work on linux. The use of macros confines system
>       dependency to into one file (ps-status.h). Users of other
>       operating systems should check this code and submit new macros.

From: Massimo Dal Zotto <dz@cs.unitn.it>

> pqpacket.patch
>
>       fixed indentation.

From: Massimo Dal Zotto <dz@cs.unitn.it>

lock.patch

        I have rewritten lock.c cleaning up the code and adding better
        assert checking I have also added some fields to the lock and
        xid tags for better support of user locks. There is also a new
        function which returns an array of pids owning a lock.
        I'm using this code from over six months and it works fine.

From: Massimo Dal Zotto <dz@cs.unitn.it>

config.patch

        remove old defines no more used in the code.

From: Massimo Dal Zotto <dz@cs.unitn.it>

assert.patch

        adds a switch to turn on/off the assert checking if enabled at compile
        time. You can now compile postgres with assert checking and disable it
        at runtime in a production environment.

Can someone please apply this portability patch to genbki.sh ?
(Mark or Bruce?) It fixes a problem when cpp gives a warning when
precompiling /dev/null like:  "/dev/null", line 1: 1506-229 (W)
File is empty.  This leads to a hangup when doing the description
load during initdb, since stderr also ends up in the global1.description
and local1_template1.description

stderr has to be redirected to /dev/null:

Andreas Zeugswetter

Add is_sequence flag to ColumnDef structure. Used to implement SERIAL type.

Support SERIAL column type. Expand column marked is_sequence into three
 statements:
 - the table definition with a default clause referencing the sequence;
 - a CREATE SEQUENCE statement;
 - a UNIQUE constraint, which expands into a CREATE INDEX statement.
This is not a perfect solution, since the sequence will remain even if
 the table is dropped. Also, there is no absolute protection on updating
 the sequence column.

Support SERIAL column type. Expand into an integer column but mark
 is_sequence in the ColumnDef structure.

Make new section header.

Clean up double-quotes around column and table names.
 Had too many in some cases because there was already a routine
 taking care of it.

Move debugging printout of the query tree to print for all cases.
 Formerly came just after early exit from loop for command nodes,
 so missed some cases.

Add nextstep to similar template file

re-integrate nextstep dynloader functionality

From: Jacek Lasecki <jacek@sound.eti.pg.gda.pl>

From: Michael Meskes <meskes@online-club.de>

+
+ Fri Aug 14 12:44:21 CEST 1998
+
+       - Added EXEC SQL DEFINE statement
+       - Set version to 2.4.0
+
+ Tue Aug 18 09:24:15 CEST 1998
+
+       - Removed keyword IS from DEFINE statement
+       - Added latest changes from gram.y
+       - Removed duplicate symbols from preproc.y
+       - Initialize sqlca structure
+       - Added check for connection to ecpglib
+       - Set version to 2.4.1
+
+ Thu Aug 20 15:31:29 CEST 1998
+
+       - Cleaned up memory allocation in ecpglib.c
+       - Set library version to 2.6
+

From: Michael Meskes <meskes@online-club.de>

+
+ Fri Aug 14 12:44:21 CEST 1998
+
+       - Added EXEC SQL DEFINE statement
+       - Set version to 2.4.0
+
+ Tue Aug 18 09:24:15 CEST 1998
+
+       - Removed keyword IS from DEFINE statement
+       - Added latest changes from gram.y
+       - Removed duplicate symbols from preproc.y
+       - Initialize sqlca structure
+       - Added check for connection to ecpglib
+       - Set version to 2.4.1
+
+ Thu Aug 20 15:31:29 CEST 1998
+
+       - Cleaned up memory allocation in ecpglib.c
+       - Set library version to 2.6
+

From: Michael Meskes <meskes@online-club.de>

+
+ Fri Aug 14 12:44:21 CEST 1998
+
+       - Added EXEC SQL DEFINE statement
+       - Set version to 2.4.0
+
+ Tue Aug 18 09:24:15 CEST 1998
+
+       - Removed keyword IS from DEFINE statement
+       - Added latest changes from gram.y
+       - Removed duplicate symbols from preproc.y
+       - Initialize sqlca structure
+       - Added check for connection to ecpglib
+       - Set version to 2.4.1
+
+ Thu Aug 20 15:31:29 CEST 1998
+
+       - Cleaned up memory allocation in ecpglib.c
+       - Set library version to 2.6
+

>Applied.

Thanks. But patches for src/backend/catalog/Makefile seems missing
in the current source tree. Please apply attached patches.

It also includes some corrections to src/backend/util/mb/wchar.c.
-- Tatsuo Ishii t-ishii@sra.co.jp

Cleanup of target file.

Make sure resdomno for update/insert match attribute number for
rewrite system.  Restructure parse_target to make it easier to
understand.

int64 fix

move plpgsql

move plpgsql to src/pl for Jan.

Fix bootstrap so it properly defines alignment of attributes.

autoconf

LONG_LONG_64 fix.

autoconf

I have found a minor problem with current configure.in.

[AC_MSG_RESULT(yes) AC_DEFINE(HAVE_LONG_INT_64)],

this line produces something like:

  echo "$ac_t""yes" 1>&6 cat >> confdefs.h <<\EOF

and would append garbage "yes cat" to confdefs.h. Of course the
result confdefs.h is not syntactically correct therefore following
tests using confdefs.h would all fail.  To avoid the problem, we
could switch the order of AC_MSG_RESULT and AC_DEFINE (see attached
patch). This happend on my LinuxPPC box.

Tatsuo Ishii t-ishii@sra.co.jp

man pages cleanup

OLD has to return CURRENT for now, Jan.

This  is the final state of the rule system for 6.4 after the
    patch is applied:

Rewrite rules on relation level work fine now.

Event qualifications on insert/update/delete  rules  work
fine now.

I  added  the  new  keyword  OLD to reference the CURRENT
tuple. CURRENT will be removed in 6.5.

Update rules can  reference  NEW  and  OLD  in  the  rule
qualification and the actions.

Insert/update/delete rules on views can be established to
let them behave like real tables.

For  insert/update/delete  rules  multiple  actions   are
supported  now.   The  actions  can also be surrounded by
parantheses to make psql  happy.   Multiple  actions  are
required if update to a view requires updates to multiple
tables.

Regular users  are  permitted  to  create/drop  rules  on
tables     they     have     RULE     permissions     for
(DefineQueryRewrite() is  now  able  to  get  around  the
access  restrictions  on  pg_rewrite).  This enables view
creation for regular users too. This  required  an  extra
boolean  parameter  to  pg_parse_and_plan() that tells to
set skipAcl on all rangetable entries  of  the  resulting
queries.       There      is      a      new     function
pg_exec_query_acl_override()  that  could  be   used   by
backend utilities to use this facility.

All rule actions (not only views) inherit the permissions
of the event relations  owner.  Sample:  User  A  creates
tables    T1    and    T2,   creates   rules   that   log
INSERT/UPDATE/DELETE on T1 in T2 (like in the  regression
tests  for rules I created) and grants ALL but RULE on T1
to user B.  User B  can  now  fully  access  T1  and  the
logging  happens  in  T2.  But user B cannot access T2 at
all, only the rule actions can. And due to  missing  RULE
permissions on T1, user B cannot disable logging.

Rules  on  the  attribute  level are disabled (they don't
work properly and since regular users are  now  permitted
to create rules I decided to disable them).

Rules  on  select  must have exactly one action that is a
select (so select rules must be a view definition).

UPDATE NEW/OLD rules  are  disabled  (still  broken,  but
triggers can do it).

There are two new system views (pg_rule and pg_view) that
show the definition of the rules or views so the db admin
can  see  what  the  users do. They use two new functions
pg_get_ruledef() and pg_get_viewdef() that are  builtins.

The functions pg_get_ruledef() and pg_get_viewdef() could
be used to implement rule and view support in pg_dump.

PostgreSQL is now the only database system I  know,  that
has rewrite rules on the query level. All others (where I
found a  rule  statement  at  all)  use  stored  database
procedures  or  the  like  (triggers as we call them) for
active rules (as some call them).

    Future of the rule system:

The now disabled parts  of  the  rule  system  (attribute
level,  multiple  actions on select and update new stuff)
require a complete new rewrite handler from scratch.  The
old one is too badly wired up.

After  6.4  I'll  start to work on a new rewrite handler,
that fully supports the attribute level  rules,  multiple
actions on select and update new.  This will be available
for 6.5 so we get full rewrite rule capabilities.

Jan

Compile ecpg by default.

just that the regression tests for rules work, please apply
    the following to regress/sql/tests.

    If applying by hand note that the setup_... must run before
    the run_... (that I splitted these two was due to the errors
    that occured when creating rules and using them then in the
    same session - I'll post another fix for this later).

    BTW: the regression tests sanity_checks and alter_table fail
    now due to the remove of some indices and the oidint4 and
    oidname types. At least expectes should be set to the current
    results.

    Thanks.

Jan

o note that now pg_database has a new attribuite "encoding" even
if MULTIBYTE is not enabled. So be sure to run initdb.

o these patches are made against the latest source tree (after
Bruce's massive patch, I think) BTW, I noticed that after running
regression, the oid field of pg_type seems disappeared.

regression=> select oid from pg_type; ERROR:  attribute
'oid' not found

this happens after the constraints test. This occures with/without
my patches. strange...

o pg_database_mb.h, pg_class_mb.h, pg_attribute_mb.h are no longer
used, and shoud be removed.

o GetDatabaseInfo() in utils/misc/database.c removed (actually in
#ifdef 0). seems nobody uses.

t-ishii@sra.co.jp

autoconf

Attached is a patch that uses autoconf to determine whether there
is a working 64-bit-int type available.

In playing around with it on my machine, I found that gcc provides
perfectly fine support for "long long" arithmetic ... but sprintf()
and sscanf(), which are system-supplied, don't work :-(.  So the
autoconf test program does a cursory test on them too.

If we find that a lot of systems are like this, it might be worth
the trouble to implement binary<->ASCII conversion of int64 ourselves
rather than relying on sprintf/sscanf to handle the data type.

regards, tom lane

cleanup

fulltext copy script.