Re: [HACKERS] Adding support for Default partition in partitioning

От

Jeevan Ladhe

Тема

Дата

17 мая 2017 г. в 12:29:31

Msg-id

CAOgcT0O6Yy9zEcHdfE5Y84A6oi2qB8PnpZUHZN+5=tzJD4kT=A@mail.gmail.com

обсуждение

Ответ на

Re: [HACKERS] Adding support for Default partition in partitioning (Ashutosh Bapat)

Список

pgsql-hackers

Дерево обсуждения

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 6 июня 2017 г. в 23:38:33

Re: [HACKERS] Adding support for Default partition in partitioning amul sul <sulamul@gmail.com> 7 июня 2017 г. в 07:53:17

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 7 июня 2017 г. в 08:00:23

Re: [HACKERS] Adding support for Default partition in partitioning amul sul <sulamul@gmail.com> 7 июня 2017 г. в 08:59:15

Re: [HACKERS] Adding support for Default partition in partitioning Robert Haas <robertmhaas@gmail.com> 8 июня 2017 г. в 19:49:37

Re: [HACKERS] Adding support for Default partition in partitioning Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> 7 июня 2017 г. в 09:28:58

Re: [HACKERS] Adding support for Default partition in partitioning Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> 8 июня 2017 г. в 12:24:33

Re: [HACKERS] Adding support for Default partition in partitioning Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> 8 июня 2017 г. в 13:34:59

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 8 июня 2017 г. в 13:59:49

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 12 июня 2017 г. в 05:59:30

Re: [HACKERS] Adding support for Default partition in partitioning Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> 12 июня 2017 г. в 07:09:25

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 12 июня 2017 г. в 09:19:00

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 14 июня 2017 г. в 15:02:42

Re: [HACKERS] Adding support for Default partition in partitioning Robert Haas <robertmhaas@gmail.com> 14 июня 2017 г. в 22:51:44

Re: [HACKERS] Adding support for Default partition in partitioning Amit Langote <Langote_Amit_f8@lab.ntt.co.jp> 15 июня 2017 г. в 09:48:55

Re: [HACKERS] Adding support for Default partition in partitioning Amit Langote <Langote_Amit_f8@lab.ntt.co.jp> 15 июня 2017 г. в 10:01:53

Re: [HACKERS] Adding support for Default partition in partitioning Kyotaro HORIGUCHI <horiguchi.kyotaro@lab.ntt.co.jp> 16 июня 2017 г. в 11:16:08

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 21 июня 2017 г. в 15:42:28

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 12 июля 2017 г. в 23:09:07

Re: [HACKERS] Adding support for Default partition in partitioning Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> 15 июня 2017 г. в 19:54:39

Re: [HACKERS] Adding support for Default partition in partitioning Robert Haas <robertmhaas@gmail.com> 15 июня 2017 г. в 22:18:39

Re: [HACKERS] Adding support for Default partition in partitioning Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> 16 июня 2017 г. в 08:16:25

Re: [HACKERS] Adding support for Default partition in partitioning Amit Langote <Langote_Amit_f8@lab.ntt.co.jp> 19 июня 2017 г. в 10:04:04

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 30 июня 2017 г. в 15:18:53

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 12 июля 2017 г. в 22:31:32

Re: [HACKERS] Adding support for Default partition in partitioning Robert Haas <robertmhaas@gmail.com> 2 августа 2017 г. в 01:21:52

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 14 августа 2017 г. в 14:51:00

Re: [HACKERS] Adding support for Default partition in partitioning Robert Haas <robertmhaas@gmail.com> 14 августа 2017 г. в 19:18:59

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 17 августа 2017 г. в 13:36:09

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 17 августа 2017 г. в 13:24:55

Re: [HACKERS] Adding support for Default partition in partitioning Robert Haas <robertmhaas@gmail.com> 17 августа 2017 г. в 21:55:49

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 18 августа 2017 г. в 08:57:41

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 20 июля 2017 г. в 16:17:37

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 26 июля 2017 г. в 15:14:52

Re: [HACKERS] Adding support for Default partition in partitioning Robert Haas <robertmhaas@gmail.com> 15 августа 2017 г. в 16:50:34

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 17 августа 2017 г. в 12:59:51

Re: [HACKERS] Adding support for Default partition in partitioning Thom Brown <thom@linux.com> 17 августа 2017 г. в 15:49:32

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 21 августа 2017 г. в 14:17:47

Re: [HACKERS] Adding support for Default partition in partitioning Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> 29 августа 2017 г. в 14:01:27

Re: [HACKERS] Adding support for Default partition in partitioning Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> 28 июля 2017 г. в 16:30:20

Re: [HACKERS] Adding support for Default partition in partitioning Robert Haas <robertmhaas@gmail.com> 29 июля 2017 г. в 00:25:07

Re: [HACKERS] Adding support for Default partition in partitioning Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> 31 июля 2017 г. в 08:49:00

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 30 июля 2017 г. в 05:37:01

Re: [HACKERS] Adding support for Default partition in partitioning Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> 31 июля 2017 г. в 09:22:30

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 17 августа 2017 г. в 13:11:40

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 17 августа 2017 г. в 13:15:22

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 12 июля 2017 г. в 23:01:08

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 21 июня 2017 г. в 15:37:43

Re: [HACKERS] Adding support for Default partition in partitioning Amit Langote <Langote_Amit_f8@lab.ntt.co.jp> 22 июня 2017 г. в 03:47:13

Re: [HACKERS] Adding support for Default partition in partitioning Robert Haas <robertmhaas@gmail.com> 22 июня 2017 г. в 22:50:07

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 21 июня 2017 г. в 15:31:01

Re: [HACKERS] Adding support for Default partition in partitioning Jeevan Ladhe <jeevan.ladhe@enterprisedb.com> 12 июля 2017 г. в 22:52:39

Re: [HACKERS] Adding support for Default partition in partitioning Beena Emerson <memissemerson@gmail.com> 14 июля 2017 г. в 13:05:18

On Wed, May 17, 2017 at 2:28 PM, Ashutosh Bapat <ashutosh.bapat@enterprisedb.com> 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?