>From Ali Alsuliman <[email protected]>:
Attention is currently required from: Murtadha Hubail, Peeyush Gupta, Ali
Alsuliman.
Hello Murtadha Hubail, Peeyush Gupta, Wail Alkowaileet, Jenkins, Anon. E. Moose
#1000171,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17928
to look at the new patch set (#2).
Change subject: [ASTERIXDB-3301][COMP] NPE for subquery with only SELECT ELEMENT
......................................................................
[ASTERIXDB-3301][COMP] NPE for subquery with only SELECT ELEMENT
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
When a subquery consists of only SELECT ELEMENT, a NullPointerException
is encountered. The exception is due to the fact that the base/input
operator for the SUBPLAN of the subquery starts with a 'null' op that
gets replaced by a NTS op later after translating the subquery. This
works fine, but when the subquery is only a SELECT ELEMENT
expression, there are no intermediate operators translated and therefore
the root op of the subplan tries to reference the input op directly which
is 'null' at the time.
Change-Id: I5b314c014798daf4890f75998138954e3d858445
---
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.05.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.13.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.14.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.09.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.10.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.15.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.02.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_sqlpp.xml
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/SqlppExpressionToPlanTranslator.java
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.06.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.10.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.11.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.16.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.01.ddl.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.02.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.11.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.13.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.03.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.12.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.05.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.07.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.04.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.15.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.03.query.sqlpp
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.14.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.99.ddl.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.09.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.08.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.17.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.04.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.12.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.16.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.06.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.17.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/subquery/select_element/select_element.07.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/results/subquery/select_element/select_element.08.adm
37 files changed, 508 insertions(+), 3 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/28/17928/2
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17928
To unsubscribe, or for help writing mail filters, visit
https://asterix-gerrit.ics.uci.edu/settings
Gerrit-Project: asterixdb
Gerrit-Branch: neo
Gerrit-Change-Id: I5b314c014798daf4890f75998138954e3d858445
Gerrit-Change-Number: 17928
Gerrit-PatchSet: 2
Gerrit-Owner: Ali Alsuliman <[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: Wail Alkowaileet <[email protected]>
Gerrit-Attention: Murtadha Hubail <[email protected]>
Gerrit-Attention: Peeyush Gupta <[email protected]>
Gerrit-Attention: Ali Alsuliman <[email protected]>
Gerrit-MessageType: newpatchset
