You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by moon soo Lee <mo...@apache.org> on 2015/06/07 00:46:13 UTC
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
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 <ds...@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 <gi...@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 <mo...@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 infrastructure@apache.org or file a JIRA
> ticket
> > with INFRA.
> > ---
> >
>
Fwd: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by Dmitriy Setrakyan <ds...@apache.org>.
Can someone in the community review Zeppelin integration?
Thanks,
D.
---------- Forwarded message ----------
From: moon soo Lee <mo...@apache.org>
Date: Sun, Jun 7, 2015 at 1:46 AM
Subject: Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
To: dev@zeppelin.incubator.apache.org
Cc: "dev@ignite.incubator.apache.org" <de...@ignite.incubator.apache.org>
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 <ds...@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 <gi...@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 <mo...@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 infrastructure@apache.org or file a JIRA
> ticket
> > with INFRA.
> > ---
> >
>
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by moon soo Lee <mo...@apache.org>.
Sure, i'm taking a look now.
Thanks!
moon
On Fri, Jun 19, 2015 at 9:39 AM Dmitriy Setrakyan <ds...@apache.org>
wrote:
> 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 <le...@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 <mo...@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 <agura@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 <mo...@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 <
> > > > > > dsetrakyan@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 <
> > git@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 <mo...@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 infrastructure@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
> >
>
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by moon soo Lee <mo...@apache.org>.
Sure, i'm taking a look now.
Thanks!
moon
On Fri, Jun 19, 2015 at 9:39 AM Dmitriy Setrakyan <ds...@apache.org>
wrote:
> 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 <le...@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 <mo...@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 <agura@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 <mo...@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 <
> > > > > > dsetrakyan@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 <
> > git@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 <mo...@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 infrastructure@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
> >
>
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by Dmitriy Setrakyan <ds...@apache.org>.
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 <le...@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 <mo...@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 <mo...@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 <
> > > > > dsetrakyan@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 <
> git@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 <mo...@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 infrastructure@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
>
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by Dmitriy Setrakyan <ds...@apache.org>.
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 <le...@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 <mo...@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 <mo...@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 <
> > > > > dsetrakyan@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 <
> git@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 <mo...@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 infrastructure@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
>
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by Andrey Gura <ag...@gridgain.com>.
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 <le...@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 <mo...@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 <mo...@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 <
> > > > dsetrakyan@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 <gi...@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 <mo...@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 infrastructure@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
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by Andrey Gura <ag...@gridgain.com>.
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 <le...@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 <mo...@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 <mo...@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 <
> > > > dsetrakyan@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 <gi...@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 <mo...@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 infrastructure@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
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by moon soo Lee <le...@gmail.com>.
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 <mo...@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 <mo...@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 <
> > > dsetrakyan@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 <gi...@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 <mo...@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 infrastructure@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
>
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by Andrey Gura <ag...@gridgain.com>.
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 <mo...@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 <mo...@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 <
> > dsetrakyan@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 <gi...@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 <mo...@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 infrastructure@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
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by Andrey Gura <ag...@gridgain.com>.
I think that %ignitesql and %ignite interpreters should work differently.
%ignitesql uses JDBC driver and creates connection using URL like
jdbc:ignite://<hostname>:<port>/<cache_name>.
If I understand correctly, %ignite interpreter allows execute Scala code.
In this case interpreter should connect to Ignite topology using discovery
(static or multicast) with setting provided by user (list of addresses).
See "Network configuration" (
http://apacheignite.readme.io/v1.1/docs/network-config) for example.
I think also that in both cases interpreter should use client mode.
BTW, Ignite provides Scalar DSL, but it doesn't available from %ignite
interpreter. Only Ignite instance is available.
"Scheduler already terminated" error is gone. Thank you very much.
On Tue, Jun 9, 2015 at 4:52 AM, moon soo Lee <mo...@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 <mo...@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 <
> > dsetrakyan@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 <gi...@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 <mo...@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 infrastructure@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
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by moon soo Lee <mo...@apache.org>.
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 <mo...@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 <
> dsetrakyan@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 <gi...@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 <mo...@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 infrastructure@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
>
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by moon soo Lee <mo...@apache.org>.
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 <mo...@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 <
> dsetrakyan@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 <gi...@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 <mo...@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 infrastructure@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
>
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by Andrey Gura <ag...@gridgain.com>.
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 <mo...@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 <ds...@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 <gi...@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 <mo...@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 infrastructure@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
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by Andrey Gura <ag...@gridgain.com>.
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 <mo...@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 <ds...@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 <gi...@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 <mo...@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 infrastructure@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
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by Andrey Gura <ag...@gridgain.com>.
Moon, thanks!
I'll try it today and look to implementation.
On Sun, Jun 7, 2015 at 1:46 AM, moon soo Lee <mo...@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 <ds...@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 <gi...@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 <mo...@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 infrastructure@apache.org or file a JIRA
>> ticket
>> > with INFRA.
>> > ---
>> >
>>
>
--
Andrey Gura
GridGain Systems, Inc.
www.gridgain.com
Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add
interpreter for...
Posted by Andrey Gura <ag...@gridgain.com>.
Moon, thanks!
I'll try it today and look to implementation.
On Sun, Jun 7, 2015 at 1:46 AM, moon soo Lee <mo...@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 <ds...@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 <gi...@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 <mo...@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 infrastructure@apache.org or file a JIRA
>> ticket
>> > with INFRA.
>> > ---
>> >
>>
>
--
Andrey Gura
GridGain Systems, Inc.
www.gridgain.com