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

Add our session extension correctly if there are other extensions configured #376

Merged

Conversation

abellina
Copy link
Collaborator

Fixes a small bug if other Spark Session Extensions (--conf spark.sql.extensions) were already specified at launch, as we mutate this collection to add our own extension.

@abellina abellina added bug Something isn't working SQL part of the SQL/Dataframe plugin labels Jul 16, 2020
@revans2
Copy link
Collaborator

revans2 commented Jul 16, 2020

build

@abellina abellina merged commit aff67ca into NVIDIA:branch-0.2 Jul 16, 2020
@abellina abellina deleted the play_nice_with_other_session_extensions branch July 16, 2020 20:17
@abellina
Copy link
Collaborator Author

Thanks @revans2

@sameerz sameerz added this to the Jul 6 - Jul 17 milestone Jul 17, 2020
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
pxLi pushed a commit to pxLi/spark-rapids that referenced this pull request May 12, 2022
* Update CI/CD requirements and tests

* Fix main ci (NVIDIA#377)

* Fix build doc process (NVIDIA#387)

* Fix app testing pt model validator

* Update tests

* Fix format issues

* Fix integration tests

* Update premerge.yml

* Fix format

* Fix comments
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
Signed-off-by: spark-rapids automation <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working SQL part of the SQL/Dataframe plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants