Re: [HACKERS] Adding support for Default partition in partitioning
От | Amit Langote |
---|---|
Тема | Re: [HACKERS] Adding support for Default partition in partitioning |
Дата | |
Msg-id | e1abea68-ddf1-db12-c61b-39a0efc1dc19@lab.ntt.co.jp обсуждение исходный текст |
Ответ на | Re: [HACKERS] Adding support for Default partition in partitioning (Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>) |
Список | pgsql-hackers |
On 2017/05/17 17:58, Ashutosh Bapat wrote: > On Tue, May 16, 2017 at 9:01 PM, Robert Haas <robertmhaas@gmail.com> wrote: >> On Tue, May 16, 2017 at 8:57 AM, Jeevan Ladhe >> <jeevan.ladhe@enterprisedb.com> wrote: >>> I have fixed the crash in attached patch. >>> Also the patch needed bit of adjustments due to recent commit. >>> I have re-based the patch on latest commit. >> >> + bool has_default; /* Is there a default partition? >> Currently false >> + * for a range partitioned table */ >> + int default_index; /* Index of the default list >> partition. -1 for >> + * range partitioned tables */ >> > > We have has_null and null_index for list partitioning. There > null_index == -1 = has_null. May be Rahila and/or Jeevan just copied > that style. Probably we should change that as well? Probably a good idea. Thanks, Amit
В списке pgsql-hackers по дате отправления: