[CALCITE-6637] Date type results should not be automatically converted to timestamp in Arrow adapters

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 1.38.0
• Fix Version/s: 1.39.0
• Component/s: arrow-adapter
• Labels: pull-request-available
• Assignee: Caican Cai
• Reporter: Caican Cai
• Created: 4 days ago 10:44
• Updated: Yesterday 20:35
• Resolved: Yesterday 20:35

Description

Currently, Calcite returns the timestamp type for the arrow date type, which is wrong. It should return the date type.

Currently

 @Test void testDateProject() {
    String sql = "select HIREDATE from EMP";
    String plan = "PLAN=ArrowToEnumerableConverter\n"
        + "  ArrowProject(HIREDATE=[$4])\n"
        + "    ArrowTableScan(table=[[ARROW, EMP]], fields=[[0, 1, 2, 3, 4, 5, 6, 7]])\n\n";
    String result = "HIREDATE=1970-01-01 00:00:04\n"
        + "HIREDATE=1970-01-01 00:00:04\n"
        + "HIREDATE=1970-01-01 00:00:04\n";
    CalciteAssert.that()
        .with(arrow)
        .query(sql)
        .limit(3)
        .returns(result)
        .explainContains(plan);

The correct one is

  @Test void testDateProject() {
    String sql = "select HIREDATE from EMP";
    String plan = "PLAN=ArrowToEnumerableConverter\n"
        + "  ArrowProject(HIREDATE=[$4])\n"
        + "    ArrowTableScan(table=[[ARROW, EMP]], fields=[[0, 1, 2, 3, 4, 5, 6, 7]])\n\n";
    String result = "HIREDATE=1980-12-17\nHIREDATE=1981-02-20\nHIREDATE=1981-02-22\n";

    CalciteAssert.that()
        .with(arrow)
        .query(sql)
        .limit(3)
        .returns(result)
        .explainContains(plan);
  }

Issue Links

• links to: GitHub Pull Request #4010