Re: "bad key in cancel request"

Neil Conway <neilc@samurai.com> writes:
>     ereport(DEBUG2,
>             (errmsg_internal("bad key in cancel request for process %d",
>              backendPID)));

> I think this ought to be logged at a higher level than DEBUG2: for one
> thing, it is a potential security issue the DBA might want to be aware
> of. It could also indicate a misconfigured or buggy client application.
> Therefore, I think we should raise the level to WARNING, or perhaps
> NOTICE. Any objections?

No, if it's intended for the log it should be LOG.  Your other proposals
are actually *less* likely to get to where the DBA could see them.
        regards, tom lane