Add support for binding parameters

[meyskens]

I am using the github.com/go-sql-driver/mysql MySQL driver for Go. When I try to execute a query containing a variable i get the error that it is not implemented yet.

Error 1047: command handling not implemented yet: 22

q, err := db.Query(`
	SELECT 
		COUNT(*)
	FROM
		commit_files
	NATURAL JOIN
		ref_commits
	WHERE
		ref_commits.ref_name = 'HEAD'
		AND ref_commits.history_index = 0
		AND commit_files.repository_id = ?
		AND commit_files.file_path NOT REGEXP '^vendor.*'
	`, repoID)

[erizocosmico]

Yep, that's not implemented yet AFAIK

[meyskens]

Then see this as a feature request 😉I could not find it listed here.

[erizocosmico]

After a little bit of investigation, turns out COM_STMT_PREPARE and COM_STMT_EXECUTE are not implemented in vitess.

I'm not sure if we want to do this and contribute it to vitess, give it's not a requirement for now in the OKRs. How should we proceed? @ajnavarro

More info:

• https://dev.mysql.com/doc/internals/en/com-stmt-execute.html
• https://dev.mysql.com/doc/internals/en/com-stmt-prepare.html
• https://github.com/vitessio/vitess/blob/68f33e4ca6c78e7b58f063b1543b1f67cec2d995/go/mysql/server.go#L509

[ajnavarro]

@erizocosmico we should focus on OKRs.