|From:||Andreas Karlsson <andreas(at)proxel(dot)se>|
|To:||David Rowley <dgrowleyml(at)gmail(dot)com>|
|Cc:||Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>|
|Subject:||Re: Using 128-bit integers for sum, avg and statistics aggregates|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On 12/16/2014 11:04 AM, David Rowley wrote:> These are some very
promising performance increases.
> I've done a quick pass of reading the patch. I currently don't have a
> system with a 128bit int type, but I'm working on that.
Sorry for taking some time to get back. I have been busy before
Christmas. A new version of the patch is attached.
> This fragment needs fixed to put braces on new lines
> It also looks like your OIDs have been nabbed by some jsonb stuff.
> I'm also wondering why in numeric_int16_sum() you're doing:
> return numeric_sum(fcinfo);
> but you're not doing return int8_accum() in the #else part
> of int8_avg_accum()
> The same goes for int8_accum_inv() and int8_avg_accum_inv(), though
> perhaps you're doing it here because of the elog() showing the wrong
> function name. Although that's a pretty much "shouldn't ever happen"
> case that mightn't be worth worrying about.
No strong reason. I did it for symmetry with int2_accum() and int4_accum().
> Also since I don't currently have a machine with a working int128, I
> decided to benchmark master vs patched to see if there was any sort of
> performance regression due to numeric_int16_sum calling numeric_sum, but
> I'm a bit confused with the performance results as it seems there's
> quite a good increase in performance with the patch, I'd have expected
> there to be no change.
Weird, I noticed similar results when doing my benchmarks, but given
that I did not change the accumulator function other than adding an
ifdef I am not totally sure if this difference is real.
tps = 1.001984 (excluding connections establishing)
tps = 1.014511 (excluding connections establishing)
tps = 3.185956 (excluding connections establishing)
|Next Message||Bruce Momjian||2014-12-24 03:51:22||Re: Misaligned BufferDescriptors causing major performance problems on AMD|
|Previous Message||Fabrízio de Royes Mello||2014-12-24 02:49:40||Re: Proposal "VACUUM SCHEMA"|