Removed Exception Test (#669) #35
Merged
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
* Added more information to error messages To help debug an irreproducable/random mismatch error if it occurs in the future. * Fix for the issue when using setMaxRows() with SHOWPLAN ON (#666) * Dont throw exception for colmetadata token * Adding a comment * Update comment * Adding a warning message * remove ignoreLengthPrefixedToken * Fix for uncaught/unhandled exception (#664) * Added more information to error messages To help debug an irreproducable/random mismatch error if it occurs in the future. * Revert "Added information to error message" This reverts commit 25301e6. * Fix for #659 Added error handling logic for special cases. * Read message length Read the message length instead of reading until terminating character * Unsigned byte update Message length is an unsigned byte, converting before using. * Removed clunky hex conversions convert the byte straight to an int and use existing constants instead of making new ones * Narrowed trigger conditions fixed an issue where column names who had the hex token 'AA' would cause an error to be thrown. * Spacing fixes * Added test case * spacing adjustment * Edited test drop procedures Changed IF EXISTS DROP commands to be compatible with sql server 2008 * github spacing misalignment fixes * Changed test condition now only runs on compatible database or higher * Removed error check Removed a previous implementation in favor of one that changes the TDS parser * tdsreader change * removing test for now * enabled tests * github spacing fix * removed array import * removed "arrays" instead of "array" * spacing changes * Use Socket instead of SocketChannel when multiSubnetFailover=true (#662) * Upped SQL Server requirement to 2017 * Removing Exception Test Implement a more generic and compatible test in the future * Removed imports Used in removed test
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.
To help debug an irreproducable/random mismatch error if it occurs in the future.
Fix for the issue when using setMaxRows() with SHOWPLAN ON (Fix for the issue when using setMaxRows() with SHOWPLAN ON microsoft/mssql-jdbc#666)
Dont throw exception for colmetadata token
Adding a comment
Update comment
Adding a warning message
remove ignoreLengthPrefixedToken
Fix for uncaught/unhandled exception (Fix for uncaught/unhandled exception microsoft/mssql-jdbc#664)
Added more information to error messages
To help debug an irreproducable/random mismatch error if it occurs in the future.
This reverts commit 25301e6.
Added error handling logic for special cases.
Read the message length instead of reading until terminating character
Message length is an unsigned byte, converting before using.
convert the byte straight to an int and use existing constants instead of making new ones
fixed an issue where column names who had the hex token 'AA' would cause an error to be thrown.
Spacing fixes
Added test case
spacing adjustment
Edited test drop procedures
Changed IF EXISTS DROP commands to be compatible with sql server 2008
github spacing misalignment fixes
Changed test condition
now only runs on compatible database or higher
Removed a previous implementation in favor of one that changes the TDS parser
tdsreader change
removing test for now
enabled tests
github spacing fix
removed array import
removed "arrays" instead of "array"
spacing changes
Use Socket instead of SocketChannel when multiSubnetFailover=true (Use Socket instead of SocketChannel when multiSubnetFailover=true microsoft/mssql-jdbc#662)
Upped SQL Server requirement to 2017
Removing Exception Test
Implement a more generic and compatible test in the future
Used in removed test