Comments (3)
I agree that b
is too weird. Another alternative (let's call it d
) would be to input the bound arguments only to the last statement. I wonder what the other DuckDB binding libraries (e.g. Python or Node) do?
Intuitively, I think a
would make the most sense – I took a look, and I think we can use duckdb_nparams to distribute the args correctly over the statements as they're prepared and executed.
If there are multiple SELECT
statements in one query string (which I would never do), I would expect to get the result of only the last SELECT
statement (but the error of the first that failed, if any).
Generally though, I don't have much of a use case for doing multiple statements with bound args. The main use cases I can think of for multi-statement queries are when loading queries from a file (e.g. migrations) or receiving them from the user – in both cases, bound args are unlikely to be used. So c
should also be fine.
from go-duckdb.
Thanks for your detailed opinion. I agree with you, that having multiple SELECTs in one statement is kinda weird.
It seems, that the Python driver applies the given parameters to each query instead of distributing them.
The C++ API seems to have the identical behavior. This is similar to b
.
And the Python driver also returns the result of the last statement.
I think I need to do more research on this to prevent total confusion.
from go-duckdb.
If that's what the official drivers do, I would probably just do the same for Go :)
from go-duckdb.
Related Issues (20)
- Add CI script to run gofumpt
- hope create a new tag for last update HOT 1
- Build failures on redhat linux HOT 1
- Support missing timestamp types in the appender HOT 1
- Detect incorrect column counts in the appender
- transaction.go:6:5: undefined: conn HOT 9
- Exposing filesystem interface HOT 1
- Make Apache Arrow Optional HOT 2
- Add support for the DATE type in the appender
- Does `go-duckdb` has a glibc version dependency? HOT 2
- Silent primary key violation in the appender HOT 2
- Appending a NULL in UUID[] column panics HOT 1
- found architecture 'arm64', required architecture 'x86_64' on mac os HOT 3
- libduckdb.a, building for macOS-x86_64 but attempting to link with file built for macOS-arm64 HOT 2
- Info: Deploying go-duckdb In A Distroless Docker Container
- DuckDB version mismatch in v1.6.5 release HOT 1
- Auto-load / auto-install in go-duckdb? And packaging JSON? HOT 1
- Cannot run duckdb v1.6.6 . Apache arrow lib broken HOT 1
- macos `go run` works but unable to execute the output of `go build` HOT 3
- Proposal: Support better error messages HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from go-duckdb.