aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
* Cause planner to include costs of InitPlans in startup cost of parentTom Lane2002-12-19
| | | | plan node. Not sure why this oversight has persisted so long ...
* pgindent fe-connect.c --- done to make IPv6 patch easier to apply.Bruce Momjian2002-12-19
|
* Update RELEASE checklist.Bruce Momjian2002-12-18
|
* From the SSL_CTX_new man page:PostgreSQL Daemon2002-12-18
| | | | | | | | | | | | | "SSLv23_method(void), SSLv23_server_method(void), SSLv23_client_method(void) A TLS/SSL connection established with these methods will understand the SSLv2, SSLv3, and TLSv1 protocol. A client will send out SSLv2 client hello messages and will indicate that it also understands SSLv3 and TLSv1. A server will understand SSLv2, SSLv3, and TLSv1 client hello messages. This is the best choice when compatibility is a concern." This will maintain backwards compatibility for those us that don't use TLS connections ...
* Update EvalPlanQual() to work with new executor memory management method.Tom Lane2002-12-18
| | | | It doesn't leak memory anymore ...
* Prevent core dump when ExecMarkPos is called before any tuples have beenTom Lane2002-12-18
| | | | | retrieved. This cannot happen in ordinary execution, but it can happen under EvalPlanQual().
* Skip unnecessary plan-copying now that plan trees are read-only in theTom Lane2002-12-17
| | | | executor.
* Fix typo.Tom Lane2002-12-17
|
* To suppress memory leakage in long-lived Lists, lremove() should pfreeTom Lane2002-12-17
| | | | | the cons cell it's deleting from the list. Do this, and fix a few callers that were bogusly assuming it wouldn't free the cons cell.
* Fix GEQO to work again in CVS tip, by being more careful about memoryTom Lane2002-12-16
| | | | | | | | | | | allocation in best_inner_indexscan(). While at it, simplify GEQO's interface to the main planner --- make_join_rel() offers exactly the API it really wants, whereas calling make_rels_by_clause_joins() and make_rels_by_clauseless_joins() required jumping through hoops. Rewrite gimme_tree for clarity (sometimes iteration is much better than recursion), and approximately halve GEQO's runtime by recognizing that tours of the forms (a,b,c,d,...) and (b,a,c,d,...) are equivalent because of symmetry in make_join_rel().
* Fix ALTER TABLE ADD COLUMN to disallow the same column types that areTom Lane2002-12-16
| | | | | | | | disallowed by CREATE TABLE (eg, pseudo-types); also disallow these types from being introduced by the range-function syntax. While at it, allow CREATE TABLE to create zero-column tables, per recent pghackers discussion. I am back-patching this into 7.3 since failure to disallow pseudo-types is arguably a security hole.
* Code review for palloc0 patch --- avoid dangerous and unnecessaryTom Lane2002-12-16
| | | | | | | | practice of evaluating MemSet's arguments multiple times, except for the special case of newNode(), where we can assume the argument is a constant sizeof() operator. Also, add GetMemoryChunkContext() to mcxt.c's API, in preparation for fixing recent GEQO breakage.
* Tweak default memory context allocation policy so that a context is notTom Lane2002-12-15
| | | | | | | | | given any malloc block until something is first allocated in it; but thereafter, MemoryContextReset won't release that first malloc block. This preserves the quick-reset property of the original policy, without forcing 8K to be allocated to every context whether any of it is ever used or not. Also, remove some more no-longer-needed explicit freeing during ExecEndPlan.
* Revise executor APIs so that all per-query state structure is built inTom Lane2002-12-15
| | | | | | a per-query memory context created by CreateExecutorState --- and destroyed by FreeExecutorState. This provides a final solution to the longstanding problem of memory leaked by various ExecEndNode calls.
* Add major/minor release changes info to RELEASE_CHANGES file.Bruce Momjian2002-12-14
|
* The actual segfault was caused by a double pfree(), but ISTM thatBruce Momjian2002-12-14
| | | | | | | | failing to find pg_hba.conf should be a fatal error anyway, so I increased the priority of the elog() from LOG to FATAL and refactored the code a little bit. Neil Conway
* Ok, I think I've gotten this figured out now. I saw this comment inBruce Momjian2002-12-14
| | | | | | | pqcomm.c, switched the ERROR logs to COMMERROR logs and it all works. I've attached a patch to be-secure.c that fixes all my problems. Nathan Mueller
* Initial version of the SQL information schemaPeter Eisentraut2002-12-14
|
* Clean up plantree representation of SubPlan-s --- SubLink does not appearTom Lane2002-12-14
| | | | | | | | in the planned representation of a subplan at all any more, only SubPlan. This means subselect.c doesn't scribble on its input anymore, which seems like a good thing; and there are no longer three different possible interpretations of a SubLink. Simplify node naming and improve comments in primnodes.h. No change to stored rules, though.
* Increment libpq major number for 7.3.1 and minor for 7.4.Bruce Momjian2002-12-13
|
* Admit defeat on the prospect of keeping straight exactly which platformsTom Lane2002-12-13
| | | | | | | | produce which output in the geometry test, even with the problem narrowed down to only whether they print minus zero or not. Instead, use pg_regress' locale-variant mechanism to automatically consider the test to pass if it matches either supplied comparison file. geometry_1.out replaces the former geometry-positive-zeros.out.
* Protect from null param on ecpg disconnect, verified by Roland KarchBruce Momjian2002-12-13
|
* Add a basic regression test for IS DISTINCT FROM, which has spent way tooTom Lane2002-12-13
| | | | much time in a broken state for lack of anyone noticing.
* Phase 3 of read-only-plans project: ExecInitExpr now builds expressionTom Lane2002-12-13
| | | | | | | execution state trees, and ExecEvalExpr takes an expression state tree not an expression plan tree. The plan tree is now read-only as far as the executor is concerned. Next step is to begin actually exploiting this property.
* Adjust costsize calculations to avoid introducing unnecessary roundoffTom Lane2002-12-13
| | | | | error. This seems to explain the differing choice of plan that's been causing geometry regress test to fail for the last few days.
* Remove strerror output for openssl SYSCALL error check.Bruce Momjian2002-12-13
|
* Here's the patch I mentioned on HACKERS to fix up ALTER command tabBruce Momjian2002-12-13
| | | | | | | | | | completion. Note that it's based on 7.3 tarball, not CVS HEAD, or 7.3rel branch. Damn, looking at CVS, this will patch into 7.3rel (just tested, it does) probably collide with Rod Taylor's patch adding ALTER TRIGGER stuff. O.K, second patch attached against HEAD - not tested, hand merged. Ross Reedstrom
* > There is an ugly little problem with the DB wrapper class.Bruce Momjian2002-12-12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | > > In pg.py the attributes of DB are defined as being the same as > the attributes of the corresponding pgobject "db", using the following ... > The problem is that the attributes of db (which are read only) > are not static (they are actually function calls to PostgreSQL), > especially "status" and "error", but those attributes are copied > and this is done only once when initializing the DB object. > > So, in effect, only the attribute "db.error" of a DB instance > will be updated, but not the attribute "error". Same with "status". > Don't copy the (read only) attributes of the pgobject to the > DB object, but only the methods, and all of them, like this: > > --------------- change in pg.py ------------------ > # Create convience methods, in a way that is still overridable. > for e in self.db.__methods__: > setattr(self, e, getattr(self.db, e)) > ---------------------------------------------------- > > Furthermore, make an addition to the documentation of the > DB wrapper class (i.e. in pygresql-pg-db.html): > After the sentence "All pgobject methods are included in this class also." > add the following sentence "The pgobject read-only attributes can be > accessed py adding the prefix 'db.' to them." Christoph Zwerschke
* Check SSL_get_error() value SSL_ERROR_SYSCALL to see if SSL_read()Bruce Momjian2002-12-12
| | | | | | returned -1, per SSL_get_error() documentation. Nathan Mueller
* While porting my TPC-C implementation from Oracle, I discovered theBruce Momjian2002-12-12
| | | | | | attached problem with the EXEC SQL COMMIT RELEASE statement. Roland Karch
* Don't produce bogus COPY command when there are no undropped columnsTom Lane2002-12-12
| | | | in a table.
* TODO marked as done:Bruce Momjian2002-12-12
| | | | | | | | | | | | | | | | | | | | | * Add schema, cast, and conversion backslash commands to psql I had to create a new publically available function, pg_conversion_is_visible, as it seemed to be missing from the catalogs. This required me to do no small amount of hacking around in namespace.c I have updated the \? help and sgml docs. \dc - list conversions [PATTERN] \dC - list casts \dn list schemas I didn't support patterns with casts as there's nothing obvious to match against. Catalog version incremented --- initdb required. Christopher Kings-Lynne
* Don't force rebuild of all of backend/parser when we only needTom Lane2002-12-12
| | | | keywords.o.
* Preliminary code review for domain CHECK constraints patch: add documentation,Tom Lane2002-12-12
| | | | | | | | make VALUE a non-reserved word again, use less invasive method of passing ConstraintTestValue into transformExpr, fix problems with nested constraint testing, do correct thing with NULL result from a constraint expression, remove memory leak. Domain checks still need much more work if we are going to allow ALTER DOMAIN, however.
* Guard against overrunning CTZName buffer when TZ is bogus.Tom Lane2002-12-12
|
* Phase 2 of read-only-plans project: restructure expression-tree nodesTom Lane2002-12-12
| | | | | | | | | so that all executable expression nodes inherit from a common supertype Expr. This is somewhat of an exercise in code purity rather than any real functional advance, but getting rid of the extra Oper or Func node formerly used in each operator or function call should provide at least a little space and speed improvement. initdb forced by changes in stored-rules representation.
* Move psql \? Formatting above Large Object/Copy.Bruce Momjian2002-12-11
|
* psql \? rearrangement, suggested by Tom.Bruce Momjian2002-12-11
|
* Reorder psql \? help into groupings, idea from Harald Armin Massa.Bruce Momjian2002-12-11
|
* Mark SCO Openserver 5.0.4. supported by 7.3.1 as per chat report from user.Bruce Momjian2002-12-11
| | | | template/sco updated.
* Fix line count error reporting in config files, like pg_hba.conf, perBruce Momjian2002-12-11
| | | | | | report from Oliver Elphick. Backpatch to 7.3.
* patch for getTables to fix problems with backends before 7.3 by Kris JurkaDave Cramer2002-12-11
|
* Mike Beachy's build patch to allow ant builds without makeDave Cramer2002-12-11
|
* Aaron's patch for Pooled ConnectionsDave Cramer2002-12-11
|
* Back out psql help change. Not done yet.Bruce Momjian2002-12-11
|
* Bump version for 7.3 and 7.4.Bruce Momjian2002-12-11
|
* Reverse out unintended resultmap change.Tom Lane2002-12-10
|
* If typecmds is going to have its own header file, said header fileTom Lane2002-12-10
| | | | should include what it needs.
* Add an unnecessary assignment to remove a bogus warning. I checked theD'Arcy J.M. Cain2002-12-10
| | | | | logic carefully and I am sure that the test against n happens after it is assigned to.
* Some quick fixes for ALTER DOMAIN patch. It still needs a lot of work,Tom Lane2002-12-09
| | | | but at least it doesn't generate gcc warnings.