prestodb / RPresto

DBI-based adapter for Presto for the statistical programming language R.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

`DECIMAL` types should be translated to numeric rather than character

jarodmeng opened this issue · comments

Currently RPresto converts DECIMAL types to character (see #60 and #64) because the previous translation code (implemented in Rcpp) has limitations. Presto treats DECIMAL type columns like a number (i.e. you can do arithmetic operations on DECIMAL) rather than a string. So it makes sense to translate it into numeric in R. Since the refactoring of the translation code (see #158), this should be quite straightforward.