Skip to content

Conversation

ivoson
Copy link
Contributor

@ivoson ivoson commented Oct 6, 2025

What changes were proposed in this pull request?

This is a follow up of #50230 to fix the code style issues in MapStatusEndToEndSuite.

Why are the changes needed?

Fix indentation issue.

Does this PR introduce any user-facing change?

No

How was this patch tested?

NA

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

No

@github-actions github-actions bot added the SQL label Oct 6, 2025
@ivoson
Copy link
Contributor Author

ivoson commented Oct 6, 2025

cc @cloud-fan

@HyukjinKwon
Copy link
Member

Let's avoid making a PR for such style changes. It's not really worth while spending time on this in a separate PR. we can fix it together when we happen to touch the code around here.

@ivoson
Copy link
Contributor Author

ivoson commented Oct 13, 2025

Let's avoid making a PR for such style changes. It's not really worth while spending time on this in a separate PR. we can fix it together when we happen to touch the code around here.

Sure. Closing this then.

@ivoson ivoson closed this Oct 13, 2025
@cloud-fan
Copy link
Contributor

This is a quick followup to fix the wrong code style introduced by a recently merged PR from the same PR author, I think we should accept it? @HyukjinKwon

@ivoson ivoson deleted the SPARK-51756-FollowUp branch October 15, 2025 01:55
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.

3 participants