Skip to content

For IR type of splits, if the timestamp column is stored in date string, the pushdown will not work #32

Description

@anlowee

Bug description

Presto stores the TIMESTAMP as integer or a packed number for TIMESTAMP WITH TIMEZONE. So I expect that the timestamp pushdown passed to Velox will never contain date string. Therefore, if the timestamp field in the IR file is actually a string, then it will have issues according to @LinZhihao-723 and @gibber9809 's comments.

System information

Velox System Info v0.0.2
Commit: 7ae5119
CMake Version: 3.28.3
System: Linux-4.15.0-197-generic
Arch: x86_64
C++ Compiler: /usr/bin/c++
C++ Compiler Version: 11.4.0
C Compiler: /usr/bin/cc
C Compiler Version: 11.4.0
CMake Prefix Path: /usr/local;/usr;/;/usr/local/lib/python3.10/dist-packages/cmake/data;/usr/local;/usr/X11R6;/usr/pkg;/opt

Relevant logs

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions