Skip to content

AL-4273 fix bugs for ReduceExpressionsRule #260

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
frearb wants to merge 1 commit into
base: kycalcite-1.16.0.x-3.x
Choose a base branch
from
Open

AL-4273 fix bugs for ReduceExpressionsRule #260

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions cassandra/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-cassandra</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Cassandra</name>
<description>Cassandra adapter for Calcite</description>

Expand Down
4 changes: 2 additions & 2 deletions core/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-core</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Core</name>
<description>Core Calcite APIs and engine.</description>

Expand Down
2 changes: 2 additions & 0 deletions core/src/main/java/org/apache/calcite/rex/RexBuilder.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1509,6 +1509,8 @@ private static Object clean(Object o, RelDataType type) {
throw new AssertionError();
}
return DateString.fromCalendarFields((Calendar) o);
} else if (o instanceof Long) {
return DateString.fromDaysSinceEpoch(((Long) o).intValue());
} else {
return DateString.fromDaysSinceEpoch((Integer) o);
}
Expand Down
1 change: 1 addition & 0 deletions core/src/main/java/org/apache/calcite/rex/RexExecutable.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,7 @@ public void reduce(RexBuilder rexBuilder, List<RexNode> constExps,
} catch (RuntimeException e) {
// One or more of the expressions failed.
// Don't reduce any of the expressions.
reducedValues.clear();
Copy link

@songzhxlh-max songzhxlh-max Oct 21, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个需要加个开关不

reducedValues.addAll(constExps);
values = new Object[constExps.size()];
}
Expand Down
4 changes: 2 additions & 2 deletions druid/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-druid</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Druid</name>
<description>Druid adapter for Calcite</description>

Expand Down
4 changes: 2 additions & 2 deletions elasticsearch2/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-elasticsearch2</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Elasticsearch</name>
<description>Elasticsearch adapter for Calcite</description>

Expand Down
4 changes: 2 additions & 2 deletions elasticsearch5/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-elasticsearch5</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Elasticsearch5</name>
<description>Elasticsearch5 adapter for Calcite</description>

Expand Down
4 changes: 2 additions & 2 deletions example/csv/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite-example</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-example-csv</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Example CSV</name>
<description>An example Calcite provider that reads CSV files</description>

Expand Down
4 changes: 2 additions & 2 deletions example/function/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite-example</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-example-function</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Example Function</name>
<description>Examples of user-defined Calcite functions</description>

Expand Down
4 changes: 2 additions & 2 deletions example/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,13 +20,13 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<!-- The basics. -->
<artifactId>calcite-example</artifactId>
<packaging>pom</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Examples</name>
<description>Calcite examples</description>

Expand Down
4 changes: 2 additions & 2 deletions file/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,13 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<!-- The basics. -->
<artifactId>calcite-file</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite File</name>
<description>Calcite provider that reads files and URIs</description>

Expand Down
4 changes: 2 additions & 2 deletions geode/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-geode</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Geode</name>
<description>Geode adapter for Calcite</description>

Expand Down
4 changes: 2 additions & 2 deletions linq4j/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-linq4j</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Linq4j</name>
<description>Calcite APIs for LINQ (Language-Integrated Query) in Java</description>

Expand Down
4 changes: 2 additions & 2 deletions mongodb/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-mongodb</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite MongoDB</name>
<description>MongoDB adapter for Calcite</description>

Expand Down
4 changes: 2 additions & 2 deletions pig/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-pig</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Pig</name>
<description>Pig adapter for Calcite</description>

Expand Down
4 changes: 2 additions & 2 deletions piglet/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-piglet</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Piglet</name>
<description>Pig-like language built on top of Calcite algebra</description>

Expand Down
4 changes: 2 additions & 2 deletions plus/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-plus</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Plus</name>
<description>Miscellaneous extras for Calcite</description>

Expand Down
4 changes: 2 additions & 2 deletions pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ limitations under the License.
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<packaging>pom</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>

<!-- More project information. -->
<name>Calcite</name>
Expand Down Expand Up @@ -58,7 +58,7 @@ limitations under the License.

<!-- This list is in alphabetical order. -->
<airlift-tpch.version>0.1</airlift-tpch.version>
<avatica.version>1.11.0.x-3.x-r2</avatica.version>
<avatica.version>1.11.0.x-3.x-r3</avatica.version>
<build-helper-maven-plugin.version>3.0.0</build-helper-maven-plugin.version>
<cassandra-driver-core.version>3.4.0</cassandra-driver-core.version>
<checksum-maven-plugin.version>1.2</checksum-maven-plugin.version>
Expand Down
4 changes: 2 additions & 2 deletions server/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-server</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Server</name>
<description>Calcite Server</description>

Expand Down
4 changes: 2 additions & 2 deletions spark/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-spark</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Spark</name>

<properties>
Expand Down
4 changes: 2 additions & 2 deletions splunk/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,12 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<artifactId>calcite-splunk</artifactId>
<packaging>jar</packaging>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
<name>Calcite Splunk</name>
<description>Splunk adapter for Calcite; also a JDBC driver for Splunk</description>

Expand Down
2 changes: 1 addition & 1 deletion ubenchmark/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ limitations under the License.
<parent>
<groupId>org.apache.calcite</groupId>
<artifactId>calcite</artifactId>
<version>1.16.0-kylin-3.x-r58</version>
<version>1.16.0-kylin-3.x-r59</version>
</parent>

<properties>
Expand Down