Skip to content

SQLite amalgamation to 3.8.9 #171

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

Closed
wants to merge 1 commit into from
Closed

SQLite amalgamation to 3.8.9 #171

wants to merge 1 commit into from

Conversation

denji
Copy link

@denji denji commented Feb 10, 2015

The new release also conducted performance optimization of full-text queries fts3/fts4, used in combination with the OR operator. Fixed a potential integer overflow in functions sqlite3_blob_read() and sqlite3_blob_write().

Before (go test -bench=. -timeout=30s):

BenchmarkExec           200000     107113 req/s
BenchmarkQuery           50000      34443 req/s
BenchmarkParams          50000      29273 req/s
BenchmarkStmt           100000      49890 req/s
BenchmarkRows             2000       1059 req/s
BenchmarkStmtRows         2000       1088 req/s
PASS
ok      _/tmp/go/go-sqlite3-master  16.157s

After:

BenchmarkExec           200000     110548 req/s
BenchmarkQuery           50000      35105 req/s
BenchmarkParams          50000      29825 req/s
BenchmarkStmt           100000      51111 req/s
BenchmarkRows             2000       1076 req/s
BenchmarkStmtRows         2000       1100 req/s
PASS
ok      _/tmp/go/go-sqlite3-sqlite388   15.905s

@coveralls
Copy link

Coverage Status

Coverage remained the same at 84.01% when pulling 5b6351e on denji:sqlite388 into 25d045f on mattn:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 84.01% when pulling 027bc7d on denji:sqlite388 into 25d045f on mattn:master.

@mattn
Copy link
Owner

mattn commented Feb 24, 2015

Thank you. But currently, golang have issue about this.

golang/go#9356

@denji denji changed the title SQLite amalgamation to 3.8.8.2 SQLite amalgamation to 3.8.9 Apr 9, 2015
@coveralls
Copy link

Coverage Status

Coverage remained the same at 84.48% when pulling 48b2bc4 on denji:sqlite388 into e28cd44 on mattn:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 84.48% when pulling 84386b7 on denji:sqlite388 into e28cd44 on mattn:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 84.48% when pulling 9acd35d on denji:sqlite388 into e28cd44 on mattn:master.

@mattn
Copy link
Owner

mattn commented Jan 23, 2016

I didn't merge this PR, but I could decide to merge latest amalgamation code to latest. thanks!
#264

@mattn mattn closed this Jan 23, 2016
@denji denji deleted the sqlite388 branch January 23, 2016 17:07
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.

3 participants