| From: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
|---|---|
| To: | Peter Geoghegan <pg(at)bowt(dot)ie> |
| Cc: | Dmitry Dolgov <9erthalion6(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>, Claudio Freire <klaussfreire(at)gmail(dot)com>, Anastasia Lubennikova <a(dot)lubennikova(at)postgrespro(dot)ru>, "Andrey V(dot) Lepikhov" <a(dot)lepikhov(at)postgrespro(dot)ru> |
| Subject: | Re: Making all nbtree entries unique by having heap TIDs participate in comparisons |
| Date: | 2019-03-05 11:37:08 |
| Message-ID: | 948c9381-6c75-d8f1-e896-39fd5cfe4890@iki.fi |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
I'm looking at the first patch in the series now. I'd suggest that you
commit that very soon. It's useful on its own, and seems pretty much
ready to be committed already. I don't think it will be much affected by
whatever changes we make to the later patches, anymore.
I did some copy-editing of the code comments, see attached patch which
applies on top of v14-0001-Refactor-nbtree-insertion-scankeys.patch.
Mostly, to use more Plain English: use active voice instead of passive,
split long sentences, avoid difficult words.
I also had a few comments and questions on some details. I added them in
the same patch, marked with "HEIKKI:". Please take a look.
- Heikki
| Attachment | Content-Type | Size |
|---|---|---|
| v14-0001-Refactor-nbtree-insertion-scankeys-HEIKKI-comments.patch | text/x-patch | 19.9 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | David Steele | 2019-03-05 12:36:44 | Re: Re: Patch for SortSupport implementation on inet/cdir |
| Previous Message | Thomas Munro | 2019-03-05 11:15:18 | Re: Rare SSL failures on eelpout |