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.

As a bonus:

Use text instead of arbitrary varchar(4000) fields and add, e.g., check
(length(url) < 4000) or something better.

Needing 38 digit integers for "id" fields seems odd.

Not sure what kind of date would need 128 characters to represent; ideally
you'd just make that a  "date" typed field and normalize on input.

David J.

Reply via email to