Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-202727314
@jeffsteinmetz I had a typo in the default template, it should be fixed now
and %r show up. Thanks for catching!
You are correct that build-distr wo
I’m not affiliated to either author nor have any attachment to an specific
outcome - and happy to continue being as objective and unbiased as possible.
I would say they now have different philosophical approaches (as of the March
23rd merge of datalayer#7 to 702).
I agree with Amos Elberg tha
Github user AhyoungRyu commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/801#issuecomment-202726499
@prabhjyotsingh Absolutely better than before :+1:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/798
---
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
+1 on merging 208...
I am also using 208 repo from long time...
Regards,
Ankur
-Original Message-
From: Joel Van Veluwen [mailto:joel.vanvelu...@quantium.com.au]
Sent: 29 March 2016 07:21 AM
To: dev@zeppelin.incubator.apache.org
Subject: RE: [DISCUSS] Back to PRs 208 & 702...
+1
We've
Github user felizbear commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/796#issuecomment-202717837
Maybe you should do `bower install`?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If you
Github user johnnyws commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/796#issuecomment-202716213
It seems some recent change breaks the karma unit tests. I saw such
failures while running unit tests:
```
Error: [$injector:modulerr] Failed to in
Github user jeffsteinmetz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-202711880
As a reminder, building the distribution with `-Pbuild-distr` doesn't
package the interpreter as expected.
---
If your project is set up for it, you c
Github user jeffsteinmetz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-202711574
Should the interpreter list %r in the header?
Right now it shows %knitr, but not %r
**spark** %spark (default) , %sql , %knitr , %pyspa
Github user johnnyws commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/797#issuecomment-202698867
@felixcheung sorry for the delay. I've uploaded the screenshots
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-202695901
Things that left to be done in order to merge this PR:
- [x] ~~master and this branch have diverged: we need to either merge
master in here, or rebase i
Github user prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/801#issuecomment-202691934
Thanks for the feedback @Leemoonsoo, have restrict access to notebook(s)
without login.
---
If your project is set up for it, you can reply to this e
Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-202682652
The push earlier tonight *should* (if it doesn't, let me know) resolve all
comments I've received about this PR.
It currently passes CI for Spark 1.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/784
---
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
+1
We've been super keen for 208 to be merged for some time at my organisation. I
think if 702 is really adding value then we can assess it once 208 has been
merged.
Amos has put a lot of work into this and it delivers a feature that many
Zeppelin users have wanted.
Can we finally merge and
All said and done we had enough discussion on this point for many months
now. As far as I know, 208 is the PR which community/people have so far
used mostly and successfully (at least me and whoever I introduced to 208
for SparkR support). I thought it was going to be merged a long time ago.
May b
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/803#issuecomment-202604280
looks good to me. any more comment?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If yo
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/803#issuecomment-202604105
test failure was:
```
+tar zxf spark-1.3.1-bin-hadoop2.3.tgz
gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Err
Github user rawkintrevo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/794#issuecomment-202599734
After reviewing the other InterpreterTest files, I find that none utilize a
test where the output '.contains' a specific string. That specific test in
I
Sarah Quigley created ZEPPELIN-771:
--
Summary: GUI issue displaying unicode characters
Key: ZEPPELIN-771
URL: https://issues.apache.org/jira/browse/ZEPPELIN-771
Project: Zeppelin
Issue Type:
Github user jeffsteinmetz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-202590457
This Virtual Machine PR
https://github.com/apache/incubator-zeppelin/pull/751 has also been updated to
support R and all libraries required for the exa
Github user jeffsteinmetz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-202583441
This still shows `Error` (paragraph header upper right), but doesn't really
break anything per se:
```
%r
foo <-1
```
---
If
Github user jeffsteinmetz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-202583079
Should `%r` be added to the interpreter headers?
See the following attachments:
https://cloud.githubusercontent.com/assets/5488059/14090453/44342
Github user jeffsteinmetz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-202582736
I tested the build and distribution as follows:
`mvn clean package -Pspark-1.6 -Ppyspark -Phadoop-2.4 -Psparkr -DskipTests
-Pbuild-distr`
T
Github user fireboy1919 commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/782#issuecomment-202581884
The second one.
And I have a solution that does that in this pull request now.
I didn't remove the interpreter serializer from this pull
Diving into the mail archive, I found this one :
https://mail-archives.apache.org/mod_mbox/incubator-zeppelin-dev/201508.mbox/%3CCABePtM0cu4SZSMJLq=steckawup3i_5x9xhjw4fmxtg4s5i...@mail.gmail.com%3E
Followed the advice: using HTML interpreter type instead of SVG. It's
working (well I still need to
Github user cjmatta commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/803#issuecomment-202570853
@felixcheung As far as I know they work without mapr-sepcific libraries.
---
If your project is set up for it, you can reply to this email and have your
repl
Hi Eric,
thx for the prompt answer. much appreciated.Simply said I tried to load a
basic SVG sample from a static file as interpreter output, just to see how
it behave (code sample http://pastebin.com/vaC0B3M5 )
I got nothing in the notebook (no error, no rendering of the SVG). A new
empty paragr
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/803#issuecomment-202556152
Would other components like HBase, Hive work with MapR 5.1?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user rawkintrevo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/794#issuecomment-202538580
I moved the test to the end on a hunch that max fix the build issue.
@Leemoonsoo can you run again on your machine? I think the code is solid- it
just s
GitHub user cjmatta opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/803
[ZEPPELIN-770] Add support for MapR 5.1
### What is this PR for?
Adding support for MapR 5.1 via profile
### What type of PR is it?
Improvement
### Todos
GitHub user rawkintrevo reopened a pull request:
https://github.com/apache/incubator-zeppelin/pull/794
[ZEPPELIN-760] Companion object bug fix for Flink Interpretter
### What is this PR for?
This implements the solution presented for the Spark interpreter in PR-780
for the Flink
Github user rawkintrevo closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/794
---
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 th
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/794#issuecomment-202517074
@rawkintrevo Please feel free to confine this change if it would help.
We could follow up separately for Ignite.
---
If your project is set up for i
Chris Matta created ZEPPELIN-770:
Summary: Need MapR 5.1 Build profile
Key: ZEPPELIN-770
URL: https://issues.apache.org/jira/browse/ZEPPELIN-770
Project: Zeppelin
Issue Type: Bug
Co
I would go into SVG what you want to display can be achieved with SVG.
Can you share the issue you encounter?
On 28/03/16 18:17, Gérard Dupont wrote:
Hi guys,
I'm currently working (night & week-end not full time) on a neo4J
interpreter to try q=and test few things with zeppelin. For context : n
Github user EronWright commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/795#issuecomment-202470891
How about the options are renamed as follows:
- "shared" -> "shared"
- "instance" -> "scoped"
- "process" -> "isolated"
---
If your
Hi guys,
I'm currently working (night & week-end not full time) on a neo4J
interpreter to try q=and test few things with zeppelin. For context : neo4j
is a graph database which is fun to work with.
So far, everything good and I managed to get json results in the notebook.
Next step would be to int
Github user rawkintrevo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/794#issuecomment-202463668
I really don't. I don't know much of anything about Ignite. I really was
only doing this for Flink, but Felix asked that I apply the fix to
Scalding/Ig
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/782#issuecomment-202459832
Thanks for the understanding.
I think term 'resource', 'remote resource' are bit confusing in this
conversation. if i define,
Resource -
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/801#issuecomment-202445244
@prabhjyotsingh Cool improvement!
I tried. And found that i can access notebook without login after enables
authentication by editing shiro.ini.
Github user fireboy1919 commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/782#issuecomment-202411857
That makes sense. The issue is that "ResourcePoolUtils" currently doesn't
use that enhanced method, and if you're trying to get a specific resource, that
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/802
replace standard alert and confirm with BootstrapDialog
### What is this PR for?
Replace standard alert and confirm with BootstrapDialog.
Most of these were already take ca
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/801
Replace standard basic-auth dialog with BootstrapDialog for user
authentication
### What is this PR for?
Instead of showing standard basic-auth dialog, replace the same with
Prabhjyot Singh created ZEPPELIN-769:
Summary: Replace standard basic-auth dialog with BootstrapDialog
for user authentication
Key: ZEPPELIN-769
URL: https://issues.apache.org/jira/browse/ZEPPELIN-769
@Elberg, If I were you I would ask myself why isn't the community taking
part in this debate?
Personally I prefer a team player as a contributor over the best developer.
just my 2c
Eran
On Mon, 28 Mar 2016 at 09:52 Amos B. Elberg wrote:
> Moon - I opened this discussion so it could take place wi
46 matches
Mail list logo