This is a continuation of
CALCITE-2081. The problem wasn't really solved in scope of that ticket:
- It produces incorrect results, even in the unit-test which was included into the fix (function last_value(empid) shouldn't produce NULL in such query).
- NPE is still present for more complex queries with joined window functions (e.g. if window functions with the same name are used in both part of the join).
Here is the example of the query, which produces NPE in 1.16.0:
The problem is still the same - windows produces several declarations with non-unique names, which are then incorrectly merged in BlockBuilder.append method (declaration name substitution conflicts with optimization logic).
Two variables were fixed in scope of
CALCITE-2081 - 'prevStart' and 'prevEnd', but other variables are still non-unique: result and state variables (aXwY and aXsYwZ which cause NPE during initialization) and 'list' variable (causing wrong results as both aggregates are storing results into the same variable).