Skip to content

Commit

Permalink
[KYUUBI #6283] Improve the message of distribution built without enab…
Browse files Browse the repository at this point in the history
…ling web ui

# 🔍 Description
Improve message of distribution built without enabling web ui.
## Issue References 🔗

This pull request fixes #6283

## Describe Your Solution 🔧

Here is my implement:

<img width="1512" alt="image" src="https://github.com/apache/kyuubi/assets/34719039/fc5e859e-1c92-44a1-8e27-316986c5d9c3">

## Types of changes 🔖

- [x] Bugfix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to change)

## Test Plan 🧪

#### Behavior Without This Pull Request ⚰️

#### Behavior With This Pull Request 🎉

#### Related Unit Tests

---

# Checklist 📝

- [ ] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html)

**Be nice. Be informative.**

Closes #6288 from dupen01/issue-6283.

Closes #6288

a5886e2 [dupeng] revise the message of page
bad14a7 [dupeng] improve the message of distribution built without enabling web ui

Authored-by: dupeng <[email protected]>
Signed-off-by: Cheng Pan <[email protected]>
  • Loading branch information
dupen01 authored and pan3793 committed Apr 11, 2024
1 parent 117d77a commit 64e673d
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion kyuubi-server/src/main/resources/dist/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@
<title>Apache Kyuubi Dashboard</title>
</head>
<body>
<div>This is a dummy page for development.</div>
<h2 style="text-align: center;">🙁 The Web UI is currently unavailable.</h2>
<h4 style="text-align: center;">👉 To enable Web UI, `--web-ui` is required when using `build/dist` tool to create binary distributions, e.g.</h4>
<pre style="text-align: center;"><code class="language-shell">
./build/dist --tgz --web-ui --spark-provided --flink-provided --hive-provided
</code></pre>
</body>
</html>

0 comments on commit 64e673d

Please sign in to comment.