Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

39 MySQL prepared statements #98

Merged
merged 4 commits into from
Feb 19, 2020
Merged

39 MySQL prepared statements #98

merged 4 commits into from
Feb 19, 2020

Conversation

sm-shaw
Copy link
Contributor

@sm-shaw sm-shaw commented Feb 18, 2020

Adds user configurable option to use server side prepared statements in the driver script. Also adds option to connect by socket when testing on a local server.

@sm-shaw
Copy link
Contributor Author

sm-shaw commented Feb 18, 2020

Pull request also fixed warnings in stored procedures for date formatting using str_to_date and added return value to slev stored procedure. Added Fetch values to driver scripts to ensure consistency with other implementations.

@abondvt89
Copy link
Contributor

Accepting pull request as voted on February 18, 2020 in the TPC-OSS subcommittee.

@abondvt89 abondvt89 merged commit fe1e0d6 into TPC-Council:master Feb 19, 2020
@sm-shaw sm-shaw deleted the 39 branch February 19, 2020 15:53
@sm-shaw sm-shaw linked an issue Mar 2, 2020 that may be closed by this pull request
set Name $Parent.f1.e8a
set Prompt $Parent.f1.p8a
ttk::label $Prompt -text "PostgreSQL Tablespace :"
ttk::entry $Name -width 30 -textvariable pg_tspace
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems this PR accidentally undoes the table space changes for PostgreSQL in #95.

That causes errors when trying to run TPC-C against PostgreSQL:

Error: can't read "pg_tspace": no such variable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update mysqltcl MySQL prepared statements and update
3 participants