From: | "D'Arcy J(dot)M(dot) Cain" <darcy(at)druid(dot)net> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: MemoryContextAlloc: invalid request size 1934906735 |
Date: | 2002-08-29 12:16:35 |
Message-ID: | 20020829121635.ACEF11BB4@druid.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On August 28, 2002 11:07 pm, Tom Lane wrote:
> "D'Arcy J.M. Cain" <darcy(at)druid(dot)net> writes:
> > Hmm. I did give it a harder look and look what jumped out. Both
> > chkpass_out and chkpass_rout return PG_RETURN_CSTRING but chkpass_out
> > builds a standard c string while chkpass_rout builds a variable text
> > structure. That can't be right.
>
> It's not, but chkpass_rout is declared to return text, so it should be
> saying PG_RETURN_TEXT_P. It turns out both macros do the same thing,
> so this is just a cosmetic issue.
OK, I will make that cosmetic change.
> > Oh, one more datapoint - the error only happens on vacuum analyze, not
> > just vacuum. Not sure what that means exactly.
>
> That is odd. You only have the chkpass operators shown in the contrib
> module, right? No "chkpass = chkpass" operator? Without one, vacuum
> analyze should pretty much ignore the chkpass column ...
YES! Well, sort of. I didn't have any other operators but while I thought
that both were the same (after all, I contributed it) someone must have fixed
the one in CVS before adding it. The one I was working with had the
operators working with chkpass on both sides. As soon as I fixed that it
worked again.
In 7.2 the cstring and chkpass types fail in the function definitions because
they have not been defined so I had to stay with opaque. In fact, how will
that work in 7.3 anyway? We declare the functions to take or return a
chkpass before we define it.
--
D'Arcy J.M. Cain <darcy(at){druid|vex}.net> | Democracy is three wolves
http://www.druid.net/darcy/ | and a sheep voting on
+1 416 425 1212 (DoD#0082) (eNTP) | what's for dinner.
From | Date | Subject | |
---|---|---|---|
Next Message | Oleg Bartunov | 2002-08-29 12:49:14 | README.tsearch.diff for CVS |
Previous Message | Sir Mordred The Traitor | 2002-08-29 09:45:11 | Misc replies |