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

[ZEPPELIN-3476] change description "auto-restart interpreter on cron execution" #2991

Closed
wants to merge 2 commits into from

Conversation

Savalek
Copy link
Contributor

@Savalek Savalek commented May 29, 2018

What is this PR for?

Description "auto-restart interpreter on cron execution" is does not quite fit in the sense.

What type of PR is it?

Documentation

What is the Jira issue?

ZEPPELIN-3476

Screenshot

cron_scheduler_dialog_box

@Leemoonsoo
Copy link
Member

LGTM

@@ -45,7 +45,7 @@ You can set the cron schedule by filling in this form. Please see [Cron Trigger

You can set the cron executing user by filling in this form and press the enter key.

### auto-restart interpreter on cron execution
### After execution close the interpreter
Copy link
Member

Choose a reason for hiding this comment

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

maybe "stop" is better than "close"? I think that's the terminology used elsewhere
https://zeppelin.apache.org/docs/0.7.3/manual/interpreters.html#what-is-interpreter-group

@Savalek
Copy link
Contributor Author

Savalek commented May 30, 2018

@felixcheung, agree with you. Replaced the close on stop.

@r-kamath
Copy link
Member

LGTM

@r-kamath
Copy link
Member

will merge this if there are no more discussions

@felixcheung
Copy link
Member

@Savalek can you update the title of this PR?
something like "change Description "auto-restart interpreter on cron execution""

@Savalek Savalek changed the title [ZEPPELIN-3476] change description [ZEPPELIN-3476] change description "auto-restart interpreter on cron execution" Jun 13, 2018
@Savalek
Copy link
Contributor Author

Savalek commented Jun 13, 2018

@felixcheung, I did it.

asfgit pushed a commit that referenced this pull request Jun 13, 2018
…execution"

Description "auto-restart interpreter on cron execution"  is does not quite fit in the sense.

Documentation

[ZEPPELIN-3476](https://issues.apache.org/jira/browse/ZEPPELIN-3476)

![cron_scheduler_dialog_box](https://user-images.githubusercontent.com/30798933/40646934-4d110526-6333-11e8-90d2-c4feaa1f2c3d.png)

Author: Savalek <def113@mail.ru>

Closes #2991 from Savalek/ZEPPELIN-3476 and squashes the following commits:

4916b0d [Savalek] [ZEPPELIN-3476] replace "close" on "stop"
d2e9c78 [Savalek] [ZEPPELIN-3476] change description

Change-Id: Ic8a0d283268f843f0ec440b2203c027fea0ffb2b
(cherry picked from commit ee06cf0)
Signed-off-by: Renjith Kamath <rkamath@apache.org>
@asfgit asfgit closed this in ee06cf0 Jun 13, 2018
@Savalek Savalek deleted the ZEPPELIN-3476 branch June 13, 2018 09:00
prabhjyotsingh pushed a commit to prabhjyotsingh/zeppelin that referenced this pull request Jul 4, 2018
…execution"

Description "auto-restart interpreter on cron execution"  is does not quite fit in the sense.

Documentation

[ZEPPELIN-3476](https://issues.apache.org/jira/browse/ZEPPELIN-3476)

![cron_scheduler_dialog_box](https://user-images.githubusercontent.com/30798933/40646934-4d110526-6333-11e8-90d2-c4feaa1f2c3d.png)

Author: Savalek <def113@mail.ru>

Closes apache#2991 from Savalek/ZEPPELIN-3476 and squashes the following commits:

4916b0d [Savalek] [ZEPPELIN-3476] replace "close" on "stop"
d2e9c78 [Savalek] [ZEPPELIN-3476] change description

Change-Id: Ic8a0d283268f843f0ec440b2203c027fea0ffb2b
(cherry picked from commit ee06cf0)
Signed-off-by: Renjith Kamath <rkamath@apache.org>
mckartha pushed a commit to syntechdev/zeppelin that referenced this pull request Aug 9, 2018
…execution"

Description "auto-restart interpreter on cron execution"  is does not quite fit in the sense.

Documentation

[ZEPPELIN-3476](https://issues.apache.org/jira/browse/ZEPPELIN-3476)

![cron_scheduler_dialog_box](https://user-images.githubusercontent.com/30798933/40646934-4d110526-6333-11e8-90d2-c4feaa1f2c3d.png)

Author: Savalek <def113@mail.ru>

Closes apache#2991 from Savalek/ZEPPELIN-3476 and squashes the following commits:

4916b0d [Savalek] [ZEPPELIN-3476] replace "close" on "stop"
d2e9c78 [Savalek] [ZEPPELIN-3476] change description

Change-Id: Ic8a0d283268f843f0ec440b2203c027fea0ffb2b
mckartha pushed a commit to syntechdev/zeppelin that referenced this pull request Aug 9, 2018
…execution"

Description "auto-restart interpreter on cron execution"  is does not quite fit in the sense.

Documentation

[ZEPPELIN-3476](https://issues.apache.org/jira/browse/ZEPPELIN-3476)

![cron_scheduler_dialog_box](https://user-images.githubusercontent.com/30798933/40646934-4d110526-6333-11e8-90d2-c4feaa1f2c3d.png)

Author: Savalek <def113@mail.ru>

Closes apache#2991 from Savalek/ZEPPELIN-3476 and squashes the following commits:

4916b0d [Savalek] [ZEPPELIN-3476] replace "close" on "stop"
d2e9c78 [Savalek] [ZEPPELIN-3476] change description

Change-Id: Ic8a0d283268f843f0ec440b2203c027fea0ffb2b
(cherry picked from commit ee06cf0)
Signed-off-by: Renjith Kamath <rkamath@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
4 participants