Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1990
Thanks for taking care of @Leemoonsoo :+1:
---
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 featu
Thanks moon.
moon soo Lee 于2017年2月10日周五 下午2:21写道：
> Done.
>
> On Fri, Feb 10, 2017 at 3:19 PM Jeff Zhang wrote:
>
> > Yes
> >
> >
> > moon soo Lee 于2017年2月10日周五 下午2:17写道：
> >
> > > Your cwiki id is zjffdu, right?
> > >
> > > On Fri, Feb 10, 2017 at 11:05 AM Jeff Zhang wrote:
> > >
> > > > Hi
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1990
Thanks @soralee for the fix!
Tested and LGTM. CI failure looks not related.
Merge to master if no further discussions.
---
If your project is set up for it, you can reply to this e
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1978
Merging this to master and branch-0.7 if no more discussion.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user 1ambda closed the pull request at:
https://github.com/apache/zeppelin/pull/1995
---
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 e
GitHub user 1ambda reopened a pull request:
https://github.com/apache/zeppelin/pull/1994
[MINOR] Replace angular inject with ngInject
### What is this PR for?
Replaced angular inject with ngInject to reduce duplication and keep
simple. There is no reason to use angular inje
Github user 1ambda closed the pull request at:
https://github.com/apache/zeppelin/pull/1994
---
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 e
GitHub user 1ambda reopened a pull request:
https://github.com/apache/zeppelin/pull/1995
[ZEPPELIN-2086] Move zeppelin-web related test files into src/ for
efficiency
### What is this PR for?
Since we are using a separated test directory,
- it's uncomfortable to c
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1988
Thanks. LGTM. If CI finished and become green, will merge it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Done.
On Fri, Feb 10, 2017 at 3:19 PM Jeff Zhang wrote:
> Yes
>
>
> moon soo Lee 于2017年2月10日周五 下午2:17写道：
>
> > Your cwiki id is zjffdu, right?
> >
> > On Fri, Feb 10, 2017 at 11:05 AM Jeff Zhang wrote:
> >
> > > Hi I am working on PIG-5126 to add document about running pig in
> > zeppelin.
> >
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1980
@felixcheung Is this okay to go?
---
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
enab
Yes
moon soo Lee 于2017年2月10日周五 下午2:17写道：
> Your cwiki id is zjffdu, right?
>
> On Fri, Feb 10, 2017 at 11:05 AM Jeff Zhang wrote:
>
> > Hi I am working on PIG-5126 to add document about running pig in
> zeppelin.
> > But it is not good practise to include corporate link in apache project.
> So
Your cwiki id is zjffdu, right?
On Fri, Feb 10, 2017 at 11:05 AM Jeff Zhang wrote:
> Hi I am working on PIG-5126 to add document about running pig in zeppelin.
> But it is not good practise to include corporate link in apache project. So
> I would like to copy the content of below link to zeppel
Github user DmytroShkvyra commented on the issue:
https://github.com/apache/zeppelin/pull/2002
I have had similar issues. The main problem with this plugin. It can run
scripts in package.json via shell (postinstall, pretest etc) but not provide
that they accessible for current shell
Hoon Park created ZEPPELIN-2098:
---
Summary: Weird progress bar position when editor is hidden
Key: ZEPPELIN-2098
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2098
Project: Zeppelin
Issue
Hoon Park created ZEPPELIN-2097:
---
Summary: artifact field in Helium JSON doens't support the
relative path
Key: ZEPPELIN-2097
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2097
Project: Zeppelin
Github user yiming187 commented on the issue:
https://github.com/apache/zeppelin/pull/1986
Thanks, Moon.
---
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
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1986
CI failure looks not related.
LGTM and merge to master if no further discussions
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1886
LGTM and merge to master if no more discussions.
---
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 th
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1998
Looks great to me!
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1992
---
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 e
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1892
---
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 e
Jongyoul Lee created ZEPPELIN-2096:
--
Summary: Change LazyInterpreter to ClassloaderInterpreter when
creating interpreter into RemoteInterpreterServer
Key: ZEPPELIN-2096
URL: https://issues.apache.org/jira/browse/
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1911
@DrIgor I agree with you. It's very easily broken. But on the other hands,
I will make a new PR for changing `ClassloaderInterpreter` because another
interpreter will fails with same issue if it
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1886
@sotnich
now Well working!
good works!
Thank you for great feature!
 | After (rough avg
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1997
Tested and works as expected.
---
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
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2000
Looks nice :)
---
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
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/2000
Hi~ @AhyoungRyu @1ambda
I updated this as your comment!
Thanks for that could you review again, please? :smiley:
---
If your project is set up for it, you can reply to this email and ha
Kevin (Sangwoo) Kim created ZEPPELIN-2093:
-
Summary: Simple Spark code is not running with shell-defined class
Key: ZEPPELIN-2093
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2093
Projec
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1999
@jongyoul Thanks for verification!
Only one CI test case failed because of timeout. Merge if there are no more
comments.
---
If your project is set up for it, you can reply to this email
Hi I am working on PIG-5126 to add document about running pig in zeppelin.
But it is not good practise to include corporate link in apache project. So
I would like to copy the content of below link to zeppelin wiki page and
refer the zeppelin wiki in pig doc.
https://community.hortonworks.com/arti
AFAIK, no one has permission to cancel the job except for ASF-INFRA.
On Fri, Feb 10, 2017 at 9:56 AM Jeff Zhang wrote:
> There's lots of pending build job in travis build queue, I can cancel some
> of my build as I would push a new commit. But I didn't find way to cancel
> that. Does anyone know
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/2000
Thanks for review! @AhyoungRyu @1ambda
I'll update this soon!
---
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 d
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2002
FYI - https://github.com/eirslett/frontend-maven-plugin/issues/562
---
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 do
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1952
---
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 e
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2000
Hi @soralee :)
I would like to suggest using `'ngInject'l` instead of `$inject =
['$location'];` as like https://github.com/apache/zeppelin/pull/1994
>
http://zeppelin.apache.org/
Github user 1ambda closed the pull request at:
https://github.com/apache/zeppelin/pull/1991
---
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 e
GitHub user 1ambda reopened a pull request:
https://github.com/apache/zeppelin/pull/1991
[ZEPPELIN-2081] Prettify input, textarea in interpreter setting page
### What is this PR for?
Fix small details in interpreter setting page about inputboxes, textareas
- Set ali
There's lots of pending build job in travis build queue, I can cancel some
of my build as I would push a new commit. But I didn't find way to cancel
that. Does anyone know how to cancel it ? Thanks
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1992
LGTM
---
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
GitHub user DmytroShkvyra opened a pull request:
https://github.com/apache/zeppelin/pull/2002
[ZEPPELIN-1928]: Get a rid of preinstalled external dependencies (nodejs
and npm) for maven build
### What is this PR for?
Get a rid of preinstalled external dependencies (nodejs and np
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1992
LGTM
---
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 fe
Jongyoul Lee created ZEPPELIN-2092:
--
Summary: Introduce new class to maintain interpreter processes
Key: ZEPPELIN-2092
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2092
Project: Zeppelin
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1993
---
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 e
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1999
Looks nice
---
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
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/2001
ZEPPELIN-2057 Extract InterpreterSetting functions from InterpreterFactory
### What is this PR for?
Reducing size of InterpreterFactory and divide some functions from
InterpreterFactory. Cur
GitHub user 1ambda reopened a pull request:
https://github.com/apache/zeppelin/pull/1995
[ZEPPELIN-2086] Move zeppelin-web related test files into src/ for
efficiency
### What is this PR for?
Since we are using a separated test directory,
- it's uncomfortable to c
Github user 1ambda closed the pull request at:
https://github.com/apache/zeppelin/pull/1995
---
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 e
Github user 1ambda closed the pull request at:
https://github.com/apache/zeppelin/pull/1994
---
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 e
GitHub user 1ambda reopened a pull request:
https://github.com/apache/zeppelin/pull/1994
[MINOR] Replace angular inject with ngInject
### What is this PR for?
Replaced angular inject with ngInject to reduce duplication and keep
simple. There is no reason to use angular inje
Hi all,
I have a question about working with a 'Job' class. Why do we change a job
status to RUNNING, FINISHED, ERROR, ABORTED outside of the class (mostly from
schedulers)?
This approach makes impossible to synchronize 'Job' status changing with an
actual 'Job' state (final result, finished t
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/2000
I found another typo in
[contribution/zeppelinweb/goodPracticeGuide04.html](http://zeppelin.apache.org/contribution/zeppelinweb/goodPracticeGuide04.html).
Could you fix this one as well? It sho
Sora Lee created ZEPPELIN-2091:
--
Summary: should not show another paragraph in "Link this paragraph"
Key: ZEPPELIN-2091
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2091
Project: Zeppelin
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/2000
@AhyoungRyu @1ambda Could you review this, please whether I missed
something or not?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
GitHub user soralee opened a pull request:
https://github.com/apache/zeppelin/pull/2000
[gh-pages] update contribution/webapplication.html
### What is this PR for?
This PR is for matching current web environment.
I changed `dev mode` part from `./grunt serve` to `yarn run de
Github user sotnich commented on the issue:
https://github.com/apache/zeppelin/pull/1886
@cloverhearts @astroshim, check it now, please. If it's still not working
send me details: configuration of the interpreter and jdbc log file.
---
If your project is set up for it, you can reply
GitHub user AhyoungRyu opened a pull request:
https://github.com/apache/zeppelin/pull/1999
[MINOR][ZEPPELIN-2090] Remove "zeppelin.interpreters" property related
guide msg
### What is this PR for?
As we won't support `zeppelin.interpreters` property anymore (from
`0.7.0`), the
61 matches
Mail list logo
