Re: Add LogicalTapeSetExtend() to logtape.c - Mailing list pgsql-hackers

From Jeff Davis
Subject Re: Add LogicalTapeSetExtend() to logtape.c
Date
Msg-id 619e61919fd604c2caa629536ee87615b732e054.camel@j-davis.com
Whole thread Raw
In response to Re: Add LogicalTapeSetExtend() to logtape.c  (Adam Lee <ali@pivotal.io>)
Responses Re: Add LogicalTapeSetExtend() to logtape.c  (Adam Lee <ali@pivotal.io>)
List pgsql-hackers
On Fri, 2020-02-28 at 14:16 +0800, Adam Lee wrote:
> I noticed another difference, I was using palloc0(), which could be
> one of the
> reason, but not sure.

I changed the palloc0()'s in your code to plain palloc(), and it didn't
make any perceptible difference. Still slower than the version I posted
that keeps the flexible array.

Did you compare all 3? Master, with your patch, and with my patch? I'd
like to see if you're seeing the same thing that I am.

> Tested your hashagg-20200226.patch on my laptop(Apple clang version
> 11.0.0),
> the average time is 25.9s:

That sounds high -- my runs are about half that time. Is that with a
debug build or an optimized one?

Regards,
    Jeff Davis






pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Allowing ALTER TYPE to change storage strategy
Next
From: Alvaro Herrera
Date:
Subject: Re: Portal->commandTag as an enum