Skip to content

[ZEPPELIN-6439] Fix typos and non-ASCII lookalike characters in interpreter documentation - #5506

Merged
voidmatcha merged 1 commit into
apache:masterfrom
JangAyeon:ZEPPELIN-6439
Sep 30, 2026
Merged

voidmatcha merged 1 commit into
apache:masterfrom
JangAyeon:ZEPPELIN-6439

Conversation

@JangAyeon

Copy link
Copy Markdown
Contributor

What is this PR for?

Interpreter settings and documentation are shown directly to users in Zeppelin's UI and docs, but some of this text contains typos and non-ASCII lookalike characters.

The most notable case is the JDBC default.completer.schemaFilters description, which starts with Сomma using a Cyrillic С (U+0421) instead of the Latin C. It looks identical on screen, but it can break text search, copy-paste, and encoding-sensitive tooling. This PR replaces it with the Latin C in both the interpreter setting (interpreter-setting.json) and the corresponding docs (jdbc.md), so the two stay consistent.

This PR also fixes the following typos:

  • seperated → separated in the notebook authorization docs
  • inteprreters → interpreters in the zeppelin.interpreter.include / zeppelin.interpreter.exclude descriptions of conf/zeppelin-site.xml.template
  • phatomJs → PhantomJS in the Vagrant dev environment Ansible task

Wording changes are kept minimal. Only description text is changed; no property keys, default values, or runtime behavior are affected. Following the issue guidance, generated files (e.g. zeppelin-server/src/main/resources/zeppelin-site.xml) and vendored/minified assets are not modified.

What type of PR is it?

Documentation

Todos

  • Replace the Cyrillic С (U+0421) in Сomma separated with the Latin C in the JDBC interpreter setting and docs
  • Fix seperated → separated in the notebook authorization docs
  • Fix inteprreters → interpreters (2 occurrences) in conf/zeppelin-site.xml.template
  • Fix phatomJs → PhantomJS in the Vagrant dev environment Ansible task

What is the Jira issue?

ZEPPELIN-6439

How should this be tested?

Search the whole repository and confirm there are no matches:

rg -n "Сomma|seperated|Thats|phatomJs|inteprreters" .

Questions:

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

voidmatcha

This comment was marked as outdated.

@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 👍

@voidmatcha
voidmatcha merged commit a424174 into apache:master Sep 30, 2026
27 of 29 checks passed
@voidmatcha

Copy link
Copy Markdown
Member

Merged into master (a424174).

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