On Thu, Mar 13, 2025 at 1:37 AM Steven Niu wrote:
>
>
>
> 在 2025/3/12 6:31, Masahiko Sawada 写道:
> > On Mon, Mar 10, 2025 at 3:08 AM Steven Niu wrote:
> >>
> >>
> >>
> >> Hi, Masahiko
> >>
> >> Thanks for your comments! I understand your concern as you stated.
> >> However, my initial patch was
在 2025/3/12 6:31, Masahiko Sawada 写道:
On Mon, Mar 10, 2025 at 3:08 AM Steven Niu wrote:
Hi, Masahiko
Thanks for your comments! I understand your concern as you stated.
However, my initial patch was split into two parts as Kirill suggested.
This thread is about the first part. Another p
On Mon, Mar 10, 2025 at 3:08 AM Steven Niu wrote:
>
>
>
> Hi, Masahiko
>
> Thanks for your comments! I understand your concern as you stated.
> However, my initial patch was split into two parts as Kirill suggested.
> This thread is about the first part. Another part is here:
> https://commitfes
Masahiko Sawada 于2025年3月8日周六 12:04写道：
> Hi,
>
> On Sun, Oct 27, 2024 at 12:05 PM Kirill Reshke
> wrote:
> >
> > On Wed, 14 Aug 2024 at 11:35, Steven Niu wrote:
> > >
> > > Junwang, Kirill,
> > >
> > > The split work has been done. I created a new patch for removing
> redundant smgrclose() funct
>
> Hi, Masahiko
>
> Thanks for your comments! I understand your concern as you stated.
> However, my initial patch was split into two parts as Kirill suggested.
> This thread is about the first part. Another part is here:
> https://commitfest.postgresql.org/patch/5149/
> Or you can take a look a
On Sat, Mar 8, 2025 at 12:04 PM Masahiko Sawada wrote:
>
> Hi,
>
> On Sun, Oct 27, 2024 at 12:05 PM Kirill Reshke wrote:
> >
> > On Wed, 14 Aug 2024 at 11:35, Steven Niu wrote:
> > >
> > > Junwang, Kirill,
> > >
> > > The split work has been done. I created a new patch for removing
> > > redund
On Sat, Mar 8, 2025 at 1:37 AM Junwang Zhao wrote:
>
> Hi Masahiko,
>
> > > I've looked at the patch and have some comments:
> > >
> > > The patch removes smgrclose() calls following smgrdounlinkall(), for
> > > example:
> > >
> > > --- a/src/backend/catalog/storage.c
> > > +++ b/src/backend/cata
Hi Masahiko,
> > I've looked at the patch and have some comments:
> >
> > The patch removes smgrclose() calls following smgrdounlinkall(), for
> > example:
> >
> > --- a/src/backend/catalog/storage.c
> > +++ b/src/backend/catalog/storage.c
> > @@ -686,9 +686,6 @@ smgrDoPendingDeletes(bool isCommi
Hi,
On Sun, Oct 27, 2024 at 12:05 PM Kirill Reshke wrote:
>
> On Wed, 14 Aug 2024 at 11:35, Steven Niu wrote:
> >
> > Junwang, Kirill,
> >
> > The split work has been done. I created a new patch for removing redundant
> > smgrclose() function as attached.
> > Please help review it.
> >
> > Than
On Wed, 14 Aug 2024 at 11:35, Steven Niu wrote:
>
> Junwang, Kirill,
>
> The split work has been done. I created a new patch for removing redundant
> smgrclose() function as attached.
> Please help review it.
>
> Thanks,
> Steven
>
> Steven Niu 于2024年8月12日周一 18:11写道：
>>
>> Kirill,
>>
>> Good cat
The following review has been posted through the commitfest application:
make installcheck-world: not tested
Implements feature: not tested
Spec compliant: not tested
Documentation:not tested
Hi
the patch looks good to me as well. Calling smgrclose() right after calli
Junwang Zhao 于2024年8月15日周四 18:03写道：
> On Wed, Aug 14, 2024 at 2:35 PM Steven Niu wrote:
> >
> > Junwang, Kirill,
> >
> > The split work has been done. I created a new patch for removing
> redundant smgrclose() function as attached.
> > Please help review it.
>
> Patch looks good, actually you ca
On Wed, Aug 14, 2024 at 2:35 PM Steven Niu wrote:
>
> Junwang, Kirill,
>
> The split work has been done. I created a new patch for removing redundant
> smgrclose() function as attached.
> Please help review it.
Patch looks good, actually you can make the refactoring code as v3-0002-xxx
by using:
Junwang, Kirill,
The split work has been done. I created a new patch for removing
redundant smgrclose() function as attached.
Please help review it.
Thanks,
Steven
Steven Niu 于2024年8月12日周一 18:11写道：
> Kirill,
>
> Good catch!
> I will split the patch into two to cover both cases.
>
> Thanks,
> S
Kirill,
Good catch!
I will split the patch into two to cover both cases.
Thanks,
Steven
Junwang Zhao 于2024年8月9日周五 18:19写道：
> On Fri, Aug 9, 2024 at 5:20 PM Kirill Reshke
> wrote:
> >
> > On Thu, 1 Aug 2024 at 17:32, Junwang Zhao wrote:
> > >
> > > Hi Steven,
> > >
> > > On Wed, Jul 31, 2024
On Fri, Aug 9, 2024 at 5:20 PM Kirill Reshke wrote:
>
> On Thu, 1 Aug 2024 at 17:32, Junwang Zhao wrote:
> >
> > Hi Steven,
> >
> > On Wed, Jul 31, 2024 at 11:16 AM Steven Niu wrote:
> > >
> > > Hello, hackers,
> > >
> > > I think there may be some duplicated codes.
> > > Function smgrDoPendingD
On Thu, 1 Aug 2024 at 17:32, Junwang Zhao wrote:
>
> Hi Steven,
>
> On Wed, Jul 31, 2024 at 11:16 AM Steven Niu wrote:
> >
> > Hello, hackers,
> >
> > I think there may be some duplicated codes.
> > Function smgrDoPendingDeletes() calls both smgrdounlinkall() and
> > smgrclose().
> > But both fu
Thanks, I have set my name in the Authors column of CF.
Steven
Junwang Zhao 于2024年8月2日周五 13:22写道：
> Hi Steven,
>
> On Fri, Aug 2, 2024 at 12:12 PM Steven Niu wrote:
> >
> > Hi, Junwang,
> >
> > Thank you for the review and excellent summary in commit message!
> >
> > This is my first contribut
Hi Steven,
On Fri, Aug 2, 2024 at 12:12 PM Steven Niu wrote:
>
> Hi, Junwang,
>
> Thank you for the review and excellent summary in commit message!
>
> This is my first contribution to community, and not so familiar with the
> overall process.
> After reading the process again, it looks like tha
Hi, Junwang,
Thank you for the review and excellent summary in commit message!
This is my first contribution to community, and not so familiar with the
overall process.
After reading the process again, it looks like that I'm not qualified to
submit the patch to commitfest as I never had reviewed
Hi Steven,
On Wed, Jul 31, 2024 at 11:16 AM Steven Niu wrote:
>
> Hello, hackers,
>
> I think there may be some duplicated codes.
> Function smgrDoPendingDeletes() calls both smgrdounlinkall() and smgrclose().
> But both functions would close SMgrRelation object, it's dupliacted behavior?
>
> So
Hello, hackers,
I think there may be some duplicated codes.
Function smgrDoPendingDeletes() calls both smgrdounlinkall() and
smgrclose().
But both functions would close SMgrRelation object, it's dupliacted
behavior?
So I make this patch. Could someone take a look at it?
Thanks for your help,
Ste
22 matches
Mail list logo
