On 2018-03-14 13:32:10 -0400, Tom Lane wrote: > Andres Freund <and...@anarazel.de> writes: > > Only thing I wonder is if we shouldn't just *remove* > > ExplainPropertyLong and make ExplainPropertyInteger accept 64bits of > > input - the effort of passing and printing a 64bit integer will never be > > relevant for explain. > > -0.5 ... everywhere else, we mean "int32" when we say "int", and I don't > think it's worth the potential confusion to do it differently here.
Since I'm +1 on this (I don't think Integer is the same as int, there's no benefit in having two functions, and I've already written the patch this way), I'm inclined to go with one function. Does anybody else have an opinion? - Andres
