| From: | Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> |
|---|---|
| To: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: partition -> partitioned |
| Date: | 2018-06-19 09:02:22 |
| Message-ID: | 7d206ed0-f3e3-12d8-16b9-da38ea32e91c@lab.ntt.co.jp |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 2018/06/19 17:51, Amit Langote wrote:
> On 2018/05/17 11:48, Amit Langote wrote:
>> On 2018/05/17 11:40, David Rowley wrote:
>>> On 17 May 2018 at 13:52, Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp> wrote:
>>>> Commit 499be013de6 used 'partition' where it really meant 'partitioned' in
>>>> a few places including in a variable name. For example, what almost all
>>>> places call 'partitioned_rels', make_partition_pruneinfo called
>>>> 'partition_rels'.
>>>>
>>>> Attached a patch to make that uniform to avoid confusion.
>>>
>>> Looks good to me.
>>
>> Thanks for taking a look at it.
>
> Did this perhaps get forgotten?
Noticed that the relevant code changed, so I rebased the patch. Also,
made a minor update to a nearby comment.
Thanks,
Amit
| Attachment | Content-Type | Size |
|---|---|---|
| use-partitioned-not-partition-v2.patch | text/plain | 2.8 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Konstantin Knizhnik | 2018-06-19 09:08:27 | Re: WAL prefetch |
| Previous Message | Amit Langote | 2018-06-19 08:51:05 | Re: partition -> partitioned |