Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/671#issuecomment-188129729
@peeyar Wrong PR?
---
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 h
Github user jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/703#issuecomment-188128970
@Leemoonsoo As I mentioned in some codes, how about passing
InterpreterContext to the RemoteInterpreterServer? It would be useful and we
don't any any other
Github user hsaputra commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-188128964
Oh my goodness @elbamos , you have so many aliases =P
I apologize about the confusion.
---
If your project is set up for it, you can reply to this email
Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-188128084
I just pushed fixes for the issue spotted by @jeffsteinmetz, the extraneous
R Studio file, and (I think) the misalligned function. @hsaputra can you
explain
Github user khalidhuseynov commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/703#issuecomment-188122438
I see, that makes sense. CI is green, LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as w
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/737
---
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 asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/743
---
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 asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/720
---
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 Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/703#issuecomment-188110608
@felixcheung Thanks for the review. I have addressed the comments.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/703#issuecomment-188103224
@khalidhuseynov Thanks for the opinion. Both UX and Performance point of
view, which make sense to be default will be totally depends on the
interpreters.
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/703#issuecomment-188101021
big change - only a few minor comments.
---
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 jongyoul commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/712#issuecomment-188084114
heartbeat
---
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 prabhjyotsingh commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-188083016
Let me look into it, but by screen shot, it looks like spark is down.
java.net.ConnectException: Connection refused
at java.net.
Github user ravicodder commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/693#issuecomment-188082804
@corneadoug corrected bad rebase.
---
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
Github user felixcheung commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/719#issuecomment-188080562
In the post commit build, it looks like Selenium tests are failing because
of "Connection refused" from screen shots. Any idea?
https://s3.amazonaws.c
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/719
---
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 khalidhuseynov commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/703#issuecomment-188035398
Thanks for the great improvement!
What do you think would be the default use case for the interpreter context
sharing? Currently the checkbox is u
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-zeppelin/pull/727
---
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/743#issuecomment-187992008
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 f
Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-187961103
@jeffsteinmetz Thanks for your comment. By default, `r` and `knitr` are in
the `spark` interpreter group, so Zeppelin lets them be called by
`%r`/`%spark.r`
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/703#issuecomment-187924861
@felixcheung @AhyoungRyu @echarles I have changed to "Separate interpreter
for each note"
---
If your project is set up for it, you can reply to this ema
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/736#issuecomment-187889549
Looks good to me. and merge if there're no more discussions
---
If your project is set up for it, you can reply to this email and have your
reply appear o
DOAN DuyHai created ZEPPELIN-696:
Summary: Add notification system for AngularJS z functions
Key: ZEPPELIN-696
URL: https://issues.apache.org/jira/browse/ZEPPELIN-696
Project: Zeppelin
Issue
I continue to not see a point to engaging in this as a debate.
The user acceptance speaks for itself. (As just one thing, the only person who
hasn't gotten the display system working in 208, is Eric.) So does the rate of
change - there have been a series of pushes to 702 in the past few month o
On 23/02/16 19:52, Amos B. Elberg wrote:
Eric, they're not equivalent. 208 continues to have functionality 702 doesn't,
including the display system.
I'm not going to tell you what you're doing wrong in your implementation and
"test" of 208, because the users don't seem to have the same confu
Eric, they're not equivalent. 208 continues to have functionality 702 doesn't,
including the display system.
I'm not going to tell you what you're doing wrong in your implementation and
"test" of 208, because the users don't seem to have the same confusion, and
I've essentially been guiding you
The community is working toward merging 208. %spark.r and %r are the same thing
- just two different ways Zeppelin let's you identify an interpreter.
> On Feb 23, 2016, at 1:09 PM, Jeff Steinmetz
> wrote:
>
> Thank you Amos Elberg & Eric Charles:
> Is the goal of the community to merge both 2
It would make no sense merging both.
From an end-user perspective, I guess both are equivalent, although
with the last commit I made, the Zeppelin Display system is supported in
702 (I had no luck when testing this functionality with 208). As I said,
feel free to test both and send feature req
Thank you Amos Elberg & Eric Charles:
Is the goal of the community to merge both 208 and 702 at some point as two
“different” R interpreters?
One that is
%r
And another that is
%spark.r
Still trying to wrap my head around the difference.
On 2/23/16, 9:34 AM, "Amos B. Elberg" wrote:
>Je
Github user ankurmitujjain commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-187820595
may be this URL can help...
http://jtleek.com/protocols/travis_bioc_devel/
:+1:
---
If your project is set up for it, you can reply to t
Jeff - 702 isn't a fork, it's an alternative based on 208 that has a subset of
208's features. 208 is the superset. 208 is also what the community is now
attempting to integrate.
R does support serialization of functions.
208 does support passing a spark table back and forth between R and sca
On 23/02/16 18:03, Jeff Steinmetz wrote:
Hello zeppelin dev group,
Regarding the R Interpreter Pull requests 208 and 702. I am trying to figure
out if the functionality between these are overlapping, or one supports
something different than the other. Is 702 a super set of 208 (702 is a fo
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/743#issuecomment-187798457
Thanks @fhalim. LGTM and merge if there're no more discussions.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user doanduyhai commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/743#issuecomment-187796635
LGTM, +1
The patch is trivial
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Hello zeppelin dev group,
Regarding the R Interpreter Pull requests 208 and 702. I am trying to figure
out if the functionality between these are overlapping, or one supports
something different than the other. Is 702 a super set of 208 (702 is a fork
of 208)?
Can you pass the reference of a
Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/208#issuecomment-187787778
@bzz Hey just checking-in to see how we're doing and if there's anything I
can do at this point to help move us along toward merge.
---
If your project is
Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-187786770
@echarles Its one thing for you to wait for me to identify the missing
functionality in your PR for you to implement it. *Making up* that features
are missi
GitHub user fhalim opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/743
Use spark_cassandra_connection 1.5.0
Update dependency to release version of spark-cassandra-connector.
You can merge this pull request into a Git repository by running:
$ git pull h
Github user Leemoonsoo commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/737#issuecomment-187783980
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/737#issuecomment-187784053
Merge if there're 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 y
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-187781300
@elbamos just pulled the 208 patch, built and tested with "%r print("%html
hello")". Got
```
"[1] "%html hello"
```
printed as res
Github user elbamos commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-187764059
@btiernay support for that has been in 208 all along...
> On Feb 23, 2016, at 9:27 AM, Bob Tiernay wrote:
>
> @echarles This is great! Than
Github user corneadoug commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/739#issuecomment-187750881
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 doanduyhai opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/742
[ZEPPELIN-695] Add AngularJS z.runParagraph()
### What is this PR for?
Add AngularJS `z.runParagraph()` and `z.runParagraphs()` methods
@corneadoug
On the client-side
DOAN DuyHai created ZEPPELIN-695:
Summary: Add AngularJS z.runParagraph()
Key: ZEPPELIN-695
URL: https://issues.apache.org/jira/browse/ZEPPELIN-695
Project: Zeppelin
Issue Type: Improvement
GitHub user doanduyhai opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/741
[Zeppelin-693] Add AngularJS z.angularUnbind()
### What is this PR for?
Add AngularJS `z.angularUnbind()` method
The signature of the method is `angularUnbind(varName, par
Github user btiernay commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-187719246
@echarles This is great! Thanks for all your hard work. Very much
appreciated!
---
If your project is set up for it, you can reply to this email and have y
Github user echarles commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/702#issuecomment-187717158
@btiernay Just pushed the HTML, TABLE and IMG display support (the dynamic
form is not yet fully operational).
[
Key: ZEPPELIN-693
URL: https://issues.apache.org/jira/browse/ZEPPELIN-693
Project: Zeppelin
Issue Type: Improvement
GitHub user doanduyhai opened a pull request:
https://github.com/apache/incubator-zeppelin/pull/740
[Zeppelin 689] Add AngularJS z object and z.angularBind()
### What is this PR for?
Add client-side `z` object with method `angularBind()`
@Leemoonsoo
Compared to
Github user ravicodder commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/689#issuecomment-187600870
@corneadoug Corrected bad rebase.
---
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
52 matches
Mail list logo
