You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by mebelousov <gi...@git.apache.org> on 2018/01/30 19:09:47 UTC
[GitHub] zeppelin pull request #2752: [ZEPPELIN-3195] Remove the limit on the number ...
GitHub user mebelousov opened a pull request:
https://github.com/apache/zeppelin/pull/2752
[ZEPPELIN-3195] Remove the limit on the number of run paragraphs at cron
### What is this PR for?
Increase interpreter pool size for unlimit the number of paragraph that run at cron schedule.
### What type of PR is it?
[Hot Fix]
### What is the Jira issue?
*https://issues.apache.org/jira/browse/ZEPPELIN-3195
### How should this be tested?
* Create note with 15 paragraphs
* Check that at cron will be executed only 10 paragraphs
* Checkout this branch and rebuild app
* Check that all paragraphs are executed at cron
### Questions:
* Does the licenses files need update? NA
* Is there breaking changes for older versions? NA
* Does this needs documentation? NA
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tinkoff-dwh/zeppelin ZEPPELIN-3195
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2752.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 #2752
----
commit 827054ce769b7c61ff8344c4df19f09ef5598e80
Author: mebelousov <me...@...>
Date: 2018-01-30T18:58:00Z
ZEPPELIN-3195 Increase interpreter pool size
----
---
[GitHub] zeppelin issue #2752: [ZEPPELIN-3195] Remove the limit on the number of run ...
Posted by mebelousov <gi...@git.apache.org>.
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2752
@zjffdu @felixcheung
What will we do with this bug?
---
[GitHub] zeppelin issue #2752: [ZEPPELIN-3195] Remove the limit on the number of run ...
Posted by zjffdu <gi...@git.apache.org>.
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2752
Actually, you can change the configuration in `zeppelin-site.xml`, and change it to 65535 seems too large for me.
---
[GitHub] zeppelin issue #2752: [ZEPPELIN-3195] Remove the limit on the number of run ...
Posted by mebelousov <gi...@git.apache.org>.
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2752
@felixcheung
Where I could document about ZEPPELIN_INTERPRETER_MAX_POOL_SIZE?
There is no such cases in documentation.
Also I cannot imagine than Zeppelin administrator will go to documentation when users will complain that manual note run is OK but only 10 first paragraphs are done at cron.
---
[GitHub] zeppelin issue #2752: [ZEPPELIN-3195] Remove the limit on the number of run ...
Posted by mebelousov <gi...@git.apache.org>.
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2752
It's not easy to understand that "ZEPPELIN_INTERPRETER_MAX_POOL_SIZE" affects the number of paragraphs executed at cron. I would be glad if nobody will meet this bug and no need to change zeppelin-site.xml to fix note scheduling.
@zjffdu which number will be more appropriate? =)
---
[GitHub] zeppelin issue #2752: [ZEPPELIN-3195] Remove the limit on the number of run ...
Posted by mebelousov <gi...@git.apache.org>.
Github user mebelousov commented on the issue:
https://github.com/apache/zeppelin/pull/2752
I renewed docs: remove old "ZEPPELIN_INTERPRETERS" and added ZEPPELIN_INTERPRETER_MAX_POOL_SIZE.
The default value for ZEPPELIN_INTERPRETER_MAX_POOL_SIZE remains 10.
---
[GitHub] zeppelin pull request #2752: [ZEPPELIN-3195] Remove the limit on the number ...
Posted by mebelousov <gi...@git.apache.org>.
Github user mebelousov closed the pull request at:
https://github.com/apache/zeppelin/pull/2752
---