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

We have tested the patches on three platforms:

NetBSD/macppc
LinuxPPC
FreeBSD 2.2.6-RELEASE

All of them seem happy with the regression test. Note that, however,
compiling with optimization enabled on NetBSD/macppc causes an initdb
failure (other two platforms are ok). After checking the asm code, we
are suspecting that might be a compiler(egcs) bug.

Tatsuo Ishii

This patch improves the destroyuser script's behaviour in checking whether
the user is allowed to detroy users.  It also allows for a possible incorrect
empty reply to a yes/no question.

Oliver Elphick

Patches for the SGML documentation relating
to the tutorial code changes.

NOTE:  I couldn't get ngsmls to find the document
type so that the changes could be checked.  However,
the changes were very minor:

Best,

Clark

Fixup for src/tutorial/func.c and src/tutorial/func.source

Removed char16 and replaced with an example using Point
as suggested by Tom Lane.  The dept field was changed to
the cubicle field denoting the row(x) and column(y) of
the employee's cube in the corporate jungle.  The C function
builds a 'compromise' cubicle from two suggested ones.

I'll try and patchup the documentation next.

Clark

Hello,

I've been working on the following TODO list item:
  * psql \d on index with char()/varchar() fields shows improper length

I've attached a simple patch to fix this.

-Ryan

The isWritable method in ResultSetMetaData returns the logically
incorrect result. This bug goes back to at least 6.3.

Alvin

I have a problem with Access97 not working properly when entering new
records using a sub form, i.e. entering a new order/orderlines or master
and
detail tables.  The problem is caused by a SQL statement that Access97
makes
involving NULL.  The syntax that fails is "column_name" = NULL.  The
following attachment was provided by -Jose'-.  It contains a very small
enhancement to gram.y that will allow Access97 to work properly with sub
forms.  Can this enhancement be added to release 6.5?

 <<gram.patch>>
Thanks, Michael

There are errors in the PGmoney class in the conversion routines over
the handling of negative numbers and commas. The attached path attempts
to fix these.
However the getValue method does not yet insert commas into the
generated string.
Also in getValue there is an incorrect assumption that the currency
symbol is '$', it should of course be '£'!. I have no idea on how to go
about fixing this one.

Alvin

Enclosed below I have a patch to allow a btree index on the int8 type.

I would like some feedback on what the hash function for the int8 hash
function
in the ./backend/access/hash/hashfunc.c should return.

Also, could someone (maybe Tomas Lockhart?) look-over the patch and make
sure
the system table entries are correct?  I've tried to research them as
much as I
could, but some of them are still not clear to me.

Thanks,
-Ryan

*** empty log message ***

We have to return dummy tuple for empty targetlist!
Try select t1.x from t1, t2 where t1.y = 1 and t2.y = 1
- t2 scan target list will be empty and so no one tuple
will be returned...

Make type 'regproc' binary-compatible with int4 and oid,
so that it has some SQL operators available.  It's difficult to write
automated tests of a data type that you haven't even got == for...

Fix some typos in argument and result types in pg_proc
and pg_operator.  The lone error in pg_operator was reported as a bug
by Michael Reifenberger; the multiple errors in pg_proc would only have
been noticed if one invoked the functions by name rather than using
operator syntax.  I guess few people do that.

From: Constantin Teodorescu <teo@flex.ro>

UPgrade to 0.96 ..

Changes to fix/improve the dynamic loading on NT

From: Horak Daniel <horak@mmp.plzen-city.cz>

Fix OR index selectivity problem.

Fix for Alpha OR selectivity bug.

Have configure check for use of %lld for int64, and if that fails, check for
use of %qd...a more generic solution then having #ifdef __<INSERT OS HERE>__
in the code...

Export include/utils/mcxt.h so that external stuff can include
palloc.h again.  Move exporting of backend header files out of libpq's
Makefile (whatever was it doing there in the first place?) and into
backend/Makefile.

Modify mcxt.h so that it doesn't pull in half of creation.

Cleaner autoconf tests for int8 support.

geqo now at 11 tables

*** empty log message ***

Really remove Recipe.

Eliminate duplicated code in dt.h

Reduce DEF_MAXBACKENDS to 32 so that default configuration
is less likely to fail for lack of semaphores.

A few further tweaks to shared memory space estimation.
This change brings the default size of the main shmem block back under 1MB,
which is a fairly popular value for the kernel's SHMMAX parameter.

*** empty log message ***

*** empty log message ***