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

2022-01-04-check_spelling_dl_en #6706

Conversation

jsl-models
Copy link
Collaborator

No description provided.

@maziyarpanahi
Copy link
Member

@egenc @josejuanmartinez

we have these pipelines for spark 2.4 already, the spark 3.0 is missing. These pipelines must be saved/uploaded for spark_version: 3.0 (not just choose 3.0, but actually by using spark 3.0)

@egenc
Copy link
Contributor

egenc commented Jan 4, 2022

@maziyarpanahi , that is exactly what I did. Created these pipelines using pyspark==3.1.2

@maziyarpanahi
Copy link
Member

The metadata says spark_version: 2.4 and it saved them as 2.4 on S3. It won't download these if a user is on spark3.x, so probably the spark version was manually changed in Models Hub UI.

@egenc
Copy link
Contributor

egenc commented Jan 4, 2022

so I created these pipelines using pyspark==3.1.2 . However, when I upload the model, I picked spark compatibility as 2 otherwise these models will not be compatible with spark 2..x

@maziyarpanahi
Copy link
Member

The original issue was that these pipelines are missing for Spark 3.x: #2738

@egenc
Copy link
Contributor

egenc commented Jan 4, 2022

ok, pushing them again. thanks!

@egenc
Copy link
Contributor

egenc commented Jan 4, 2022

#6709

@maziyarpanahi
Copy link
Member

Thanks @egenc
it's the same for the other 2 pipelines I closed.

@egenc
Copy link
Contributor

egenc commented Jan 4, 2022

totally got it. Fixing them all

@maziyarpanahi maziyarpanahi deleted the 2022-01-04-check_spelling_dl_en_tBU2TQAe9YDnartagvpWesvN branch September 13, 2022 12:44
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