Skip to content

Conversation

@kumarUjjawal
Copy link
Contributor

@kumarUjjawal kumarUjjawal commented Dec 27, 2025

Which issue does this PR close?

Rationale for this change

As per the goal stated that we should avoid using the user_defined in useful places.

What changes are included in this PR?

  • Refactored Spark ascii to not use user_defined.
  • Checked the Spark code to make sure it follows the same convention

Are these changes tested?

  • This is a refactoring so no new tests were added but all previous test pass.

Are there any user-facing changes?

@github-actions github-actions bot added the spark label Dec 27, 2025
@Jefffrey Jefffrey added this pull request to the merge queue Dec 29, 2025
Merged via the queue into apache:main with commit 83ed192 Dec 29, 2025
29 checks passed
@Jefffrey
Copy link
Contributor

Thanks @kumarUjjawal

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