Handle the case if we set a timestamp field default to current time

[mmcomp]

No description provided.

[oguimbal]

Hi, thanks for this PR !

That said, the diff is too complicated, I cant see what has changed.

Would it possible to re-push this, only with actual changes ?

1. I use single quotes for strings
2. Please don't reorder properties
3. I dont wrap single argument lambdas with parenthesis
4. You've changed identation

[oguimbal]

ah, and it would be nice to have a unit test that demonstrate what you are trying to solve (and prevent regressions in the future 😉)

[abenhamdine]

Looks like the actual change is limited to these 2 lines :

@mmcomp perhaps you can submit only this change and a unit test ?

[Thore1954]

Nice eye @abenhamdine, but I don't understand the point of it anyway.
If they mean something like "myColumn" timestamp default now() then it already works.

[abenhamdine]

Nice eye @abenhamdine, but I don't understand the point of it anyway.
If they mean something like "myColumn" timestamp default now() then it already works

Actually I don't know if this change is needed, I was just asking for adding a test.
If this change is controversial, then a first PR should demonstrate the bug by adding a failing test.