[storm] Lack of primary key and None

Gustavo Niemeyer gustavo at niemeyer.net
Thu Dec 18 23:19:16 GMT 2008


Hey Grahame,

> Pretty confusing and hard to track down. I think it'd be better if
> storm raised an exception in this case. I can send a patch in if this
> is thought to be a good change?

That's not a bad idea, but I'm not sure if it's such an easy thing to
do, considering that certain databases accept NULL values in the
primary key, possibly with special behavior (e.g. sqlite). When should
Storm raise that error?  If you have a patch, we can certainly
consider it for inclusion.

-- 
Gustavo Niemeyer
http://niemeyer.net



More information about the storm mailing list