New version: ODBC v1.1.0 #66084
Merged
New version: ODBC v1.1.0 #66084
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
JuliaRegistrator
commented
Aug 11, 2022
JuliaRegistrator
commented
- Registering package: ODBC
- Repository: https://github.com/JuliaDatabases/ODBC.jl
- Created by: @quinnj
- Version: v1.1.0
- Commit: a7ce1365d6ab070196dedc8dfab4ea8f30213587
- Reviewed by: @quinnj
- Reference: JuliaDatabases/ODBC.jl@a7ce136#commitcomment-80926783
- Description: An ODBC interface for the Julia programming language
UUID: be6f12e9-ca4f-5eb2-a339-a4f995cc0291 Repo: https://github.com/JuliaDatabases/ODBC.jl.git Tree: ab457f6208fa4faa5675c81d32275e661a06ad9e Registrator tree SHA: 8e1a5ac2695627143951512d700c7e3c445102ec
JuliaRegistrator
referenced
this pull request
in JuliaDatabases/ODBC.jl
Aug 11, 2022
* Fix case when SQLGetData data size exceeds column size Fixes #328. Ok, a bit nasty, but here's the rundown: * This is mostly outlined [here](https://docs.microsoft.com/en-us/sql/odbc/reference/syntax/sqlgetdata-function?view=sql-server-ver16#retrieving-data-with-sqlgetdata) * The issue is that for our result set, the cursor may be told that a give column has a certain size in bytes, particularly for variable-sized columns like VARCHAR * BUT, the driver then may have to "convert to the target type" and this conversion may result in the _actual_ data size being larger than we orignally allocated for * But wait, you may ask, don't we already account for this case [here](https://github.com/JuliaDatabases/ODBC.jl/blob/f38f771557a5763f777ed0f441c2cc5a72f41c70/src/utils.jl#L328) by resizing the buffer and calling SQLGetData again to retrieve the rest of the data? * Why yes, we do, but only for "long" data types, which is the primary use case for the multiple SQLGetData calls * So basically this is _another_ case where, totally depending on the db driver, we might end up with truncated data which will result in `API.SQL_SUCCESS_WITH_INFO` being returned from SQLGetData Thankfully the fix is relatively easy: we just need to use our buffer-resizing branch if the data type is long OR SQLGetData returns this success with info status code. * bump some versions and update ci
|
Your If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.