Tim Armstrong created IMPALA-5572:
-------------------------------------

             Summary: Support timestamp codegen for text scanner
                 Key: IMPALA-5572
                 URL: https://issues.apache.org/jira/browse/IMPALA-5572
             Project: IMPALA
          Issue Type: Improvement
          Components: Backend
    Affects Versions: Impala 2.10.0
            Reporter: Tim Armstrong
            Priority: Minor


Codegen is disabled when scanning text tables with timestamp columns. The 
message is "Timestamp not yet supported for codegen."

The supported types use cross-compiled conversion functions in 
TextConverter::CodegenWriteSlot():
{code}
    IRFunction::Type parse_fn_enum;
    Function* parse_fn = NULL;
    switch (slot_desc->type().type) {
      case TYPE_BOOLEAN:
        parse_fn_enum = IRFunction::STRING_TO_BOOL;
        break;
      case TYPE_TINYINT:
        parse_fn_enum = IRFunction::STRING_TO_INT8;
        break;
      case TYPE_SMALLINT:
        parse_fn_enum = IRFunction::STRING_TO_INT16;
        break;
      case TYPE_INT:
        parse_fn_enum = IRFunction::STRING_TO_INT32;
        break;
      case TYPE_BIGINT:
        parse_fn_enum = IRFunction::STRING_TO_INT64;
        break;
      case TYPE_FLOAT:
        parse_fn_enum = IRFunction::STRING_TO_FLOAT;
        break;
      case TYPE_DOUBLE:
        parse_fn_enum = IRFunction::STRING_TO_DOUBLE;
        break;
      default:
        DCHECK(false);
        return NULL;
    }
{code}

TimestampValue::Parse() does not implement the same interface as the other 
functions but I think we could either change the interface or cross-compile an 
adapter function.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to