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

[SEDONA-636] fix some bugs when sedona parser running with other parser #1706

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

freamdx
Copy link
Contributor

@freamdx freamdx commented Nov 29, 2024

  • fix some blocking problem when running sql concurrently
  • fix some error when setting "spark.sql.extensions" with different order
  • improve performance

@jiayuasu
Copy link
Member

jiayuasu commented Dec 3, 2024

@freamdx can you address the comment from @Kontinuation ? Thanks! Then I will merge the PR.

@freamdx
Copy link
Contributor Author

freamdx commented Dec 8, 2024

@freamdx can you address the comment from @Kontinuation ? Thanks! Then I will merge the PR.

done

@jiayuasu
Copy link
Member

jiayuasu commented Dec 8, 2024

@freamdx

Can you do this? Thanks!

ParserRegistrator.scala is not needed anymore, as for now we register parsers using the extension interface instead of using the reflection trick. We can delete this file.

@freamdx
Copy link
Contributor Author

freamdx commented Dec 10, 2024

@freamdx

Can you do this? Thanks!

ParserRegistrator.scala is not needed anymore, as for now we register parsers using the extension interface instead of using the reflection trick. We can delete this file.

done

@jiayuasu jiayuasu added this to the sedona-1.7.1 milestone Dec 11, 2024
@jiayuasu jiayuasu merged commit 12903e1 into apache:master Dec 11, 2024
39 checks passed
@freamdx
Copy link
Contributor Author

freamdx commented Dec 17, 2024

@jiayuasu
can you release a new version before 2024/12/31? such as 1.7.1, i need this patch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants