ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From moon soo Lee <m...@apache.org>
Subject Re: [GitHub] incubator-zeppelin pull request: [ZEPPELIN-63] add interpreter for...
Date Fri, 19 Jun 2015 17:03:11 GMT
Sure, i'm taking a look now.

Thanks!
moon

On Fri, Jun 19, 2015 at 9:39 AM Dmitriy Setrakyan <dsetrakyan@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 <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