[prev in list] [next in list] [prev in thread] [next in thread] 

List:       sr-dev
Subject:    Re: [sr-dev] [kamailio/kamailio] p_usrloc: add UTC_timestamps parameter (PR #3036)
From:       Henning Westerholt <notifications () github ! com>
Date:       2022-02-23 9:12:24
Message-ID: kamailio/kamailio/pull/3036/c1048576608 () github ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Thanks for the pull request. As we are using "DATETIME" values at about 20 other \
places in the database scheme, i am wondering if a specific usrloc module parameter \
is really the best solution for this problem. Maybe a core parameter would be \
benefitial, or just having the server running on UTC timezone? Interested about the \
comments from other developers.

We are using the DATETIME in MySQL which does not keep track of timezones, TIMESTAMP \
does. Not sure why it was choosen this way, probably in the past it was the \
easiest/fastest way of doing it.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/3036#issuecomment-1048576608
You are receiving this because you are subscribed to this thread.

Message ID: <kamailio/kamailio/pull/3036/c1048576608@github.com>


[Attachment #5 (text/html)]

<p></p>
<p dir="auto">Thanks for the pull request. As we are using "DATETIME" values at about \
20 other places in the database scheme, i am wondering if a specific usrloc module \
parameter is really the best solution for this problem. Maybe a core parameter would \
be benefitial, or just having the server running on UTC timezone? Interested about \
the comments from other developers.</p> <p dir="auto">We are using the DATETIME in \
MySQL which does not keep track of timezones, TIMESTAMP does. Not sure why it was \
choosen this way, probably in the past it was the easiest/fastest way of doing \
it.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br \
/>Reply to this email directly, <a \
href="https://github.com/kamailio/kamailio/pull/3036#issuecomment-1048576608">view it \
on GitHub</a>, or <a \
href="https://github.com/notifications/unsubscribe-auth/ABO7UZJZDOVOWODLCVJE6JLU4SQHRANCNFSM5PDSIHHQ">unsubscribe</a>.<br \
/>Triage notifications on the go with GitHub Mobile for <a \
href="https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&amp;mt=8&amp;pt=524675">iOS</a> \
or <a href="https://play.google.com/store/apps/details?id=com.github.android&amp;refer \
rer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub">Android</a>.
 <br />You are receiving this because you are subscribed to this thread.<img \
src="https://github.com/notifications/beacon/ABO7UZMJ4BPHFKH2PXFC6N3U4SQHRA5CNFSM5PDSIHH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOH2AAEYA.gif" \
height="1" width="1" alt="" /><span style="color: transparent; font-size: 0; display: \
none; visibility: hidden; overflow: hidden; opacity: 0; width: 0; height: 0; \
max-width: 0; max-height: 0; mso-hide: all">Message ID: \
<span>&lt;kamailio/kamailio/pull/3036/c1048576608</span><span>@</span><span>github</span><span>.</span><span>com&gt;</span></span></p>
 <script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/kamailio/kamailio/pull/3036#issuecomment-1048576608",
"url": "https://github.com/kamailio/kamailio/pull/3036#issuecomment-1048576608",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>



_______________________________________________
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[prev in list] [next in list] [prev in thread] [next in thread]