nf-sqldb icon indicating copy to clipboard operation
nf-sqldb copied to clipboard

Published 20 hours ago • verified publisher icon nextflow-io

Add `trailingSemicolon` option

Open bentsherman opened this issue 2 years ago • 2 comments

Close #16

bentsherman avatar Jul 07 '23 15:07 bentsherman

I wonder if a semicolon is needed for a single SQL statement. May it can just be removed

pditommaso avatar Jul 10 '23 07:07 pditommaso

I know that MySQL expects it, because of how many times it complained to me in the mysql shell 😆

I would be fine with removing it or adding this option to disable it. Because we aim to support any SQL-like backend, we should not try to adjust the query to fit any particular syntax, or at least we should make the adjustment optional.

bentsherman avatar Jul 10 '23 15:07 bentsherman