feat: Streamline SQL INTERVAL
handling and improve related error messages, update sqlparser-rs
lib
#16744
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Double-checking against a PostgreSQL instance confirmed that unary ops are invalid when applied to interval strings (eg:
INTERVAL -'7d'
instead ofINTERVAL '-7d'
), so can simplify the parsing and raise a clear error here instead.Also upgraded
sqlparser-rs
from 0.39 to 0.45; the very latest release (0.47) will require a bit more work to upgrade to so I'll tackle that later. Still pick up a few good additions/fixes with this jump.