[ https://issues.apache.org/jira/browse/IGNITE-22739?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Maksim Zhuravkov updated IGNITE-22739: -------------------------------------- Description: AccumulatorsFactory use `RexToLitTranslator` from `org.apache.ignite.internal.sql.engine.exec.exp` instead of the one provided by calcite. Because the fixed version correctly handles cast overflows: {noformat} SELECT AVG(12345678::TINYINT) {noformat} Code generated by calcite: {noformat} inTypes: [TINYINT] // AVG argument cast COMPILE public Object apply(Object in) { return Double.valueOf(org.apache.calcite.runtime.SqlFunctions.toByte(in)); } inType: DOUBLE, outType: TINYINT // AVG output cast COMPILE public Object apply(Object in) { return (Double) in == null ? null : (Byte) org.apache.calcite.linq4j.tree.Primitive.integerCast(org.apache.calcite.linq4j.tree.Primitive.BYTE, ((Double) in).doubleValue()); } COMPILE public void execute(org.apache.ignite.internal.sql.engine.exec.ExecutionContext ctx, Object in1, org.apache.ignite.internal.sql.engine.exec.RowHandler.RowBuilder out) { try { out.addField(org.apache.ignite.internal.sql.engine.util.IgniteMath.convertToByteExact(12345678)); } catch (Exception e) { throw new org.apache.ignite.sql.SqlException( 262151, e); } } {noformat} Code generated by the updated version of the RexToLitTranslator {noformat} inTypes: [TINYINT] COMPILE public Object apply(Object in) { return Double.valueOf(org.apache.calcite.runtime.SqlFunctions.toByte(in)); } inType: DOUBLE, outType: TINYINT COMPILE public Object apply(Object in) { return (Double) in == null ? null : Byte.valueOf(org.apache.ignite.internal.sql.engine.util.IgniteMath.convertToByteExact(((Double) in).doubleValue())); } COMPILE public void execute(org.apache.ignite.internal.sql.engine.exec.ExecutionContext ctx, Object in1, org.apache.ignite.internal.sql.engine.exec.RowHandler.RowBuilder out) { try { out.addField(org.apache.ignite.internal.sql.engine.util.IgniteMath.convertToByteExact(12345678)); } catch (Exception e) { throw new org.apache.ignite.sql.SqlException( 262151, e); } } {noformat} was: 1. AccumulatorsFactory use `RexToLitTranslator` from `org.apache.ignite.internal.sql.engine.exec.exp` instead of the one provided by calcite. Because the fixed version correctly handles cast overflows: {noformat} SELECT AVG(12345678::TINYINT) {noformat} Code generated by calcite: {noformat} inTypes: [TINYINT] // AVG argument cast COMPILE public Object apply(Object in) { return Double.valueOf(org.apache.calcite.runtime.SqlFunctions.toByte(in)); } inType: DOUBLE, outType: TINYINT // AVG output cast COMPILE public Object apply(Object in) { return (Double) in == null ? null : (Byte) org.apache.calcite.linq4j.tree.Primitive.integerCast(org.apache.calcite.linq4j.tree.Primitive.BYTE, ((Double) in).doubleValue()); } COMPILE public void execute(org.apache.ignite.internal.sql.engine.exec.ExecutionContext ctx, Object in1, org.apache.ignite.internal.sql.engine.exec.RowHandler.RowBuilder out) { try { out.addField(org.apache.ignite.internal.sql.engine.util.IgniteMath.convertToByteExact(12345678)); } catch (Exception e) { throw new org.apache.ignite.sql.SqlException( 262151, e); } } {noformat} Code generated by the updated version of the RexToLitTranslator {noformat} inTypes: [TINYINT] COMPILE public Object apply(Object in) { return Double.valueOf(org.apache.calcite.runtime.SqlFunctions.toByte(in)); } inType: DOUBLE, outType: TINYINT COMPILE public Object apply(Object in) { return (Double) in == null ? null : Byte.valueOf(org.apache.ignite.internal.sql.engine.util.IgniteMath.convertToByteExact(((Double) in).doubleValue())); } COMPILE public void execute(org.apache.ignite.internal.sql.engine.exec.ExecutionContext ctx, Object in1, org.apache.ignite.internal.sql.engine.exec.RowHandler.RowBuilder out) { try { out.addField(org.apache.ignite.internal.sql.engine.util.IgniteMath.convertToByteExact(12345678)); } catch (Exception e) { throw new org.apache.ignite.sql.SqlException( 262151, e); } } {noformat} 2. RexExecutorImpl used by RexSimplifier also uses `RexToLitTranslator` provided by calcite. See: {code:java} Frameworks.newConfigBuilder().executor(new RexExecutorImpl(DataContexts.EMPTY)) // in org.apache.ignite.internal.sql.engine.util.Commons {code} > Sql. Use modified version of the RexToLitTranslator > --------------------------------------------------- > > Key: IGNITE-22739 > URL: https://issues.apache.org/jira/browse/IGNITE-22739 > Project: Ignite > Issue Type: Improvement > Components: sql > Reporter: Maksim Zhuravkov > Priority: Major > Labels: ignite-3 > > AccumulatorsFactory use `RexToLitTranslator` from > `org.apache.ignite.internal.sql.engine.exec.exp` instead of the one provided > by calcite. Because the fixed version correctly handles cast overflows: > {noformat} > SELECT AVG(12345678::TINYINT) > {noformat} > Code generated by calcite: > {noformat} > inTypes: [TINYINT] // AVG argument cast > COMPILE > public Object apply(Object in) { > return Double.valueOf(org.apache.calcite.runtime.SqlFunctions.toByte(in)); > } > inType: DOUBLE, outType: TINYINT // AVG output cast > COMPILE > public Object apply(Object in) { > return (Double) in == null ? null : (Byte) > org.apache.calcite.linq4j.tree.Primitive.integerCast(org.apache.calcite.linq4j.tree.Primitive.BYTE, > ((Double) in).doubleValue()); > } > COMPILE > public void > execute(org.apache.ignite.internal.sql.engine.exec.ExecutionContext ctx, > Object in1, org.apache.ignite.internal.sql.engine.exec.RowHandler.RowBuilder > out) { > try { > > out.addField(org.apache.ignite.internal.sql.engine.util.IgniteMath.convertToByteExact(12345678)); > } catch (Exception e) { > throw new org.apache.ignite.sql.SqlException( > 262151, > e); > } > } > {noformat} > Code generated by the updated version of the RexToLitTranslator > {noformat} > inTypes: [TINYINT] > COMPILE > public Object apply(Object in) { > return Double.valueOf(org.apache.calcite.runtime.SqlFunctions.toByte(in)); > } > inType: DOUBLE, outType: TINYINT > COMPILE > public Object apply(Object in) { > return (Double) in == null ? null : > Byte.valueOf(org.apache.ignite.internal.sql.engine.util.IgniteMath.convertToByteExact(((Double) > in).doubleValue())); > } > COMPILE > public void > execute(org.apache.ignite.internal.sql.engine.exec.ExecutionContext ctx, > Object in1, org.apache.ignite.internal.sql.engine.exec.RowHandler.RowBuilder > out) { > try { > > out.addField(org.apache.ignite.internal.sql.engine.util.IgniteMath.convertToByteExact(12345678)); > } catch (Exception e) { > throw new org.apache.ignite.sql.SqlException( > 262151, > e); > } > } > {noformat} -- This message was sent by Atlassian Jira (v8.20.10#820010)