Skip to content

[SPARK-49017][SQL][3.5] Insert statement fails when multiple parameters are being used#47814

Closed
mihailomilosevic2001 wants to merge 1 commit intoapache:branch-3.5from
mihailomilosevic2001:fixParametersBackport
Closed

[SPARK-49017][SQL][3.5] Insert statement fails when multiple parameters are being used#47814
mihailomilosevic2001 wants to merge 1 commit intoapache:branch-3.5from
mihailomilosevic2001:fixParametersBackport

Conversation

@mihailomilosevic2001
Copy link
Contributor

@mihailomilosevic2001 mihailomilosevic2001 commented Aug 20, 2024

What changes were proposed in this pull request?

Fix for multiple parameters support.

This PR is a backport of #47501.

Why are the changes needed?

The use of multiple parameters with identifiers were broken

Does this PR introduce any user-facing change?

Yes, look at tests.

How was this patch tested?

New tests added to ParametersSuite.

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

No.

@mihailomilosevic2001
Copy link
Contributor Author

@MaxGekk Opened this backport PR.

Copy link
Member

@MaxGekk MaxGekk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mihailom-db Could you add the original PR description (additionally to the ref to the original PR) and remove (Backport) and add the [3.5] tag to PR's title.

@mihailomilosevic2001 mihailomilosevic2001 changed the title [SPARK-49017][SQL] Insert statement fails when multiple parameters are being used (Backport) [SPARK-49017][SQL][3.5] Insert statement fails when multiple parameters are being used Aug 20, 2024
@github-actions github-actions bot added the SQL label Aug 20, 2024
@mihailomilosevic2001 mihailomilosevic2001 changed the title [SPARK-49017][SQL][3.5] Insert statement fails when multiple parameters are being used [3.5][SPARK-49017][SQL] Insert statement fails when multiple parameters are being used Aug 20, 2024
@MaxGekk MaxGekk changed the title [3.5][SPARK-49017][SQL] Insert statement fails when multiple parameters are being used [SPARK-49017][SQL][3.5] Insert statement fails when multiple parameters are being used Aug 20, 2024
@MaxGekk
Copy link
Member

MaxGekk commented Aug 20, 2024

+1, LGTM. Merging to 3.5.
Thank you, @mihailom-db.

MaxGekk pushed a commit that referenced this pull request Aug 20, 2024
…rs are being used

### What changes were proposed in this pull request?
Fix for multiple parameters support.

This PR is a backport of #47501.

### Why are the changes needed?
The use of multiple parameters with identifiers were broken

### Does this PR introduce _any_ user-facing change?
Yes, look at tests.

### How was this patch tested?
New tests added to ParametersSuite.

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

Closes #47814 from mihailom-db/fixParametersBackport.

Authored-by: Mihailo Milosevic <mihailo.milosevic@databricks.com>
Signed-off-by: Max Gekk <max.gekk@gmail.com>
@MaxGekk MaxGekk closed this Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants