Two single or double quote escapes single or double quote when string is surrounded by same type of quote#696

Merged

joshuali925 merged 37 commits intoopensearch-project:mainfrom

Bit-Quill:Integ-DoubleQuote

Aug 2, 2022

Commits on Jul 22, 2022

Added integration test for escape character in select statement.
MitchellGale
committed
AOS-637: PoC remove escape characters

Andrew Carbonetto
authored and
MitchellGale
committed
Fixed number of escape characters returned by plugin. Number returned is now expected number.
MitchellGale
committed
Addedd test for 'im' in literals.txt.
MitchellGale
committed
Added unit test for unquoteText in StringUtils.java.
MitchellGale
committed
Removed old unescapeBackslashes from StringUtils which was replaced by unquoteText.
MitchellGale
committed
Removed * from import of Assertions in StringUtilsTest.java.
MitchellGale
committed
Fixed indentation. Moved to 2 space indents to replace 4 space indents.
MitchellGale
committed
Put static imports before regular imports. Maintained alphabetical order in StringUtilsTest.java
MitchellGale
committed
Added case for double single quote which converts to a single single quote.
MitchellGale
committed
Added new line at end of StringUtilsTest.java and added space between static and non-static imports
MitchellGale
committed
Replaced statement lambda with expression lambda.
MitchellGale
committed
Allows for quotes(") and backticks(`) to be around strings in result.
MitchellGale
committed
Added additional test cases to unquoteTest() in StringUtilsTest.java.
MitchellGale
committed
Added additional test cases to unquoteTest() in StringUtilsTest.java. Added case for double, double quote being converted to single, double quote.
MitchellGale
committed
Made double single and double double quotes only reduce down to a single of their respective type when the string is quoted by the same type. Added some unit tests to test for that condition in Str…
MitchellGale
committed
Fixed whitespace found before comma in StringUtilsTest.java.
MitchellGale
committed
Added more tests. Removed formatting changes in OpenSearchExecutionEngine.java.
MitchellGale
committed
Reverting ProjectOperator.java.
MitchellGale
committed
Resolved issue with overlapping replacement of quotes (double and single).
MitchellGale
committed
Moved `replace` periods to new line.
MitchellGale
committed
Changed StringUtils.java to use String Builder. Updated some tests.
MitchellGale
committed
Changed over to fully use string builder and works with escape as well as double quotes.
MitchellGale
committed
Fixed formatting of StringUtilsTest.java, removed code in comments in StringUtils.java.
MitchellGale
committed
Double quote implemented to convert to single quote when string is surrounded by same string. Consistent with MySQL standard.
MitchellGale
committed
Cleaned up redundant code from StringUtils.java.
MitchellGale
committed
Removed IT test added.
MitchellGale
committed
Moved whichquote inside unquote text. Removed large portion of code from if statement to break off for a return. Added unit test, removed unit tests for whichquote function. Removed whichquote func…
MitchellGale
committed
Added some additional unit tests. Added branch for case of quoted by back tick to skip over String Builder. Fixed issue with strings passed to unquotetext being too small casuing array bounds acces…
MitchellGale
committed
Fixed test case expected.
MitchellGale
committed
Reverting changes to OpenSearchExecutionEngine.java.
MitchellGale
committed
Reverting changes to ProjectOperator.java.java.
MitchellGale
committed
Added test cases in literals.txt.
MitchellGale
committed
Added back tick and moved String builder declaration down.
MitchellGale
committed
Added better description regarding unquoteText function in regards to double quote escape implementation.
MitchellGale
committed
Merge branch 'main' of https://github.com/opensearch-project/sql into Integ-DoubleQuote
MitchellGale
committed

Commits on Jul 28, 2022

Fixed line length of comments to not exceed 100 characters
MitchellGale
committed