Moon, Is there any way you can take a look at the Ignite pull request soon?
Thanks, D. On Thu, Jun 18, 2015 at 9:17 AM, Andrey Gura <ag...@gridgain.com> wrote: > Hi, Moon > > I have pull request with implementation of Apache Ignite interpreters ( > https://github.com/apache/incubator-zeppelin/pull/108). > Could you please review it. > > Thanks. > > On Sat, Jun 13, 2015 at 8:32 PM, moon soo Lee <leemoon...@gmail.com> > wrote: > > > Hi, Andrey > > > > Sure, that would be awesome! > > > > Thanks! > > moon > > > > On Thu, Jun 11, 2015 at 6:17 AM Andrey Gura <ag...@gridgain.com> wrote: > > > > > Moon, > > > > > > Can I join to you and provide patch for ignite branch? I believe it > > faster > > > way to finish Zepplein-Ignite integration. > > > > > > Thanks, > > > Andrey > > > > > > On Tue, Jun 9, 2015 at 4:52 AM, moon soo Lee <m...@apache.org> wrote: > > > > > > > Hi, > > > > > > > > Thanks for the feedback. > > > > > > > > I was thinking it make sense to have one connection address for > %ignite > > > and > > > > %ignitesql. what do you think? which style of connection address > would > > > more > > > > familiar to ignite users? > > > > > > > > Scheduler already terminated error has been fixed and i just rebased > > > ignite > > > > branch to master. It should be fixed by now. > > > > > > > > Let me look in to error propagation. > > > > > > > > Thanks, > > > > moon > > > > > > > > > > > > On Mon, Jun 8, 2015 at 5:47 PM Andrey Gura <ag...@gridgain.com> > wrote: > > > > > > > > > Hi, > > > > > > > > > > I looked for the Ignite SQL Interpreter and it works. Awesome! :) > > > > > > > > > > Unfortunately I was not able to run ignite (scalar version). Moon, > > > could > > > > > you please provide your test example? > > > > > > > > > > Some notes/comments: > > > > > > > > > > 1. Maybe it would be better if Ignite URL will look like JDBC URL > for > > > > > consistency with other inerpreters like Tajo or Hive? Or is it > > > deliberate > > > > > decision in order to provide one URL for both (%ignite and > > %ignitesql) > > > > > interpreters? > > > > > > > > > > 2. "Scheduler already terminated" error occurs after configuration > > > > settings > > > > > changes and interpreter restart. Is it normal behavior? Only > restart > > of > > > > > Zeppelin daemon fix this problem. > > > > > > > > > > 3. Interpreter doesn't retrieve any error if SQL query refers to > > > unknown > > > > > table. > > > > > > > > > > Thanks, > > > > > Andrey. > > > > > > > > > > On Mon, Jun 8, 2015 at 12:40 PM, Andrey Gura <ag...@gridgain.com> > > > wrote: > > > > > > > > > > > Moon, thanks! > > > > > > > > > > > > I'll try it today and look to implementation. > > > > > > > > > > > > On Sun, Jun 7, 2015 at 1:46 AM, moon soo Lee <m...@apache.org> > > > wrote: > > > > > > > > > > > >> Hi, > > > > > >> > > > > > >> It's okay to try jdbc based one using %ignitesql. > > > > > >> > > > > > >> %ignite is the one for ignite API, which embeds scala REPL. > > > > > >> 'ignite' is automatically injected. if you can try this one and > > give > > > > any > > > > > >> feedback, it'll be very much appreciated. > > > > > >> > > > > > >> > > > > > >> Thanks, > > > > > >> moon > > > > > >> > > > > > >> On Tue, May 26, 2015 at 3:07 AM Dmitriy Setrakyan < > > > > > dsetrak...@apache.org> > > > > > >> wrote: > > > > > >> > > > > > >>> Moon, > > > > > >>> > > > > > >>> Do you think we can try this out already, or is it still too > > early? > > > > > >>> > > > > > >>> D. > > > > > >>> > > > > > >>> On Sat, May 23, 2015 at 1:45 AM, Leemoonsoo < > g...@git.apache.org> > > > > > wrote: > > > > > >>> > > > > > >>> > GitHub user Leemoonsoo opened a pull request: > > > > > >>> > > > > > > >>> > https://github.com/apache/incubator-zeppelin/pull/78 > > > > > >>> > > > > > > >>> > [ZEPPELIN-63] add interpreter for Apache Ignite > > > > > >>> > > > > > > >>> > Currently it is very early stage of implementation. > > > > > >>> > My idea is implementing 2 interpreter for ignite, > > > > > >>> > > > > > > >>> > * [ ] IgniteInterpreter - Let user use ignite API > > (scala) > > > by > > > > > >>> > embedding scala compiler. > > > > > >>> > * [ ] IgniteSqlInterpreter - use JDBC driver to make > sql > > > > query > > > > > >>> > > > > > > >>> > While IgniteSqlInterpreter can be simply implemented > using > > > JDBC > > > > > >>> > driver, i'd like to work on IgniteSqlInterpreter first and > than > > > > work > > > > > on > > > > > >>> > IgniteInterpreter. > > > > > >>> > > > > > > >>> > Work in progress .. > > > > > >>> > > > > > > >>> > You can merge this pull request into a Git repository by > > running: > > > > > >>> > > > > > > >>> > $ git pull > > https://github.com/Leemoonsoo/incubator-zeppelin > > > > > ignite > > > > > >>> > > > > > > >>> > Alternatively you can review and apply these changes as the > > patch > > > > at: > > > > > >>> > > > > > > >>> > > https://github.com/apache/incubator-zeppelin/pull/78.patch > > > > > >>> > > > > > > >>> > To close this pull request, make a commit to your > master/trunk > > > > branch > > > > > >>> > with (at least) the following in the commit message: > > > > > >>> > > > > > > >>> > This closes #78 > > > > > >>> > > > > > > >>> > ---- > > > > > >>> > commit 10654215bdf0cf52a06887aca8a8c4cd38eeee37 > > > > > >>> > Author: Lee moon soo <m...@apache.org> > > > > > >>> > Date: 2015-05-23T08:25:30Z > > > > > >>> > > > > > > >>> > add interpreter for Apache Ignite > > > > > >>> > > > > > > >>> > ---- > > > > > >>> > > > > > > >>> > > > > > > >>> > --- > > > > > >>> > 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. > > > > > >>> > --- > > > > > >>> > > > > > > >>> > > > > > >> > > > > > > > > > > > > > > > > > > -- > > > > > > Andrey Gura > > > > > > GridGain Systems, Inc. > > > > > > www.gridgain.com > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > Andrey Gura > > > > > GridGain Systems, Inc. > > > > > www.gridgain.com > > > > > > > > > > > > > > > > > > > > > -- > > > Andrey Gura > > > GridGain Systems, Inc. > > > www.gridgain.com > > > > > > > > > -- > Andrey Gura > GridGain Systems, Inc. > www.gridgain.com >
