Re: [HACKERS] Re: varchar() troubles (fwd)

Forwarded message:
> > Why attlen should be -1 ?
> > attlen in pg_attribute for v in table t is 84, why run-time attlen
> > should be -1 ? How else maxlen constraint could be checked ?
> > IMHO, you have to change heap_getattr() to check is atttype == VARCHAROID
> > and use vl_len if yes. Also, other places where attlen is used must be
> > changed too - e.g. ExecEvalVar():
> >
> > {
> > len = tuple_type->attrs[attnum - 1]->attlen;
> > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> > byval = tuple_type->attrs[attnum - 1]->attbyval ? true : false;
> > }
> >
> > execConstByVal = byval;
> > execConstLen = len;
> > ^^^^^^^^^^^^^^^^^^ - used in nodeHash.c
> >
>
> The major problem is that TupleDesc comes from several places, and
> attlen means several things.
>
> There are some cases where TupleDesc (int numatt, Attrs[]) is created
> on-the-fly (tupdesc.c), and the attlen is the length of the type. In
> other cases, we get attlen from opening the relation, heap_open(), and
> in these cases it is the length as defined for the particular attribute.
>
> Certainly a bad situation. I am not sure about a fix.

OK, here is a temporary fix to the problem. It does the heap_open(),
then replaces the attrs for VARCHAR with attlen of -1. You can't just
change the field, because the data is in a cache and you are just
returned a pointer.

Can I add an 'attdeflen' for "attributed defined length" field to
pg_attribute, and change the attlen references needed to the new field?
This is the only proper way to fix it.

---------------------------------------------------------------------------

*** ./backend/executor/execAmi.c.orig Thu Jan 15 22:42:13 1998
--- ./backend/executor/execAmi.c Thu Jan 15 23:54:37 1998
***************
*** 42,47 ****
--- 42,48 ----
#include "access/genam.h"
#include "access/heapam.h"
#include "catalog/heap.h"
+ #include "catalog/pg_type.h"

static Pointer
ExecBeginScan(Relation relation, int nkeys, ScanKey skeys,
***************
*** 124,129 ****
--- 125,155 ----
if (relation == NULL)
elog(DEBUG, "ExecOpenR: relation == NULL, heap_open failed.");

+ {
+ int i;
+ Relation trel = palloc(sizeof(RelationData));
+ TupleDesc tdesc = palloc(sizeof(struct tupleDesc));
+ AttributeTupleForm *tatt =
+ palloc(sizeof(AttributeTupleForm*)*relation->rd_att->natts);
+
+ memcpy(trel, relation, sizeof(RelationData));
+ memcpy(tdesc, relation->rd_att, sizeof(struct tupleDesc));
+ trel->rd_att = tdesc;
+ tdesc->attrs = tatt;
+
+ for (i = 0; i < relation->rd_att->natts; i++)
+ {
+ if (relation->rd_att->attrs[i]->atttypid != VARCHAROID)
+ tdesc->attrs[i] = relation->rd_att->attrs[i];
+ else
+ {
+ tdesc->attrs[i] = palloc(sizeof(FormData_pg_attribute));
+ memcpy(tdesc->attrs[i], relation->rd_att->attrs[i],
+ sizeof(FormData_pg_attribute));
+ tdesc->attrs[i]->attlen = -1;
+ }
+ }
+ }
return relation;
}

--
Bruce Momjian
maillist(at)candle(dot)pha(dot)pa(dot)us