<table><tr><td style="">pablo added a comment.
</td></tr></table><br /><div><div><p><strong>Flags should be <em>NOT NULL</em></strong></p>

<p>I noticed <strong>[ResourceTable]</strong> has a flag named <strong>isVirtual</strong> and it is NULL'able.  It is defined as a <strong>TINYINT(1)</strong> (effectively a Boolean).  We'll want to fix this column so it is <strong>NOT NULL</strong>  The DDL for the column has a default (zero).</p>

<p><strong>Key/Value Pair anti-pattern </strong></p>

<p>The <strong>[CollectionAttributeTable]</strong> is being used as a generic key/value pair table - <a href="https://stackoverflow.com/questions/126271/key-value-pairs-in-relational-database" class="remarkup-link" target="_blank" rel="noreferrer">see</a> why this is not a good way to go if there is heavy demand on this table.</p></div></div><br /><div><strong>TASK DETAIL</strong><div><a href="https://phabricator.kde.org/T7846" rel="noreferrer">https://phabricator.kde.org/T7846</a></div></div><br /><div><strong>To: </strong>pablo<br /><strong>Cc: </strong>dvratil, kde-pim, pablo<br /></div>