[GitHub] zeppelin pull request #1369: [ZEPPELIN-1376]

2016-08-28 Thread doanduyhai
GitHub user doanduyhai reopened a pull request: https://github.com/apache/zeppelin/pull/1369 [ZEPPELIN-1376] ### What is this PR for? When using Zeppelin behind corporate firewall, sometimes the dependencies download just fails silently. This PR has 2 objectives: * add

[GitHub] zeppelin pull request #746: [ZEPPELIN-699] Add new synchronous paragraph run...

2016-08-28 Thread doanduyhai
GitHub user doanduyhai reopened a pull request: https://github.com/apache/zeppelin/pull/746 [ZEPPELIN-699] Add new synchronous paragraph run REST API ### What is this PR for? Right now, when calling the REST API `http://:/api/notebook/job//` Zeppelin always returns **OK** as

[GitHub] zeppelin issue #1376: [ZEPPELIN-449] Enhance log messages when interpreter i...

2016-08-28 Thread doanduyhai
Github user doanduyhai commented on the issue: https://github.com/apache/zeppelin/pull/1376 @bzz I don't know why the tests fail on this error message, because this is indeed **a test which expects a failure** because `system.complex_table` does not exist, see below:

[GitHub] zeppelin issue #1378: [ZEPPELIN-1358] Add support to display Pandas DataFram...

2016-08-28 Thread bustios
Github user bustios commented on the issue: https://github.com/apache/zeppelin/pull/1378 Sure @bzz, it can be optional. I just thought it was more appropriate, since a dataframe index is always printed, but I can add a parameter like `show_index=False` to make it optional. --- If

[GitHub] zeppelin pull request #1367: [ZEPPELIN-1040] Show the time when the result i...

2016-08-28 Thread rajarajan-g
GitHub user rajarajan-g reopened a pull request: https://github.com/apache/zeppelin/pull/1367 [ZEPPELIN-1040] Show the time when the result is updated ### What is this PR for? As per existing usage, the time shown in end of each paragraph is the time the paragraph is updated

[GitHub] zeppelin pull request #1367: [ZEPPELIN-1040] Show the time when the result i...

2016-08-28 Thread rajarajan-g
Github user rajarajan-g closed the pull request at: https://github.com/apache/zeppelin/pull/1367 --- 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 #1377: ZEPPELIN-1384. Spark interpreter binary compatibility ...

2016-08-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1377 LGTM. CI is green as well. --- 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 #1378: [ZEPPELIN-1358] Add support to display Pandas DataFram...

2016-08-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1378 @bustios thank you for improvements! Could you please explain a bit the benefits of having print index behaviour enabled by default? Do you think it should not be optional? It's

[GitHub] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-08-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1358 Looks great to me! Merging to master if there is no further discussion. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] zeppelin pull request #1378: [ZEPPELIN-1358] Add support to display Pandas D...

2016-08-28 Thread bustios
GitHub user bustios opened a pull request: https://github.com/apache/zeppelin/pull/1378 [ZEPPELIN-1358] Add support to display Pandas DataFrame index using z.show() ### What is this PR for? Add support to display Pandas DataFrame index using z.show() in python interpreter

[GitHub] zeppelin issue #1370: [ZEPPELIN-1379] Flink interpreter is missing scala lib...

2016-08-28 Thread bzz
- run paragraph 20160828-003254_783795023 using md org.apache.zeppelin.interpreter.LazyOpenInterpreter@22778f7f 00:32:54,195 INFO org.apache.zeppelin.interpreter.remote.RemoteInterpreterManagedProcess:116 - Run interpreter process [..//bin/interpreter.sh, -d, ../interpreter/md, -p, 46521, -l

[GitHub] zeppelin issue #1333: [ZEPPELIN-1334] Environment variable defined in interp...

2016-08-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1333 @zjffdu Looks great to me, modulo minor style issue above, thank you and :+1: for tests! One more thing - it's a bit strange ZEPPELIN-1334 has fix-for version ONLY 0.6.2 which implies

[GitHub] zeppelin issue #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

2016-08-28 Thread Peilin-Yang
Github user Peilin-Yang commented on the issue: https://github.com/apache/zeppelin/pull/1363 @bzz I agree --- 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,

[GitHub] zeppelin issue #1363: [ZEPPELIN-1371]add text/numeric conversion support to ...

2016-08-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1363 Let's merge #1371 first, and then it makes perfect sense to adjust this one to match user expectations of sorting changes according to column type (so it lets user override auto-detection).

[GitHub] zeppelin issue #1371: [ZEPPELIN-1372]Automatically Detect the data type in t...

2016-08-28 Thread echarles
Github user echarles commented on the issue: https://github.com/apache/zeppelin/pull/1371 Upon numeric values, the automatic detection of dates... would be great. Detection at client (javascript) side could be misleading as you only get a subset of the data. On the

[GitHub] zeppelin issue #1371: [ZEPPELIN-1372]Automatically Detect the data type in t...

2016-08-28 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1371 Thank you @Peilin-Yang ! It looks like it's `O(N)` from the table size - do you think there might be a performance implication here? Just curious, but if it's a table - should not analysing

[GitHub] zeppelin issue #1376: [ZEPPELIN-449] Enhance log messages when interpreter i...

2016-08-28 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/1376 @lresende maybe ZEPPELIN-1187 is useful for you. You can edit log4j.properties to add the followign line. This is helpful for me to debug the remote interpreter process. ```

[GitHub] zeppelin issue #1371: [ZEPPELIN-1372]Automatically Detect the data type in t...

2016-08-28 Thread Peilin-Yang
Github user Peilin-Yang commented on the issue: https://github.com/apache/zeppelin/pull/1371 @arunsoman Yes. in this PR the code can automatically detect the type of the cells, but restricted to strings and numbers. Dates can be added either and this is a good idea. In my other

[GitHub] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-08-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1358 @astroshim I rebuild with `mvn clean package -DskipTests -Pexamples` and it works. I should've attached `-Pexamples`. Now I can see the Helium icon :) Anyway I checked

[GitHub] zeppelin pull request #1351: [ZEPPELIN/1356] The graph legend truncates at t...

2016-08-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1351 --- 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 #1377: ZEPPELIN-1384. Spark interpreter binary compati...

2016-08-28 Thread zjffdu
GitHub user zjffdu opened a pull request: https://github.com/apache/zeppelin/pull/1377 ZEPPELIN-1384. Spark interpreter binary compatibility to scala 2.10 / 2.11 broken ### What is this PR for? As described in ZEPPELIN-1384, we may hit the following error when running

[GitHub] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-08-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1358 I think it's not an issue related with this PR but Helium itself. --- 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

[GitHub] zeppelin pull request #1376: [ZEPPELIN-449] Enhance log messages when interp...

2016-08-28 Thread lresende
GitHub user lresende opened a pull request: https://github.com/apache/zeppelin/pull/1376 [ZEPPELIN-449] Enhance log messages when interpreter is initializing ### What is this PR for? Enhance log messages when interpreter is initializing to avoid user confusion. ###

[GitHub] zeppelin pull request #1347: ZEPPELIN-1284. Unable to run paragraph with def...

2016-08-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1347 --- 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 #1347: ZEPPELIN-1284. Unable to run paragraph with default in...

2016-08-28 Thread lresende
Github user lresende commented on the issue: https://github.com/apache/zeppelin/pull/1347 LGTM, Do we need to wait on anything else to get this merged ? --- 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 #1373: [ZEPPELIN-1385] Avoid NPE when create SparkSess...

2016-08-28 Thread lresende
Github user lresende closed the pull request at: https://github.com/apache/zeppelin/pull/1373 --- 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 #1373: [ZEPPELIN-1385] Avoid NPE when create SparkSession

2016-08-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue: https://github.com/apache/zeppelin/pull/1373 Thanks @lresende for the fix and verification. But i think we already have exactly the same fix at #1347. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin pull request #1366: ZEPPELIN-1342. Adding dependencies via SPARK_SU...

2016-08-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/zeppelin/pull/1366 --- 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 #1374: [MINOR] Add new line before logging paragraph c...

2016-08-28 Thread lresende
GitHub user lresende opened a pull request: https://github.com/apache/zeppelin/pull/1374 [MINOR] Add new line before logging paragraph content ### What is this PR for? Add new line before logging paragraph content ### What type of PR is it? [Improvement] You can

[GitHub] zeppelin pull request #746: [ZEPPELIN-699] Add new synchronous paragraph run...

2016-08-28 Thread doanduyhai
GitHub user doanduyhai reopened a pull request: https://github.com/apache/zeppelin/pull/746 [ZEPPELIN-699] Add new synchronous paragraph run REST API ### What is this PR for? Right now, when calling the REST API `http://:/api/notebook/job//` Zeppelin always returns **OK** as

[GitHub] zeppelin pull request #746: [ZEPPELIN-699] Add new synchronous paragraph run...

2016-08-28 Thread doanduyhai
Github user doanduyhai closed the pull request at: https://github.com/apache/zeppelin/pull/746 --- 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 #1334: [ZEPPELIN-682] New interpreter for Apache Beam (incuba...

2016-08-28 Thread mfelgamal
Github user mfelgamal commented on the issue: https://github.com/apache/zeppelin/pull/1334 @bzz Gentle reminder! --- 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 issue #1373: [ZEPPELIN-1385] Avoid NPE when create SparkSession

2016-08-28 Thread lresende
Github user lresende commented on the issue: https://github.com/apache/zeppelin/pull/1373 After this fix is applied, have verified that can connect and execute a job on a remote Spark cluster. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] zeppelin pull request #746: [ZEPPELIN-699] Add new synchronous paragraph run...

2016-08-28 Thread doanduyhai
GitHub user doanduyhai reopened a pull request: https://github.com/apache/zeppelin/pull/746 [ZEPPELIN-699] Add new synchronous paragraph run REST API ### What is this PR for? Right now, when calling the REST API `http://:/api/notebook/job//` Zeppelin always returns **OK** as

[GitHub] zeppelin issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-08-28 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1358 @AhyoungRyu Thank you for reviewing!! Did you clear your browser cache? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-28 Thread astroshim
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1302 I just changed to 'icon'. Thank you. --- 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 issue #1358: [ZEPPELIN-1365] Error of Zeppelin Application in devel...

2016-08-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1358 @astroshim Definitely in the latest master branch, I faced the below exception when I tried to run `org.apache.zeppelin.interpreter.dev.ZeppelinApplicationDevServer` in my IDE. ```

[GitHub] zeppelin issue #1302: [ZEPPELIN-1306] Interpreter restarts on a note.

2016-08-28 Thread AhyoungRyu
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/1302 @astroshim Great! Personally I would prefer "just icon" like the reload notes icon in Zeppelin home. But it's just my personal opinion. Final decision is up to you :) --- If your project

[GitHub] zeppelin pull request #746: [ZEPPELIN-699] Add new synchronous paragraph run...

2016-08-28 Thread doanduyhai
Github user doanduyhai closed the pull request at: https://github.com/apache/zeppelin/pull/746 --- 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 #1369: [ZEPPELIN-1376]

2016-08-28 Thread doanduyhai
Github user doanduyhai commented on the issue: https://github.com/apache/zeppelin/pull/1369 @felixcheung That's the main issue I'm facing now, how to **automate** test with corporate firewall ... I'm going to ask someone to test the this PR at his place (he has a corporate