Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-189144007
I think it is "complete" and "working" -- the convenience binary is not
part of the release, and a new one isn't going to be made until there is a new
releas
GitHub user zhongneu opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/749
Remove duplicated java option concats in common.sh
### What is this PR for?
There are some java option concats in common.sh, which are executed twice
when start an interpreter. This
Zhong Wang created ZEPPELIN-702:
---
Summary: Duplicated java option concats in common.sh
Key: ZEPPELIN-702
URL: https://issues.apache.org/jira/browse/ZEPPELIN-702
Project: Zeppelin
Issue Type: Bu
Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-189140201
@elbamos please ping me if you need any help adding a profile for R in
order to avoid failing default Zeppelin build for users who do not need R.
---
If your p
Github user jeffsteinmetz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-189133244
Nothing I need pre-merge.
I was mainly curious how close this was to working end to end.
I wasn't familiar with the pre-merge / post-merge ex
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/703#issuecomment-189130930
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 feat
Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-189129155
Jeff - well that log explains it, the R package isn't being included in the
distribution binary.
I'm not sure I understand either the goal or the "
Hi devs,
I suggest the method of running Zeppelin's interpreters in a cluster mode.
Review this proposal and give me comments. I will start from supporting
standalone cluster.
https://cwiki.apache.org/confluence/display/ZEPPELIN/Cluster+Manager+Proposal
Regards,
Jongyoul
--
이종열, Jongyoul Lee,
Github user zhongneu commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/713#issuecomment-189120617
All tasks are done. Ready for final review
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. I
Github user jeffsteinmetz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-189117683
ah, I see. Well, if the pom's aren't setup to include it in the
distribution binary that certainly the issue.
I didn't set SPARK_HOME intenti
Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-189116032
@jeffsteinmetz A couple of things are happening here --
First, I would expect it to fail if you run it from zeppelin-distribution
because the pom's
Github user jeffsteinmetz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-189102577
@elbamos I tested a full build of a Zeppelin distribution using this PR in
a virtual machine.
When running from the source directory (not the distri
Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/276#issuecomment-189099153
@runyontr thanks for standing up!
> Given that I can fix the list above, how do I update this pull request
with my repository?
It's not really p
Github user runyontr commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/276#issuecomment-189093591
@felixcheung I'll be happy to take this up. From what I can see there is
the following left:
- Change string creation to use StringBuilder
- Th
Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/693#issuecomment-189077149
Merging if there is 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
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/738
---
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 fea
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/727#issuecomment-189016612
Maybe pre-caching is the only way to make this more stable.. I have seen
more than a few times download fails even after 3 tries either from timing out
o
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-189009246
that we would need a job run status API...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/748#issuecomment-18857
Then lets remove the bookmarklet and update the template in the
contributing.md
---
If your project is set up for it, you can reply to this email and hav
Github user doanduyhai commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-188999787
Humm, your remark about future extension of the existing API with new
parameter makes sense. We need to design API carefully to make evolution easier.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-188989671
I don't have strong opinion here for adding parameter vs new api.
But i want to mention that adding parameter make the api return two
different things.
Github user doanduyhai commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/744#issuecomment-188981265
Right, didn't know that we had ngToast. I never see it in action. Will
remove angular-growl-2
---
If your project is set up for it, you can reply to thi
Github user doanduyhai commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-188977035
@vgromakowski
Something like
`http://:/api/notebook/job//?sync=true` ?
---
If your project is set up for it, you can reply to this email and ha
Greetings, {podling} developers!
This is a reminder that your report is due next Wednesday, March
2nd. Details below.
Best,
Marvin Humphrey, Report Manager for March, on behalf of the
Incubator PMC
---
Dear podling,
This email was sent by an automated system on behalf of the Apac
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/748#issuecomment-188932693
I think it's useful to have that in contributing.md so newcomers would have
an idea what to expect before actually opening a new PR. We might want to leav
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/738#issuecomment-188868549
I'm merging it into the master.
---
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 pr
Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/744#issuecomment-188856053
We already have ngtoast for that
---
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 p
Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/748#issuecomment-188853739
Should we remove the PR template part from the Contributing.md then?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user vgromakowski commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-188844963
I would suggest to use the same method as for the spark job server,
asynchronous is the default, synchronous is obtain with an additional parameter.
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-188842837
@doanduyhai Right.
for asynchronous behavior, multiple API is required to control the job,
like submitting job (POST), canceling job (DELETE), ge
Github user doanduyhai commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-188828325
@Leemoonsoo so you're suggesting to create another API for pagraph
execution ? Fine for me, what's about
`http://:/api/notebook/synchronous_job//` ?
---
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/748#issuecomment-188816748
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 fe
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-188816674
This PR change behavior of REST api `job/{notebookId}/{paragraphId}` from
asynchronous to synchronous.
To me it's better we keep asynchronous beha
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/86#issuecomment-188806844
Can someone provide additional review?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user doanduyhai commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-188791929
Fixed
---
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 f
Mauro Cortellazzi created ZEPPELIN-701:
--
Summary: Upgrade Tachyon Interpreter to Alluxio Interpreter
Key: ZEPPELIN-701
URL: https://issues.apache.org/jira/browse/ZEPPELIN-701
Project: Zeppelin
Github user vgromakowski commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/746#issuecomment-188683473
I would prefer to not return OK in case of failure
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/748#issuecomment-188663996
yea ;)
---
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
Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/748#issuecomment-188663789
CI failure is due to
```
Unapproved licenses:
.github/PULL_REQUEST_TEMPLATE
***
```
.. but you got
Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/748#issuecomment-188663097
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
40 matches
Mail list logo
