Magnus Hagander <mag...@hagander.net> writes:
> The reason for the crash is 6d842be6c11, where Tom added an assert for
> passing null into %s. But I don't think we can blame that patch for the
> problem -- it's passing the NULL there in the first place that's the
> problem.
Indeed; this crash existed on some platforms all along (which means
we'd better back-patch the fix).
> AFAICT the actual drop works fine, it's just the logging that crashes. So
> maybe we should just add a check and make it log something like "<dropped>"
> if pg_namespace_name() returns null?
+1 ... maybe "(dropped)", because we tend to use parens for this sort
of thing, I think.
regards, tom lane
