Bigint data type for id in the table for due messages

rebus-org / Rebus.SqlServer

:bus: Microsoft SQL Server transport and persistence for Rebus

https://mookid.dk/category/rebus

Other

44 stars 45 forks source link

Bigint data type for id in the table for due messages #46

Closed rsivanov closed 5 years ago

rsivanov commented 5 years ago

SqlServerTimeoutManager creates a table with id defined as int.

[id] [int] IDENTITY(1,1) NOT NULL

It seems that bigint would be a better choice for id as the table might be used in a production system with many inserts into due messages during many years. I had to make a fork of SqlServerTimeoutManager just to change data type for id.

mookid8000 commented 5 years ago

You're absolutely right – the ID should be a BIGINT

mookid8000 commented 5 years ago

fixed by #47 – available in Rebus.SqlServer 5.1.0