Re: [HACKERS] Current sources?

>> I do not believe that this could ever have passed regression. Do we have
>> the whole patch to back out, or do we need to just "fix what we have now"?
>>
>> Also, perhaps we need to be more selective about checkins?
>
>Not sure. Marc and I reviewed it, and it looked very good. In fact, I
>would like to see more of such patches, of course, without the destroydb
>problem, but many patches have little quirks the author could not have
>anticipated.
>
>> {
>> JunkFilter *j = (JunkFilter *) ExecInitJunkFilter(targetList);
>> estate->es_junkFilter = j;
>> >>>> tupType = j->jf_cleanTupType; /* Added by daveh(at)insightdist(dot)com 5/20/98 */
>> }
>> else
>> estate->es_junkFilter = NULL;
>>
>> Here is my debug transcript for "drop database regression"
>
>Here is the original patch. I got it with the command:

I have just removed the patch using patch -R and confirmed that "drop
table", and "delete from" works again. regression tests also look
good, except char/varchar/strings.

Now I can start to create patches for snapshot...
--
Tatsuo Ishii
t-ishii(at)sra(dot)co(dot)jp