Skip to content

[ZEPPELIN-6430] Clean up removed R interpreter docs#5282

Merged
jongyoul merged 1 commit into
apache:masterfrom
jongyoul:codex/cleanup-r-docs
Jul 8, 2026
Merged

[ZEPPELIN-6430] Clean up removed R interpreter docs#5282
jongyoul merged 1 commit into
apache:masterfrom
jongyoul:codex/cleanup-r-docs

Conversation

@jongyoul

@jongyoul jongyoul commented Jul 7, 2026

Copy link
Copy Markdown
Member

What changed

This cleans up documentation references left behind after the R interpreter, SparkR, and rlang code removal in ZEPPELIN-6430.

  • Removed R build/runtime requirements from docs where they are no longer Zeppelin requirements.
  • Removed %spark.r and R example blocks, and updated the generic interpreter example to use %spark.sql.
  • Updated stale env_python_3_with_R.yml links to the current Python environment file.
  • Removed R example links and support-list entries from interpreter, quickstart, Livy, VM, ZeppelinContext, and SDK docs.

Why

The code removal left docs that still implied R interpreter or SparkR support. These updates avoid sending users toward removed functionality while keeping supported Python, Spark, SQL, and Mahout docs intact.

Validation

  • git diff --check
  • rg -n "SparkR|R interpreter|r-base|r-cran|IRkernel|R Tutorial|%spark\\.r|%r\\b|env_python_3_with_R" docs --glob "*.md"
  • Additional strict search for actual R feature remnants returned no matches; the broad search only leaves false positives such as interpreter-base / interpreter-based.

@jongyoul jongyoul force-pushed the codex/cleanup-r-docs branch from 3910227 to 873ef36 Compare July 8, 2026 02:27
@jongyoul jongyoul marked this pull request as ready for review July 8, 2026 02:29
Copilot AI review requested due to automatic review settings July 8, 2026 02:29

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates Zeppelin’s documentation to remove or replace references to the removed R interpreter / SparkR functionality (per ZEPPELIN-6430), ensuring the docs only describe currently supported interpreters and environments.

Changes:

  • Removes R-related example links, build/runtime requirements, and documentation sections that referenced %spark.r / R interpreter usage.
  • Updates generic interpreter examples to use supported %spark.sql (and similar) instead of R.
  • Fixes stale Docker Python environment links by pointing to the current env_python_3.yml.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
docs/usage/zeppelin_sdk/session_api.md Removes the R client example link from the SDK session API page.
docs/usage/other_features/zeppelin_context.md Removes R-specific ZeppelinContext usage example and updates interpreter list wording.
docs/usage/interpreter/overview.md Replaces the %spark.r interpreter example with %spark.sql.
docs/setup/deployment/virtual_machine.md Removes R packages from the VM contents list.
docs/setup/basics/how_to_build.md Removes R packages from build requirements/install steps.
docs/quickstart/install.md Removes R from the “Spark, Flink, SQL, Python…” quickstart section header.
docs/interpreter/spark.md Removes R mentions from Spark interpreter docs and updates Docker env link to env_python_3.yml.
docs/interpreter/python.md Updates Docker env link to env_python_3.yml.
docs/interpreter/mahout.md Removes the Mahout→R visualization section and adjusts the related heading.
docs/interpreter/livy.md Removes R mentions from Livy interpreter capability and shared SparkContext description.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/usage/other_features/zeppelin_context.md
Comment thread docs/interpreter/spark.md
@jongyoul

jongyoul commented Jul 8, 2026

Copy link
Copy Markdown
Member Author

@pan3793 Hello, I found some more places mentioning R. Could you please review it?

@jongyoul jongyoul merged commit d9021ef into apache:master Jul 8, 2026
18 of 19 checks passed
@jongyoul

jongyoul commented Jul 8, 2026

Copy link
Copy Markdown
Member Author

Merged into master (d9021ef).

@jongyoul jongyoul deleted the codex/cleanup-r-docs branch July 8, 2026 02:40
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