Float/real type cannot be read

[OssiLehtinen]

Hi Dyfan,

it appears that noctua doesn't know how to interpret a query result with the type 'real'.

For example:

res <- DBI::dbSendStatement(cona, "create table temp.realexample as (select cast(1 as real) as realcol)")
# Wait until table is created
dplyr::tbl(cona, dbplyr::in_schema("temp", "realexample"))

produces the error: "Error: Unknown shortcut: real"

Should AthenaToRDataType-function have mapping for 'real' included (instead of 'float')?

noctua/R/DataTypes.R

Line 23 in 3b9d5d5

AthenaToRDataType.athena_data.table <-

noctua/R/DataTypes.R

Line 47 in 3b9d5d5

AthenaToRDataType.athena_vroom <-

[DyfanJones]

Hi @OssiLehtinen,

You are quiet right, this is an oversight. As I didn't realise that Athena only uses float in DDL but real in SQL functions like select cast (https://docs.aws.amazon.com/athena/latest/ug/data-types.html).

[DyfanJones]

When float has been create in the tables ddl what does that column data type return as: float or real?

I could always just add real to the switch function to make sure noctua covers all bases :)

[DyfanJones]

@OssiLehtinen is there any other Data types that are missing? I am not a 100% sure if binary, map, array and struct can be simply added without some extra thinking it through.

Potentially cast them as strings for the file readrs and then parse them using jsonlite 🤔

[OssiLehtinen]

When float has been create in the tables ddl what does that column data type return as: float or real?

I could always just add real to the switch function to make sure noctua covers all bases :)

I'm afk atm but I think it returns real. Will have to check, but probably only tomorrow.

[OssiLehtinen]

@OssiLehtinen is there any other Data types that are missing? I am not a 100% sure if binary, map, array and struct can be simply added without some extra thinking it through.

Potentially cast them as strings for the file readrs and then parse them using jsonlite 🤔

Didn't bump into issues with other baisc types.

Arrays etc definitely don't work so I have been avoiding them. An interesting idea to read them in as strings and parse afterwards!

[DyfanJones]

I will raise another ticket around the array data type, as I might not be able to get it working :P