Append only time

[gkdr]

Description

contains the changes i suggested in #1223

Checklist

• Code compiles correctly
• Created tests which fail without the change (if possible)
• All tests passing
• Extended the README / documentation, if necessary
• Added myself / the copyright holder to the AUTHORS file

[dolmen]

This is on hold. See #1223 (comment)

I agree with @shogo82148. TIME values are not handled by database/sql. They should be handled as strings when user code communicate with database/sql. User code can use the sql.Scanner and driver.Value interface for convenience to implement automatic conversion.

[gkdr]

Sure, I can just close the PR.

It's fine if you don't agree with my argument. What I cared about was the seemingly unintended behaviour change, and I simply wanted a decision.

[methane]

It is too old change. No more discussion is needed for the unintended change.

This pull request is bit ugly, and have bug about handling TZ, as @shogo82148 mentioned in the issue.

Like this example, "date only" would be "-0001-12-31" or "0000-01-02". "date only" should be checked before calling t.In(loc).

https://go.dev/play/p/VBqOewWwJsI