Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SQLServerTransport uses DateTime and not DateTimeOffset, #546

Closed
xenoputtss opened this issue Oct 17, 2016 · 2 comments
Closed

SQLServerTransport uses DateTime and not DateTimeOffset, #546

xenoputtss opened this issue Oct 17, 2016 · 2 comments

Comments

@xenoputtss
Copy link
Contributor

I'm dealing with high latency network (WAN) and the single message at a time pattern is too slow. I decided to steal the shoveling concept from RabbitMQ (basicly bulk copy messages to/from two queue systems https://www.rabbitmq.com/shovel.html ).

as soon as I did this I realized that Expiration and Visibility are stored as DateTime values. The local and remote machines are in different timezones. When I moved messages around, messages wouldn't be processed for hours.

If I make the changes to make Visibility and Expiration stored as DateTimeOffset, do you think this is worth while to bring into Rebus proper or does this not go into a direction you want to take Rebus?

@mookid8000
Copy link
Member

I guess it would not hurt to use DateTimeOffset instead of DateTime. Don't you think it would be possible to implement in a way where only newly-created tables get the DateTimeOffset type columns, while the existing send/receive SQL can work with both types?

@mookid8000
Copy link
Member

(PS: I have created an issue here – the SQL transport has a new home 😉 )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants