-
Notifications
You must be signed in to change notification settings - Fork 21
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
bug docker image path #916
Comments
Just checked locally it seems to work for me. Can you provide more details, how are you trying to set I've tested it with this this value: |
on
due to my code has extra comma at the back. after removed, issue solved. on |
Are you sure you did not have an extra comma when testing on 0.1.0? It seems to work locally for me. |
I reran again. i checked, no extra comma. i used same deployment.yaml able to run on
did you tested on k8s? |
Thanks for detailed response. I see that statement api returns http status 200, while sparkmagic is expecting 201. It is not related to lighter_session_default_conf. It was probably broken, with one of new features introduced in 0.1.0. Will try to fix it and release as soon as possible. |
I've prepared an MR for the fix, and built an image for it: ghcr.io/exacaster/lighter:919-spark3.5.1 |
solved on ghcr.io/exacaster/lighter:919-spark3.5.1 |
i specified
"spark.kubernetes.container.image"
inLIGHTER_SESSION_DEFAULT_CONF
, but the juypter notebook can't identify it. in order to work, i need to insert this key in%%configure
jupyter notebook. Also tested with latest docker image0.1.0-spark3.5.0
and0.0.50-spark3.5.0
. DoesLIGHTER_SESSION_DEFAULT_CONF
key work on latest version of lighter?The text was updated successfully, but these errors were encountered: