Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Numerical Values Aren't Converted as Literals when Necessary #27

Closed
ckatsulis opened this issue Aug 20, 2017 · 1 comment
Closed
Labels

Comments

@ckatsulis
Copy link

Behavior

###Error:

> influx_select(con, "dbdata", measurement = "data", field_keys = "L1BP,L1BV,L1AP,L1AV", where = 'symbol::tag=\'tag/name\' and time >= now() - 2d', limit = 100000000)[[1]]
NULL
Warning message:
http: error parsing query: found 1e, expected integer at line 1, char 115 

No error: (notice L at the end of limit)

influx_select(con, "dbdata", measurement = "data", field_keys = "L1BP,L1BV,L1AP,L1AV", where = 'symbol::tag=\'tag/name\' and time >= now() - 2d', limit = 100000000L)[[1]]

Suggested Fix

As you have done in other portions of the package, simply format items such as limit and slimit with:

### Current issue:
> format(123456789012345)
[1] "1.234568e+14"
### Desired result:
> format(123456789012345,scientific = FALSE)
[1] "123456789012345"
@ckatsulis
Copy link
Author

Thx 👍

dleutnant added a commit that referenced this issue Aug 21, 2017
* cran submission

* update description to indicate dev version

* Fix coercion error in influx_write()` in case of sub-second accuracy (#25).

* `influx_select()` correctly parses integer arguments (#27)

* typo

* significant parsing speed improvement (naming series columns at later stage)

* Update NEWS.md

* prepare merge with master

* prepare merge with master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants