On Tue, Sep 10, 2019 at 07:51:08AM +0530, Amit Kapila wrote: > It will look bit strange to use heapam as a prefix for this function > when all others use heap. I guess if we want to keep it AM specific, > then the proposed name (heap_infomask_flags) is better or > alternatively we can consider heap_tuple_infomask_flags?
Using "heap_" as prefix of the function looks like the best choice to me and that's more consistent with the other functions we have already. Using "tuple" looks sensible as well so the last name you are proposing sounds like a good alternative. > At the beginning of pageinspect documentation page, we have a line > "All of these functions may be used only by superusers.". We need to > change that and then maybe give some explanation of why this > particular function will be allowed to non-superusers. BTW, do you > have any use case in mind for the same because anyway we need > superuser privileges to get the page contents and I think this > function can't be used independently? I would still keep it as superuser-restricted, to avoid any risks with people playing with the internals of this function. pageinspect is sensitive enough. -- Michael
signature.asc
Description: PGP signature
