[GitHub] asfgit closed pull request #3287: IPythonInterpreter delete temp file and close stream

2019-02-02 Thread GitBox
asfgit closed pull request #3287: IPythonInterpreter delete temp file and close stream URL: https://github.com/apache/zeppelin/pull/3287 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] felixcheung commented on a change in pull request #3300: ZEPPELIN-3983. Travis fails due to downloading spark takes a lot of time

2019-02-03 Thread GitBox
felixcheung commented on a change in pull request #3300: ZEPPELIN-3983. Travis fails due to downloading spark takes a lot of time URL: https://github.com/apache/zeppelin/pull/3300#discussion_r253330574 ## File path:

[GitHub] felixcheung commented on issue #3300: ZEPPELIN-3983. Travis fails due to downloading spark takes a lot of time

2019-02-03 Thread GitBox
felixcheung commented on issue #3300: ZEPPELIN-3983. Travis fails due to downloading spark takes a lot of time URL: https://github.com/apache/zeppelin/pull/3300#issuecomment-460106676 LGTM This is an automated message from

[GitHub] Leemoonsoo opened a new pull request #3302: [ZEPPELIN-3988] Paragraph Text output includes \r\n is not displayed correctly.

2019-02-03 Thread GitBox
Leemoonsoo opened a new pull request #3302: [ZEPPELIN-3988] Paragraph Text output includes \r\n is not displayed correctly. URL: https://github.com/apache/zeppelin/pull/3302 ### What is this PR for? When paragraph text output includes `\r\n`, it is not displayed correctly in paragraph

[GitHub] Leemoonsoo commented on a change in pull request #3302: [ZEPPELIN-3988] Paragraph Text output includes \r\n is not displayed correctly.

2019-02-03 Thread GitBox
Leemoonsoo commented on a change in pull request #3302: [ZEPPELIN-3988] Paragraph Text output includes \r\n is not displayed correctly. URL: https://github.com/apache/zeppelin/pull/3302#discussion_r253344252 ## File path:

[GitHub] Leemoonsoo commented on a change in pull request #3302: [ZEPPELIN-3988] Paragraph Text output includes \r\n is not displayed correctly.

2019-02-03 Thread GitBox
Leemoonsoo commented on a change in pull request #3302: [ZEPPELIN-3988] Paragraph Text output includes \r\n is not displayed correctly. URL: https://github.com/apache/zeppelin/pull/3302#discussion_r253344467 ## File path:

[GitHub] Leemoonsoo commented on a change in pull request #3302: [ZEPPELIN-3988] Paragraph Text output includes \r\n is not displayed correctly.

2019-02-03 Thread GitBox
Leemoonsoo commented on a change in pull request #3302: [ZEPPELIN-3988] Paragraph Text output includes \r\n is not displayed correctly. URL: https://github.com/apache/zeppelin/pull/3302#discussion_r253344639 ## File path:

[GitHub] Leemoonsoo commented on a change in pull request #3302: [ZEPPELIN-3988] Paragraph Text output includes \r\n is not displayed correctly.

2019-02-03 Thread GitBox
Leemoonsoo commented on a change in pull request #3302: [ZEPPELIN-3988] Paragraph Text output includes \r\n is not displayed correctly. URL: https://github.com/apache/zeppelin/pull/3302#discussion_r253344719 ## File path: zeppelin-web/src/app/notebook/paragraph/result/result.js

[GitHub] Leemoonsoo opened a new pull request #3298: [ZEPPELIN-3982] add how to setup development mode for Kubernetes support

2019-02-03 Thread GitBox
Leemoonsoo opened a new pull request #3298: [ZEPPELIN-3982] add how to setup development mode for Kubernetes support URL: https://github.com/apache/zeppelin/pull/3298 ### What is this PR for? This pr adds docs that explains development mode for Kubernetes support. ### What

[GitHub] Leemoonsoo closed pull request #3298: [ZEPPELIN-3982] add how to setup development mode for Kubernetes support

2019-02-03 Thread GitBox
Leemoonsoo closed pull request #3298: [ZEPPELIN-3982] add how to setup development mode for Kubernetes support URL: https://github.com/apache/zeppelin/pull/3298 This is an automated message from the Apache Git Service. To

[GitHub] Leemoonsoo commented on issue #3298: [ZEPPELIN-3982] add how to setup development mode for Kubernetes support

2019-02-03 Thread GitBox
Leemoonsoo commented on issue #3298: [ZEPPELIN-3982] add how to setup development mode for Kubernetes support URL: https://github.com/apache/zeppelin/pull/3298#issuecomment-460127410 Merge to master if no more comment. This

[GitHub] Leemoonsoo commented on issue #3299: [ZEPPELIN-3920] [FOLLOWUP] get correct hyperlink address when num job url is one

2019-02-03 Thread GitBox
Leemoonsoo commented on issue #3299: [ZEPPELIN-3920] [FOLLOWUP] get correct hyperlink address when num job url is one URL: https://github.com/apache/zeppelin/pull/3299#issuecomment-460127666 @liuxunorg No problem :) Thanks for the review and confirming fix! Merging to master

[GitHub] gignac-cha opened a new pull request #3295: Fix misspelled word

2019-01-31 Thread GitBox
gignac-cha opened a new pull request #3295: Fix misspelled word URL: https://github.com/apache/zeppelin/pull/3295 ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out the contributing guide -

[GitHub] egorklimov commented on issue #3205: [ZEPPELIN-3814] visualization - Add apply button to table settings

2019-01-31 Thread GitBox
egorklimov commented on issue #3205: [ZEPPELIN-3814] visualization - Add apply button to table settings URL: https://github.com/apache/zeppelin/pull/3205#issuecomment-459261974 Time to merge?) This is an automated message

[GitHub] zjffdu commented on issue #3278: [ZEPPELIN-3931] Redisplay angularObjectBind when the note is reopened

2019-01-31 Thread GitBox
zjffdu commented on issue #3278: [ZEPPELIN-3931] Redisplay angularObjectBind when the note is reopened URL: https://github.com/apache/zeppelin/pull/3278#issuecomment-459260923 LGTM @liuxunorg will merge if no more comments. And feel free to create tickets for the TODO task you

[GitHub] asfgit closed pull request #3205: [ZEPPELIN-3814] visualization - Add apply button to table settings

2019-01-31 Thread GitBox
asfgit closed pull request #3205: [ZEPPELIN-3814] visualization - Add apply button to table settings URL: https://github.com/apache/zeppelin/pull/3205 This is an automated message from the Apache Git Service. To respond to

[GitHub] zjffdu opened a new pull request #3296: ZEPPELIN-3974. Reorganize zeppelin integration test

2019-01-31 Thread GitBox
zjffdu opened a new pull request #3296: ZEPPELIN-3974. Reorganize zeppelin integration test URL: https://github.com/apache/zeppelin/pull/3296 ### What is this PR for? I create a new module zeppelin-interpreter-integration which contains all the interpreter related integration test. The

[GitHub] monsieurp commented on issue #3241: ZEPPELIN-3880: Code refactoring: remove class files using a Java stream

2019-01-31 Thread GitBox
monsieurp commented on issue #3241: ZEPPELIN-3880: Code refactoring: remove class files using a Java stream URL: https://github.com/apache/zeppelin/pull/3241#issuecomment-459424335 @zjffdu Hi! This is my user account: https://issues.apache.org/jira/secure/ViewProfile.jspa?name=monsieurp.

[GitHub] Leemoonsoo closed pull request #3303: [ZEPPELIN-3989] Configure IPython Interpreter in Docker image

2019-02-04 Thread GitBox
Leemoonsoo closed pull request #3303: [ZEPPELIN-3989] Configure IPython Interpreter in Docker image URL: https://github.com/apache/zeppelin/pull/3303 This is an automated message from the Apache Git Service. To respond to

[GitHub] Leemoonsoo opened a new pull request #3303: [ZEPPELIN-3989] Configure IPython Interpreter in Docker image

2019-02-04 Thread GitBox
Leemoonsoo opened a new pull request #3303: [ZEPPELIN-3989] Configure IPython Interpreter in Docker image URL: https://github.com/apache/zeppelin/pull/3303 ### What is this PR for? Configure IPython interpreter in Docker image by installing necessary dependencies. ### What type

[GitHub] asfgit closed pull request #3299: [ZEPPELIN-3920] [FOLLOWUP] get correct hyperlink address when num job url is one

2019-02-04 Thread GitBox
asfgit closed pull request #3299: [ZEPPELIN-3920] [FOLLOWUP] get correct hyperlink address when num job url is one URL: https://github.com/apache/zeppelin/pull/3299 This is an automated message from the Apache Git Service.

[GitHub] maziyarpanahi opened a new pull request #3304: Add spark csv and json tests

2019-02-05 Thread GitBox
maziyarpanahi opened a new pull request #3304: Add spark csv and json tests URL: https://github.com/apache/zeppelin/pull/3304 ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out the contributing guide -

[GitHub] egorklimov commented on issue #3292: ZEPPELIN-3970. JdbcInterpreter is broken

2019-01-29 Thread GitBox
egorklimov commented on issue #3292: ZEPPELIN-3970. JdbcInterpreter is broken URL: https://github.com/apache/zeppelin/pull/3292#issuecomment-458467566 Please, add also the simple completion fix: when paragraph wasn't executed, completion finishes with error, I think it could be fixed this

[GitHub] prabhjyotsingh commented on issue #3288: [ZEPPELIN-2274] - update cron UI

2019-01-29 Thread GitBox
prabhjyotsingh commented on issue #3288: [ZEPPELIN-2274] - update cron UI URL: https://github.com/apache/zeppelin/pull/3288#issuecomment-458486069 The theme does not match our notebook UI, otherwise looks good. This is an

[GitHub] egorklimov commented on a change in pull request #3292: ZEPPELIN-3970. JdbcInterpreter is broken

2019-01-29 Thread GitBox
egorklimov commented on a change in pull request #3292: ZEPPELIN-3970. JdbcInterpreter is broken URL: https://github.com/apache/zeppelin/pull/3292#discussion_r251752050 ## File path: jdbc/pom.xml ## @@ -255,6 +255,12 @@ test + + org.apache.commons

[GitHub] liuxunorg commented on issue #3273: [ZEPPELIN-3920] Paragraph support custom menu display with multiple URL link

2019-01-28 Thread GitBox
liuxunorg commented on issue #3273: [ZEPPELIN-3920] Paragraph support custom menu display with multiple URL link URL: https://github.com/apache/zeppelin/pull/3273#issuecomment-458102309 **Complete single-menu & multi-menu test.** Custom single-menu test code: ``` Map infos

[GitHub] zjffdu commented on issue #3273: [ZEPPELIN-3920] Paragraph support custom menu display with multiple URL link

2019-01-28 Thread GitBox
zjffdu commented on issue #3273: [ZEPPELIN-3920] Paragraph support custom menu display with multiple URL link URL: https://github.com/apache/zeppelin/pull/3273#issuecomment-458050820 LGTM, will merge if no more comment This

[GitHub] zjffdu commented on issue #3290: [ZEPPELIN-3939] Spark 2.4 uses a more recent commons-lang3 than Zeppelin-0.8

2019-01-28 Thread GitBox
zjffdu commented on issue #3290: [ZEPPELIN-3939] Spark 2.4 uses a more recent commons-lang3 than Zeppelin-0.8 URL: https://github.com/apache/zeppelin/pull/3290#issuecomment-458067719 @maziyarpanahi I have committed it to branch-0.8, could you close this PR ?

[GitHub] egorklimov opened a new pull request #3107: [ZEPPELIN-3646] Add note for updating user permissions

2019-01-28 Thread GitBox
egorklimov opened a new pull request #3107: [ZEPPELIN-3646] Add note for updating user permissions URL: https://github.com/apache/zeppelin/pull/3107 ### What is this PR for? This is @mebelousov workaround to @maziyarpanahi

[GitHub] egorklimov commented on issue #3107: [ZEPPELIN-3646] Add note for updating user permissions

2019-01-28 Thread GitBox
egorklimov commented on issue #3107: [ZEPPELIN-3646] Add note for updating user permissions URL: https://github.com/apache/zeppelin/pull/3107#issuecomment-458071772 I think it would be a nice solution ;) @maziyarpanahi WDYT?

[GitHub] Leemoonsoo commented on a change in pull request #3303: [ZEPPELIN-3989] Configure IPython Interpreter in Docker image

2019-02-04 Thread GitBox
Leemoonsoo commented on a change in pull request #3303: [ZEPPELIN-3989] Configure IPython Interpreter in Docker image URL: https://github.com/apache/zeppelin/pull/3303#discussion_r253588410 ## File path: scripts/docker/zeppelin/bin/Dockerfile ## @@ -63,12 +63,11 @@ RUN

[GitHub] Leemoonsoo opened a new pull request #3299: [ZEPPELIN-3920] [FOLLOWUP] get correct hyperlink address when num job url is one

2019-02-04 Thread GitBox
Leemoonsoo opened a new pull request #3299: [ZEPPELIN-3920] [FOLLOWUP] get correct hyperlink address when num job url is one URL: https://github.com/apache/zeppelin/pull/3299 ### What is this PR for? After change https://github.com/apache/zeppelin/pull/3273, invalid value is assigned

[GitHub] Leemoonsoo closed pull request #3299: [ZEPPELIN-3920] [FOLLOWUP] get correct hyperlink address when num job url is one

2019-02-04 Thread GitBox
Leemoonsoo closed pull request #3299: [ZEPPELIN-3920] [FOLLOWUP] get correct hyperlink address when num job url is one URL: https://github.com/apache/zeppelin/pull/3299 This is an automated message from the Apache Git

[GitHub] Leemoonsoo opened a new pull request #3303: [ZEPPELIN-3989] Configure IPython Interpreter in Docker image

2019-02-04 Thread GitBox
Leemoonsoo opened a new pull request #3303: [ZEPPELIN-3989] Configure IPython Interpreter in Docker image URL: https://github.com/apache/zeppelin/pull/3303 ### What is this PR for? Configure IPython interpreter in Docker image by installing necessary dependencies. ### What type

[GitHub] Savalek closed pull request #3281: [ZEPPELIN-3943] - Add button "stop notebook execution"

2019-02-04 Thread GitBox
Savalek closed pull request #3281: [ZEPPELIN-3943] - Add button "stop notebook execution" URL: https://github.com/apache/zeppelin/pull/3281 This is an automated message from the Apache Git Service. To respond to the

[GitHub] Leemoonsoo closed pull request #3299: [ZEPPELIN-3920] [FOLLOWUP] get correct hyperlink address when num job url is one

2019-02-04 Thread GitBox
Leemoonsoo closed pull request #3299: [ZEPPELIN-3920] [FOLLOWUP] get correct hyperlink address when num job url is one URL: https://github.com/apache/zeppelin/pull/3299 This is an automated message from the Apache Git

[GitHub] Leemoonsoo opened a new pull request #3299: [ZEPPELIN-3920] [FOLLOWUP] get correct hyperlink address when num job url is one

2019-02-04 Thread GitBox
Leemoonsoo opened a new pull request #3299: [ZEPPELIN-3920] [FOLLOWUP] get correct hyperlink address when num job url is one URL: https://github.com/apache/zeppelin/pull/3299 ### What is this PR for? After change https://github.com/apache/zeppelin/pull/3273, invalid value is assigned

[GitHub] Leemoonsoo commented on issue #3295: Fix misspelled word

2019-02-04 Thread GitBox
Leemoonsoo commented on issue #3295: Fix misspelled word URL: https://github.com/apache/zeppelin/pull/3295#issuecomment-460332823 Thanks @gignac-cha for the fix. LGTM This is an automated message from the Apache Git Service.

[GitHub] asfgit closed pull request #3298: [ZEPPELIN-3982] add how to setup development mode for Kubernetes support

2019-02-04 Thread GitBox
asfgit closed pull request #3298: [ZEPPELIN-3982] add how to setup development mode for Kubernetes support URL: https://github.com/apache/zeppelin/pull/3298 This is an automated message from the Apache Git Service. To

[GitHub] Leemoonsoo commented on issue #3300: ZEPPELIN-3983. Travis fails due to downloading spark takes a lot of time

2019-02-04 Thread GitBox
Leemoonsoo commented on issue #3300: ZEPPELIN-3983. Travis fails due to downloading spark takes a lot of time URL: https://github.com/apache/zeppelin/pull/3300#issuecomment-460332250 Looks great! +1 This is an automated

[GitHub] felixcheung commented on a change in pull request #3302: [ZEPPELIN-3988] Paragraph Text output includes \r\n is not displayed correctly.

2019-02-04 Thread GitBox
felixcheung commented on a change in pull request #3302: [ZEPPELIN-3988] Paragraph Text output includes \r\n is not displayed correctly. URL: https://github.com/apache/zeppelin/pull/3302#discussion_r253374351 ## File path: zeppelin-web/src/app/notebook/paragraph/result/result.js

[GitHub] [zeppelin] felixcheung commented on issue #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch

2019-04-10 Thread GitBox
felixcheung commented on issue #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch URL: https://github.com/apache/zeppelin/pull/3346#issuecomment-481972289 I think one of comment might have disappear: should there be a TERMINATED state and a COMPLETED state to

[GitHub] [zeppelin] asfgit closed pull request #3345: [ZEPPELIN-4043]. Create shell script for moving note permission info from notebook-authorization.json to note file

2019-04-10 Thread GitBox
asfgit closed pull request #3345: [ZEPPELIN-4043]. Create shell script for moving note permission info from notebook-authorization.json to note file URL: https://github.com/apache/zeppelin/pull/3345 This is an automated

[GitHub] [zeppelin] zjffdu commented on issue #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch

2019-04-11 Thread GitBox
zjffdu commented on issue #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch URL: https://github.com/apache/zeppelin/pull/3346#issuecomment-481984565 Thanks for the review and suggestion, I have addressed the comments

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch

2019-04-09 Thread GitBox
zjffdu commented on a change in pull request #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch URL: https://github.com/apache/zeppelin/pull/3346#discussion_r273790891 ## File path:

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch

2019-04-09 Thread GitBox
zjffdu commented on a change in pull request #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch URL: https://github.com/apache/zeppelin/pull/3346#discussion_r273762591 ## File path:

[GitHub] [zeppelin] asfgit closed pull request #3341: ZEPPELIN-4038. Deprecate spark 2.2 and earlier

2019-04-09 Thread GitBox
asfgit closed pull request #3341: ZEPPELIN-4038. Deprecate spark 2.2 and earlier URL: https://github.com/apache/zeppelin/pull/3341 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [zeppelin] felixcheung commented on a change in pull request #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch

2019-04-09 Thread GitBox
felixcheung commented on a change in pull request #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch URL: https://github.com/apache/zeppelin/pull/3346#discussion_r273780948 ## File path:

[GitHub] [zeppelin] felixcheung commented on a change in pull request #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch

2019-04-09 Thread GitBox
felixcheung commented on a change in pull request #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch URL: https://github.com/apache/zeppelin/pull/3346#discussion_r273780922 ## File path:

[GitHub] [zeppelin] felixcheung commented on a change in pull request #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch

2019-04-09 Thread GitBox
felixcheung commented on a change in pull request #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch URL: https://github.com/apache/zeppelin/pull/3346#discussion_r273781041 ## File path:

[GitHub] [zeppelin] asfgit closed pull request #3347: [ZEPPELIN-4040]. Changed links in documentation to use Gitbox

2019-04-10 Thread GitBox
asfgit closed pull request #3347: [ZEPPELIN-4040]. Changed links in documentation to use Gitbox URL: https://github.com/apache/zeppelin/pull/3347 This is an automated message from the Apache Git Service. To respond

[GitHub] [zeppelin] AyWa commented on issue #3337: [ZEPPELIN-4090] Ipython queue performance

2019-04-10 Thread GitBox
AyWa commented on issue #3337: [ZEPPELIN-4090] Ipython queue performance URL: https://github.com/apache/zeppelin/pull/3337#issuecomment-481575952 Will try to rebase and fix this pr in the coming week :) This is an automated

[GitHub] [zeppelin] zjffdu commented on issue #3345: [ZEPPELIN-4043]. Create shell script for moving note permission info from notebook-authorization.json to note file

2019-04-08 Thread GitBox
zjffdu commented on issue #3345: [ZEPPELIN-4043]. Create shell script for moving note permission info from notebook-authorization.json to note file URL: https://github.com/apache/zeppelin/pull/3345#issuecomment-481068328 @felixcheung @liu Could you help review it ? Thanks

[GitHub] [zeppelin] Savalek closed pull request #3179: [ZEPPELIN-3773] - add check permission on write.

2019-04-15 Thread GitBox
Savalek closed pull request #3179: [ZEPPELIN-3773] - add check permission on write. URL: https://github.com/apache/zeppelin/pull/3179 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [zeppelin] Savalek closed pull request #3281: [ZEPPELIN-3943] - Add button "stop notebook execution"

2019-04-15 Thread GitBox
Savalek closed pull request #3281: [ZEPPELIN-3943] - Add button "stop notebook execution" URL: https://github.com/apache/zeppelin/pull/3281 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [zeppelin] hyonaldo opened a new pull request #3350: Branch 0.8

2019-04-16 Thread GitBox
hyonaldo opened a new pull request #3350: Branch 0.8 URL: https://github.com/apache/zeppelin/pull/3350 ### What is this PR for? A few sentences describing the overall goals of the pull request's commits. First time? Check out the contributing guide -

[GitHub] [zeppelin] hyonaldo closed pull request #3350: Branch 0.8

2019-04-16 Thread GitBox
hyonaldo closed pull request #3350: Branch 0.8 URL: https://github.com/apache/zeppelin/pull/3350 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [zeppelin] hyonaldo opened a new pull request #3351: [ZEPPELIN-4127] There is no way to list all registered cron jobs

2019-04-16 Thread GitBox
hyonaldo opened a new pull request #3351: [ZEPPELIN-4127] There is no way to list all registered cron jobs URL: https://github.com/apache/zeppelin/pull/3351 ### What is this PR for? Make list all registered zeppelin's cron jobs ### What type of PR is it? [Bug Fix |

[GitHub] [zeppelin] hyonaldo closed pull request #3351: [ZEPPELIN-4127] There is no way to list all registered cron jobs

2019-04-16 Thread GitBox
hyonaldo closed pull request #3351: [ZEPPELIN-4127] There is no way to list all registered cron jobs URL: https://github.com/apache/zeppelin/pull/3351 This is an automated message from the Apache Git Service. To respond to

[GitHub] [zeppelin] conker84 commented on issue #3034: ZEPPELIN-3552. Support Scala 2.12 of SparkInterpreter

2019-04-17 Thread GitBox
conker84 commented on issue #3034: ZEPPELIN-3552. Support Scala 2.12 of SparkInterpreter URL: https://github.com/apache/zeppelin/pull/3034#issuecomment-484287827 Any ETA for this? This is an automated message from the Apache

[GitHub] [zeppelin] asfgit closed pull request #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch

2019-04-18 Thread GitBox
asfgit closed pull request #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch URL: https://github.com/apache/zeppelin/pull/3346 This is an automated message from the Apache Git Service. To

[GitHub] [zeppelin] zjffdu commented on issue #3352: [ZEPPELIN-4127] There is no way to list all registered cron jobs

2019-04-16 Thread GitBox
zjffdu commented on issue #3352: [ZEPPELIN-4127] There is no way to list all registered cron jobs URL: https://github.com/apache/zeppelin/pull/3352#issuecomment-483896463 Thanks @hyonaldo , have you thought about provide api for this and list the cron jobs in frontend ?

[GitHub] [zeppelin] hyonaldo opened a new pull request #3352: [ZEPPELIN-4127] There is no way to list all registered cron jobs

2019-04-16 Thread GitBox
hyonaldo opened a new pull request #3352: [ZEPPELIN-4127] There is no way to list all registered cron jobs URL: https://github.com/apache/zeppelin/pull/3352 ### What is this PR for? Make list all registered zeppelin's cron jobs ### What type of PR is it? [Bug Fix |

[GitHub] [zeppelin] zjffdu commented on issue #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch

2019-04-17 Thread GitBox
zjffdu commented on issue #3346: [ZEPPELIN-4066]. Introduce ProcessLauncher to encapsulate process launch URL: https://github.com/apache/zeppelin/pull/3346#issuecomment-483971606 Will merge if no more comment This is an

[GitHub] [zeppelin] bd2019us opened a new pull request #3349: [ZEPPELIN-4124] Use optJSONArray to avoid exception

2019-04-13 Thread GitBox
bd2019us opened a new pull request #3349: [ZEPPELIN-4124] Use optJSONArray to avoid exception URL: https://github.com/apache/zeppelin/pull/3349 ### What is this PR for? Avoid unexpected JSONException if no key exist. ### What type of PR is it? [Bug Fix | Improvement |

[GitHub] [zeppelin] zjffdu commented on issue #3349: [ZEPPELIN-4124] Use optJSONArray to avoid exception

2019-04-14 Thread GitBox
zjffdu commented on issue #3349: [ZEPPELIN-4124] Use optJSONArray to avoid exception URL: https://github.com/apache/zeppelin/pull/3349#issuecomment-482923309 @bd2019us Thanks for the contribution, could you add unit test and update the PR description ?

[GitHub] [zeppelin] asfgit closed pull request #3332: ZEPPELIN-4062. Don't wait ipython kernel if python process failed

2019-03-13 Thread GitBox
asfgit closed pull request #3332: ZEPPELIN-4062. Don't wait ipython kernel if python process failed URL: https://github.com/apache/zeppelin/pull/3332 This is an automated message from the Apache Git Service. To respond to

[GitHub] [zeppelin] liuxunorg commented on a change in pull request #3331: [Zeppelin-4049] Hadoop Submarine (Machine Learning) interpreter

2019-03-13 Thread GitBox
liuxunorg commented on a change in pull request #3331: [Zeppelin-4049] Hadoop Submarine (Machine Learning) interpreter URL: https://github.com/apache/zeppelin/pull/3331#discussion_r265397174 ## File path: scripts/docker/submarine/tensorflow_1.10-hadoop_3.12-cpu-1.0.0/Dockerfile

[GitHub] [zeppelin] felixcheung commented on a change in pull request #3320: [ZEPPELIN-3977]. Create shell script for converting old note file to new file

2019-03-13 Thread GitBox
felixcheung commented on a change in pull request #3320: [ZEPPELIN-3977]. Create shell script for converting old note file to new file URL: https://github.com/apache/zeppelin/pull/3320#discussion_r265339213 ## File path:

[GitHub] [zeppelin] felixcheung commented on a change in pull request #3320: [ZEPPELIN-3977]. Create shell script for converting old note file to new file

2019-03-13 Thread GitBox
felixcheung commented on a change in pull request #3320: [ZEPPELIN-3977]. Create shell script for converting old note file to new file URL: https://github.com/apache/zeppelin/pull/3320#discussion_r265338872 ## File path: docs/setup/operation/upgrading.md ## @@ -35,6

[GitHub] [zeppelin] liuxunorg commented on issue #3334: [ZEPPELIN-4072] Fix CreateNote() throw Note existed exception caused travis to fail

2019-03-15 Thread GitBox
liuxunorg commented on issue #3334: [ZEPPELIN-4072] Fix CreateNote() throw Note existed exception caused travis to fail URL: https://github.com/apache/zeppelin/pull/3334#issuecomment-473289621 CI pass. https://travis-ci.org/liuxunorg/zeppelin/builds/506717726 @felixcheung, @zjffdu ,

[GitHub] [zeppelin] liuxunorg commented on issue #3313: [ZEPPELIN-4004] add a systemd unit file to launch the Zeppelin daemon via systemd commands

2019-03-15 Thread GitBox
liuxunorg commented on issue #3313: [ZEPPELIN-4004] add a systemd unit file to launch the Zeppelin daemon via systemd commands URL: https://github.com/apache/zeppelin/pull/3313#issuecomment-473251114 LGTM, thanks for the contribution @monsieurp :-)

[GitHub] [zeppelin] monsieurp commented on a change in pull request #3313: [ZEPPELIN-4004] add a systemd unit file to launch the Zeppelin daemon via systemd commands

2019-03-15 Thread GitBox
monsieurp commented on a change in pull request #3313: [ZEPPELIN-4004] add a systemd unit file to launch the Zeppelin daemon via systemd commands URL: https://github.com/apache/zeppelin/pull/3313#discussion_r265930599 ## File path: bin/zeppelin-systemd-service.sh ## @@

[GitHub] [zeppelin] monsieurp commented on issue #3313: [ZEPPELIN-4004] add a systemd unit file to launch the Zeppelin daemon via systemd commands

2019-03-15 Thread GitBox
monsieurp commented on issue #3313: [ZEPPELIN-4004] add a systemd unit file to launch the Zeppelin daemon via systemd commands URL: https://github.com/apache/zeppelin/pull/3313#issuecomment-473241976 @zjffdu @liuxunorg: PR updated with your suggestions. Please review.

[GitHub] [zeppelin] monsieurp commented on a change in pull request #3313: [ZEPPELIN-4004] add a systemd unit file to launch the Zeppelin daemon via systemd commands

2019-03-15 Thread GitBox
monsieurp commented on a change in pull request #3313: [ZEPPELIN-4004] add a systemd unit file to launch the Zeppelin daemon via systemd commands URL: https://github.com/apache/zeppelin/pull/3313#discussion_r265932542 ## File path: bin/zeppelin-systemd-service.sh ## @@

[GitHub] [zeppelin] liuxunorg commented on a change in pull request #3313: [ZEPPELIN-4004] add a systemd unit file to launch the Zeppelin daemon via systemd commands

2019-03-15 Thread GitBox
liuxunorg commented on a change in pull request #3313: [ZEPPELIN-4004] add a systemd unit file to launch the Zeppelin daemon via systemd commands URL: https://github.com/apache/zeppelin/pull/3313#discussion_r265933407 ## File path: bin/zeppelin-systemd-service.sh ## @@

[GitHub] [zeppelin] liuxunorg opened a new pull request #3334: [ZEPPELIN-4072] Fix CreateNote() throw Note existed exception caused travis to fail

2019-03-15 Thread GitBox
liuxunorg opened a new pull request #3334: [ZEPPELIN-4072] Fix CreateNote() throw Note existed exception caused travis to fail URL: https://github.com/apache/zeppelin/pull/3334 ### What is this PR for? Fix CreateNote() function throw Note existed IOException caused travis to fail.

[GitHub] [zeppelin] liuxunorg commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter

2019-05-15 Thread GitBox
liuxunorg commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter URL: https://github.com/apache/zeppelin/pull/3359#issuecomment-492696041 # The function is exactly the same as the terminal emulator ## curl + tar ScreenRecord

[GitHub] [zeppelin] felixcheung commented on a change in pull request #3364: [ZEPPELIN-4155] Unable get available HostAddress in multi-NIC environment

2019-05-15 Thread GitBox
felixcheung commented on a change in pull request #3364: [ZEPPELIN-4155] Unable get available HostAddress in multi-NIC environment URL: https://github.com/apache/zeppelin/pull/3364#discussion_r284328179 ## File path:

[GitHub] [zeppelin] felixcheung commented on a change in pull request #3365: [ZEPPELIN-4160] Fixed Move this note to trash icon always show

2019-05-15 Thread GitBox
felixcheung commented on a change in pull request #3365: [ZEPPELIN-4160] Fixed Move this note to trash icon always show URL: https://github.com/apache/zeppelin/pull/3365#discussion_r284327576 ## File path: zeppelin-web/src/app/notebook/notebook.controller.js ## @@ -237,7

[GitHub] [zeppelin] Leemoonsoo commented on issue #3363: [ZEPPELIN-4144] After refresh the revision tools are sometimes hidden

2019-05-15 Thread GitBox
Leemoonsoo commented on issue #3363: [ZEPPELIN-4144] After refresh the revision tools are sometimes hidden URL: https://github.com/apache/zeppelin/pull/3363#issuecomment-492770214 Thanks @florpor for the contribution! Could you check message from Jenkins and try setup travis, and make

[GitHub] [zeppelin] liuxunorg opened a new pull request #3365: [ZEPPELIN-4160] Fixed Move this note to trash icon always show

2019-05-15 Thread GitBox
liuxunorg opened a new pull request #3365: [ZEPPELIN-4160] Fixed Move this note to trash icon always show URL: https://github.com/apache/zeppelin/pull/3365 ### What is this PR for? when the note moved to trash, the note should show 'Remove this note permanently', it does not work

[GitHub] [zeppelin] liuxunorg commented on issue #3365: [ZEPPELIN-4160] Fixed Move this note to trash icon always show

2019-05-15 Thread GitBox
liuxunorg commented on issue #3365: [ZEPPELIN-4160] Fixed Move this note to trash icon always show URL: https://github.com/apache/zeppelin/pull/3365#issuecomment-492710080 @zjffdu , Please help me review code. Thanks! This

[GitHub] [zeppelin] liuxunorg commented on issue #3364: [ZEPPELIN-4155] Unable get available HostAddress in multi-NIC environment

2019-05-15 Thread GitBox
liuxunorg commented on issue #3364: [ZEPPELIN-4155] Unable get available HostAddress in multi-NIC environment URL: https://github.com/apache/zeppelin/pull/3364#issuecomment-492710247 @zjffdu , Please help me review code. Thanks!

[GitHub] [zeppelin] felixcheung commented on issue #3360: [ZEPPELIN-4152]. Use spark 2.2 as the default profile

2019-05-16 Thread GitBox
felixcheung commented on issue #3360: [ZEPPELIN-4152]. Use spark 2.2 as the default profile URL: https://github.com/apache/zeppelin/pull/3360#issuecomment-493283757 sure, this could be a big problem... This is an automated

[GitHub] [zeppelin] felixcheung commented on issue #3360: [ZEPPELIN-4152]. Use spark 2.2 as the default profile

2019-05-16 Thread GitBox
felixcheung commented on issue #3360: [ZEPPELIN-4152]. Use spark 2.2 as the default profile URL: https://github.com/apache/zeppelin/pull/3360#issuecomment-493283789 LGTM This is an automated message from the Apache Git

[GitHub] [zeppelin] asfgit closed pull request #3360: [ZEPPELIN-4152]. Use spark 2.2 as the default profile

2019-05-16 Thread GitBox
asfgit closed pull request #3360: [ZEPPELIN-4152]. Use spark 2.2 as the default profile URL: https://github.com/apache/zeppelin/pull/3360 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [zeppelin] asfgit closed pull request #3366: [ZEPPELIN-4148]. Move GitNotebookRepo into zeppelin-zengine as default NotebookRepo

2019-05-16 Thread GitBox
asfgit closed pull request #3366: [ZEPPELIN-4148]. Move GitNotebookRepo into zeppelin-zengine as default NotebookRepo URL: https://github.com/apache/zeppelin/pull/3366 This is an automated message from the Apache Git

[GitHub] [zeppelin] liuxunorg opened a new pull request #3359: [ZEPPELIN-4131] Refactoring shell interpreter

2019-05-12 Thread GitBox
liuxunorg opened a new pull request #3359: [ZEPPELIN-4131] Refactoring shell interpreter URL: https://github.com/apache/zeppelin/pull/3359 ### What is this PR for? Realize the same function as the terminal, The shell-by-command can be executed by the shell interpreter.

[GitHub] [zeppelin] zjffdu commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter

2019-05-12 Thread GitBox
zjffdu commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter URL: https://github.com/apache/zeppelin/pull/3359#issuecomment-491682054 @liuxunorg Is it possible to download hterm_all.js at building time instead of put into repo ? And could you write a simple design doc for

[GitHub] [zeppelin] liuxunorg commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter

2019-05-12 Thread GitBox
liuxunorg commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter URL: https://github.com/apache/zeppelin/pull/3359#issuecomment-491677070 @zjffdu , @felixcheung , Please help me review code. Thanks! :-) This

[GitHub] [zeppelin] liuxunorg commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter

2019-05-12 Thread GitBox
liuxunorg commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter URL: https://github.com/apache/zeppelin/pull/3359#issuecomment-491682950 @zjffdu , 《Refactoring zeppelin shell interpreter Design Document》 :

[GitHub] [zeppelin] liuxunorg commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter

2019-05-12 Thread GitBox
liuxunorg commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter URL: https://github.com/apache/zeppelin/pull/3359#issuecomment-491684736 @zjffdu , `hterm_all.js` is exported via the chromium libapps project. `hterm_all.js` relies on several submodules in chromium

[GitHub] [zeppelin] Tagar commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter

2019-05-15 Thread GitBox
Tagar commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter URL: https://github.com/apache/zeppelin/pull/3359#issuecomment-492824721 As part of Zeppelin notes we have static %sh paragraphs (non-interactive) with or without variable embeddings (variable interpolations

[GitHub] [zeppelin] asfgit closed pull request #3353: [ZEPPELIN-4132]. Spark Interpreter has issue of SPARK-22393

2019-05-17 Thread GitBox
asfgit closed pull request #3353: [ZEPPELIN-4132]. Spark Interpreter has issue of SPARK-22393 URL: https://github.com/apache/zeppelin/pull/3353 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [zeppelin] liuxunorg commented on issue #3365: [ZEPPELIN-4160] Fixed Move this note to trash icon always show

2019-05-15 Thread GitBox
liuxunorg commented on issue #3365: [ZEPPELIN-4160] Fixed Move this note to trash icon always show URL: https://github.com/apache/zeppelin/pull/3365#issuecomment-492884302 > can you explain the change from `[0]` to `[1]`? @felixcheung , This is because of the problem caused by this

[GitHub] [zeppelin] zjffdu commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter

2019-05-16 Thread GitBox
zjffdu commented on issue #3359: [ZEPPELIN-4131] Refactoring shell interpreter URL: https://github.com/apache/zeppelin/pull/3359#issuecomment-492935457 @liuxunorg Based on the above comments, I think we should keep the old shell interpreter as the batch mode and the new shell interpreter

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3364: [ZEPPELIN-4155] Unable get available HostAddress in multi-NIC environment

2019-05-15 Thread GitBox
zjffdu commented on a change in pull request #3364: [ZEPPELIN-4155] Unable get available HostAddress in multi-NIC environment URL: https://github.com/apache/zeppelin/pull/3364#discussion_r284523852 ## File path:

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3364: [ZEPPELIN-4155] Unable get available HostAddress in multi-NIC environment

2019-05-15 Thread GitBox
zjffdu commented on a change in pull request #3364: [ZEPPELIN-4155] Unable get available HostAddress in multi-NIC environment URL: https://github.com/apache/zeppelin/pull/3364#discussion_r284523852 ## File path:

[GitHub] [zeppelin] zjffdu commented on a change in pull request #3364: [ZEPPELIN-4155] Unable get available HostAddress in multi-NIC environment

2019-05-15 Thread GitBox
zjffdu commented on a change in pull request #3364: [ZEPPELIN-4155] Unable get available HostAddress in multi-NIC environment URL: https://github.com/apache/zeppelin/pull/3364#discussion_r284523852 ## File path:

<    4   5   6   7   8   9   10   11   12   13   >