aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
* I guess I'd vote for changing the code to beBruce Momjian2002-08-14
| | | | | | | | | | | | | | sys = malloc(strlen(editorName) + strlen(fname) + 10 + 1); if (!sys) return false; sprintf(sys, "exec '%s' '%s'", editorName, fname); (note the added quotes to provide a little protection against spaces and such). Then it's perfectly obvious what the calculation is doing. I don't care about wasting 20-some bytes, but confusing readers of the code is worth avoiding. regards, tom lane
* Fix memory leak in SetClientEncoding().Tatsuo Ishii2002-08-14
|
* Add Cyrillic and other encodings for encoding conversion.Tatsuo Ishii2002-08-14
| | | | Patches submitted by Kaori Inaba (i-kaori@sra.co.jp).
* Put back old code.Bruce Momjian2002-08-13
|
* Change NAMEDATALEN to 64, INDEX_MAX_KEYS/MAX_FUNC_ARGS to 32, per ↵Bruce Momjian2002-08-13
| | | | discussion on hackers.
* Fix tuple-chain-moving tests to handle marked-for-update tuples correctlyTom Lane2002-08-13
| | | | | | | | (they are not part of a chain). When failing to find a parent tuple in an update chain, emit a warning and abandon repair_frag, but do not give an error as before. This should eliminate the infamous 'No one parent tuple was found' failure, which we now realize is not a can't-happen condition but a perfectly valid database state. Per recent pghackers discussion.
* Make sure that t_ctid is reset to equal t_self in heap_delete andTom Lane2002-08-13
| | | | | | | | | heap_mark4update; this avoids situations where a deleted tuple might look like it is chained to something else. Also, cause all the WAL redo routines to set t_ctid to equal t_self, rather than leaving it undefined as before. Make heap_xlog_clean set the page's LSN and SUI correctly. All per past discussions in pghackers, ranging back to last December.
* Remove obsolete uses of lanispl. Only used in pg_dump now, but can bePeter Eisentraut2002-08-13
| | | | removed altogether if pg_dump doesn't need it anymore.
* Remove use of __P so that <sys/cdefs.h> is not needed. Per suggestionTom Lane2002-08-12
| | | | from Martin Renters.
* Convert possibly-unsafe sprintf's to snprintf's.Tom Lane2002-08-12
|
* Buffer-overrun paranoia.Tom Lane2002-08-12
|
* Restructure rowtype-parameter handling to eliminate need for possibly-Tom Lane2002-08-12
| | | | | overflowable buffer for 'name%rowtype'; not to mention avoid problems with mixed-case type names and other special cases.
* Change memory-space accounting mechanism in tuplesort.c and tuplestore.cTom Lane2002-08-12
| | | | | | | | to make a reasonable attempt at accounting for palloc overhead, not just the requested size of each memory chunk. Since in many scenarios this will make for a significant reduction in the amount of space acquired, partially compensate by doubling the default value of SORT_MEM to 1Mb. Per discussion in pgsql-general around 9-Jun-2002..
* Code review of CLUSTER patch. Clean up problems with relcache gettingTom Lane2002-08-11
| | | | confused, toasted data getting lost, etc.
* Create/drop cast now requires ownership of at least one of the types.Peter Eisentraut2002-08-11
|
* Make cluster regress test functional.Tom Lane2002-08-11
|
* Neil's patch was supposed to remove this, too.Tom Lane2002-08-11
|
* Now that we allow ANALYZE to run inside a transaction block, the locksTom Lane2002-08-11
| | | | | | | | | | | it takes could be held for quite awhile after the analyze step completes. Rethink locking of pg_statistic in light of this fact. The original scheme took an exclusive lock on pg_statistic, which was okay when the lock could be expected to be released shortly, but that doesn't hold anymore. Back off to a normal writer's lock (RowExclusiveLock). This allows concurrent ANALYZE of nonoverlapping sets of tables, at the price that concurrent ANALYZEs of the same table may fail with 'tuple concurrently updated'.
* Update with newer version of CLUSTER patch posted August 8.Bruce Momjian2002-08-10
| | | | | | | | | | | | | | | | | > Looks like Alvaro got sideswiped by the system catalog indexing changes > I made over the weekend. It's a simple change, just reduce the whole > mess to a "CatalogUpdateIndexes()" call. I update two tuples, so I manually CatalogOpenIndexes() and CatalogIndexInsert() two times, as per comments in CatalogUpdateIndexes(). I also removed a couple of useless CommandCounterIncrement(), some useless definitions in src/include/commands/cluster.h and useless includes in src/backend/commands/cluster.c. This version passes the regression test I had made for previous versions. Alvaro Herrera
* Remove error files.Bruce Momjian2002-08-10
|
* > create view pg_locks as select l.relation, l.database, l.backendpid,Bruce Momjian2002-08-10
| | | | | | | | | | | | | > l.mode, l.isgranted from pg_lock_info() as l(relation oid, database oid, > backendpid int4, mode text, isgranted bool); > ERROR: badly formatted planstring "COLUMNDEF "... > Reported by Neil Conway -- I never implemented readfuncs.c support for ColumnDef or TypeName, which is needed so that views can be created on functions returning type RECORD. Here's a patch. Joe Conway
* Major improvement in CLUSTER which preserves table characteristics usingBruce Momjian2002-08-10
| | | | | | | | | | | | relfilenode. I sent the CLUSTER patch a few days ago and I think it was missed. I append it again, this time including the regression test files. For the committer, please note that you have to cvs add the files as they don't exist. Maybe add to the parallel and serial schedules also, but I don't know such stuff. Alvaro Herrera (<alvherre[a]atentus.com>)
* This patch removes a lot of unused code related to assertions andBruce Momjian2002-08-10
| | | | | | | | | | | | error handling, and simplifies the code that remains. Apparently, the code that left Berkeley had a whole "error handling subsystem", which exceptions and whatnot. Since we don't use that anymore, there's no reason to keep it around. The regression tests pass with the patch applied. Unless anyone sees a problem, please apply. Neil Conway
* psql prints its version number in its startup message, per recentTom Lane2002-08-10
| | | | | discussion. Also, cause the \timing command to display time in a format consistent with the backend's EXPLAIN ANALYZE output.
* Allow schema-qualified operator names to be used in the optionalTom Lane2002-08-10
| | | | arguments of CREATE OPERATOR.
* Fix failure to relink postmaster executable in the first make run if only aPeter Eisentraut2002-08-10
| | | | single source file a few directories deep in the backend tree has changed.
* Make sure all clients have the same understanding of default user namePeter Eisentraut2002-08-10
| | | | | and database. In particular, make script wrappers understand the PGDATABASE environment variable.
* Get rid of double PQclear(res) in error path.Tom Lane2002-08-10
|
* Add a sanity check to make sure that all system catalogs that have OIDsTom Lane2002-08-10
| | | | also have a unique index on OID.
* psql backslash commands are schema-aware. Pattern matching behaviorTom Lane2002-08-10
| | | | | follows recent pghackers discussion. This commit includes all the relevant fixes from Greg Mullane's patch of 24-June.
* Make sure monetary, numeric, and time locale categories are set to C andPeter Eisentraut2002-08-09
| | | | are only activated temporarily to read out formatting information.
* Column ordering in \d output should agree with sort ordering;Tom Lane2002-08-09
| | | | per Joe Conway's patch of 20-July.
* has_table_privilege spawns scions has_database_privilege, ↵Tom Lane2002-08-09
| | | | | | | | | | | has_function_privilege, has_language_privilege, has_schema_privilege to let SQL queries test all the new privilege types in 7.3. Also, add functions pg_table_is_visible, pg_type_is_visible, pg_function_is_visible, pg_operator_is_visible, pg_opclass_is_visible to test whether objects contained in schemas are visible in the current search path. Do some minor cleanup to centralize accesses to pg_database, as well.
* Change LOCK statements to start new transactionHiroshi Inoue2002-08-09
| | | | in auto-commit off mode.
* Make the standard system views schema-aware.Tom Lane2002-08-08
| | | | | I did not force an initdb via catversion ... but the rules regression test will fail until you do an initdb.
* Arrange to collect statistics on a few more system catalog columns.Tom Lane2002-08-08
|
* Cause view/rule display to work as expected after rename of an underlyingTom Lane2002-08-08
| | | | table or column, or of an output column of the view itself.
* Address build problems on cygwin and (hopefully) AIX.Tatsuo Ishii2002-08-08
|
* Load and keep conversion function info when SET CLIENT_ENCODING TO isTatsuo Ishii2002-08-08
| | | | | executed to prevent database access while performing encoding conversion.
* Remove #ifdef MULTIBYTETatsuo Ishii2002-08-08
|
* Cause schema-qualified FROM items and schema-qualified variable referencesTom Lane2002-08-08
| | | | | | to behave according to SQL92 (or according to my current understanding of same, anyway). Per pghackers discussion way back in March 2002: thread 'Do FROM items of different schemas conflict?'
* Clean up plpgsql identifier handling: process quoted identifiersTom Lane2002-08-08
| | | | | | | | | correctly, truncate to NAMEDATALEN where needed, allow whitespace around dots in qualified identifiers. Get rid of T_RECFIELD and T_TGARGV token categories, which weren't accomplishing anything except to create room for sins of omission in the grammar, ie, places that should have allowed them and didn't. Fix a few other bugs en passant.
* parseTypeString wasn't quite paranoid enough.Tom Lane2002-08-08
|
* Fix permission checking for temp-table namespace.Tom Lane2002-08-07
|
* Improve 'failed to initialize' messages.Tom Lane2002-08-07
|
* Updated translation from Dennis Bjorklund <db@zigo.dhs.org>Peter Eisentraut2002-08-06
|
* Still more paranoia in PageAddItem: disallow specification of an itemTom Lane2002-08-06
| | | | | | | | | offset past the last-used-item-plus-one, since that would result in leaving uninitialized holes in the item pointer array. AFAICT the only place that was depending on this was btree index build, which was being cavalier about when to fill in the P_HIKEY pointer; easily fixed. Also a small performance improvement: shuffle itemid's by means of memmove, not a one-at-a-time loop.
* Remove no-longer-used PageManagerMode enum.Tom Lane2002-08-06
|
* Move pg_convert3 declaration to suppress compile warning. ReallyTom Lane2002-08-06
| | | | | pg_convert3 does not belong in a backend/catalog file at all, IMHO; it should be in utils/adt.
* Add SQL99 CONVERT() function.Tatsuo Ishii2002-08-06
|