Skip to content

Conversation

@JohanMabille
Copy link
Member

This adds the missing specification for the accepted JEPS of these last years, in order to make a new release of the specification. I'll make a separate PR for the handshaking and the regular connection patterns.

Copy link
Member

@minrk minrk left a comment

Choose a reason for hiding this comment

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

Great, thank you!

@rgbkrk
Copy link
Member

rgbkrk commented Dec 17, 2025

For previous client bindings that used PUB/SUB sockets for IOPub, will they be backwards compatible or do they need to be able to detect that the version is 5.5 somehow?

Based on reading Impact on Existing Implementations, it seems like I should add this into the jupyter rust crates (jupyter-protocol, runtimed) and get it into Deno and Zed.

For now I'll add the message in to the Rust crate in runtimed/runtimed#203 and come up with a plan for how I can support XPUB kernels maybe optionally at first (?).

@minrk
Copy link
Member

minrk commented Dec 17, 2025

If a kernel speaks protocol version 5.4 and uses a PUB socket, the iopub_welcome message will be missing when subscribers connect. Clients expecting protocol version 5.5 should understand this and not wait for this message. It is otherwise indistinguishable from the client side. If a client doesn't wait for iopub_welcome, there is no difference.

between the frontend and the kernel to ensure that they are still connected.

.. versionchanged:: 5.5
The **IOPub** PUB socket is replaced with an XPUB socket.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The **IOPub** PUB socket is replaced with an XPUB socket.
The **IOPub** PUB socket is replaced with an XPUB socket,
to enable the `iopub_welcome` message.
There is no other difference between kernel PUB and XPUB sockets from the client perspective.

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