ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dmitriy Setrakyan <dsetrak...@apache.org>
Subject Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add interpreter for...
Date Fri, 19 Jun 2015 16:38:41 GMT
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 <agura@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 <leemoonsoo@gmail.com>
> wrote:
>
> > Hi, Andrey
> >
> > Sure, that would be awesome!
> >
> > Thanks!
> > moon
> >
> > On Thu, Jun 11, 2015 at 6:17 AM Andrey Gura <agura@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 <moon@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 <agura@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 <moon@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 <moon@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
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message