pg.ddx.io  pgsql-hackers@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Nathan Bossart <nathandbossart@gmail.com>
To: Peter Eisentraut <peter@eisentraut.org>
Cc: Dean Rasheed <dean.a.rasheed@gmail.com>
Cc: John Naylor <john.naylor@enterprisedb.com>
Cc: Kirk Wolak <wolakk@gmail.com>
Cc: Eric Radman <ericshane@eradman.com>
Cc: pgsql-hackers@postgresql.org
Subject: Re: [PATCH] Add function to_oct
Date: Wed, 23 Aug 2023 07:54:30 -0700
Message-ID: <20230823145430.GA1014432@nathanxps13> (raw)
In-Reply-To: <c1a0e94b-98b5-6162-cd4d-faa5f87c3de2@eisentraut.org>
References: <20230815171715.GA2312399@nathanxps13>
	<CAFBsxsEm0VGAipmQNK-EZ1a8ej6PDF+2aEkYGbktKH9aHFb4Dw@mail.gmail.com>
	<20230816142410.GA2651383@nathanxps13>
	<CAFBsxsEVR_M4_eGgP_zdugerq8iTUHeVJP-DBRj7yPh2xcuh4g@mail.gmail.com>
	<20230817152628.GA3148583@nathanxps13>
	<CAEZATCWaQ+-7A9AxxUcxdhdOa4p54rSvKreHMXhnTJg6VuAURQ@mail.gmail.com>
	<20230820152551.GB290672@nathanxps13>
	<0161f4fa-5cd4-ea9b-01a6-8b6b535bd812@eisentraut.org>
	<20230822142608.GA651404@nathanxps13>
	<c1a0e94b-98b5-6162-cd4d-faa5f87c3de2@eisentraut.org>

On Tue, Aug 22, 2023 at 04:57:02PM +0200, Peter Eisentraut wrote:
> On 22.08.23 16:26, Nathan Bossart wrote:
>> > I don't understand the reason for this handling of negative values.  I would
>> > expect that, say, to_hex(-1234) would return '-' || to_hex(1234).
>> For this patch set, I was trying to retain the current behavior, which is
>> to return the two's complement representation.  I'm open to changing this
>> if there's agreement on what the output should be.
> 
> Ah, if there is existing behavior then we should probably keep it.

Committed.

-- 
Nathan Bossart
Amazon Web Services: https://aws.amazon.com





view thread (34+ messages)

Message-ID: <20230823145430.GA1014432@nathanxps13>
Permalink:  ../20230823145430.GA1014432@nathanxps13/
Also on:    postgresql.org/message-id/20230823145430.GA1014432@nathanxps13

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-hackers@postgresql.org
  Cc: nathandbossart@gmail.com, peter@eisentraut.org, dean.a.rasheed@gmail.com, john.naylor@enterprisedb.com, wolakk@gmail.com, ericshane@eradman.com
  Subject: Re: [PATCH] Add function to_oct
  In-Reply-To: <20230823145430.GA1014432@nathanxps13>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox