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 ***

Partial fix for copied-plan bugs reported by Hiroshi Inoue:
_copyResult didn't copy subPlan structure completely.  _copyAgg is still
busted, apparently because of changes from EXCEPT/INTERSECT patch
(get_agg_tlist_references is no longer sufficient to find all aggregates).
No time to look at that tonight, however.

Executor no longer cares about mergejoinop, mergerightorder, mergeleftorder,
so remove them from MergeJoin node.  Hack together a partial
solution for commuted mergejoin operators --- yesterday
a mergejoin int4 = int8 would crash if the planner decided to
commute it, today it works.  The planner's representation of
mergejoins really needs a rewrite though.
Also, further testing of mergejoin ops in opr_sanity regress test.

From: Michael Meskes <Michael_Meskes@topmail.de>

+
+ Tue Feb 23 17:32:25 CET 1999
+
+       - Other than a struct a union itself cannot be specified as variable.
+
+ Fri Feb 26 07:18:25 CET 1999
+
+       - Synced preproc.y with gram.y.
+
+ Sat Feb 27 20:30:03 CET 1999
+
+       - Added automatic allocating for NULL pointers.

Fix pg_operator entries for mergejoinable operators with different left and
right side data types.  Correct the opr_sanity regress test to check these
entries properly.  NOTE that opr_sanity will now fail until you do an initdb!

Fix executor to work correctly with mergejoins where left and
right sides have different data types.

Remove recipe.o from Makefile ... dead code not quite dead enough,
it seems.

Put back T_RecipeStmt, since the system doesn't actually build
without it.  Sloppy, sloppy.

Someone forgot to commit gram.c and parse.h after his latest
set of updates to gram.y.

Remove reference to recipe.h.

Fix problem with selectivity error in added columns with ALTER TABLE.
Move files to deadcode.

update comments

Thank you for the advice.  I concluded that current inet code has a
portability problem. Included patches should be applied to both
current and 6.4 tree. I have tested on LinuxPPC, FreeBSD and Solaris
2.6. Now the inet regression tests on these platforms are all happy.
---
Tatsuo Ishii

From: Michael Meskes <Michael_Meskes@topmail.de>

+
+ Son Feb 21 14:10:47 CET 1999
+
+       - Fixed variable detection in libecpg.
+
+ Mon Feb 22 19:47:45 CET 1999
+
+       - Added 'at <db_connection>' option to all commands it is apllicable
+         to. Due to changing the API of some libecpg functions this
+         requires me to increase the major version number.
+       - Synced pgc.l with scan.l.
+       - Added support for unions.
+       - Set library version to 3.0.0
+       - Set ecpg version to 3.0.0

Add parameters to function calls to pass back table qualifications.

Add a few other parser-only nodes for debugging help.
Define the JoinExpr node.

Define JoinExpr structure for outer join syntax.
Clean up comments in execnodes.h.

Clean up error messages.

Do a better job of selecting candidates among functions
 when no exact match.
Clean up elog error messages.

Include some new code for outer joins. Disabled by default, but enable by
 including the following in your Makefile.custom:
 CFLAGS+= -DENABLE_OUTER_JOINS -DEXEC_MERGEJOINDEBUG

Put in explicit checks for implicit index name lengths.
Put in hooks for outer joins by passing a few parameters back and forth
 in function calls. May not be close to working yet.

Modify the outer join placeholder code with something closer to working
 code. Works here, but not completely implemented past this point.

Bring debugging print statement declarations up to date.
Comment-out the #undef default declarations to allow the parameters to be
 set on the compiler command line.

Add constants for outer join states in executor.