Hello Anon. E. Moose #1000171, Jenkins,

I'd like you to reexamine a change.  Please visit

    https://asterix-gerrit.ics.uci.edu/2980

to look at the new patch set (#3).

Change subject: [NO ISSUE][COMP] allow optimization rule to inspect other 
functions + fixes
......................................................................

[NO ISSUE][COMP] allow optimization rule to inspect other functions + fixes

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
1. allow optimization rule "InjectTypeCastForSwitchCaseRule"
(renamed to "InjectTypeCastForFunctionArgumentsRule" now) to consider other
functions specified by the user in addition to the ones already considered
by the rule.
2. fix an issue related to ConstantFoldingRule that affects some existing
functions where the rule would have the function compute the value but fail
to deserialize it (read it) back due to differences in compile-time type
and runtime type.
3. fix static casting of a list constructor function which fails to cast
open its items when the item is a function that returns a closed derived
type.

Change-Id: I65679e934a41e00b04bc3fd479ed404af293eb21
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/ConstantFoldingRule.java
R 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/InjectTypeCastForFunctionArgumentsRule.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/typecast/StaticTypeCastUtil.java
4 files changed, 57 insertions(+), 27 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/80/2980/3
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2980
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I65679e934a41e00b04bc3fd479ed404af293eb21
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Ali Alsuliman <ali.al.solai...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Dmitry Lychagin <dmitry.lycha...@couchbase.com>
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>

Reply via email to