3 ms·
A lot of people are talking about the bug but not the fix. Is anyone else incredibly surprised that the fix was just adding auto_increment to the primary key c
by herpderperator 5y ago
A lot of people are talking about the bug but not the fix.
Is anyone else incredibly surprised that the fix was just adding auto_increment to the primary key column for two tables[0][1]? Not having these as auto_increment seems like incredible oversight to me. In what common scenario would you want a setup like that?
[0] https://github.com/signalapp/Signal-Android/commit/83086a5a2b8cae3ac40dea75d8c9533457a31858 https://github.com/signalapp/Signal-Android/commit/83086a5a2...
[1] https://github.com/signalapp/Signal-Android/commit/b9657208fea1c7bcfb90b7400037a7858ba56516 https://github.com/signalapp/Signal-Android/commit/b9657208f...