Split the trim function inputCovering() and update the output type
Address the TRegex (REGEX) function input parameters.
TLike to get values correctly.
Tweak TRegex for input values.
Serial sequence -> Leave the covering on all the varchar columns, update
the output type.
Sequence NEXT|CUR VAL -> leave the covering. Add test to verify.
Fix QuoteIdent, with test
Remove dead code from Trim
Inputs to L|R PAD function
Fix Locate/Position, update test.
Add test for Coalesce, but leave it alone.
Add test for CONCAT, verifies it works as expected.
Add tests for replace, strcmp. No changes to the code recommended.
Fix Substring_index, with tests.
Last set of reviews. Tweak QuoteIdent, Trim to get nulls correct. 
Add more tests for other potentally affected, but not updated,
functions.
More test tweaks to resolve problems when the code changes propigate.
Fix the pom.xml

Address merge comments - Fix length of QuoteIdent, SerialSequence.
Extract method from Trim to commnon ancestor.
Restore the defaults for the protobuf properties in the pom.
Fix three broken tests.
Fix quote_ident() by removing the collation. Nathan's point is now
proven.
merge tjoneslo: Function varchar review

Merge pull request #1319 from tjoneslo/sql-layer/function-varchar-review
https://github.com/FoundationDB/sql-layer/pull/1319