Correct way to do deletes?

From: Bill Studenmund <wrstuden(at)netbsd(dot)org>
To: <pgsql-hackers(at)postgresql(dot)org>
Subject: Correct way to do deletes?
Date: 2001-10-28 03:31:09
Message-ID: Pine.NEB.4.33.0110272026010.2858-100000@vespasia.home-net.internetconnect.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

I'm working on DROP OPERATOR CLASS, and have a question about how to
actually do the deletes. I ask as my main method has been to copy other
bits of the backend, assuming they do things right.

But I've found two different examples, and don't know which is right? Or
are they both?

Specifically I'm at the step of cleaning out the entries in pg_amop and
pg_amproc.

Example 1 from backend/commands/remove.c for aggregates:

relation = heap_openr(AggregateRelationName, RowExclusiveLock);

tup = SearchSysCache(AGGNAME,
PointerGetDatum(aggName),
ObjectIdGetDatum(basetypeID),
0, 0);

if (!HeapTupleIsValid(tup))
agg_error("RemoveAggregate", aggName, basetypeID);

/* Remove any comments related to this aggregate */
DeleteComments(tup->t_data->t_oid, RelationGetRelid(relation));

simple_heap_delete(relation, &tup->t_self);

ReleaseSysCache(tup);

heap_close(relation, RowExclusiveLock);

Another apparently contrary example from backend/catalog/heap.c:

pg_class_desc = heap_openr(RelationRelationName, RowExclusiveLock);

tup = SearchSysCacheCopy(RELOID,
ObjectIdGetDatum(rel->rd_id),
0, 0, 0);
if (!HeapTupleIsValid(tup))
elog(ERROR, "Relation \"%s\" does not exist",
RelationGetRelationName(rel));

/*
* delete the relation tuple from pg_class, and finish up.
*/
simple_heap_delete(pg_class_desc, &tup->t_self);
heap_freetuple(tup);

heap_close(pg_class_desc, RowExclusiveLock);

In one we SearchSysCache(), simple_heap_delete(), and then
ReleaseSysCache(). In the other we SearchSysCacheCopy(),
simple_heap_delete, and heap_freetuple().

Are they two parallel ways, or is one better?

Take care,

Bill

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message mlw 2001-10-28 03:38:53 Optimizer, index use, good news for 7.2b1
Previous Message Joe Conway 2001-10-28 01:02:57 Re: storing binary data