Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Dmitry Lychagin has submitted this change and it was merged. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. [ASTERIXDB-2376][FUN] Negative offset for substring() - user model changes: yes - storage format changes: no - interface changes: no Details: - Support negative offset as an argument to substring() (that means: count from the end of the string) Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Reviewed-on: https://asterix-gerrit.ics.uci.edu/2626 Sonar-Qube: JenkinsTested-by: Jenkins Contrib: Jenkins Integration-Tests: Jenkins Reviewed-by: Till Westmann --- M asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql M asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substring2-4/substring2-4.3.query.sqlpp M asterixdb/asterix-app/src/test/resources/runtimets/results/string/substr01/substr01.1.adm M asterixdb/asterix-app/src/test/resources/runtimets/results/string/substring2-4/substring2-4.1.adm M asterixdb/asterix-app/src/test/resources/runtimets/results/stringoffset/substring/substring.1.adm M asterixdb/asterix-app/src/test/resources/runtimets/results/stringoffset/substring2/substring2.1.adm M asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/string/substr01/substr01.3.ast M asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/string/substring2-4/substring2-4.3.ast M asterixdb/asterix-doc/src/main/markdown/builtins/2_string_common.md M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/Substring2Descriptor.java M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/SubstringDescriptor.java 13 files changed, 18 insertions(+), 17 deletions(-) Approvals: Anon. E. Moose #1000171: Till Westmann: Looks good to me, approved Jenkins: Verified; No violations found; ; Verified diff --git a/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql b/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql index bc9e7f7..a45b24c 100644 --- a/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql +++ b/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql @@ -36,7 +36,7 @@ let $str8:=substring($str7,0) let $str9:="This is a test string" -let $str10:=substring($str9,-1) +let $str10:=substring($str9,-6) let $str11:="This is a test string" let $str12:="This is a another test string" diff --git a/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql b/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql index 075af82..ecfcd94 100644 --- a/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql +++ b/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql @@ -18,5 +18,5 @@ */ use dataverse test; -let $c1 := substring("HEllow",-1) +let $c1 := substring("HEllow",-3) return {"result1": $c1} diff --git a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp index b9c79d0..6abe9ff 100644 --- a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp +++ b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp @@ -23,4 +23,4 @@ * Date : 18th April 2012 */ -{'str2':substring('Hello World',9),'str4':substring('This is a test string',20),'str6':substring('This is a test string',21),'str8':substring('This is a test string',0),'str10':substring('This is a test string',-1),'str13':substring(`string-concat`(['This is a test string','This is a another test string']),20),'str14':substring('UC Irvine',(`string-length`('UC Irvine') / 2 - 1))}; +{'str2':substring('Hello World',9),'str4':substring('This is a test string',20),'str6':substring('This is a test string',21),'str8':substring('This is a test string',0),'str10':substring('This is a test string',-6),'str13':substring(`string-concat`(['This is a test string','This is a another test string']),20),'str14':substring('UC Irvine',(`string-length`('UC Irvine') / 2 -
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Till Westmann has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Code-Review+2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-Reviewer: Till Westmann Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/6334/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Contrib+1 Analytics Compatibility Tests Successful https://goo.gl/jussJC : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Contrib+1 BAD Compatibility Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/3124/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: BAD Compatibility Tests Started https://asterix-jenkins.ics.uci.edu/job/asterixbad-compat/3124/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Analytics Compatibility Compilation Successful https://goo.gl/xL8zVa : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/6334/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/3916/ (5/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/3997/ (11/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/3465/ (3/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/1472/ (10/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/3377/ (8/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/4181/ (9/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/7404/ (2/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Hello Anon. E. Moose #1000171, Jenkins, I'd like you to reexamine a change. Please visit https://asterix-gerrit.ics.uci.edu/2626 to look at the new patch set (#2). Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. [ASTERIXDB-2376][FUN] Negative offset for substring() - user model changes: yes - storage format changes: no - interface changes: no Details: - Support negative offset as an argument to substring() (that means: count from the end of the string) Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 --- M asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql M asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substring2-4/substring2-4.3.query.sqlpp M asterixdb/asterix-app/src/test/resources/runtimets/results/string/substr01/substr01.1.adm M asterixdb/asterix-app/src/test/resources/runtimets/results/string/substring2-4/substring2-4.1.adm M asterixdb/asterix-app/src/test/resources/runtimets/results/stringoffset/substring/substring.1.adm M asterixdb/asterix-app/src/test/resources/runtimets/results/stringoffset/substring2/substring2.1.adm M asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/string/substr01/substr01.3.ast M asterixdb/asterix-app/src/test/resources/runtimets/results_parser_sqlpp/string/substring2-4/substring2-4.3.ast M asterixdb/asterix-doc/src/main/markdown/builtins/2_string_common.md M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/Substring2Descriptor.java M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/SubstringDescriptor.java 13 files changed, 18 insertions(+), 17 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/26/2626/2 -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/3453/ (6/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/3896/ (4/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/3424/ (12/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/8928/ (1/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 2: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/3680/ (7/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 2 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Integration-Tests+1 Integration Tests Successful https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/6324/ : SUCCESS -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Anon. E. Moose #1000171 has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Contrib-2 Analytics Compatibility Compilation Failed https://goo.gl/oeEcKb : UNSTABLE -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Integration Tests Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/6324/ -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-asterix-app/3906/ (6/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-ensure-ancestor/1462/ (11/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-verify-no-installer-app/3886/ (8/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app/4171/ (2/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-verify-storage/3987/ (12/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Dmitry Lychagin has uploaded a new change for review. https://asterix-gerrit.ics.uci.edu/2626 Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. [ASTERIXDB-2376][FUN] Negative offset for substring() - user model changes: yes - storage format changes: no - interface changes: no Details: - Support negative offset as an argument to substring() (that means: count from the end of the string) Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 --- M asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql M asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substring2-4/substring2-4.3.query.sqlpp M asterixdb/asterix-app/src/test/resources/runtimets/results/string/substr01/substr01.1.adm M asterixdb/asterix-app/src/test/resources/runtimets/results/string/substring2-4/substring2-4.1.adm M asterixdb/asterix-doc/src/main/markdown/builtins/2_string_common.md M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/Substring2Descriptor.java M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/evaluators/functions/SubstringDescriptor.java 9 files changed, 14 insertions(+), 13 deletions(-) git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/26/2626/1 diff --git a/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql b/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql index bc9e7f7..a45b24c 100644 --- a/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql +++ b/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substr01/substr01.3.query.aql @@ -36,7 +36,7 @@ let $str8:=substring($str7,0) let $str9:="This is a test string" -let $str10:=substring($str9,-1) +let $str10:=substring($str9,-6) let $str11:="This is a test string" let $str12:="This is a another test string" diff --git a/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql b/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql index 075af82..ecfcd94 100644 --- a/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql +++ b/asterixdb/asterix-app/src/test/resources/runtimets/queries/string/substring2-4/substring2-4.3.query.aql @@ -18,5 +18,5 @@ */ use dataverse test; -let $c1 := substring("HEllow",-1) +let $c1 := substring("HEllow",-3) return {"result1": $c1} diff --git a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp index b9c79d0..6abe9ff 100644 --- a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp +++ b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substr01/substr01.3.query.sqlpp @@ -23,4 +23,4 @@ * Date : 18th April 2012 */ -{'str2':substring('Hello World',9),'str4':substring('This is a test string',20),'str6':substring('This is a test string',21),'str8':substring('This is a test string',0),'str10':substring('This is a test string',-1),'str13':substring(`string-concat`(['This is a test string','This is a another test string']),20),'str14':substring('UC Irvine',(`string-length`('UC Irvine') / 2 - 1))}; +{'str2':substring('Hello World',9),'str4':substring('This is a test string',20),'str6':substring('This is a test string',21),'str8':substring('This is a test string',0),'str10':substring('This is a test string',-6),'str13':substring(`string-concat`(['This is a test string','This is a another test string']),20),'str14':substring('UC Irvine',(`string-length`('UC Irvine') / 2 - 1))}; diff --git a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substring2-4/substring2-4.3.query.sqlpp b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substring2-4/substring2-4.3.query.sqlpp index 8ea572c..ae943d4 100644 --- a/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substring2-4/substring2-4.3.query.sqlpp +++ b/asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/string/substring2-4/substring2-4.3.query.sqlpp @@ -20,4 +20,4 @@ use test; -{'result1':test.substring('HEllow',-1)}; +{'result1':test.substring('HEllow',-3)}; diff --git a/asterixdb/asterix-app/src/test/resources/runtimets/results/string/substr01/substr01.1.adm
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-sonar/7394/ (10/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-cancellation-test/3443/ (4/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-format/3414/ (3/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-source-assemblies/3670/ (9/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/hyracks-gerrit/3367/ (7/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-asterix-app-sql-execution/3455/ (5/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No
Change in asterixdb[master]: [ASTERIXDB-2376][FUN] Negative offset for substring()
Jenkins has posted comments on this change. Change subject: [ASTERIXDB-2376][FUN] Negative offset for substring() .. Patch Set 1: Build Started https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/8918/ (1/12) -- To view, visit https://asterix-gerrit.ics.uci.edu/2626 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: I849c38819ea2df2c203fd1d794c6b0b3a0eca9d1 Gerrit-PatchSet: 1 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: Dmitry LychaginGerrit-Reviewer: Jenkins Gerrit-HasComments: No