Skip to content

[ZEPPELIN-6702] Fix shell terminal startup hang on macOS with JDK 17.0.13 - #5467

Open
seung-00 wants to merge 1 commit into
apache:masterfrom
seung-00:ZEPPELIN-6702
Open

[ZEPPELIN-6702] Fix shell terminal startup hang on macOS with JDK 17.0.13#5467
seung-00 wants to merge 1 commit into
apache:masterfrom
seung-00:ZEPPELIN-6702

Conversation

@seung-00

@seung-00 seung-00 commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

What is this PR for?

Upgrade pty4j from 0.12.10-jdk8 to 0.13.12 to avoid excessive shell terminal startup delays with high FD limits.

Reproduced on macOS with JDK 17.0.13, where the FD limit was INT_MAX. The old pty4j iterates through the entire range before starting the shell.

JDK 17.0.14 reduced the macOS FD limit, mitigating this issue. Upgrading pty4j avoids excessive startup delays when the FD limit remains high.

Also remove the JetBrains repository declaration, as the updated dependency is available from Maven Central.

What type of PR is it?

Bug Fix

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-6702

How should this be tested?

Verified locally:
On macOS with JDK 17.0.13:

  • Run terminal tests:
    ./mvnw test -pl shell -Dtest=TerminalInterpreterTest
  • In the Zeppelin UI, run a %sh.terminal paragraph. Confirm that a shell prompt appears.

Screenshots (if appropriate)

Questions:

  • Does the license files need to update? N
  • Is there breaking changes for older versions? N
  • Does this needs documentation? N

@voidmatcha voidmatcha left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM 👍

@pan3793 pan3793 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, I checked the upgraded pty4j version is built against JDK 11, and it's great to get rid of a 3rd-party maven repo.

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.

3 participants