prestodb / RPresto

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

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

src_presto does not work with trino

ADraginda opened this issue · comments

This works:

con <- dbConnect(
  RPresto::Presto(),
  use.trino.headers=TRUE,
  ...
)

This fails

RPresto::src_presto(
  use.trino.headers=TRUE,
  ...
)

with

Error: lexical error: invalid char in json text.
                                       java.lang.IllegalArgumentExcept
                     (right here) ------^

And I think that it is because in #147 the arg was not added to src_presto?

The docs suggest that either should work, at least as I interpreted

To connect to Trino you must set the use.trino.headers parameter so RPresto knows to send the correct headers to the server. Otherwise all the same functionality is supported.

@ADraginda Are you able to send a PR to fix?

@ADraginda I can't reproduce the error. Both dbConnect() and src_presto() work fine with the use.trino.headers parameter on my machine. Can you provide a reprex for us to further troubleshoot here?

Closing for now as I can't reproduce the error. Please reopen if there's a reproducible example to troubleshoot.