search

nfultz / AWR.Athena

Short R Wrapper for Athena JDBC connections
23 stars 13 forks source link

Dealing with fetchSize internally #7

Closed tonytonov closed 6 years ago

tonytonov commented 6 years ago

It's probably a good idea to include the fix proposed at https://stackoverflow.com/questions/44379318/the-requested-fetchsize-is-more-than-the-allowed-value-in-athena-with-jdbc-dri to deal with "The requested fetchSize is more than the allowed value in Athena." issue.

The solution itself is quite trivial, however incorporating it into the package is a bit trickier, since one needs to reexport the appropriate S4 method. I have a working solution at hand, so let me know if you'd like a PR.

daroczig commented 6 years ago

I just bumped into this issue -- can you please check if #8 solves this problem?

nfultz commented 6 years ago

I've posted an alternative fix in #9 taking the S4 route.

nfultz commented 6 years ago

Fixed, new version should be going to cran this week.

tonytonov commented 6 years ago

Thanks!