Skip to content

[ZEPPELIN-6480] Stop logging terminal dashboard HTML - #5365

Open
vividbaek wants to merge 1 commit into
apache:masterfrom
vividbaek:ZEPPELIN-6480-remove-terminal-dashboard-html-log
Open

[ZEPPELIN-6480] Stop logging terminal dashboard HTML#5365
vividbaek wants to merge 1 commit into
apache:masterfrom
vividbaek:ZEPPELIN-6480-remove-terminal-dashboard-html-log

Conversation

@vividbaek

Copy link
Copy Markdown
Contributor

What is this PR for?

Remove the INFO log that writes the full rendered terminal dashboard HTML.

The dashboard HTML is already written to the paragraph output, so logging it duplicates the content, adds unnecessary log volume, and exposes terminal URL parameters in logs.

The paragraph output rendering behavior remains unchanged.

What type of PR is it?

Improvement

Todos

  • Remove the full dashboard HTML log
  • Preserve the paragraph output rendering
  • Run the shell module build and tests

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-6480

How should this be tested?

./mvnw test -pl shell

The build completed successfully.

### Screenshots (if appropriate)

Not applicable.

### Questions:

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

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