You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by oxygen311 <gi...@git.apache.org> on 2018/08/14 09:50:58 UTC
[GitHub] zeppelin pull request #3137: [ZEPPELIN-3709] Add releaseResource parameter t...
GitHub user oxygen311 opened a pull request:
https://github.com/apache/zeppelin/pull/3137
[ZEPPELIN-3709] Add releaseResource parameter to Cron REST API
### What is this PR for?
Add `releaseResource` parameter to Cron REST API `Post` and `Get` methods.
### What type of PR is it?
Improvement
### What is the Jira issue?
[ZEPPELIN-3709](https://issues.apache.org/jira/browse/ZEPPELIN-3709)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? Yes, updated
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/TinkoffCreditSystems/zeppelin DW-17903
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/3137.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #3137
----
commit dd872f93892de1b2f54626d470385400112453bf
Author: oxygen311 <al...@...>
Date: 2018-08-13T16:36:39Z
Add `releaseResource` param to cron api methods
----
---
[GitHub] zeppelin issue #3137: [ZEPPELIN-3709] Add releaseResource parameter to Cron ...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/3137
How `releaseResource` works, actually?
---
[GitHub] zeppelin issue #3137: [ZEPPELIN-3709] Add releaseResource parameter to Cron ...
Posted by oxygen311 <gi...@git.apache.org>.
Github user oxygen311 commented on the issue:
https://github.com/apache/zeppelin/pull/3137
@jongyoul
You can find usage in [docs](https://zeppelin.apache.org/docs/0.8.0/usage/other_features/cron_scheduler.html#after-execution-stop-the-interpreter).
---
[GitHub] zeppelin issue #3137: [ZEPPELIN-3709] Add releaseResource parameter to Cron ...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/3137
Aha~ this changes the property only, correct?
---
[GitHub] zeppelin pull request #3137: [ZEPPELIN-3709] Add releaseResource parameter t...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/3137
---
[GitHub] zeppelin issue #3137: [ZEPPELIN-3709] Add releaseResource parameter to Cron ...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/3137
Yes, right. https://issues.apache.org/jira/browse/ZEPPELIN-3605
---
[GitHub] zeppelin issue #3137: [ZEPPELIN-3709] Add releaseResource parameter to Cron ...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/3137
Merged. Sorry for the late reply as I'm on my vacation.
---
[GitHub] zeppelin issue #3137: [ZEPPELIN-3709] Add releaseResource parameter to Cron ...
Posted by oxygen311 <gi...@git.apache.org>.
Github user oxygen311 commented on the issue:
https://github.com/apache/zeppelin/pull/3137
@jongyoul Yes
---
[GitHub] zeppelin issue #3137: [ZEPPELIN-3709] Add releaseResource parameter to Cron ...
Posted by zjffdu <gi...@git.apache.org>.
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/3137
@jongyoul I remember you have a ticket which try to refactor cron job, right ?
---
[GitHub] zeppelin issue #3137: [ZEPPELIN-3709] Add releaseResource parameter to Cron ...
Posted by Savalek <gi...@git.apache.org>.
Github user Savalek commented on the issue:
https://github.com/apache/zeppelin/pull/3137
What about merge?
---