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 CallableStatement default value regression (2452) #2456

Merged
merged 2 commits into from
Jun 20, 2024

Conversation

tkyc
Copy link
Member

@tkyc tkyc commented Jun 19, 2024

Backport #2452

tkyc and others added 2 commits June 19, 2024 16:38
* Fixed default value regression for cstmts

* Adjusted regex

* Test update

* Updated test p2

* Updated regex
@Jeffery-Wasty Jeffery-Wasty changed the title [Backport] Fixed CallableStatement default value regression [Backport] Fixed CallableStatement default value regression (2452) Jun 19, 2024
@Jeffery-Wasty Jeffery-Wasty added this to the 12.6.3 milestone Jun 19, 2024
@Jeffery-Wasty Jeffery-Wasty merged commit 6e9c756 into release/12.6 Jun 20, 2024
4 checks passed
@Jeffery-Wasty Jeffery-Wasty deleted the 12.6.3-cstmt-default branch June 20, 2024 17:59
dongjoon-hyun pushed a commit to apache/spark that referenced this pull request Jul 22, 2024
### What changes were proposed in this pull request?
The pr aims to upgrade `mssql-jdbc` from `12.6.2.jre11` to `12.6.3.jre11`

### Why are the changes needed?
https://github.com/microsoft/mssql-jdbc/releases/tag/v12.6.3
Hotfix & Stable Release:
- Fixed issue where TokenCredential class was required to be imported microsoft/mssql-jdbc#2453
- Fixed timestamp string conversion regression microsoft/mssql-jdbc#2455
- Fixed SQLServerCallableStatement default value regression microsoft/mssql-jdbc#2456

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Pass GA.

### Was this patch authored or co-authored using generative AI tooling?
No.

Closes #47075 from panbingkun/SPARK-48703.

Authored-by: panbingkun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
attilapiros pushed a commit to attilapiros/spark that referenced this pull request Oct 4, 2024
### What changes were proposed in this pull request?
The pr aims to upgrade `mssql-jdbc` from `12.6.2.jre11` to `12.6.3.jre11`

### Why are the changes needed?
https://github.com/microsoft/mssql-jdbc/releases/tag/v12.6.3
Hotfix & Stable Release:
- Fixed issue where TokenCredential class was required to be imported microsoft/mssql-jdbc#2453
- Fixed timestamp string conversion regression microsoft/mssql-jdbc#2455
- Fixed SQLServerCallableStatement default value regression microsoft/mssql-jdbc#2456

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Pass GA.

### Was this patch authored or co-authored using generative AI tooling?
No.

Closes apache#47075 from panbingkun/SPARK-48703.

Authored-by: panbingkun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
himadripal pushed a commit to himadripal/spark that referenced this pull request Oct 19, 2024
### What changes were proposed in this pull request?
The pr aims to upgrade `mssql-jdbc` from `12.6.2.jre11` to `12.6.3.jre11`

### Why are the changes needed?
https://github.com/microsoft/mssql-jdbc/releases/tag/v12.6.3
Hotfix & Stable Release:
- Fixed issue where TokenCredential class was required to be imported microsoft/mssql-jdbc#2453
- Fixed timestamp string conversion regression microsoft/mssql-jdbc#2455
- Fixed SQLServerCallableStatement default value regression microsoft/mssql-jdbc#2456

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Pass GA.

### Was this patch authored or co-authored using generative AI tooling?
No.

Closes apache#47075 from panbingkun/SPARK-48703.

Authored-by: panbingkun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Closed/Merged PRs
Development

Successfully merging this pull request may close these issues.

4 participants