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 user guide for accessing spark Jupyter lab #33

Merged
merged 1 commit into from
Jun 17, 2024
Merged

Conversation

Tianhao-Gu
Copy link
Collaborator

No description provided.

Copy link

codecov bot commented Jun 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 48.57%. Comparing base (45cef54) to head (d839ecb).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #33   +/-   ##
=======================================
  Coverage   48.57%   48.57%           
=======================================
  Files           4        4           
  Lines         105      105           
=======================================
  Hits           51       51           
  Misses         54       54           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Comment on lines +96 to +97
Please be aware that your session will automatically close after `1 hour`. Should you require an extension, simply invoke
`get_spark_session()` to initiate a new session.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this means the prior session is still sitting around wasting resources, right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no. after 1 hr, the prior session will be closed by the timer.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

But for that hour it is, that's what I'm talking about

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yea. for the hour the session is open, it's taking resources.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suppose we could keep track of open sessions and close them when a new session is requested but that's probably overkill for now

Comment on lines +131 to +133
This error occurs when there is an issue with the SparkMonitor kernel extension, specifically related to the
communication between the kernel and the front end. The error indicates that the comm attribute is missing from the
ScalaMonitor object.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What causes this?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From the extension developer:

This is a difficult error to debug; the kernel extension is complaining that the Scala Listener has no comms; this is usually because the listener wasn't able to find a Spark Instance.
itsjafer/jupyterlab-sparkmonitor#16

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, that ticket says the bug is fixed

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yea. It didn't happen very often. And restart kernel always works.

@Tianhao-Gu Tianhao-Gu merged commit 72ad000 into main Jun 17, 2024
9 checks passed
@Tianhao-Gu Tianhao-Gu deleted the dev_user_guide branch June 17, 2024 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants