You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Suggested change to wrap the param names in square brackets in the method build_param_list. This would enable us to auto-create tables with periods in the column names (i.e. for the sample R "iris" dataset for instance). However, I don't know if this would have adverse effects elsewhere in the code, so leaving it to you to figure out whether this is the right fix, thanks!
Hello there!
Suggested change to wrap the param names in square brackets in the method build_param_list. This would enable us to auto-create tables with periods in the column names (i.e. for the sample R "iris" dataset for instance). However, I don't know if this would have adverse effects elsewhere in the code, so leaving it to you to figure out whether this is the right fix, thanks!
build_param_list <- function(query_params)
{
ps <- mapply(function(name, value)
{
type <- switch(class(value)[1],
"logical"="bool",
"numeric"="real",
"integer64"="long",
"integer"="int",
"Date"="datetime",
"POSIXct"="datetime",
"string"
)
paste(name, type, sep=":")
}, paste0('[', names(query_params), ']'), query_params)
}
The text was updated successfully, but these errors were encountered: