[CALCITE-3477] Geospatial function (ST_MakePoint) with integer parameter fails

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.22.0
• Component/s: core
• Labels: geospatial pull-request-available
• Assignee: Feng Zhu
• Reporter: Kirils Mensikovs
• Created: 05/Nov/19 12:17
• Updated: 06/Mar/20 04:34
• Resolved: 09/Dec/19 02:24

Description

Geospatial function with integer parameter fails. The expected behavior is to cast automatically all number values to BigDecimal.

Example: 'select  ST_MAKEPOINT(1.0, 1)'

Return:

Error: Error while executing SQL "select  ST_MAKEPOINT(1.0, 1)": Error while compiling generated Java code:
public org.apache.calcite.linq4j.Enumerable bind(final org.apache.calcite.DataContext root) {
  final org.apache.calcite.linq4j.Enumerable _inputEnumerable = org.apache.calcite.linq4j.Linq4j.asEnumerable(new Integer[] {
    0});
  return new org.apache.calcite.linq4j.AbstractEnumerable(){
      public org.apache.calcite.linq4j.Enumerator enumerator() {
        return new org.apache.calcite.linq4j.Enumerator(){
            public final org.apache.calcite.linq4j.Enumerator inputEnumerator = _inputEnumerable.enumerator();
            public void reset() {
              inputEnumerator.reset();
            }


            public boolean moveNext() {
              return inputEnumerator.moveNext();
            }


            public void close() {
              inputEnumerator.close();
            }


            public Object current() {
              final java.math.BigDecimal v = $L4J$C$new_java_math_BigDecimal_1_0_;
              return org.apache.calcite.runtime.GeoFunctions.ST_MakePoint(v, 1);
            }


            static final java.math.BigDecimal $L4J$C$new_java_math_BigDecimal_1_0_ = new java.math.BigDecimal(
              "1.0");
          };
      }


    };
}




public Class getElementType() {
  return org.apache.calcite.runtime.GeoFunctions.Geom.class;
} (state=,code=0)

 

Issue Links

• is blocked by: CALCITE-3565 Explicitly cast assignable operand types to decimal for udf
• links to: GitHub Pull Request #1583