On Tue, Jul 28, 2020 at 10:35:54AM -0400, Tom Lane wrote:
> Yeah, given that it should be an unreachable case, there's likely
> no need to back-patch.
Thanks. Fixed on HEAD then.
--
Michael
signature.asc
Description: PGP signature
Michael Paquier writes:
> Good catch. Yep, Oids are unsigned. We don't backpatch such things
> usually, do we? Particularly, this one should not be triggerable
> normally because no code paths should call JsonEncodeDateTime() with
> an unsupported type Oid.
Yeah, given that it should be an unr
On Mon, Jul 27, 2020 at 08:55:23PM -0500, Justin Pryzby wrote:
> +JsonEncodeDateTime(char *buf, Datum value, Oid typid)
> ...
> + elog(ERROR, "unknown jsonb value datetime type oid
> %d", typid);
>
> I think this should be %u.
Good catch. Yep, Oids are unsigned. We don't
