>From Ritik Raj <[email protected]>:
Attention is currently required from: Murtadha Hubail, Peeyush Gupta, Till
Westmann, Ali Alsuliman, Ritik Raj.
Hello Murtadha Hubail, Peeyush Gupta, Till Westmann, Ali Alsuliman, Jenkins,
Anon. E. Moose #1000171,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17450
to look at the new patch set (#5).
Change subject: [ASTERIXDB-3155][SQL] Supporting escape backticks and
singleQuote in SQL++
......................................................................
[ASTERIXDB-3155][SQL] Supporting escape backticks and singleQuote in SQL++
- user model changes: yes
- storage format changes: no
- interface changes: no
Details:
This patch contains change to allow escaping of backticks
so that field containing backticks don't throw
parsing error when they are used in the query.
Allows to
1. escape backticks using backticks(`) and reverse solidus (\)
2. escape singleQuotes(') using (')
Change-Id: I5d9069c6aaa1365545f7e0ca728be6ea2ca4641d
---
A
asterixdb/asterix-app/src/test/resources/runtimets/results/select-star/escaping/escape.2.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/select-star/escaping/failure/escape.4.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/select-star/escaping/success/escape.1.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/select-star/escaping/success/escape.2.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/results/select-star/escaping/escape.3.adm
M asterixdb/asterix-lang-sqlpp/src/main/javacc/SQLPP.jj
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/select-star/escaping/success/escape.3.query.sqlpp
M
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/parser/ScopeChecker.java
A
asterixdb/asterix-app/src/test/resources/runtimets/results/select-star/escaping/escape.1.adm
10 files changed, 212 insertions(+), 41 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/50/17450/5
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17450
To unsubscribe, or for help writing mail filters, visit
https://asterix-gerrit.ics.uci.edu/settings
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I5d9069c6aaa1365545f7e0ca728be6ea2ca4641d
Gerrit-Change-Number: 17450
Gerrit-PatchSet: 5
Gerrit-Owner: Ritik Raj <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Murtadha Hubail <[email protected]>
Gerrit-Reviewer: Peeyush Gupta <[email protected]>
Gerrit-Reviewer: Till Westmann <[email protected]>
Gerrit-Attention: Murtadha Hubail <[email protected]>
Gerrit-Attention: Peeyush Gupta <[email protected]>
Gerrit-Attention: Till Westmann <[email protected]>
Gerrit-Attention: Ali Alsuliman <[email protected]>
Gerrit-Attention: Ritik Raj <[email protected]>
Gerrit-MessageType: newpatchset
