Greetings,

* David G. Johnston (david.g.johns...@gmail.com) wrote:
> On Thu, May 16, 2019 at 8:31 AM Daulat Ram <daulat....@exponential.com>
> wrote:
> 
> > url_hash        | bigint                  |           | not null |
> 
> Change the type of url_hash; make it text instead of bigint.

Making it text wastes a bunch of space actually, since it's really a
binary value.  I tend to recommend using a bytea for storing hashes.
Given that hashes are really fixed width, it'd be nice if we had a set
of proper datatypes for them, perhaps not so much to avoid the 1-byte
overhead from storing as a variable-length bytea, but because we could
then avoid having a 7-byte hole due to padding if the hash is followed
by a bigint or such.

Thanks,

Stephen

Attachment: signature.asc
Description: PGP signature

Reply via email to