DateTime in UTC?

Question

DateTime in UTC?

Closed this issue 8 years ago · 9 comments

I am using SqlFu v2 with mysql and would like all DateTime's to be set to DateTimeKind.Utc. Can you recommend a way to do this for all SqlFu queries?

I can certainly force UTC manually on the DateTime properties for the objects I get back from SqlFu - but we have many different objects and I would rather not have to do the same thing over and over.

Thanks for your awesome library!
Cameron

Answer 1 · 2016-06-03T09:22:51.000Z

Forgot to mention that all dates are being saved as UTC.

Answer 2 · 2016-06-03T13:14:48.000Z

Hm, I always work with GMT dates and never had a problem, but anyways... try this in your app's start up class

PocoFactory.RegisterConverterFor<DateTime>(obj=>{ /* ensure it's UTC */})

Answer 3 · 2016-06-04T00:41:59.000Z

Thanks for the quick response!

I have tried what you suggested but the converter never gets called:

   PocoFactory.RegisterConverterFor<DateTime>(o =>

        {

            return DateTime.SpecifyKind((DateTime) o, DateTimeKind.Utc);

        });

I placed this in my Application_Start and a few other places (just in case) with no luck. Any other thoughts?

Cheers

Cameron

From: Mike Mogosanu [mailto:notifications@github.com]
Sent: Friday, 3 June 2016 21:15
To: sapiens/SqlFu SqlFu@noreply.github.com
Cc: cameronmr cam@ctr.id.au; Author author@noreply.github.com
Subject: Re: [sapiens/SqlFu] DateTime in UTC? (#86)

Hm, I always work with GMT dates and never had a problem, but anyways... try this in your app's start up class

PocoFactory.RegisterConverterFor(obj=>{ /* ensure it's UTC */})

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub #86 (comment) , or mute the thread https://github.com/notifications/unsubscribe/AC8AS6y5yihs-_fQgCADUGUsHFiawGDhks5qICjKgaJpZM4ItWZ- .

Answer 4 · 2016-06-04T14:15:13.000Z

Can you show me a code sample of the query itself?

Answer 5 · 2016-06-04T14:23:13.000Z

Ok, I think I've found the problem. The mapper invokes directly reader.GetDateTime as an optimization and it doesn't check the converters. I'm thinking if I can come up with a simple solution.

Answer 6 · 2016-06-04T15:23:29.000Z

Fixed it. You can update your nuget pack now (v.2.3.14).

Answer 7 · 2016-06-07T00:12:23.000Z

Thanks Mike,

I tried the new package and the DateTime converter worked. Unfortunately, I was seeing consistent database timeouts (mysql/mariadb) when issuing multiple updates in a single transaction. Rolling back to SqlFu 2.3.12 resolved the timeout issue.

I haven’t had a chance to dig deeper. Any changes you are aware of that may be causing the timeout/deadlock – or anything I can do to help you identify the problem?

Thanks

Cameron

From: Mike Mogosanu [mailto:notifications@github.com]
Sent: Saturday, 4 June 2016 23:24
To: sapiens/SqlFu SqlFu@noreply.github.com
Cc: cameronmr cam@ctr.id.au; Author author@noreply.github.com
Subject: Re: [sapiens/SqlFu] DateTime in UTC? (#86)

Fixed it. You can update your nuget pack now (v.2.3.14).

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub #86 (comment) , or mute the thread https://github.com/notifications/unsubscribe/AC8ASxbS-yxOVhkGPl93oyM_0_uuANDlks5qIZhygaJpZM4ItWZ- .

Answer 8 · 2016-06-07T10:16:07.000Z

Hi Mike,
I suspect the timeout I mentioned below has nothing to do with your changes. I will get back to you with a confirmation tomorrow.
Thanks
Cameron
On 7 Jun 2016 8:12 am, Cameron Rochester cam@ctr.id.au wrote:Thanks Mike, I tried the new package and the DateTime converter worked. Unfortunately, I was seeing consistent database timeouts (mysql/mariadb) when issuing multiple updates in a single transaction. Rolling back to SqlFu 2.3.12 resolved the timeout issue. I haven’t had a chance to dig deeper. Any changes you are aware of that may be causing the timeout/deadlock – or anything I can do to help you identify the problem? ThanksCameron From: Mike Mogosanu [mailto:notifications@github.com] Sent: Saturday, 4 June 2016 23:24To: sapiens/SqlFu SqlFu@noreply.github.comCc: cameronmr cam@ctr.id.au; Author author@noreply.github.comSubject: Re: [sapiens/SqlFu] DateTime in UTC? (#86) Fixed it. You can update your nuget pack now (v.2.3.14).—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or mute the thread.

Answer 9 · 2016-06-08T23:33:33.000Z

Hi Mike,

I can confirm there was a problem on our side and your changes work perfectly. Thanks again for the rapid response!

Cheers

Cameron

From: Cameron Rochester [mailto:cam@ctr.id.au]
Sent: Tuesday, 7 June 2016 18:16
To: 'sapiens/SqlFu' reply@reply.github.com
Subject: RE: [sapiens/SqlFu] DateTime in UTC? (#86)

Hi Mike,

I suspect the timeout I mentioned below has nothing to do with your changes. I will get back to you with a confirmation tomorrow.

Thanks
Cameron

On 7 Jun 2016 8:12 am, Cameron Rochester <cam@ctr.id.au mailto:cam@ctr.id.au > wrote:

Thanks Mike,

I tried the new package and the DateTime converter worked. Unfortunately, I was seeing consistent database timeouts (mysql/mariadb) when issuing multiple updates in a single transaction. Rolling back to SqlFu 2.3.12 resolved the timeout issue.

I haven’t had a chance to dig deeper. Any changes you are aware of that may be causing the timeout/deadlock – or anything I can do to help you identify the problem?

Thanks

Cameron

From: Mike Mogosanu [mailto:notifications@github.com]
Sent: Saturday, 4 June 2016 23:24
To: sapiens/SqlFu <SqlFu@noreply.github.com mailto:SqlFu@noreply.github.com >
Cc: cameronmr <cam@ctr.id.au mailto:cam@ctr.id.au >; Author <author@noreply.github.com mailto:author@noreply.github.com >
Subject: Re: [sapiens/SqlFu] DateTime in UTC? (#86)

Fixed it. You can update your nuget pack now (v.2.3.14).

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub #86 (comment) , or mute the thread https://github.com/notifications/unsubscribe/AC8ASxbS-yxOVhkGPl93oyM_0_uuANDlks5qIZhygaJpZM4ItWZ- .