On Mon, Mar 05, 2018 at 12:44:32AM -0500, Tom Lane wrote:
> Andrew Gierth <andrew@tao11.riddles.org.uk> writes:
>> "Michael" == Michael Paquier <michael@paquier.xyz> writes:
>> Michael> Perhaps it would be the occasion to introduce a T_Integer64
>> Michael> type for Value which gets stored as a string? And as far as I
>> Michael> can see defGetInt64 is only used by sequences.
>
>> The slightly misnamed T_Float is what's currently used for Value nodes
>> which contain numeric values as strings. So there'd be no point in a new
>> type tag if you're still going to store the value as a string.
>
> Going forward, maybe we should change the T_Integer case to either int64
> or int32, so that it's not got a platform-dependent range.
Serial columns using bigint as type would need int64 anyway, no? Why
int32?
> That's not a workable solution for back-patching into v10, though (and
> neither is T_Integer64, really).
Sure. For v10, using just T_Float should be doable at quick glance. I
have not checked though.
--
Michael
