Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Zeppelin #87

Merged
merged 2 commits into from
Jan 17, 2025
Merged

Add Zeppelin #87

merged 2 commits into from
Jan 17, 2025

Conversation

ehoner
Copy link
Collaborator

@ehoner ehoner commented Jan 17, 2025

Adds Zeppelin configured for use with Hopitmator. Includes new make targets: build-zeppelin and run-zeppelin.

Global changes

  • Modifies Kubernetes config resolution.
    • Uses environment overrides to replace config values.
  • Adds shadowJar to make install

Modification to upstream

  • Sets default interpreter as Hoptimator.
  • Removes all other non-jdbc connectors.
  • Connects to local K8s services.

Local Zeppelin

steps:

# requires: make install build deploy-kafka deploy-samples
# build zeppelin artifacts
$ make build-zeppelin

# start local zepplin-server
$ make run-zeppelin

open browser: localhost:8080
create:
default-interpreter

query:
notebook

@ehoner ehoner requested review from jogrogan and ryannedolan and removed request for jogrogan January 17, 2025 02:50
Copy link
Collaborator

@ryannedolan ryannedolan left a comment

Choose a reason for hiding this comment

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

Looks great! I think we just need to add the Apache copyright header to a file or two.

deploy/docker/zeppelin/zeppelin-site.xml Show resolved Hide resolved
deploy/docker/zeppelin/Dockerfile-zeppelin Show resolved Hide resolved
@ehoner ehoner requested a review from ryannedolan January 17, 2025 16:40
@ryannedolan ryannedolan merged commit f10e6e8 into main Jan 17, 2025
1 check passed
@ryannedolan ryannedolan deleted the zeppelin branch January 17, 2025 16:57
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.

2 participants