Re: B-Tree support function number 3 (strxfrm() optimization)
| От | Greg Stark |
|---|---|
| Тема | Re: B-Tree support function number 3 (strxfrm() optimization) |
| Дата | |
| Msg-id | CAM-w4HPOoY3UZv8Wd4EhGbTPKmOD0425hsGfZsdi8=A7wUjyzw@mail.gmail.com обсуждение исходный текст |
| Ответ на | Re: B-Tree support function number 3 (strxfrm() optimization) (Robert Haas <robertmhaas@gmail.com>) |
| Ответы |
Re: B-Tree support function number 3 (strxfrm() optimization)
Re: B-Tree support function number 3 (strxfrm() optimization) Re: B-Tree support function number 3 (strxfrm() optimization) |
| Список | pgsql-hackers |
On Fri, Apr 4, 2014 at 12:15 PM, Robert Haas <robertmhaas@gmail.com> wrote:
>> Perhaps I shouldn't lay my own guilt trip on other committers --- but+1
>> I think it would be a bad precedent to not deal with the existing patch
>> queue first.
>
> +1
I don't think we have to promise a strict priority queue and preempt all other development. But I agree loosely that processing patches that have been around should be a higher priority.
I've been meaning to do more review for a while and just took a skim through the queue. There are only a couple I feel I can contribute with so I'm going to work on those and then if it's still before the feature freeze I would like to go ahead with Peter's patch. I think it's generally a good patch.
Two questions I have:
1) Would it make more sense to use a floating point instead of an integer? I saw a need for a function like this when I was looking into doing GPU sorts. But GPUs expect floating point values.
2) I would want to see a second data type, probably numeric, before committing to be sure we had a reasonably generic api. But it's pretty simply to do so.
greg
В списке pgsql-hackers по дате отправления: