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

[Backport] Fixed timestamp string conversion error for cstmt #2454

Closed
wants to merge 13 commits into from

Conversation

tkyc
Copy link
Member

@tkyc tkyc commented Jun 19, 2024

Backport #2449

tkyc and others added 13 commits February 16, 2024 16:39
…eStatements (#2325) (#2329)

* EXEC system stored procedure regression fix

* Additional test

* Additional test

* Indenting

* Switched error string to TestResource error string

* CR comments

* Test update p1

* Test update p2

* CR comment changes; Test update

* call escape syntax check

* CR changes

* formatting
* 12.6.1 version bump

* Update src/samples/dataclassification/pom.xml

Co-authored-by: David Engel <v-davidengel@microsoft.com>

---------

Co-authored-by: David Engel <v-davidengel@microsoft.com>
…2427)

* Fix calling procedures with output parameters by their four-part syntax (#2349)

* Corrected four part syntax regression

* JDK 8 correction

* Execute stored procedures directly for RPC calls (#2410)

* RPC fix

* Assertion index correction

* Removed magic number

* Removed login drop command

* Code review changes

* Formatting

* Test config changes
* Updates for running tests with managed identity (#2416)

* Updated tests for removal of secrets

* Reverted TimeoutTests

* AE config port

* TimeoutTests update

---------

Co-authored-by: lilgreenbird <v-susanh@microsoft.com>
…successful connection open (2355) (#2431)

* socketTimeout should reset to original value after a successful connection open (#2355)

* SocketTimeout should be unbounded by loginTimeout after a successful connection open
* Fixed timestamp string conversion error for cstmt

* Code review comments p1

* Fixed sproc used in test
@tkyc tkyc closed this Jun 19, 2024
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.

2 participants