On Mon, Sep 5, 2022 at 2:56 PM Nathan Bossart <nathandbossart@gmail.com> wrote:
> There are 2 bits remaining at the moment, so I didn't redesign the ACL
> system in the attached patch. However, I did some research on a couple
> options. Using a distinct set of bits for each catalog table should free
> up a handful of bits, which should indeed kick the can down the road a
> little. Another easy option is to simply make AclMode a uint64, which
> would immediately free up another 16 privilege bits. I was able to get
> this approach building and passing tests in a few minutes, but there might
> be performance/space concerns.
I believe Tom has expressed such concerns in the past, but it is not
clear to me that they are well-founded. I don't think we have much
code that manipulates large numbers of aclitems, so I can't quite see
where the larger size would be an issue. There may well be some
places, so I'm not saying that Tom or anyone else with concerns is
wrong, but I'm just having a hard time thinking of where it would be a
real issue.
--
Robert Haas
EDB: http://www.enterprisedb.com