[CALCITE-5755] Add to Sarg serialization/deserialization TimestampString

• Type: Sub-task
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.35.0
• Labels: pull-request-available
• Assignee: Oliver Lee
• Reporter: Oliver Lee
• Created: 05/Jun/23 23:55
• Updated: 27/Jul/23 10:22
• Resolved: 10/Jun/23 20:37

Description

Currently Sarg serialization/deserialization allows for 6 types https://github.com/apache/calcite/blob/967bb5acc5448bc8d6ee9b9f5fa3c5f0d71405c2/core/src/main/java/org/apache/calcite/rel/externalize/RelJson.java#L113 and doesn't include TimestampString

 

Currently when a timestamp string comes in, it will not throw an error, but will deserialize incorrectly as a Date set to the epoch beginning.

Desired outcome: Be able to serialize / deserialize SEARCH operators that contain Timestamp string literals.

 

 

Issue Links

• links to: GitHub Pull Request #3255