Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ASTERIXDB-2170][SQL] Fix resolution order of implicit field access

- user model changes: yes

- storage format changes: no

- interface changes: no

Details:

- Improved name resolution rules

- Resolve field access to the nearest variable in scope

instead of raising compile-time error

- Do not rely on type information when resolving names

- Cleanup group variable handling in GroupBy clause,

no longer use ‘with’ map for it

- Fix ByNameToByIndexFieldAccessRule to use type environment

of its input operator when analyzing its expression

- Fix ExternalGroupByPOperator to use input schema of its

aggregate function when generating runtime for that function

- Fix invalid free variable computation for GroupBy clause

Change-Id: I50bc823ff53da06507a5454b30f4f500b862d4bf

Reviewed-on: https://asterix-gerrit.ics.uci.edu/2207

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Dmitry Lychagin <dmitry.lychagin@couchbase.com>

  1. … 314 more files in changeset.
[ASTERIXDB-2050][SQL] Enforce a Semicolon After Each SQL++ Statement

- user model changes: a semicolon must be added after

every SQL++ statement.

- storage format changes: no

- interface changes: no

Details:

- Enforce a semicolon after each SQL++ statement.

- Adapt existing SQL++ test cases to new model.

Change-Id: I27e9e8fde5ff867ab569c8d443ba1522738046e3

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1954

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

    • -2
    • +2
    ./q2-2-negative/q2-2-negative.1.ddl.sqlpp
  1. … 2063 more files in changeset.
ASTERIXDB-1539: add aliases for several builtin types and builtin functions.

- tinyint for int8;

- smallint for int16;

- integer(or, int) for int32;

- bigint for int64;

- year_month_duration for year-month-duration;

- day_time_duration for day-time-duration;

- timestamp for datetime;

- double precision for double;

int8, int16, int32, int64, year-month-duration and day-time-duration

is going to be deprecated.

In addition, it

- converts underscores to hypens for SQL++ functions;

- uses "array_" prefix for builtin aggregation functions.

Change-Id: I5c4a933f9775e89c2d503d9ea1cb98c5934d1475

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1199

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Yingyi Bu <buyingyi@gmail.com>

    • -2
    • +2
    ./q2-2-negative/q2-2-negative.1.ddl.sqlpp
  1. … 3547 more files in changeset.
Clean up GROUP BY and WITH clause.

- fix ASTERIXDB-971, i.e., the expression to plan translator for FLWOGR without a "for";

- fix the scoping for nested WITH clause;

- fix the scoping for nested GROUP BY clause;

- fix default fields in group variables;

- fix/unify the expression substition AST visitor;

- enhance RemoveUnusedAssignAndAggregateRule to be able to remove unused GROUP BY

decoration varaibles;

- clearly sperate SQL++ aggregate functions and SQL-92 aggregate functions, i.e.,

SQL-92 aggregate sugars cannot apply to SQL++ aggregate functions;

- fix PushAggregateIntoGroupbyRule for nested plan deletions;

- enhance RemoveCartesianProductWithEmptyBranchRule;

- add ExtractGroupByDecorVariablesRule;

- add an aggregate function first-element;

- fix ASTERIXDB-1560, an error message issue.

Change-Id: I62fca7f937aa007d97ed87c75cef19f6aa3e5ade

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1050

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

    • -0
    • +55
    ./q2-10/q2-10.1.ddl.sqlpp
    • -0
    • +19
    ./q2-10/q2-10.2.update.sqlpp
    • -0
    • +37
    ./q2-10/q2-10.3.query.sqlpp
    • -0
    • +55
    ./q2-11/q2-11.1.ddl.sqlpp
    • -0
    • +19
    ./q2-11/q2-11.2.update.sqlpp
    • -0
    • +40
    ./q2-11/q2-11.3.query.sqlpp
    • -0
    • +55
    ./q2-12/q2-12.1.ddl.sqlpp
    • -0
    • +19
    ./q2-12/q2-12.2.update.sqlpp
    • -0
    • +39
    ./q2-12/q2-12.3.query.sqlpp
    • -0
    • +55
    ./q2-2-negative/q2-2-negative.1.ddl.sqlpp
    • -0
    • +19
    ./q2-2-negative/q2-2-negative.2.update.sqlpp
    • -0
    • +40
    ./q2-2-negative/q2-2-negative.3.query.sqlpp
    • -0
    • +55
    ./q2-7/q2-7.1.ddl.sqlpp
    • -0
    • +19
    ./q2-7/q2-7.2.update.sqlpp
  1. … 221 more files in changeset.
Renaming: "table"->"dataset", "database"->"dataverse".

Change-Id: Ia80193f08651fc99341d9a8cf156d0feb6d0575f

Reviewed-on: https://asterix-gerrit.ics.uci.edu/1036

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

  1. … 2766 more files in changeset.
Support SQL-compliant group-by syntax.

1. Add AST-level deep equality and expression subsutitions;

2. Fix DeepCopyVisitor and InlineColumnAliasVisitor for edge cases;

3. Add deep equality tests;

4. Add group-by tests for the SQL-compliant syntax.

Change-Id: Ia1cbe1fab216b5f47577d75fd870a537cfe1e84f

Reviewed-on: https://asterix-gerrit.ics.uci.edu/990

Sonar-Qube: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

    • -0
    • +55
    ./q2-6/q2-6.1.ddl.sqlpp
    • -0
    • +19
    ./q2-6/q2-6.2.update.sqlpp
    • -0
    • +35
    ./q2-6/q2-6.3.query.sqlpp
  1. … 115 more files in changeset.
Support implicit variable name and column name.

1. Support WITH clause for general cases;

2. Fixed open/closed type optimization;

3. Fixed scoping for nested aggregations;

4. Fixed side effects during partition property analysis;

5. Fixed type propagation policy for Subplan operator.

Change-Id: I409b9ba139c9f000a6b9b84d519d172d0069e4bb

Reviewed-on: https://asterix-gerrit.ics.uci.edu/950

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

    • -0
    • +55
    ./q2-2/q2-2.1.ddl.sqlpp
    • -0
    • +19
    ./q2-2/q2-2.2.update.sqlpp
    • -0
    • +40
    ./q2-2/q2-2.3.query.sqlpp
    • -0
    • +55
    ./q2-3/q2-3.1.ddl.sqlpp
    • -0
    • +19
    ./q2-3/q2-3.2.update.sqlpp
    • -0
    • +38
    ./q2-3/q2-3.3.query.sqlpp
    • -0
    • +55
    ./q2-4/q2-4.1.ddl.sqlpp
    • -0
    • +19
    ./q2-4/q2-4.2.update.sqlpp
    • -0
    • +39
    ./q2-4/q2-4.3.query.sqlpp
    • -0
    • +55
    ./q2-5/q2-5.1.ddl.sqlpp
    • -0
    • +19
    ./q2-5/q2-5.2.update.sqlpp
    • -0
    • +39
    ./q2-5/q2-5.3.query.sqlpp
  1. … 220 more files in changeset.
Use backticks instead of double quotes for delimited identifiers in SQL++.

Change-Id: I2db58d68e8e28befa4105504c261b101fa0285af

Reviewed-on: https://asterix-gerrit.ics.uci.edu/909

Reviewed-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Till Westmann <tillw@apache.org>

  1. … 2129 more files in changeset.
Merge branch 'master' into hyracks-merge2

  1. … 484 more files in changeset.