Hi Alvaro,
On 12/29/2017 12:59 PM, Alvaro Herrera wrote:
This seems to work pretty well, much to my surprise. I was a bit scared
of adding a new deptype, but actually the only affected code is
findDependentObjects() and the semantics of the new type is a subset of
the existing DEPTYPE_INTERNAL, so I think it's okay. I need to fill in
its description in docs and comments though -- I left it out because the
real difference between INTERNAL and INTERNAL_AUTO is not something that
is covered by the existing description of INTERNAL, so maybe I'll need
to expand that one.
This version includes the fixes I posted as "delta" to the problems
Jesper reported, as well as fixes to the ones reported by Amit. It's
looking pretty good to me -- I'm seeing it as a candidate to commit
early in the upcoming commitfest.
indexing.sql contains a \di command which list the owner of the db, so
make check-world fails.
Best regards,
Jesper
