[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2016-02-24 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-188634868 @1teed please close 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

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-12-21 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-166234850 @1teed Now, I will try to contact @andrescelis to accept jdbc interpreter. I hope you would understand it. And please check the mailing(https://www.mail-arch

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-12-06 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-162319547 @1teed Hi, do you still have an interest to contribute this feature? I've found that your PR is the first one for supporting JDBC thus I think it's important

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-08-29 Thread riomario1
Github user riomario1 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-136057196 Mysql is the most awaited feature :) Sent from my iPhone > On Aug 29, 2015, at 4:18 PM, neptun2000 wrote: > > When do you plannin

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-08-29 Thread neptun2000
Github user neptun2000 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-136043139 When do you planning to release MySQL interpreter ? Thanks --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-08-11 Thread martin-g
Github user martin-g commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-129855077 At http://apache.cbox.biz/incubator/zeppelin/0.5.0-incubating/ (my neareast Apache mirror) I see two binary downloads and one source download. If you don'

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-08-07 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-128877615 > Do you pack mysql-connector.jar within the release? I think answer currently is yes, although binary packages are provided as a favor. E

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-08-07 Thread riomario1
Github user riomario1 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-128788003 Hi, How do I use MySQL with zeppelin? Please provide steps Thank you --- If your project is set up for it, you can reply to this

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-08-04 Thread martin-g
Github user martin-g commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-127580813 Do you pack mysql-connector.jar within the release? If YES then there is a problem. If the user has to provide the jar herself (e.g. by executing `mvn

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-08-03 Thread tdunning
Github user tdunning commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-127311135 Yes. The GPL license makes mysql-connector be category X. We cannot ship a release with that component, even unmodified. --- If your project is set up for

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-08-02 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-127129085 I'm not sure about merging it, while this implementation depends on `mysql-connector-java` that distributed in GPL 2 license. Please correct me if i'm wron

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-07-29 Thread rquast
Github user rquast commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-125924988 Is this getting merged still? Just tried it out and found that it could do with some default settings for the interpreter (jdbc, host, user, password, port). W

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-07-02 Thread jmrr
Github user jmrr commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-117999770 What happened to this? It hasn't been merged to master yet and I'm having problems merging it locally as the master branch is starting to diverge... Than

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-06-10 Thread jmrr
Github user jmrr commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-110856364 +1 please go ahead and merge into master On Wed, Jun 10, 2015 at 6:40 PM, riomario1 wrote: > Looks good, when are we merging it? > > —

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-06-10 Thread riomario1
Github user riomario1 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-110847719 Looks good, when are we merging it? --- 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] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-05-31 Thread jongyoul
Github user jongyoul commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-107302737 @Leemoonsoo Concerning licenses, I've heard that GPL shouldn't be included in apache projects, and known that some project in Apache also doesn't have a mysq

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-05-30 Thread ZhijieWang
Github user ZhijieWang commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-107078417 @1teed I added postgresql support and create a merge request to your repo. Limitation of this interpreter -- may crush zeppelin when running large

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-05-21 Thread jmrr
Github user jmrr commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-104373982 Are temporary tables possible with the interpreter connector? I quickly tried one and got `Can not issue data manipulation statements with executeQuery().`

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-05-12 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-101519927 @hyonaldo be4b52b changes the mysql dependency version. but it is still GPLv2 License. In my understanding, If code is depends on some GPL (depend

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-05-12 Thread jmrr
Github user jmrr commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-101208857 @1teed Thanks for the contribution. Tested with a centos-zeppeling docker image and works fine! Looking forward to the merge! --- If your project is set up for

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-05-07 Thread sylvinho81
Github user sylvinho81 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-99785556 Thanks @1teed. I had not seen the latest changes --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-05-06 Thread 1teed
Github user 1teed commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-99732488 @sylvinho81 Thank you for your interest. I think you seem to be not latest source. Please update to the latest sources. Please chec

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-05-06 Thread sylvinho81
Github user sylvinho81 commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-99417733 Hi @1teed, I am trying to apply this patch and test it, because I think is really interesting and I need it. After apply the changes and run:

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-05-06 Thread 1teed
Github user 1teed commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-99414971 @Leemoonsoo Thanks for your comment. I fix mysql dependency to meet The GNU General Public License, Version 2 in pom.xml. And add JDBCInterprete

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-05-06 Thread Leemoonsoo
Github user Leemoonsoo commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/60#issuecomment-99353415 Thanks for the contribution. For default value of zeppelin-site.xml, you'll need add JDBCInterpreter to zeppelin-zengine/src/main/java/org/apache/co

[GitHub] incubator-zeppelin pull request: Add JDBC Interpreter for Mysql

2015-05-04 Thread 1teed
GitHub user 1teed opened a pull request: https://github.com/apache/incubator-zeppelin/pull/60 Add JDBC Interpreter for Mysql I added jdbc interpreter for mysql I used Abstract Factory Pattern It has the advantage that can be separated using the JDBC Connection only argu