[14:40:49] <Amir1>	 duesen: hey, the default for rev_timestamp is '' (empty string) which is not logically correct, is there a case that the value would be empty string (like import, etc.) because I want to remove it :D
[14:57:21] <duesen>	 Amir1: I can't think of a valid reason for it to be an empty string. But that doesn#t mean it can't ;)
[15:08:56] <Amir1>	 duesen: so I query everything in wmf prod first and if it's not anywhere, I remove it. Does it sound good?
[15:09:55] <duesen>	 yes, but also check ar_timestamp, and test things like update, undlete, import, etc. 
[15:10:28] <duesen>	 we may be creating a row with an empty timestamp intermediately.
[15:10:52] <duesen>	 by the way, by "remove it", you mean remove the default, not make it nullable, right?
[15:18:58] <Amir1>	 duesen: it's already non-nullable
[15:19:15] <Amir1>	 I will check ar_timestamp, good idea