From: | "Qingqing Zhou" <zhouqq(at)cs(dot)toronto(dot)edu> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: qsort again (was Re: [PERFORM] Strange Create Index behaviour) |
Date: | 2006-02-16 03:28:37 |
Message-ID: | dt0rn0$2km1$1@news.hub.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-performance |
"Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote
>
> I did this 100 times and sorted the reported runtimes.
>
> I'd say this puts a considerable damper on my enthusiasm for using our
> qsort all the time, as was recently debated in this thread:
> http://archives.postgresql.org/pgsql-hackers/2005-12/msg00610.php
>
> 100 runtimes for glibc qsort, sorted ascending:
>
> Time: 866.814 ms
> Time: 1234.848 ms
> Time: 1267.398 ms
>
> 100 runtimes for port/qsort.c, sorted ascending:
>
> Time: 28314.182 ms
> Time: 29400.278 ms
> Time: 34142.534 ms
>
By "did this 100 times" do you mean generate a sequence of at most
200000*100 numbers, and for every 200000 numbers, the first half are all
zeros and the other half are uniform random numbers? I tried to confirm it
by patching the program mentioned in the link, but seems BSDqsort is still a
little bit leading.
Regards,
Qingqing
---
Result
sort#./sort
[3] [glibc qsort]: nelem(20000000), range(4294901760) distr(halfhalf)
ccost(2) : 18887.285000 ms
[3] [BSD qsort]: nelem(20000000), range(4294901760) distr(halfhalf) ccost(2)
: 18801.018000 ms
[3] [qsortG]: nelem(20000000), range(4294901760) distr(halfhalf) ccost(2) :
22997.004000 ms
---
Patch to sort.c
sort#diff -c sort.c sort1.c
*** sort.c Thu Dec 15 12:18:59 2005
--- sort1.c Wed Feb 15 22:21:15 2006
***************
*** 35,43 ****
{"BSD qsort", qsortB},
{"qsortG", qsortG}
};
! static const size_t d_nelem[] = {1000, 10000, 100000, 1000000, 5000000};
! static const size_t d_range[] = {2, 32, 1024, 0xFFFF0000L};
! static const char *d_distr[] = {"uniform", "gaussian", "95sorted",
"95reversed"};
static const size_t d_ccost[] = {2};
/* factor index */
--- 35,43 ----
{"BSD qsort", qsortB},
{"qsortG", qsortG}
};
! static const size_t d_nelem[] = {5000000, 10000000, 20000000};
! static const size_t d_range[] = {0xFFFF0000L};
! static const char *d_distr[] = {"halfhalf"};
static const size_t d_ccost[] = {2};
/* factor index */
***************
*** 180,185 ****
--- 180,192 ----
swap(karray[i], karray[nelem-i-1]);
}
}
+ else if (!strcmp(distr, "halfhalf"))
+ {
+ int j;
+ for (i = 0; i < nelem/200000; i++)
+ for (j = 0; j < 100000; j++)
+ karray[i*200000 + j] = 0;
+ }
return array;
}
From | Date | Subject | |
---|---|---|---|
Next Message | Ron | 2006-02-16 04:30:54 | Re: qsort again (was Re: [PERFORM] Strange Create Index |
Previous Message | Neil Conway | 2006-02-16 02:12:52 | Re: qsort again (was Re: [PERFORM] Strange Create Index |
From | Date | Subject | |
---|---|---|---|
Next Message | Ron | 2006-02-16 04:30:54 | Re: qsort again (was Re: [PERFORM] Strange Create Index |
Previous Message | Neil Conway | 2006-02-16 02:12:52 | Re: qsort again (was Re: [PERFORM] Strange Create Index |