Thread

  1. Incorrect GetDatum() macros not match with SQL function types

    Michael Paquier <michael@paquier.xyz> — 2026-04-30T05:46:17Z

    Hi all,
    (Aleksander in CC, who has authored some of them in pageinspect.)
    
    While looking at the business in [1], I have noticed a bunch of areas
    in the code base where the *GetDatum() macros used do not match with
    the data type of the SQL functions they relate to, leading me to the
    attached.
    
    I am creating a new thread for clarity, as the patch was hidden in the
    middle of a different discussion.
    
    When it comes to the numbers touched here, the signedness should not
    matter, so this most likely a cleanup worth doing for v20, not v19.
    
    Thanks,
    
    [1]: https://www.postgresql.org/message-id/CAJ7c6TMcGu8qmRe1gZfJ-gOzVnZq-t=fwn-UuyStx1w6ZyydMw@mail.gmail.com
    --
    Michael
    
  2. Re: Incorrect GetDatum() macros not match with SQL function types

    Michael Paquier <michael@paquier.xyz> — 2026-07-08T04:25:34Z

    On Thu, Apr 30, 2026 at 02:46:17PM +0900, Michael Paquier wrote:
    > I am creating a new thread for clarity, as the patch was hidden in the
    > middle of a different discussion.
    > 
    > When it comes to the numbers touched here, the signedness should not
    > matter, so this most likely a cleanup worth doing for v20, not v19.
    
    I had this patch in my bucket of things for a couple of weeks, and
    applied now as of d92e98340fcb.
    --
    Michael