Why the default value is no longer left to '' (empty string)? Allowing NULL 
should be ok also with having a default value.

Then, changing the database structure for this situation is not going to be 
backported, all updates of minor 5.0.x and 5.1.x will fail to just restart.

I think a better option is just to allow NULL, let the default to be '' and 
eventually fix in the code if there is a case in 5.0.x or 5.1.x when NULL is 
inserted and insert '' -- can you point in the code where the issue was exposed 
for you? Anyhow, this won't change the structure dramatically to require a 
version number update.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/1431#issuecomment-364101887
_______________________________________________
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to