[GitHub] zeppelin pull request #2544: [ZEPPELIN-2724] Another Attempt to solve the de...

2017-11-29 Thread sadhen
Github user sadhen closed the pull request at: https://github.com/apache/zeppelin/pull/2544 ---

[GitHub] zeppelin pull request #2539: [ZEPPELIN-2863] change notebook-authorization.j...

2017-11-29 Thread sadhen
Github user sadhen closed the pull request at: https://github.com/apache/zeppelin/pull/2539 ---

[GitHub] zeppelin issue #2544: [ZEPPELIN-2724] Another Attempt to solve the dead lock...

2017-08-24 Thread sadhen
Github user sadhen commented on the issue: https://github.com/apache/zeppelin/pull/2544 No, this PR is for the master, we recently upgraded from 0.6.0 to master. Still to be confirmed and verified. However, we are running my PR in production for over 50 analysts (we mainly

[GitHub] zeppelin issue #2539: [ZEPPELIN-2863] change notebook-authorization.json and...

2017-08-23 Thread sadhen
Github user sadhen commented on the issue: https://github.com/apache/zeppelin/pull/2539 @zjffdu It won't satisfy my need. You may understand my concerns when you start dockerizing Zeppelin. And these days, I encounter another dockerization problem. If

[GitHub] zeppelin pull request #2544: [ZEPPELIN-2724] Another Attempt to solve the de...

2017-08-21 Thread sadhen
GitHub user sadhen opened a pull request: https://github.com/apache/zeppelin/pull/2544 [ZEPPELIN-2724] Another Attempt to solve the dead lock problem ### What is this PR for? https://github.com/apache/zeppelin/pull/2465 This is my attempt under the help with my

[GitHub] zeppelin issue #2539: [ZEPPELIN-2863] change notebook-authorization.json and...

2017-08-20 Thread sadhen
Github user sadhen commented on the issue: https://github.com/apache/zeppelin/pull/2539 @zjffdu Your solution didn't solve the problem in the quoted: `The critical part is that files under conf/ such as shiro.ini is part of the docker image and should be immu

[GitHub] zeppelin issue #2539: [ZEPPELIN-2863] change notebook-authorization.json and...

2017-08-16 Thread sadhen
Github user sadhen commented on the issue: https://github.com/apache/zeppelin/pull/2539 We deployed Zeppelin in K8S cluster, and are using volumes for persistence as you stated. A1. I will provide a solution later. A possible solution: mv the files in `conf/` if exists in

[GitHub] zeppelin pull request #2539: [ZEPPELIN-2863] change notebook-authorization.j...

2017-08-16 Thread sadhen
GitHub user sadhen reopened a pull request: https://github.com/apache/zeppelin/pull/2539 [ZEPPELIN-2863] change notebook-authorization.json and interpreter.json to NotebookDir ### TODO + [ ] Fix travis ### What is this PR for? When dockerizing Zeppelin, we need to

[GitHub] zeppelin pull request #2539: [ZEPPELIN-2863] change notebook-authorization.j...

2017-08-16 Thread sadhen
Github user sadhen closed the pull request at: https://github.com/apache/zeppelin/pull/2539 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] zeppelin issue #2539: [ZEPPELIN-2863] change notebook-authorization.json and...

2017-08-15 Thread sadhen
Github user sadhen commented on the issue: https://github.com/apache/zeppelin/pull/2539 @1ambda `interpreter.json` and `notebook-authorization.json` will be changed during the use of Zeppelin. If the docker instance is killed and restarted, the updated json files will be lost

[GitHub] zeppelin pull request #2539: [ZEPPELIN-2863] change notebook-authorization.j...

2017-08-15 Thread sadhen
GitHub user sadhen reopened a pull request: https://github.com/apache/zeppelin/pull/2539 [ZEPPELIN-2863] change notebook-authorization.json and interpreter.json to NotebookDir ### What is this PR for? When dockerizing Zeppelin, we need to persist the mutable part of Zeppelin

[GitHub] zeppelin pull request #2539: [ZEPPELIN-2863] change notebook-authorization.j...

2017-08-15 Thread sadhen
Github user sadhen closed the pull request at: https://github.com/apache/zeppelin/pull/2539 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] zeppelin pull request #2539: [ZEPPELIN-533] change notebook-authorization.js...

2017-08-15 Thread sadhen
GitHub user sadhen opened a pull request: https://github.com/apache/zeppelin/pull/2539 [ZEPPELIN-533] change notebook-authorization.json and interpreter.json to NotebookDir ### What is this PR for? When dockerizing Zeppelin, we need to persist the mutable part of Zeppelin on

[GitHub] zeppelin issue #319: ZEPPELIN-312: use threadpoll to do broadcast in zeppeli...

2017-03-23 Thread sadhen
Github user sadhen commented on the issue: https://github.com/apache/zeppelin/pull/319 @Leemoonsoo the delivery orders do not matter. Since the call of broadcast on message 1 and the call of broadcast on message 2 would be processed one by one since `noteSocketMap` is synchronized