refactor: change rowtime rewrite conditions and add error handling

[jzaralim]

Description

There were some concerns about #3160 after it had been merged. This PR makes the following changes:

1. Instead of wrapping datestrings in STRINGTOTIMESTAMP, convert them into longs. This allows for error handling to be handled in the rewrite instead of with STRINGTOTIMESTAMP
2. Added some more test cases to cover some edge cases
3. Change the conditions for rewriting to be just if one side is ROWTIME and the other a string.

Testing done

Passes unit and QTT tests

Reviewer checklist

• Ensure docs are updated if necessary. (eg. if a user visible feature is being added or changed).
• Ensure relevant issues are linked (description should include text like "Fixes #")

[hjafarpour]

LGTM.