On Tue, Nov 30, 2010 at 9:34 AM, Julian Foad <julian.foad_at_wandisco.com> wrote:
> In r1040574 I noted that the 'size' and 'md5_checksum' columns in the
> PRISTINE table are never currently null. The latter is required by
> current code to be non-null.
>
> I feel it would be better to have the schema specify 'NOT NULL' for
> these two columns. Just a feeling of Goodness, no concrete reasons.
>
> I imagine it should be possible to add 'NOT NULL' to these columns
> without performing a format bump or writing any upgrade code. Am I
> right?
If we're already enforcing it in the C code, I see no problem with
doing so in the sql schema. Additionally, I don't really see how you
could add the 'NOT NULL' clause in a format bump without doing table
duplication, which doesn't seem worth the trouble for something like
this.
-Hyrum
>
>
> CREATE TABLE PRISTINE (
> [...]
> NULL if unknown, and (thus) the pristine copy is incomplete/unusable.
> + ### We always set this; there may be no need to allow NULL. Shall we
> + add "NOT NULL" to the schema? */
> size INTEGER,
>
> [...]
> + ### We always set this and require it to be non-null. Can we add
> + "NOT NULL" to the schema? */
> md5_checksum TEXT
> );
>
> - Julian
>
>
>
Received on 2010-11-30 17:25:00 CET