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

От

Jeevan Ladhe

Тема

Дата

30 июля 2017 г. в 05:37:01

Msg-id

CAOgcT0OM8MObW1EWbhUR_iztRwWD5A=xXSBc1POzxAHK6tf5BA@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

Hi Ashutosh,

0003 patch

+ parentRel = heap_open(parentOid, AccessExclusiveLock);
In [2], Amit Langote has given a reason as to why heap_drop_with_catalog()
should not heap_open() the parent relation. But this patch still calls
heap_open() without giving any counter argument. Also I don't see
get_default_partition_oid() using Relation anywhere. If you remove that
heap_open() please remove following heap_close().

I think the patch 0004 exactly does what you have said here, i.e. it gets

rid of the heap_open() and heap_close().

The question might be why I kept the patch 0004 a separate one, and the

answer is I wanted to make it easier for review, and also keeping it that

way would make it bit easy to work on a different approach if needed.

About this: "Also I don't see get_default_partition_oid() using Relation anywhere."

The get_default_partition_oid() uses parent relation to retrieve PartitionDesc

from parent.

Kindly let me know if you think I am still missing anything.

Regards,

Jeevan Ladhe

В списке pgsql-hackers по дате отправления