Re: Define DatumGetInt8 function.

Aleksander Alekseev <aleksander@tigerdata.com>

View thread

From: Aleksander Alekseev <aleksander@tigerdata.com>

To: pgsql-hackers <pgsql-hackers@postgresql.org>

Cc: Kirill Reshke <reshkekirill@gmail.com>, David Rowley <dgrowleyml@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>

Date: 2026-01-07T14:03:19Z

Lists: pgsql-hackers

Attachments

v2-0002-Remove-DatumGetUInt8-and-UInt8GetDatum.patch (text/x-patch) patch v2-0002
v2-0001-Remove-Int8GetDatum-function.patch (text/x-patch) patch v2-0001

Hi,

> Hmm, v1 looks good and self-contained to me. Like, anyway, making two
> commits (one for signed Int8 and one for unsigned) here is better for
> sake of atomicy?
> Anyway, I can see there are users of UInt8GetDatum, which are [0] and
> forks of Greenplum. So, I am not super-sure removing UInt8* is
> desirable.

Fair enough. Let it be a separate patch then.

--
Best regards,
Aleksander Alekseev

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

Adjust some incorrect *GetDatum() macros
- ac59a90bef45 19 (unreleased) landed
Remove Int8GetDatum function
- d4a080b8a190 19 (unreleased) landed