[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1248 I'm sorry for delay, @mwkang - my plate was full last week. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 ping @bzz --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1248 Thank you for your patience @mwkang ! Sorry for the late notice, but could you please take a look and make sure that code conforms the project styleguilde? Will merge to master asap, right after that, 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 your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1248 @mwkang thank you for friendly ping! --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 @bzz I think This PR have some trouble merging to master. Because When bzz leave comment, I also leave comment immediately. So bot did not recognize this PR. Could you mind if you check again? (I'm worried about this PR open forever.) --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 @bzz 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 feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1248 Thanks! In my experience, if in case of CI failure PR author posts the failure result from logs - that speeds us review a lot! CI fails now on Spark 1.6 Scala 2.10 and Scala 2.11 profiles with ``` [INFO] Zeppelin: web Application .. FAILURE [01:31 min] [INFO] make: Leaving directory `/home/travis/build/apache/zeppelin/zeppelin-web/node_modules/karma/node_modules/socket.io/node_modules/socket.io-client/node_modules/ws/build' [ERROR] npm ERR! Linux 3.13.0-40-generic [ERROR] npm ERR! argv "/home/travis/build/apache/zeppelin/zeppelin-web/node/node" "/home/travis/build/apache/zeppelin/zeppelin-web/node/node_modules/npm/bin/npm-cli.js" "install" "--color=false" [ERROR] npm ERR! node v0.12.13 [ERROR] npm ERR! npm v2.15.0 [ERROR] npm ERR! code ECONNRESET [ERROR] npm ERR! errno ECONNRESET [ERROR] npm ERR! syscall read [ERROR] [ERROR] npm ERR! network read ECONNRESET [ERROR] npm ERR! network This is most likely not a problem with npm itself [ERROR] npm ERR! network and is related to network connectivity. [ERROR] npm ERR! network In most cases you are behind a proxy or have bad network settings. [ERROR] npm ERR! network [ERROR] npm ERR! network If you are behind a proxy, please make sure that the [ERROR] npm ERR! network 'proxy' config is set properly. See: 'npm help config' ``` Which seems to be networking issue on Travis side ð Thank you @mwkang , 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 your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 @bzz Thanks for your review. I rebase on to master and force push. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1248 This looks like an issue fixed in master 10h ago: ``` [31m- should provide onclick method *** FAILED ***[0m [31m The code passed to eventually never returned normally. Attempted 1 times over 231.694296 milliseconds. Last failure message: 0 was not equal to 1. (AbstractAngularElemTest.scala:72)[0m ``` Could you please try rebasing on latest master `git pull ; git rebase master; git push -f` ? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1248 Ok, good to know, let me look into that --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 @bzz No, It does not persists every time. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 > If you re-trigger CI more (close&re-open this PR) - does the same issue persists every time? I'm not sure if it persists. I will re-tigger after fix the issue. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1248 @mwkang thank you for prompt addressing reviews! If you re-trigger CI more (close&re-open this PR) - does the same issue persists every time? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 I am not sure this error is related with this issue. ``` Failed tests: ZeppelinSparkClusterTest.zRunTest:204 expected: but was: ``` --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 re-trigger CI --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 @jongyoul Sorry, I missed it. I will check. I appreciate your review. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/1248 Do you check the error of CI? ``` Failed tests: NoteTest.insertParagraphWithLastReplName:159 expected:<%spark > but was: NoteTest.addParagraphWithLastReplName:144 expected:<%spark > but was: ``` I think it's meaningful error and it needs to be fixed. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 @bzz I changed StringUtils is static import. Can you check the code? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 @bzz I'm worried about that `static import` confused. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1248 > @bzz ~~Thanks for comment, But could you explain more why it doesn't make sense?~~ Ah.. Your meaning is that org.apache.zeppelin.interpreter.InterpreterFactory#isBinded move to Note.java. Is it right? Yes, that is exactly what I meant @mwkang ! And while doing so, readability of the piece of the code you are changing could also be improved with static import. What do you think? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 I refactor method name. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user astroshim commented on the issue: https://github.com/apache/zeppelin/pull/1248 tested. 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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user bzz commented on the issue: https://github.com/apache/zeppelin/pull/1248 ð for having a test! Looks good to me, modulo the issue raised above. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] zeppelin issue #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...
Github user mwkang commented on the issue: https://github.com/apache/zeppelin/pull/1248 @jongyoul Can you review this PR? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---