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 config to override shims provider class #765

Merged
merged 2 commits into from
Sep 15, 2020

Conversation

jlowe
Copy link
Member

@jlowe jlowe commented Sep 15, 2020

This is related to #409 but only partially addresses it.

This provides an internal config property that allows a user to override the normal, automatic shim detection logic. This can be used to force a particular shim provider class to be used in cases where the shim detection logic does not work (e.g.: when the reported Spark version is unrecognized).

A warning is emitted when the shims provider is overridden as it is very likely the combination of Spark version and shim version is untested and may not fully function properly.

@jlowe jlowe added feature request New feature or request SQL part of the SQL/Dataframe plugin labels Sep 15, 2020
@jlowe jlowe added this to the Sep 14 - Sep 25 milestone Sep 15, 2020
@jlowe jlowe self-assigned this Sep 15, 2020
@jlowe
Copy link
Member Author

jlowe commented Sep 15, 2020

build

revans2
revans2 previously approved these changes Sep 15, 2020
tgravescs
tgravescs previously approved these changes Sep 15, 2020
@jlowe jlowe dismissed stale reviews from tgravescs and revans2 via 11a2238 September 15, 2020 13:46
@jlowe
Copy link
Member Author

jlowe commented Sep 15, 2020

build

@jlowe jlowe merged commit 6300945 into NVIDIA:branch-0.2 Sep 15, 2020
NvTimLiu pushed a commit to NvTimLiu/spark-rapids that referenced this pull request Sep 15, 2020
* Add config to override shims provider class

Signed-off-by: Jason Lowe <[email protected]>

* Update docs for shims provider override config

Signed-off-by: Jason Lowe <[email protected]>
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* Add config to override shims provider class

Signed-off-by: Jason Lowe <[email protected]>

* Update docs for shims provider override config

Signed-off-by: Jason Lowe <[email protected]>
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* Add config to override shims provider class

Signed-off-by: Jason Lowe <[email protected]>

* Update docs for shims provider override config

Signed-off-by: Jason Lowe <[email protected]>
@jlowe jlowe deleted the shims-provider-override branch September 10, 2021 15:41
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
…IDIA#765)

Signed-off-by: spark-rapids automation <[email protected]>

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
feature request New feature or request SQL part of the SQL/Dataframe plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants