| From: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> |
|---|---|
| To: | hlinnaka(at)iki(dot)fi |
| Cc: | daniel(at)yesql(dot)se, peter(dot)eisentraut(at)2ndquadrant(dot)com, robertmhaas(at)gmail(dot)com, tsunakawa(dot)takay(at)jp(dot)fujitsu(dot)com, tgl(at)sss(dot)pgh(dot)pa(dot)us, ishii(at)sraoss(dot)co(dot)jp, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: Radix tree for character conversion |
| Date: | 2016-12-13 06:11:03 |
| Message-ID: | 20161213.151103.157484378.horiguchi.kyotaro@lab.ntt.co.jp |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hello, I looked on this closer.
The attached is the revised version of this patch.
At Mon, 05 Dec 2016 19:29:54 +0900 (Tokyo Standard Time), Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> wrote in <20161205(dot)192954(dot)121855559(dot)horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>
> Apart from the aboves, I have some trivial comments on the new
> version.
>
>
> 1. If we decide not to use old-style maps, UtfToLocal no longer
> need to take void * as map data. (Patch 0001)
> 2. "use Data::Dumper" doesn't seem necessary. (Patch 0002)
> 3. A comment contains a superfluous comma. (Patch 0002) (The last
> byte of the first line below)
> 4. The following code doesn't seem so perl'ish.
> 4. download_srctxts.sh is no longer needed. (No patch)
6. Fixed some inconsistent indentation/folding.
7. Fix handling of $verbose.
8. Sort segments using leading bytes.
regards,
--
Kyotaro Horiguchi
NTT Open Source Software Center
| Attachment | Content-Type | Size |
|---|---|---|
| maps-as-radix-trees-2.patch | text/x-patch | 106.3 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Ioseph Kim | 2016-12-13 06:15:20 | Proposal: add error message in backend/catalog/index.c |
| Previous Message | Petr Jelinek | 2016-12-13 05:55:31 | Re: Logical Replication WIP |