Skip to content

Comments

[ZEPPELIN-4899]. Injected variables in ZeppelinServer are lazy loaded#3816

Closed
zjffdu wants to merge 1 commit intoapache:masterfrom
zjffdu:ZEPPELIN-4899
Closed

[ZEPPELIN-4899]. Injected variables in ZeppelinServer are lazy loaded#3816
zjffdu wants to merge 1 commit intoapache:masterfrom
zjffdu:ZEPPELIN-4899

Conversation

@zjffdu
Copy link
Contributor

@zjffdu zjffdu commented Jun 22, 2020

What is this PR for?

Trivial PR to create these injected variables when starting zeppelin server. Now these variables are
created when user open zeppelin in browser if we don't get it explicitly here.
Lazy loading will cause paragraph recovery and cron job initialization is delayed.

What type of PR is it?

[Improvement ]

Todos

  • - Task

What is the Jira issue?

How should this be tested?

  • CI pass

Screenshots (if appropriate)

Questions:

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

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 22ff37b Jun 23, 2020
asfgit pushed a commit that referenced this pull request Jun 23, 2020
### What is this PR for?

Trivial PR to create these injected variables when starting zeppelin server. Now these variables are
created when user open zeppelin in browser if we don't get it explicitly here.
Lazy loading will cause paragraph recovery and cron job initialization is delayed.

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

### Todos
* [ ] - Task

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

### How should this be tested?
* CI pass

### 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 <zjffdu@apache.org>

Closes #3816 from zjffdu/ZEPPELIN-4899 and squashes the following commits:

32e4727 [Jeff Zhang] [ZEPPELIN-4899]. Injected variables in ZeppelinServer are lazy loaded

(cherry picked from commit 22ff37b)
Signed-off-by: Jeff Zhang <zjffdu@apache.org>
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