[GitHub] zeppelin pull request #1421: [ZEPPELIN-1422][zeppelin-interpreter] fix Inter...

2016-09-09 Thread WeichenXu123
GitHub user WeichenXu123 opened a pull request: https://github.com/apache/zeppelin/pull/1421 [ZEPPELIN-1422][zeppelin-interpreter] fix InterpreterProperty equals methods, add null check ### What is this PR for? in PR https://github.com/apache/zeppelin/pull/1382, there is a

[GitHub] zeppelin issue #1297: [zeppelin] add temp directories generated by zeppelin-...

2016-08-31 Thread WeichenXu123
Github user WeichenXu123 commented on the issue: https://github.com/apache/zeppelin/pull/1297 cc @jongyoul It's a very simple pr. thanks! --- 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

[GitHub] zeppelin pull request #1297: [zeppelin] add temp directories generated by ze...

2016-08-31 Thread WeichenXu123
Github user WeichenXu123 closed the pull request at: https://github.com/apache/zeppelin/pull/1297 --- 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

[GitHub] zeppelin pull request #1297: [zeppelin] add temp directories generated by ze...

2016-08-31 Thread WeichenXu123
GitHub user WeichenXu123 reopened a pull request: https://github.com/apache/zeppelin/pull/1297 [zeppelin] add temp directories generated by zeppelin-Rinterpreter to gitignore ### What is this PR for? When compile and test zeppelin-Rinterpreter, it will generate temp

[GitHub] zeppelin pull request #1372: [ZEPPELIN-1383][ Interpreters][r-interpreter] r...

2016-08-30 Thread WeichenXu123
GitHub user WeichenXu123 reopened a pull request: https://github.com/apache/zeppelin/pull/1372 [ZEPPELIN-1383][ Interpreters][r-interpreter] remove SparkInterpreter.getSystemDefault and update relative code ### What is this PR for? clean some redundant code: remove

[GitHub] zeppelin pull request #1372: [ZEPPELIN-1383][ Interpreters][r-interpreter] r...

2016-08-30 Thread WeichenXu123
Github user WeichenXu123 closed the pull request at: https://github.com/apache/zeppelin/pull/1372 --- 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

[GitHub] zeppelin issue #1382: [ZEPPELIN-1391][Interpreters] print error while existi...

2016-08-29 Thread WeichenXu123
Github user WeichenXu123 commented on the issue: https://github.com/apache/zeppelin/pull/1382 cc @jongyoul thanks! --- 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

[GitHub] zeppelin pull request #1382: [ZEPPELIN-1391][Interpreters] print error while...

2016-08-29 Thread WeichenXu123
GitHub user WeichenXu123 opened a pull request: https://github.com/apache/zeppelin/pull/1382 [ZEPPELIN-1391][Interpreters] print error while existing registedInterpreter with the same key but different settings ### What is this PR for? print error while existing

[GitHub] zeppelin issue #1372: [ZEPPELIN-1383][ Interpreters][r-interpreter] remove S...

2016-08-27 Thread WeichenXu123
Github user WeichenXu123 commented on the issue: https://github.com/apache/zeppelin/pull/1372 cc @jongyoul @Leemoonsoo Thanks! --- 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

[GitHub] zeppelin pull request #1372: [ZEPPELIN-1383][ Interpreters][r-interpreter] r...

2016-08-27 Thread WeichenXu123
GitHub user WeichenXu123 opened a pull request: https://github.com/apache/zeppelin/pull/1372 [ZEPPELIN-1383][ Interpreters][r-interpreter] remove SparkInterpreter.getSystemDefault and update relative code ### What is this PR for? clean some redundant code: remove

[GitHub] zeppelin issue #1295: [ZEPPELIN-1301] fix potential encoding problem in RInt...

2016-08-15 Thread WeichenXu123
Github user WeichenXu123 commented on the issue: https://github.com/apache/zeppelin/pull/1295 @Leemoonsoo CI is green! would you help review it ? thanks! --- 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

[GitHub] zeppelin pull request #1295: [ZEPPELIN-1301] fix potential encoding problem ...

2016-08-15 Thread WeichenXu123
GitHub user WeichenXu123 reopened a pull request: https://github.com/apache/zeppelin/pull/1295 [ZEPPELIN-1301] fix potential encoding problem in RInterpreter processHTML DataURI conversion ### What is this PR for? fix potential encoding problem in RInterpreter processHTML

[GitHub] zeppelin pull request #1295: [ZEPPELIN-1301] fix potential encoding problem ...

2016-08-15 Thread WeichenXu123
Github user WeichenXu123 closed the pull request at: https://github.com/apache/zeppelin/pull/1295 --- 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

[GitHub] zeppelin pull request #1295: [ZEPPELIN-1301] fix potential encoding problem ...

2016-08-14 Thread WeichenXu123
Github user WeichenXu123 closed the pull request at: https://github.com/apache/zeppelin/pull/1295 --- 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

[GitHub] zeppelin pull request #1295: [ZEPPELIN-1301] fix potential encoding problem ...

2016-08-14 Thread WeichenXu123
GitHub user WeichenXu123 reopened a pull request: https://github.com/apache/zeppelin/pull/1295 [ZEPPELIN-1301] fix potential encoding problem in RInterpreter processHTML DataURI conversion ### What is this PR for? fix potential encoding problem in RInterpreter processHTML

[GitHub] zeppelin issue #1296: [ZEPPELIN-1302] fix rinterpreter default prop values i...

2016-08-13 Thread WeichenXu123
Github user WeichenXu123 commented on the issue: https://github.com/apache/zeppelin/pull/1296 @Leemoonsoo CI is green now! --- 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

[GitHub] zeppelin pull request #1295: [ZEPPELIN-1301] fix potential encoding problem ...

2016-08-13 Thread WeichenXu123
Github user WeichenXu123 closed the pull request at: https://github.com/apache/zeppelin/pull/1295 --- 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

[GitHub] zeppelin pull request #1295: [ZEPPELIN-1301] fix potential encoding problem ...

2016-08-13 Thread WeichenXu123
GitHub user WeichenXu123 reopened a pull request: https://github.com/apache/zeppelin/pull/1295 [ZEPPELIN-1301] fix potential encoding problem in RInterpreter processHTML DataURI conversion ### What is this PR for? fix potential encoding problem in RInterpreter processHTML

[GitHub] zeppelin pull request #1296: [ZEPPELIN-1302] fix rinterpreter default prop v...

2016-08-13 Thread WeichenXu123
Github user WeichenXu123 closed the pull request at: https://github.com/apache/zeppelin/pull/1296 --- 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

[GitHub] zeppelin pull request #1296: [ZEPPELIN-1302] fix rinterpreter default prop v...

2016-08-13 Thread WeichenXu123
GitHub user WeichenXu123 reopened a pull request: https://github.com/apache/zeppelin/pull/1296 [ZEPPELIN-1302] fix rinterpreter default prop values init error ### What is this PR for? 1. Correct the calling to `SparkInterpreter.getSystemDefault` from

[GitHub] zeppelin issue #1295: [ZEPPELIN-1301] fix potential encoding problem in RInt...

2016-08-12 Thread WeichenXu123
Github user WeichenXu123 commented on the issue: https://github.com/apache/zeppelin/pull/1295 @Leemoonsoo thanks! I submit this PR because I meet encoding conversion problems when the including js contains non-ASCII character. --- If your project is set up for it, you can reply

[GitHub] zeppelin issue #1296: [ZEPPELIN-1302] fix rinterpreter default prop values i...

2016-08-12 Thread WeichenXu123
Github user WeichenXu123 commented on the issue: https://github.com/apache/zeppelin/pull/1296 @Leemoonsoo How to trigger CI test again ? Is there some instructions or reply some keywords ? --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] zeppelin issue #1296: [ZEPPELIN-1302] fix rinterpreter default prop values i...

2016-08-12 Thread WeichenXu123
Github user WeichenXu123 commented on the issue: https://github.com/apache/zeppelin/pull/1296 cc @Leemoonsoo would you help review this? It's an obvious code bug fix. Thanks! The CI failed case isn't related to this PR. --- If your project is set up for it, you can rep

[GitHub] zeppelin issue #1296: [ZEPPELIN-1302] fix rinterpreter default prop values i...

2016-08-07 Thread WeichenXu123
Github user WeichenXu123 commented on the issue: https://github.com/apache/zeppelin/pull/1296 cc @zjffdu Thanks! It is an obvious bug. --- 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

[GitHub] zeppelin pull request #1297: [GIT] add temp directories generated by zeppeli...

2016-08-06 Thread WeichenXu123
GitHub user WeichenXu123 opened a pull request: https://github.com/apache/zeppelin/pull/1297 [GIT] add temp directories generated by zeppelin-Rinterpreter ### What is this PR for? When compile and test zeppelin-Rinterpreter, it will generate temp directories `/R` and `/tmp

[GitHub] zeppelin pull request #1296: [ZEPPELIN-1302] fix rinterpreter default prop v...

2016-08-06 Thread WeichenXu123
GitHub user WeichenXu123 opened a pull request: https://github.com/apache/zeppelin/pull/1296 [ZEPPELIN-1302] fix rinterpreter default prop values init error ### What is this PR for? 1. Correct the calling to `SparkInterpreter.getSystemDefault` from

[GitHub] zeppelin pull request #1295: [ZEPPELIN-1301] fix potential encoding problem ...

2016-08-06 Thread WeichenXu123
GitHub user WeichenXu123 opened a pull request: https://github.com/apache/zeppelin/pull/1295 [ZEPPELIN-1301] fix potential encoding problem in RInterpreter processHTML DataURI conversion ### What is this PR for? fix potential encoding problem in RInterpreter processHTML DataURI