Skip to content

Conversation

@zjffdu
Copy link
Contributor

@zjffdu zjffdu commented Apr 29, 2020

What is this PR for?

This PR is to resolve the paragraph shaking issue further. This PR use angular instead of html, so that each time we only update the angular object instead of the whole result. This can totally resolve the paragraph shaking issue.

What type of PR is it?

[ Improvement]

Todos

  • - Task

What is the Jira issue?

How should this be tested?

  • CI

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

@zjffdu zjffdu force-pushed the ZEPPELIN-4780 branch 2 times, most recently from d1b9fcb to a901df5 Compare April 30, 2020 02:32
Copy link
Contributor

@alexott alexott left a comment

Choose a reason for hiding this comment

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

LGTM

@asfgit asfgit closed this in cb47d73 May 3, 2020
asfgit pushed a commit that referenced this pull request May 3, 2020
### What is this PR for?

This PR is to resolve the paragraph shaking issue further. This PR use angular instead of html, so that each time we only update the angular object instead of the whole result. This can totally resolve the paragraph shaking issue.

### What type of PR is it?
[ Improvement]

### Todos
* [ ] - Task

### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-4780

### How should this be tested?
* CI

### Screenshots (if appropriate)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No

Author: Jeff Zhang <[email protected]>

Closes #3754 from zjffdu/ZEPPELIN-4780 and squashes the following commits:

a901df5 [Jeff Zhang] [ZEPPELIN-4780]. Use angular instead of html for flink streaming output

(cherry picked from commit cb47d73)
Signed-off-by: Jeff Zhang <[email protected]>
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