Bug fixes and enhances to psql submitted by Masaaki Sakaida

1. Fix problems of PAGER and \? command
2. Add -E option that shows actual queries sent by \dt and friends
3. Add version number in startup banners for psql

Small cleanups.

Addition to test/locale submitted by Oleg Broytmann. Comments from him:

There are two subdirectories (ISO8859-7 and koi8-to-win1251) containing
tests for Greek locale and server<=>client recoding feature (recently
submitted by Tatsuo Ishii <t-ishii@sra.co.jp>; we've debugged his patches
together in the field of Cyrillic support).

change comparison char* and NULL to char* and '\0'. This should be
more portable way.

Modify fmgr so that internal name (compiler name) of a built-in
function is found in prosrc field of pg_proc, not proname.  This allows
multiple aliases of a built-in to all be implemented as direct builtins,
without needing a level of indirection through an SQL function.  Replace
existing SQL alias functions with builtin entries accordingly.
Save a few K by not storing string names of builtin functions in fmgr's
internal table (if you really want 'em, get 'em from pg_proc...).
Update opr_sanity with a few more cross-checks.

1. Vacuum is updated for MVCC.
2. Much faster btree tuples deletion in the case when first on page
   index tuple is deleted (no movement to the left page(s)).
3. Remember blkno of new root page in BTPageOpaque of
   left/right siblings when root page is splitted.

Expanded opr_sanity test to look at pg_proc and other
related tables.

New regression test to cross-check pg_type, pg_class,
and related tables.

Clean up various minor irregularities detected by type_sanity
and newly expanded opr_sanity tests.

Remove pg_attribute_check.sql: these checks merged into new
regress test 'type_sanity'.

Fix bogus function signature for areajoinsel.
It still doesn't do anything, but at least now it does nothing correctly.

Delete unused system table pg_parg.

No longer need this file here; superseded by oidjoins regress test.

Add results of findoidjoins as a standard regression test.

Remove a bunch of dead entries exposed by findoidjoins crosschecks.
Apparently, whatever these things used to link to got recycled into
something else ... but the dependent entries didn't.

Update findoidjoins for 6.5: remove workaround for long-dead bug,
use NOT EXISTS() which is a lot faster than NOT IN (),
update documentation.

Revise memutils.h to use alignment information gathered by
configure, instead of having a bunch of crufty platform-specific guesses.

Fix multi-byte+locale problem

Clean up att_align calculations so that XXXALIGN macros
need not be bogus.

*** empty log message ***

*** empty log message ***

Add KOI8/WIN/ALT support

These small utilities are for generating internal tables from
rcode encoding tables.

Add KOI8/WIN/ALT to set client_encoding command help

Fix typo. #ifdef MUTIBYTE --> #ifdef MULTIBYTE

Add KOI8/WIN/ALT to the multi-byte encoding selections

Add KOI8/WIN/ALT to the multi-byte encoding selections

Add KOI8/WIN/ALT to the multi-byte encoding selections

Remove Tee code, move to _deadcode.

cleanups

Hi,

I have solved some problems with dynamic loading on NT. It is possible
to
run succesfully both trigger and plpgsql regression tests. The patch is
in
the included file "diff".

                        Dan

Add % to b_expr.

Cleanup of NULL in inet types.

Correct some comments, fix a small memory wastage when datatype
is pass-by-value.

This patch fixes the bug that made it impossible to grant permissions to
a user such as "www-data".

Oliver

Add .cvsignore file so cvs doesn't complain if you have lex/yacc
output files laying about.

cleanup

Ah, *now* I understand: plpgsql lexer must be generated with flex -l ...

Fix for %4 and 4%.

Hmm, it seems some versions of flex declare yytext as extern char[]
but others declare it as extern char *.  gcc complains (quite rightly too).
Worked around it by rearranging the order of inclusions so that we don't
have to explicitly declare yytext; this should work with either variant.

*** empty log message ***

Remove yacc/lex output files from CVS repository.

Remove yacc/lex output files from CVS repository.

Clean up build procedure: do not keep raw lex/yacc files around,
only the edited ones.

make clean should remove lex.yy.c

Script for preparing derived files during tarball construction.

Reverse out pfree agg part of patch from Erik Riedel.

cleanup

cleanup

Fix for aggregate memory leaks from Erik Riedel.

Fix &g.

Fix shift/reduce for NULL = Var.

Fix for memory leak in executor with fjIsNull.

I suggest the following portability patch, which does not
change functionality, but makes the code more ANSI C'ish.
My AIX xlc compiler barfs on all of these. Can someone please
review and apply to current.

 <<port.patch>>
Thanks
Andreas

Add EXCEPT/INTERSECT doc changes.

cleanup parser

grammar cleanup'

cleanup of grammer.

Fix optimizer indexing not working for negative numbers.

cleanups

*** empty log message ***

Add new postgres -O option to allow system table structure changes.

grammar cleanup

INET/CIDR doc cleanup

Add docs about left-associativity of operators.

Add documentation change for %'s precidence.

Left associates all operators, instead of non-associating them.

Have % operator have precedence like /.

Update sgml for 6.4.* release changes.

Update nextval() code.

Fix snprintf with strings, and nextval('"Aa"');

Here is a patch.
I have changed to call pg_exec_query_dest() instead of pg_exec_query().

Thanks.

Hiroshi Inoue

contrib/array_iterator/array_iterator.c won't compile.
Included patches should fix the problem.
--
Tatsuo Ishii

autoconf

Hmmm.  It had to do with tcl/tk on FreeBSD.  The configure scripts are
in
different directories.  The patch gave the option of specifying a dir
for
the tk script and if they were both in the same directory then it didn't
mind being empty.  It's small so I'm including it.  It was tested with
autoconf 2.12.

Vince.

Change Does Not Exist to 'does not exist'.

cleanup

*** empty log message ***

Allow sequence nextval actions to be case-sensitive.

Irix fix from Jeff Johnson

bool uses one byte

Drop buffers before destroying database files.

LIKE cleanup.

Fix brain death in !!= operator ... it's still pretty bogus
but at least now it does what it's supposed to do ...

> > > This patches src/bin/psql/psql.c.
> > >
> > > This patch is in responce to the following TODO list item:
> > >  * have psql \d on a view show the query
> > > -Ryan

Fix int8 configure one more time ... prior version didn't
define INT64_FORMAT in all cases.

Hi All,
I've changed the check_primary_key() function code to allow for either
the "automatic insert key rule" or "dependent insert key rule".
Previously it restricted the addtion of a child entry if the
corresponding parent entry was not there. Now if the option is
"automatic" it will add an entry in the parent too ( it will be
successful if there are no no-null fields in the parent apart from the
primary key).
The way to use it now is:
:/*
 * check_primary_key () -- check that key in tuple being
inserted/updated
 *                       references existing tuple in "primary" table.
 * Though it's called without args You have to specify referenced
 * table/keys while creating trigger:  key field names in triggered
table,
 * referenced table name, referenced key field names,type of action
[automatic|dependent]:
 * EXECUTE PROCEDURE
 * check_primary_key ('Fkey1', 'Fkey2', 'Ptable', 'Pkey1', 'Pkey2',
'[automatic|dependent]').
 */
I am attaching the new ../contrib/spi/refint.c file which will do this.
I will be glad to help in case of any problems.

- Anand.

cleanup

Cleanup

Reversed out libpq protocol patch for Tom Lane.

src/backend/utils/adt/numeric.c fails to compile due to a string having
an
embedded new-line character.

Billy G. Allie

This is another example of why not allowing utility functions in SPI
would be a Bad Thing.

For what it's worth, I found another case in libpq where you can get a T
message without a D that my utility patch needs to handle.  I have
attached
the updated patch against the 6.4.2 version of
src/interfaces/libpq/fe-exec.c

Jerry Gay

Attempting to insert a value of 'now' into a datetime type
results in a bogus datetime value under AlphaLinux.  (Note that
the link to submit a port-specific bug on your website is broken)

-Test Case:
----------
testdb=> create table dttest (dt datetime);
testdb=> insert into dttest values ('now');

--------------------------------------------------------------------------

Solution:
---------
The basic problem is the typedefs of AbsoluteTime and RelativeTime,
which are both 'int32'.  These types appear to be used synonymously
with the 'time_t' type, which on AlphaLinux is typedef'd as a 'long
int', which is 64-bits (not 32).  The solution included here fixes
the datetime type (it now passes the regression test), but does not
pass the absolute and relative time regression tests.  Presumably, a
more thorough investigation of how these types are used is warranted.
The included patch is from the v6.3.2 source, but can be applied to
the v6.4.2 source.  Please note that there is also a RedHat-specific
patch distributed with the PostgreSQL source package from RedHat
that was applied first.

Rich Edwards

Hi,

    Just in case you'd like to see what I was talking about, I am
attaching
my patch to src/interfaces/libpq/fe-exec.c to prevent utility functions
called from SPI from locking up the client.

Jerry Gay

Here is a little syntax error found in a .y file...  A dropped semi.

DwD
--
Daryl W. Dunbar

Cleanup of hash functions in pg_proc.

compile cleanup

Compile cleanup

Done.

o overwriting an existing lobj now works
o 8KB garbage block always inserted problem is fixed

Tatsuo Ishii

    I've been having also sorts of fun trying to get kerberos 4
authentifica
tion
working with postgresql-6.4.2 and  KTH-KRB Ebones
(http://www.pdc.kth.se/kth-kr
b) on a dec alpha running DU 4.0D using the native compiler. The
following
patch does the trick.

The rationale behind this is as follows. The KTH-KRB code header files
defines
lots of lengths like INST_SZ,REALM_SZ and KRB_SENDAUTH_VLEN. It also has
a
habit of doing things like

        chararray[LENGTH] = '\0'

to ensure null terminated strings. In my instance this just happens to
blat
the kerberos principal instance string leading to error like

        pg_krb4_recvauth: kerberos error: Can't decode authenticator
(krb_rd_req
)

The application code that comes with KTH-KRB uses "KRB_SENDAUTH_VLEN +
1" and
sometimes uses "INST_SZ + 1" so it seems safest to put that 1 char
buffer in
the appropriate place.

Rodney McDuff