From: "D'Arcy J.M. Cain" <darcy@druid.net>
Subject: [HACKERS] backend/optimizer/geqo/geqo_erx.c

I sent these changes in with a bunch of others.  Some were folded in but
others, like these, were not.  I am not sure why so I am resending this
to the developers list by itself for discussion.

The readon why I suggest these changes is that the compiler can't tell
that minimum_count is initialized before it is used.  The tests that I
add in here will cause an immediate error if it doesn't.  As the comments
below suggest, if it is 100% guaranteed that the variable will always
be initialized then how this is so should be commented here.  I don't
know how much strain the actual test puts on the performance but if it
isn't too much then maybe leave it in for absolute safety anyway.

There are also a few returns just to stop warnings.

Fix for large objects and case sensitivity, from Raymond Toy.

bsdi configure detection fix.

allpaths.c:find_join_paths(): compute_rel_size() shouldn't be called
for join-relations. Sizes already computed by
prune_rel_paths():compute_joinrel_size().

joinrels.c:
<                   if ( _use_right_sided_plans_ )
---
>                   if ( _use_right_sided_plans_ &&
>                               length (outer_rel->relids) > 1 )

 - r_plans are useful when outer_rel is join-relation... It
decreases the size of search space...

Definition for RelationForgetRelation().

heap_destroy() now calls RelationForgetRelation() to really flush
the relation from the relcache.

New func RelationForgetRelation();
 *         RelationFlushRelation + if the relation is local then get rid of
 *         the relation descriptor from the newly created relation list.

UPdate TODO/HISTORY again.

Update TODO for release

Add bsdi to template detection.

Update comments for regression testing.
Include paragraph on upgrading old databases containing path or polygon types.

Use error message syntax consistant with other messages from the same
reference platform (Linux/gcc-lib/i686).

Update to reflect new "drop aggregate" syntax and messages.

Update to show new path and polygon syntax.

Update path and polygon syntax to new conventions.

Update to use new path geometric type syntax.
Order in streets.data changed and is now alphabetical (sorry).

Modify banner to suggest using PST8PDT time zone rather than full specifier
string.

Update to suggest using PST8PDT time zone rather than full time zone string.
Include section on error message differences.

Add upgradepath(), isoldpath(), upgradepoly() and revertpoly() to allow
upgrading from existing pre-v6.1 path and polygon geometric data types.

Put parens around macro arguments for safety per D'Arcy's suggestion.

Add upgradepath(), isoldpath(), upgradepoly() and revertpoly() to allow
migration from pre-v6.1 geometric data types.
Only allow new input syntax for paths and polygons.

Use finite() macro if available to check returns from pow() and exp().

Fix a few DATEDEBUG print statements.

cc1: warnings being treated as errors
dt.c: In function `timespan2tm':
dt.c:1722: warning: unused variable `funit'
dt.c:1722: warning: unused variable `iunit'

-> got rid of them.

SET geqo TO 'on' restores _use_geqo_rels_ to GEQO_RELS, not
to last specified with 'on' #.
What is better ?

cc1: warnings being treated as errors
remove.c: In function `RemoveAggregate':
remove.c:413: warning: unused variable `typename'

-> got rid of it.

SET var TO '...'
        ^^ - added

cc1: warnings being treated as errors
psql.c: In function `HandleSlashCmds':
psql.c:1141: warning: `optarg3' might be used uninitialized in this function
psql.c:1157: warning: `optarg3' might be used uninitialized in this function

-> char           *optarg3 = NULL;

Cleanups.

Add \dt and \di to psql help.

New description of currval.

#define GEQO_RELS 6 (not 7)

Fix deleted tuples re-incarnation possible when vacuum transaction
start time equal to tuple->t_tmax.

Privent shrinking if there are tuples modifyed by running transactions
(it concerns system relations only, currently).

*** empty log message ***

Update man and psqlHelp for new SET GEQO=# option.

Allow for EOF or \0 as input() return from lex.

Oracle like currval behaviour.

#define GEQO_RELS 7
moved from geqo.h to internal.h.

Use GEQO if _use_geqo_ is TRUE and # of relations is >= _use_geqo_rels_
(both are settable via SET geqo TO ...).

SET var TO 'a=b'
             ^^ is supported by get_token now.
(SET geqo TO 'on=XXX' works now).

From: Igor <igor@sba.miami.edu>
Subject: [PATCHES] patch for a memory leak

Well...I screwed up and posted the wrong patch for psql originally..
The patch for that patch wposted below will fix it..

From: David Friend <dfriend@atlsci.atlsci.com>
Subject: [PATCHES] INSTALL patch

This patch makes the following changes to the INSTALL instructions:
  - Before step 1, describe disk space requirements.
  - Step 1 now defines a "tested" platform.
  - Add step 3a on checking for disk space.
  - Added new step 27 asking for feedback.

From: Igor <igor@sba.miami.edu>
Subject: [PATCHES] Another destroydb patch

This is a patch to my previous destroydb patch cause some people wanted
slightly different behavior. After this patch is applied, destroydb
will destroy a database as usual, but if added -i flag (which could be
aliased like rm -i) would ask for confirmation.

From: Igor <igor@sba.miami.edu>
Subject: [PATCHES] pg_dump memory leak patch

This patch fixes a HUGE memory leak problem in pg_dump.
Pretty much anything that was allocated was never freed and Purify
reported about 40% possible memory leak and 6% actual leak. I added
functions to clear out all the allocated structures. After the patch
Purify returns 0 for number of bytes leaked...

Ordering of libraries changed so that -lbsd is *after* -lm

Fixes a problem Darren reports with AIX, and doesn't seem to break either
FreeBSD or Linux(David?)...

From: Darren King <aixssd!darrenk@abs.net>
Subject: [PATCHES] psql - \dt,\di commands.

I sent this a couple of months ago in re a request by Maxim
Kozin, but I had the patch reversed, creating some confusion
over applying it.

Here's a more complete version.

Adds \dt to list only tables/views and \di to list only
indicies.  \d will still work as before.

From: Phil Nelson <phil@cs.wwu.edu>
Subject: [PATCHES] Support for NetBSD/sparc

The following is a patch to allow BSD44_derived to support NetBSD/sparc.

From: Darren King <aixssd!darrenk@abs.net>
Subject: [PATCHES] aix.h needs POSIX...

From: Robert Bruccoleri <bruc@bms.com>

One more Irix related patch for Dynamic Linking

From: Igor <igor@sba.miami.edu>
Subject: [PATCHES] destroydb patch

I am including a patch for destroydb to ask for confirmation before
deleting databases (after I accidentally deleted mine)...destroydb -y
would force delete without any confirmation.

From: Igor <igor@sba.miami.edu>
Subject: [PATCHES] memory leak patches in libpq and psql

A couple of small memory leak patches (detected with Purify) primarily
in libpq.

* Fixed (NULL) border problem in psql (run psql, do \m, then select
  something from a table...row separators will be nulls)
* Fixed memory leak with the abovementioned border not being freed
  properly.
* Fixed memory leak in freePGconn() not freeing conn->port
* Fixed up PQclear() to free parts of PGresult only if these
  parts are not null.
* Fixed a decent memory leak that occured after executing every command
  in psql. PGresult *results was not freed most of the time.

There is still a leak being detected (2 bytes) in readline functions, but
I think this is old readline library. I will install new one and test it.

Added \n to error message.

Add R_PLANS to psql help,  and SET cleanup.

DROP agg psql help change from Darren.

SET R_PLANS added to manual page.

Put back old lines.

Add bsdi 3.0 to configure.

Cleanup for AIX from Darren.

Escape newline in select output, from A. Duursma.

Enable GEQO for more than six tables, from Martin.

Changed 1943 to 1947 to fix, other cleanups.

Fix for SELECT * INTO TABLE for char(), varchar() fields.

If there is no table in RangeTable for colname then
elog (WARN, "attribute %s not found", colname);

If we have to split leaf page in the chain of duplicates
then we try to look at our right sibling first, but not farther,
as it was in yesterday fix.

Added: new BTP_CHAIN flag (if hikey == firstkey then it's not
last page in chain of duplicates).

Fix duplicates handling.

Clean up support for USE_POSIX_TIME, ! HAVE_INT_TIMEZONE machines.
Remove references to modf() which is buggy on some platforms (Sparc/Linux).

Ensure double alignment for time and arrays of circles.

From: David Friend <dfriend@atlsci.atlsci.com>
Subject: [PATCHES] INSTALL changes

This patch modifies the INSTALL file.  The changes are:
  - SPARC/Linux-ELF was added to the list of supported platforms.
    The special notes for it at the bottom of the file were removed.
  - Changed "database server" to "RDBMS database server".
  - Modified step 6 so that when you restore your database the
    tables will be owned by the original owners instead of the
    postgresql superuser.
  - Modified step 19 on diagnosing the regression tests for the
    SPARC Linux platform with a beta release.
  - Other minor changes.

From: David Friend <dfriend@atlsci.atlsci.com>
Subject: [PATCHES] pqcomprim.c patch

This is the patch by Robert Bruccoleri to fix the endian problem.
(Actually, it's the reverse of his patch.  He must have gotten the
order wrong.)

-m <arg> -M weren't setup properly in getopts()...

These are undocumented in the man pages though...should they be removed?

Reported by: "A. Duursma" <it@portsite.nl>

Fix core dump on \c.

Fix \c option after breaking it.

Clean up redundant tests for valid pointers in geometric types.
Fix up decoder field masks for timespan and reltime.

Enable new date and time definitions to allow Solaris and Irix -O2 compilation.
Move date and time field type labels to less than 32 to allow use as masks.

AllocateFile():
   fdleft = pg_nofile() - allocatedFiles - nfile;
looks more realistic, but too noisy -
   fdleft = pg_nofile() - allocatedFiles;
restored.

Only free memory if environment value is replaced.

Comment on fix added.

Preserve putenv() memory in psql \connect user fix.

Fix backslash commands broken when \connect user added.

RelationPurgeLocalRelation():
            /*
             * RelationFlushRelation () below will flush relation information
             * from the cache. We must call smgrclose to flush relation
             * information from SMGR & FMGR, too. We assume that for temp
             * relations smgrunlink is already called by heap_destroyr
             * and we skip smgrclose for them.          - vadim 05/22/97
             */
            smgrclose(reln->rd_rel->relsmgr, reln);

 - it avoids memory leaks in SMGR & VFD.

RelationFlushRelation():
   there is no more call FileInvalidate(RelationGetSystemPort(relation));
   - invalid (FileInvalidate() expects File, not SMGR' fd)
   - unuseful anyway.

1. Fix md memory leak:
   mdunlink() and mdclose() (too !!!) now free MdfdVec for relation
   and add it to free list, so it may be re-used for another relation
   later.
2. Fix VFD-manager memory leak (found by Massimo ... and me):
   mdunlink() has to call FileUnlink() to free allocation for fileName
   and add the Vfd slot to the free list.

1. Cleanup (no more FreeFd - unuseful; others).
2. FreeFile() has to do nothing with nfile (# of files opened
   by VFD manager).

gist.h:
/*
** You can have as many strategies as you please in GiSTs, as
** long as your consistent method can handle them
*/
#define GISTNStrategies                 100
                                        ^^^
- too big number:

strat.h->StrategyEvaluationData->StrategyExpression  expression[12]
                                                                ^^
 - so 12 is real max # of strategies, or StrategyEvaluationIsValid
crashes backend (called if CASSER defined).

1. Pass GISTENTRYs to giststate->penaltyFn by pointers, not by vals.
2. Re-initialize keys in gistrescan (if gist used in inner scan).

From: Darren King <aixssd!darrenk@abs.net>
To: pgsql-patches@postgreSQL.org
Subject: [PATCHES] DROP AGGREGATE gram.y typo...

Somehow I dropped a comma in the gram.y part (line 129) of my
patch for drop aggregate.  Here's a correct patch for gram.y.

PS. I hope I got the right comma, manually applied :) (scrappy)

From: Darren King <aixssd!darrenk@abs.net>
Subject: [PATCHES] DROP AGGREGATE patch/fix.

Here's a patch that fixes the DROP AGGREGATE command to delete
the desired aggregate for a specific type.

Oops, missed the commit on this one by Darren King also...new Makefile.aix

From: Darren King <aixssd!darrenk@abs.net>
Subject: [PATCHES] AIX make patch resubmitted.

Misc patches for AIX from Darren:

1)  New src/makefiles/Makefile.aix  This patch should only be
    applied if the following patch (4) is applied to backend/Makefile!
    Still looking into having configure determine the last line to do
    the shared link.  The 325 code will work for 41, so I put that in
    as the default.  Included a commented out 41 line for completeness.

*and*

4)  Patch the backend Makefile.  I've reviewed this patch with respect to the
    other ports that use MAKE_EXPORTS (svr4 and univel) as closely as I could
    and I don't see where it will break them.  If it does, please let me know
    and I'll rework it somehow.

From: Darren King <aixssd!darrenk@abs.net>
Subject: [PATCHES] Re: [PORTS] AIX 6.1 fixes...

Here are the patches for the two things that wouldn't make it thru the AIX
compiler.  The geo_ops.c change is harmless I believe.  The nbtcompare.c patch
fixes me, but I don't know about any other ports.  Maybe wait on that one
until Vadim decides what to do about the unsigned vs signed chars varlena
issue.

Updated pg_dumpall and psql to preserve database owners.

Add SRCDIR to lextest

Fix for lextest return value.

Little memmory leak in RelationFlushRelation() in freeing
relation->rd_att (relation' TupleDesc).

Define FREE(x) as pfree.

Shouldn't we use palloc instead of malloc ?
Because of
 *      resetpsort  - resets (frees) malloc'd memory for an aborted Xaction
 *
 *      Not implemented yet.

1. LocalBufferSync(): set nextFreeLocalBuf to 0 - we're freeing
   all local buffers @ xact commit, so accordingly nextFreeLocalBuf
   is first local buffer now.
   It helps to avoid unnecessary local buffer allocations in LocalBufferAlloc()
   latter ("memmory leaks" in 'order by').
2. ResetLocalBufferPool() lost allocated local buffers:

   memset(LocalBufferDescriptors, 0, sizeof(BufferDesc) * NLocBuffer);

   (local buffers leak @ xact aborts).

Consider right-sided plans if _use_right_sided_plans_ is true.

New VAR r_plans added to enable turn ON/OFF
using right-sided plans.

Initialize internal keys if rtree used in inner scan.