On Mon, Aug 22, 2016 at 9:16 PM, Robert Haas <robertmh...@gmail.com> wrote: > On Mon, Aug 22, 2016 at 9:25 AM, Michael Paquier > <michael.paqu...@gmail.com> wrote: >> Another pin-point is: given a certain page, how do we identify of >> which type it is? One possibility would be again to extend the AM >> handler with some kind of is_self function with a prototype like that: >> bool handler->is_self(Page); >> If the page is of the type of the handler, this returns true, and >> false otherwise. Still here performance would suck. >> >> At the end, what we want is a clean interface, and more thoughts into it. > > I think that it makes sense to filter based on the resource manager > ID >
+1. I think the patch currently addresses only a subset of resource manager id's (mainly Heap and Index resource manager id's). Do we want to handle the complete resource manager list as defined in rmgrlist.h? Another thing that needs some thoughts is the UI of this patch, currently it is using integer mask which might not be best way, but again as it is intended to be mainly used for tests, it might be okay. Do we want to enable some tests in the regression suite by using this option? -- With Regards, Amit Kapila. EnterpriseDB: http://www.enterprisedb.com -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
