Make src/backend/port/*.c file location dependent only on configure.in.

I have committed many support files for CREATE CONVERSION. Default
conversion procs and conversions are added in initdb. Currently
supported conversions are:

UTF-8(UNICODE) <--> SQL_ASCII, ISO-8859-1 to 16, EUC_JP, EUC_KR,
    EUC_CN, EUC_TW, SJIS, BIG5, GBK, GB18030, UHC,
    JOHAB, TCVN

EUC_JP <--> SJIS
EUC_TW <--> BIG5
MULE_INTERNAL <--> EUC_JP, SJIS, EUC_TW, BIG5

Note that initial contents of pg_conversion system catalog are created
in the initdb process. So doing initdb required is ideal, it's
possible to add them to your databases by hand, however. To accomplish
this:

psql -f your_postgresql_install_path/share/conversion_create.sql your_database

So I did not bump up the version in cataversion.h.

TODO:
Add more conversion procs
Add [CASCADE|RESTRICT] to DROP CONVERSION
Add tuples to pg_depend
Add regression tests
Write docs
Add SQL99 CONVERT command?
--
Tatsuo Ishii

Done, not sure when, reported by Neil Conway:

> * -Report failure to find readline or zlib at end of configure run

Improve getopt_long search, per Peter:

AC_SEARCH_LIBS(getopt_long, [getopt])

Add more dependency insertions --- this completes the basic pg_depend
functionality.  Of note: dropping a table that has a SERIAL column
defined now drops the associated sequence automatically.

fixed bug in support for timestamp without time zone reported by Yuva Chandolu (ychandolu@ebates.com)

Oops, sometimes strtol isn't called in pg_atoi, so we do need that badp
check.

Fix tid to in/out as unsigned.

Add initdb code to set up initial contents of pg_depend, pinning all
objects created during initdb (except for the system views, which I
think do not need to be pinned).

> pgsql-bugs@postgresql.org wrote:
>  > David Clark (dclarknospam@opsi.co.za) reports a bug with a severity
>  > Table 3-7 SQL Literal escaped octets shows the input escape
>  > representation for a single quote as '\\'' , but the third paragraph
>  > below table 3-8 SQL Output Escaped Octets says that the single quote
>  > must be input as '\''
>
> Nice catch. '\'' is correct as shown in the example in Table 3-7.
>
>  >
>  > Also in the same paragraph mentioned above it says input for the
>  > single quote must be '\'' (or '\\134') shouldn't this be (or '\\047')
>
> Also a bug. Should be '\\047', as you pointed out.
>

Here's a patch to fix the binary string doc errors.

Joe Conway

Add conversion procs for CREATE CONVERSION

Fix typo

Add FindDefaultConversionProc

Add code to extract dependencies from an expression tree, and use it
to build dependencies for rules, constraint expressions, and default
expressions.  Repair some problems in the original design of
recursiveDeletion() exposed by more complex dependency sets.  Fix
regression tests that were deleting things in illegal sequences.

Add DLLINIT mention in port/Makefile.

Move few remaining src/utils files to backend/port so everything is in
one place.  Everything may be moved to src/utils eventually.

Add DLLINIT variable to simplify makfiles.

Add mention of TOAST storage for character columns.

Mark 'line' as 'not implemented' in SGML and psql \dT, per Thomas
Lockhart.  initdb not forced.

This fixes 2 inaccuracies in the recently added SQL99 feature list docs.
UNIQUE and DISTINCT predicates are both listed as implemented -- AFAIK,
neither is.

I also included another trivial patch which adds the default location
of the DSSSL stylesheets on my system (Debian unstable, docbook-dsssl
1.76) to the list of paths that configure looks for.

Neil Conway

Add comment for isbn,issn data type, from Pete St. Onge

Item not needed:

< * Use our own getopt() for FreeBSD/OpenBSD to allow --xxx flags (Bruce)

Remove indenting of Makefile comments, per Peter.\

Remove utils/getopt.c file, not needed, it only allowed "--flag"
warnings, and we now look for getopt_long in libgetopt.a.

Add search for libgetopt.a in hopes of finding getopt_long().

Remove certain Makefile dependencies by using full pathnames in
configure.in.

Use the dependency mechanism to manage column defaults.  We need this
so that dependencies in default expressions (on operators, functions,
etc) can be expressed properly.

Fix make_ctags for exhuberant tags.

Fix a bug about the handling of '.' in parse.c.

Improve relcache.c error reporting for the next guy who has to debug
this thing.

Fix \? and \pset pager handling.  \? wasn't honoring pager before.

Add COMMENT ON CONSTRAINT facility (from Rod Taylor's pg_constraint patch).
Fix comment.c to not depend on parser token values, per discussion awhile
back.

Remove no-longer-appropriate notes about lack of CASCADE/RESTRICT behavior.

In DeleteAttributeTuples, use a single indexscan instead of the multiple
scans that will most likely be caused by SearchSysCache probes.  Also,
share some code between index deletion and table deletion.

Fix testing of partial-index predicates to work correctly in cases where
varno of index's relation is not 1.  This embarrassing oversight pointed
out by Dmitry Tkach 12-Jul-02.

Mark as done, per Rod:

> o -Add ALTER TABLE DROP non-CHECK CONSTRAINT
> * -Allow psql \d to show foreign keys
> * -Auto-destroy sequence on DROP of table with SERIAL; perhaps a separate
> * -Prevent column dropping if column is used by foreign key
> * -Automatically drop constraints/functions when object is dropped
> * -Make foreign key constraints clearer in dump file
> * -Make foreign keys easier to identify

Both done:

o -Add SET or BEGIN timeout parameter to cancel query
> * -Add pg_depend table for dependency recording; use sysrelid, oid,

Add SET statement_timeout capability.  Timeout is in ms.  A value of
zero turns off the timer.

Re-add -I documentation, per Tom.

Second phase of committing Rod Taylor's pg_depend/pg_constraint patch.
pg_relcheck is gone; CHECK, UNIQUE, PRIMARY KEY, and FOREIGN KEY
constraints all have real live entries in pg_constraint.  pg_depend
exists, and RESTRICT/CASCADE options work on most kinds of DROP;
however, pg_depend is not yet very well populated with dependencies.
(Most of the ones that are present at this point just replace formerly
hardwired associations, such as the implicit drop of a relation's pg_type
entry when the relation is dropped.)  Need to add more logic to create
dependency entries, improve pg_dump to dump constraints in place of
indexes and triggers, and add some regression tests.

Changed getCrossReference to return information about each composite key seperately
It used to return them as a,b in one row, and now returns
a in one row, and b in one row

An improvement of User Interface(Setup dialog) by Dave Page.

1) Fix a memory leak in use declare/fetch mode.
2) Change default build mode to multibyte(Windows).

Not a valid item:

< * Add version file format stamp to heap and other table types

Remove duplicate:

< * Use BlockNumber rather than int where appropriate

Change error messages ExecAppend->ExecInsert and ExecReplace->ExecUpdate
as discussed on hackers.

Remove references to pre-7.1;  too old.

Update FAQ.

Add new CREATE CONVERSION/DROP CONVERSION command.
This is the first cut toward CREATE CONVERSION/DROP CONVERSION implementaion.
The commands can now add/remove tuples to the new pg_conversion system
catalog, but that's all. Still need work to make them actually working.
Documentations, regression tests also need work.

Done, according to Rod Taylor:

> * -Add ALTER TRIGGER ... RENAME

Fix getopt flags.  Man thing thing was messed up.

Fix another --help typo in pg_restore.

Make NAMEDATALEN changeable at compile time.

Fix pg_restore flags in documentation.

fix bug in getTime() with fractional seconds reported by Laurette Cisneros (laurette@nextbus.com)

Oops, proper initialization for domainTypMod was none at all, not 0.

Fix compiler warning:

int32           domainTypMod = NULL;

should be:

int32           domainTypMod = 0;

Fix typo mentioned by Rich Morin.

Done:

> * -Make one version of simple_prompt() in code (Bruce, Tom)

Fix case syntax for freebsd template.

Fix typo in xl_heaptid comment

Manfred Koizar

Move CXX platform-specific stuff into template files.

Convert expr to case, for Peter E.

I've fixed up the way domain constraints (not null and type length)
are managed as per request.

Moved from merging with table attributes to applying themselves during
coerce_type() and coerce_type_typmod.

Regression tests altered to test the cast() scenarios.

Rod Taylor

Add comments about sharing.

Move simple_prompt() into its own file to be shared with psql and pg_dump.

Don't document that UNDO is certain to be added in the future.

fixed bug reported by Michael, Dietrich (mdt@emdete.de) where a large object handle was being used after the end of the transaction and thus resulting in an error.

Add a few new lines to display recently added fields in the ControlFile
 structure.
Now includes the following new fields:
 integer/float date/time storage
 maximum length of names (+1; they must also include a null termination)
 maximum number of function arguments
 maximum length of locale name

Mark person:

> * -Allow psql \d to show temporary table structure (Tom)

Done

> * -Allow psql \d to show temporary table structure

Fix a bug reported by Zhou Han.

Add explanation of the various *_min_messages elog() values.

Cleanup.

Fix capitalization.

Add NULL space mention.

Fix compile error in assert coded added by new DISTINCT ON patch.

More clearly document in pg_dump when we are dealing with an object name
as it appears in the schema dump, and index tags.

Implement the IS DISTINCT FROM operator per SQL99.
Reused the Expr node to hold DISTINCT which strongly resembles
 the existing OP info. Define DISTINCT_EXPR which strongly resembles
 the existing OPER_EXPR opType, but with handling for NULLs required
 by SQL99.
We have explicit support for single-element DISTINCT comparisons
 all the way through to the executor. But, multi-element DISTINCTs
 are handled by expanding into a comparison tree in gram.y as is done for
 other row comparisons. Per discussions, it might be desirable to move
 this into one or more purpose-built nodes to be handled in the backend.
Define the optional ROW keyword and token per SQL99.
 This allows single-element row constructs, which were formerly disallowed
 due to shift/reduce conflicts with parenthesized a_expr clauses.
Define the SQL99 TREAT() function. Currently, use as a synonym for CAST().

Move INTERSECT DISTINCT to the supported category. Error in docs.

Document function args are required for pg_restore -P.
Fix pg_dump to not quote the function name in the storage tag.
Fix pg_dump so GRANT/REVOKE(ACL) tag entries are not quoted, for
consistency.
Fix pg_restore to properly handle quotes and some spaces in -P.

Update FreeBSD template to properly compile c++ on alpha.

Add missing pgaccess copyright file, same as ours.

Done:

>         o -Add support for CallableStatements

Fix some more boundary-case errors in psql variable substitution:
wasn't really right for case where :var is at the end of the line,
was definitely not right if var expanded to empty in that case,
and failed to recalculate thislen before jumping back to rescan.

Disable pg_upgrade for 7.3.

Allow make_ctags to work with exuberant tags.

Indent libpq++ as mentioned in email.  Format was terrible, and this
will make fixing things easier.

>the extra level of struct naming for pd_opaque has no obvious
>usefulness.
>
>> [...] should I post a patch that puts pagesize directly into
>> PageHeaderData?
>
>If you're so inclined.  Given that pd_opaque is hidden in those macros,
>there wouldn't be much of any gain in readability either, so I haven't
>worried about changing the declaration.

Thanks for the clarification.  Here is the patch.  Not much gain, but at
least it saves the next junior hacker from scratching his head ...

Manfred Koizar

Update to pgaccess 0.91.

The attached patch fixes 2 memory leaks in pg_dump, as well as corrects
a spelling mistake and deletes an antiquated comment.

Neil Conway

The attached patch fixes a minor memory leak in psql.

Neil Conway

There already was a macro PageGetItemId;  this is now used in (almost)
all places, where pd_linp is accessed.  Also introduce new macros
SizeOfPageHeaderData and BTMaxItemSize. This is just source code
cosmetic, no behaviour changed.

Manfred Koizar

Minor code cleanup in bufmgr.c and bufmgr.h, mainly by moving repeated
lines of code into internal routines (drop_relfilenode_buffers,
release_buffer) and by hiding unused routines (PrintBufferDescs,
PrintPinnedBufs) behind #ifdef NOT_USED. Remove AbortBufferIO()
declaration from bufmgr.c (already declared in bufmgr.h)

Manfred Koizar

This patch, which is built upon the "HeapTupleHeader accessor macros"
patch from 2002-06-10, is supposed to reduce the heap tuple header size
by four bytes on most architectures.  Of course it changes the on-disk
tuple format and therefore requires initdb.

This overlays cmin/cmax/xmax fields into only two fields.

Manfred Koizar

Back out regress/results changes.  Real CVS problem worked-around.

Fix removal of results directory on 'make clean'.  Just remove *.out
files.  Was causing CVS problems.

First phase of applying Rod Taylor's pg_depend patch.  This just adds
RESTRICT/CASCADE syntax to the DROP commands that need it, and propagates
the behavioral option through the parser to the routines that execute
drops.  Doesn't do anything useful yet, but I figured I'd commit these
changes so I could get out of the parser area while working on the rest.

Commit old versions into main branch again.

Michael

Committing parser changes. Note, however, that the development bison seems ot have a problem on my home machine. So these go in untested for the time being. But at least I have them in the archive.

Michael

Not done:

> * Add the functionality for WITH CHECK OPTION clause of CREATE VIEW