[jira] [Created] (ZEPPELIN-2687) Show more meaningful info in spark UI, for each job created from paragraph run

2017-06-24 Thread Karuppayya (JIRA)
Karuppayya created ZEPPELIN-2687: Summary: Show more meaningful info in spark UI, for each job created from paragraph run Key: ZEPPELIN-2687 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2687 Pr

[GitHub] zeppelin pull request #2434: ZEPPELIN-2687: Show code in description section...

2017-06-24 Thread karuppayya
GitHub user karuppayya opened a pull request: https://github.com/apache/zeppelin/pull/2434 ZEPPELIN-2687: Show code in description section of Spark UI ### What is this PR for? Show code in description section of Spark UI ### What type of PR is it? Improvement

[GitHub] zeppelin issue #2403: ZEPPELIN-2637: Make useLegacySql configurable, in Bigq...

2017-06-24 Thread karuppayya
Github user karuppayya commented on the issue: https://github.com/apache/zeppelin/pull/2403 @jongyoul CI failure looks to be not relevant. --- 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 thi

[GitHub] zeppelin issue #2434: ZEPPELIN-2687: Show code in description section of Spa...

2017-06-24 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2434 @karuppayya Thanks for this PR. It is really helpful. Besides this, with more and more features added we have more and more duplicated code in `PySparkInterpreter`, `SparkInterpreter`, `SparkSQLInt

[GitHub] zeppelin issue #2434: ZEPPELIN-2687: Show code in description section of Spa...

2017-06-24 Thread karuppayya
Github user karuppayya commented on the issue: https://github.com/apache/zeppelin/pull/2434 @zjffdu +1 on the refactoring. --- 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

[GitHub] zeppelin issue #2428: [ZEPPELIN-1470] limiting results from jdbc

2017-06-24 Thread tinkoff-dwh
Github user tinkoff-dwh commented on the issue: https://github.com/apache/zeppelin/pull/2428 failed test wich load dependencies, it is possible too long or problems with the network. try restart again --- If your project is set up for it, you can reply to this email and have your rep

[GitHub] zeppelin pull request #2268: [ZEPPELIN-2403] interpreter property widgets

2017-06-24 Thread tinkoff-dwh
GitHub user tinkoff-dwh reopened a pull request: https://github.com/apache/zeppelin/pull/2268 [ZEPPELIN-2403] interpreter property widgets ### What is this PR for? I spoiled the previous PR #2251 Added widgets (string, text, url, password, url, checkbox) to properties of

[GitHub] zeppelin pull request #2268: [ZEPPELIN-2403] interpreter property widgets

2017-06-24 Thread tinkoff-dwh
Github user tinkoff-dwh closed the pull request at: https://github.com/apache/zeppelin/pull/2268 --- 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

[GitHub] zeppelin issue #2272: ZEPPELIN-2397. Add toJson and fromJson method for seri...

2017-06-24 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2272 LGTM. For the next step, how about introducing `JsonSerializable` and `JsonDeserializable` as interfaces? --- If your project is set up for it, you can reply to this email and have your reply ap

[GitHub] zeppelin issue #2272: ZEPPELIN-2397. Add toJson and fromJson method for seri...

2017-06-24 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2272 @jongyoul Thanks for the review. Copy my comment above. ``` toJson and fromJson is used in both zeppelin-interpreter and zeppelin-zengine. I didn't find a proper place to put the common int

[GitHub] zeppelin issue #2272: ZEPPELIN-2397. Add toJson and fromJson method for seri...

2017-06-24 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2272 I saw it above. It doesn't reduce the code size but I think it will help another contributors understand code well. How do you think of it? --- If your project is set up for it, you can reply to

[GitHub] zeppelin issue #2272: ZEPPELIN-2397. Add toJson and fromJson method for seri...

2017-06-24 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2272 I am fine to add interface for it. But only `JsonSerializable` can be added, because `fromJson` is static method. It is some kind of code convention, should be easy to read and understand. ---

[GitHub] zeppelin issue #2418: [ZEPPELIN-2645] Adding way to register RemoteInterpret...

2017-06-24 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2418 @zjffdu Agreed. --- 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, o

[GitHub] zeppelin issue #2403: ZEPPELIN-2637: Make useLegacySql configurable, in Bigq...

2017-06-24 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2403 LGTM --- 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 fe

[GitHub] zeppelin issue #2309: [ZEPPELIN-2482] added font size for paragraph

2017-06-24 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2309 Missed. I'll merge it 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 enabled and w

[GitHub] zeppelin pull request #2309: [ZEPPELIN-2482] added font size for paragraph

2017-06-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/2309 --- 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 e