Hi Dean,
First of all, thanks for working on this!
The patch compiles and applies fine (though with offsets). The feature
is in the SQL standard, and further improves an existing feature.
Some stuff I've spotted and fixed in the attached patch (hope you don't
mind, thought it'd be easier to just fix these rather than describing
them in email):
- Some typos I've spotted in some of the comments
- Fixed escaping of _ in regression tests
Other than these, the patch looks good to me. Will wait for your
thoughts and an updated patch before marking ready for committer.
Regards,
Marko Tiikkaja
