[ZEPPELIN-6508] Remove outdated Mahout interpreter helper and documentation - #5430
Open
jongyoul wants to merge 1 commit into
Open
[ZEPPELIN-6508] Remove outdated Mahout interpreter helper and documentation#5430jongyoul wants to merge 1 commit into
jongyoul wants to merge 1 commit into
Conversation
jongyoul
marked this pull request as ready for review
August 19, 2026 08:15
There was a problem hiding this comment.
Pull request overview
Warning
Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.
Removes obsolete Apache Mahout integration artifacts from Zeppelin’s repository and website to align documentation and helpers with the currently supported runtime stack.
Changes:
- Removed the legacy Mahout interpreter helper script.
- Removed the Mahout interpreter documentation page.
- Removed Mahout links from the docs index and site navigation.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| scripts/mahout/add_mahout_interpreters.py | Deletes the legacy Python 2 helper that cloned interpreter settings and injected Mahout dependencies. |
| docs/interpreter/mahout.md | Removes outdated Mahout interpreter documentation tied to old Spark/Flink/Scala versions. |
| docs/index.md | Removes the Mahout interpreter entry from the documentation index. |
| docs/_includes/themes/zeppelin/_navigation.html | Removes the Mahout interpreter link from the site navigation menu. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What is this PR for?
Remove the obsolete Mahout integration artifacts from Zeppelin.
The existing
mahout.mdpage does not describe a standalone Zeppelin interpreter module. It relies on a Python 2 helper that clones the Spark and Flink interpreter settings, then adds Mahout 0.12.2 artifacts built for Scala 2.10. The accompanying documentation and tutorial are limited to Spark 1.5/1.6 and no longer match Zeppelin's supported runtime stack.This PR removes the helper, its documentation and navigation entries, and the bundled Mahout tutorial notebook. Apache Mahout's current Qumat Python packages use a separate Python integration path and are intentionally outside the scope of this cleanup PR.
What type of PR is it?
Improvement
Todos
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-6508
How should this be tested?
bundle exec jekyll build --safein the documented Ruby 4.0.6 Docker environmentruby check_external_resources.rb _sitegit diff --checkScreenshots (if appropriate)
Not applicable.
Questions: