Re: [Proposal] Global temporary tables

From: Wenjing Zeng <wjzeng2012(at)gmail(dot)com>
To: Andrew Bille <andrewbille(at)gmail(dot)com>
Cc: wenjing <wenjing(at)gmail(dot)com>, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>, Tony Zhu <Tony(dot)zhu(at)ww-it(dot)cn>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [Proposal] Global temporary tables
Date: 2022-02-25 06:26:47
Message-ID: CA+B_3V=NGCDBT2tQcRddhcaT+M3R+xEh=JXC=dpOeYQi8J-29Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Update GTT v67 to fix conflicts with the latest code.

Regards, Wenjing.

Wenjing Zeng <wjzeng2012(at)gmail(dot)com> 于2022年2月15日周二 17:03写道:

> Update GTT v66 to fix conflicts with the latest code.
>
> Regards, Wenjing.
>
>
> Wenjing Zeng <wjzeng2012(at)gmail(dot)com> 于2022年1月20日周四 17:53写道:
>
>> very glad to see your reply.
>> Thank you very much for your review of the code and found so many
>> problems.
>> There was a conflict between the latest code and patch, I have corrected
>> it and provided a new patch (V65).
>> Waiting for your feedback.
>>
>>
>> Regards, Wenjing.
>>
>>
>> Andrew Bille <andrewbille(at)gmail(dot)com> 于2022年1月10日周一 17:17写道:
>>
>>> Hi!
>>>
>>> I could not detect crashes with your last patch, so I think the patch is
>>> ready for a review.
>>> Please, also consider fixing error messages, as existing ones don't
>>> follow message writing guidelines.
>>> https://www.postgresql.org/docs/14/error-style-guide.html
>>>
>>
>> I corrected the ERROR message of GTT according to the link and the
>> existing error message.
>> Some comments and code refactoring were also done.
>>
>>
>>>
>>> Regards, Andrew
>>>
>>> On Thu, Dec 23, 2021 at 7:36 PM wenjing <wjzeng2012(at)gmail(dot)com> wrote:
>>>
>>>>
>>>>
>>>> Andrew Bille <andrewbille(at)gmail(dot)com> 于2021年12月21日周二 14:00写道:
>>>>
>>>>> Hi!
>>>>> Thanks for new patches.
>>>>> Yet another crash reproduced on master with v63 patches:
>>>>>
>>>>> CREATE TABLESPACE ts LOCATION '/tmp/ts';
>>>>> CREATE GLOBAL TEMP TABLE tbl (num1 bigint);
>>>>> INSERT INTO tbl (num1) values (1);
>>>>> CREATE INDEX tbl_idx ON tbl (num1);
>>>>> REINDEX (TABLESPACE ts) TABLE tbl;
>>>>>
>>>> This is a feature made in PG14 that supports reindex change
>>>> tablespaces.
>>>> Thank you for pointing that out and I fixed it in v64.
>>>> Waiting for your feedback.
>>>>
>>>
>>
>
>
>

Attachment Content-Type Size
0002-gtt-v67-doc.patch application/octet-stream 7.5 KB
0004-gtt-v67-regress.patch application/octet-stream 83.1 KB
0001-gtt-v67-reademe.patch application/octet-stream 7.5 KB
0003-gtt-v67-implementation.patch application/octet-stream 174.1 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Kyotaro Horiguchi 2022-02-25 06:31:12 Re: Add checkpoint and redo LSN to LogCheckpointEnd log message
Previous Message Julien Rouhaud 2022-02-25 06:25:52 Re: [PATCH] Expose port->authn_id to extensions and triggers