sqltargets icon indicating copy to clipboard operation
sqltargets copied to clipboard
verified publisher icon daranzolin

Should the connection be passed to tar_sql() instead of parsing from query?

Open daranzolin opened this issue 1 year ago • 2 comments

daranzolin avatar Oct 24 '24 23:10 daranzolin

I have somewhat of a proof of concept in my sql-connect-func branch kiwiroy/sqltargets@5089065.

kiwiroy avatar Nov 18 '25 20:11 kiwiroy

Whoa @kiwiroy, I think I prefer that to what we have now. Putting the connection function in _targets.R fits thematically with all the other targets configurations.

At least one reason for keeping it in the file is 'SQL Preview' in RStudio. It's a nice feature but useless once parameters are added. Perhaps we need an RStudio add-in that 'compiles' any parameterized query.

daranzolin avatar Nov 19 '25 23:11 daranzolin