[GitHub] incubator-zeppelin pull request: [WIP][Zeppelin-630] Introduce new...

Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/673#issuecomment-174956596 Awesome --- 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

[GitHub] incubator-zeppelin pull request: Add tachyon interpreter

Github user maocorte commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/632#issuecomment-174931772 Changed format of tachyon documentation to new guidelines and added link to navigation page. About interpreter response management, it reflects the

[GitHub] incubator-zeppelin pull request: Docs for jdbc interpreter

Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/661#issuecomment-174913559 No problem --- 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

[GitHub] incubator-zeppelin pull request: Docs for jdbc interpreter

Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/661#issuecomment-174906314 To me there is still plenty of things missing in the doc: * Database part is not mentioned anywhere (it seems it is part of the url) * `The

[GitHub] incubator-zeppelin pull request: Docs for jdbc interpreter

Github user vgmartinez commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/661#issuecomment-174911726 now I'm busy at work in 1 to 2 days to complete itthanks for review --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-zeppelin pull request: [WIP][Zeppelin-630] Introduce new...

Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/673#issuecomment-174947906 There are some bugs. 1. It seems the URL string does not apply. - as you can see, URL value to be blanked.

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/666#issuecomment-175448598 @Leemoonsoo 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

[jira] [Created] (ZEPPELIN-634) Build failure due to latest bower (1.7.5)

Renjith Kamath created ZEPPELIN-634: --- Summary: Build failure due to latest bower (1.7.5) Key: ZEPPELIN-634 URL: https://issues.apache.org/jira/browse/ZEPPELIN-634 Project: Zeppelin Issue

[GitHub] incubator-zeppelin pull request: Fix build failure due to latest b...

GitHub user r-kamath opened a pull request: https://github.com/apache/incubator-zeppelin/pull/677 Fix build failure due to latest bower (1.7.5) ### What is this PR for? Build fails due to new bower version 1.7.5. This PR tries to fix this issue by adding the previous version

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-585] Add new doc on how...

Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/672#issuecomment-175451510 @felixcheung Shamed. You're right. A verb is needed after `to`. I pushed it again : ) --- If your project is set up for it, you can reply to this email

[GitHub] incubator-zeppelin pull request: [Hot Fix] Fix build failure due t...

Github user r-kamath commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/677#issuecomment-175465448 Travis build failure is not related to this change https://cloud.githubusercontent.com/assets/2031306/12606910/1cf5d024-c4f7-11e5-99d9-0bcc6d3569f7.png;>

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-585] Add new doc on how...

Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/672#issuecomment-175445758 "Zeppelin needs to `be` reachable for that Website" is probably better --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-zeppelin pull request: Shell interpreter improvements

Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/666#issuecomment-175465193 It looks good to me. Thanks @Karuppayya for the improvement. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-zeppelin pull request: [Hot Fix] Fix build failure due t...

Github user asfgit closed the pull request at: https://github.com/apache/incubator-zeppelin/pull/677 --- 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] incubator-zeppelin pull request: Docs for jdbc interpreter

Github user corneadoug commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/661#issuecomment-174889645 @vgmartinez where is __postgres__ coming from inside `jdbc(postgres)`? Is it the name used to create the interpreter in the interpreter tab? --- If

[GitHub] incubator-zeppelin pull request: Docs for jdbc interpreter

Github user vgmartinez commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/661#issuecomment-174887167 Hi @corneadoug, Yes, you need create a diferent configurations for each db you useand you can use multiple configuration in the same notebook this

[GitHub] incubator-zeppelin pull request: Docs for jdbc interpreter

Github user vgmartinez commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/661#issuecomment-174890826 To create the configuration is like this:

[GitHub] incubator-zeppelin pull request: [ZEPPELIN-585] Add new doc on how...

Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/672#issuecomment-175056632 @corneadoug I applied your first issue in documentation. However, so far, I'm not sure how we can deal with your second issue. I just addressed

[jira] [Created] (ZEPPELIN-633) SQL with function statement fails using JDBC

Chris Matta created ZEPPELIN-633: Summary: SQL with function statement fails using JDBC Key: ZEPPELIN-633 URL: https://issues.apache.org/jira/browse/ZEPPELIN-633 Project: Zeppelin Issue

[GitHub] incubator-zeppelin pull request: Add tachyon interpreter

Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/632#issuecomment-175192708 Thanks @AhyoungRyu could you please take a quick look at the doc too? --- If your project is set up for it, you can reply to this email and have your

[GitHub] incubator-zeppelin pull request: ZEPPELIN-372 : Exporting a notebo...

Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/376#issuecomment-175193771 @bpirvu - it's probably best if you email the dev list (https://zeppelin.incubator.apache.org/community.html) for questions like this in the future - PR

[GitHub] incubator-zeppelin pull request: [WIP][Zeppelin-630] Introduce new...

Github user minahlee commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/673#issuecomment-175231786 @astroshim, @anthonycorbacho Thank you for the review. I pushed commits to address your comment. Please take a look into this again. --- If your project

[GitHub] incubator-zeppelin pull request: [WIP][Zeppelin-630] Introduce new...

Github user felixcheung commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/673#issuecomment-175232140 Would it be possible to define dependency for a notebook? I suspect some users are currently having a %dep paragraph at the top of the notebook to bring

[GitHub] incubator-zeppelin pull request: [WIP][Zeppelin-630] Introduce new...

Github user minahlee commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/673#issuecomment-175241376 @felixcheung yes load dependency using %dep still works, and I simply added the deprecated message on the use of `z.load()`, `z.reset()`, `z.addRepo()`

[GitHub] incubator-zeppelin pull request: Add tachyon interpreter

Github user AhyoungRyu commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/632#issuecomment-175352458 Hi @maocorte : ) It's really good news that we have Tachyon interpreter ! How about adding a basic Tachyon example code and result image(if

[GitHub] incubator-zeppelin pull request: [WIP][Zeppelin-630] Introduce new...

Github user astroshim commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/673#issuecomment-175354290 Tested and working well. :+1: Cool! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.