<p><a href="https://gerrit.osmocom.org/12121">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12121/5//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12121/5//COMMIT_MSG@11">Patch Set #5, Line 11:</a> <code style="font-family:monospace,monospace">in the 'subscriber' table, in granularity of seconds.</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Describe what do the value mean. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">(imho what the value means should be in a code comment and not so much in a commit log)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/12121/5/sql/hlr.sql">File sql/hlr.sql:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12121/5/sql/hlr.sql@42">Patch Set #5, Line 42:</a> <code style="font-family:monospace,monospace">      last_lu_seen TIMESTAMP default NULL</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I'm not able to find TIMESTAMP in sqlite documentation so far. https://www.sqlite.org/datatype3. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">found https://www.sqlite.org/lang_datefunc.html but am also puzzled why there seems to be no TIMESTAMP.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In https://www.sqlite.org/datatypes.html (version 2) it says "SQLite is "typeless". This means that you can store any kind of data you want in any column of any table" -- and version 2 names a TIMESTAMP type.</p><p style="white-space: pre-wrap; word-wrap: break-word;">In 3 it says "SQLite does not have a storage class set aside for storing dates and/or times. Instead, the built-in Date And Time Functions of SQLite are capable of storing dates and times as TEXT, REAL, or INTEGER values"</p><p style="white-space: pre-wrap; word-wrap: break-word;">So apparently TIMESTAMP is a shim for TEXT.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/12121/5/sql/hlr.sql@75">Patch Set #5, Line 75:</a> <code style="font-family:monospace,monospace">PRAGMA user_version = 1;</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">this 1 together with CURRENT_SCHEMA_VERSION in code looks like a candaidate to have in configure. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">also think it's nice to have a single place... ideally the other version would be derived from this file, but I guess that's too complex? Otherwise not so nice to open the unbuilt source and not be able to see the actual version? so unless some bright idea comes up we'll have it duplicated.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12121">change 12121</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/12121"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-hlr </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ibeb49d45aec18451a260a6654b8c51b8fc3bec50 </div>
<div style="display:none"> Gerrit-Change-Number: 12121 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: Stefan Sperling <stsp@stsp.name> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 06 Dec 2018 16:37:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>