Skip to content

Commit 1dc7857

Browse files
authored
Merge pull request #14 from gateway-experiments/manage-kernel-id
Let KernelManager hold kernel_id
2 parents 51251d7 + 717d522 commit 1dc7857

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

jupyter_kernel_mgmt/managerabc.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@
33
import six
44

55
class KernelManagerABC(six.with_metaclass(ABCMeta, object)):
6+
7+
kernel_id = None # Subclass should set kernel_id during initialization
8+
69
@abstractmethod
710
def is_alive(self):
811
"""Check whether the kernel is currently alive (e.g. the process exists)

jupyter_kernel_mgmt/subproc/manager.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
import subprocess
1313
import sys
1414
import time
15+
import uuid
1516

1617
log = logging.getLogger(__name__)
1718

@@ -39,6 +40,7 @@ def __init__(self, popen, files_to_cleanup=None, win_interrupt_evt=None):
3940
self.files_to_cleanup = files_to_cleanup or []
4041
self.win_interrupt_evt = win_interrupt_evt
4142
self.log = get_app_logger()
43+
self.kernel_id = str(uuid.uuid4())
4244

4345
def wait(self, timeout):
4446
""""""

0 commit comments

Comments
 (0)