<p><a href="https://gerrit.osmocom.org/11898">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11898/1/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/11898/1/sql/hlr.sql@48">Patch Set #1, Line 48:</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 understand where you are coming from. Having spent some time on this already, […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I really fail to see your arguments related to what I am asking, maybe I expressed myself in a wrong way or there's something I'm not understanding myself.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm asking to leave everything exactly as it is in this patch, but modify this hlr.sql to be a representation of v1 table, that is add the new field in the subscriber database, and add an INSERT statement that adds version=1 to "meta" table.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This file is only used in case a new database needs to be created. In that case, we know it will always be latest supported version, so in that code path we don't need to go through upgrade procedure.</p><p style="white-space: pre-wrap; word-wrap: break-word;">With this change I request, all other code paths work exactly the same.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't want requiring to run osmo-hlr process (and having to provide a related working config and setup with IP, port, etc.) to get a .db file that is up to date and which I can populate.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11898">change 11898</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/11898"/><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: I8aeaa9a404b622657cbc7138106f38aa6ad8d01b </div>
<div style="display:none"> Gerrit-Change-Number: 11898 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Stefan Sperling <ssperling@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Stefan Sperling <ssperling@sysmocom.de> </div>
<div style="display:none"> Gerrit-CC: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 23 Nov 2018 11:07:41 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>