Re: [PATCH] Datatype of OID should be VARBINARY, not INT

Tom Lane wrote:
>
> Nissim <nissim(at)nksystems(dot)com> writes:
> > The datatype returned by the ResultSetMetaData for OID should be
> > VARBINARY, not INT.
>
> Uh, why? There's nothing VAR about it that I can see, and it is
> the same size as INT ... at the moment anyway ...
>
> regards, tom lane

If you're using it to identify rows then it is used like an int, but if
you're using it to point to a Large Object, the LO is variable size
binary data, right?

We're using it with village: http://www.working-dogs.com/village which
decides which set method of the PreparedStatement to use, based on the
type in the MetaData. If the type returned is INTEGER then it does
setInt, and if the type is BINARY or VARBINARY it does setBytes. Do you
have a better solution than making the change I sent in the patch?

Will returning VARBINARY as the type of an OID column break anyone elses
apps?

-Nissim