Github user vipul1409 commented on the issue:
https://github.com/apache/zeppelin/pull/2326
Addressed review comments and moved the dependency to npm. One jenkins job
is failing but looks unrelated to my changes. I also see build is marked as
failed in master, just double checking if m
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2329
And what I meant ux is about supporting spark's native yarn-cluster mode in
Zeppelin :-)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2329
I mean 'not in case of spark'. And this implementation satisfies your
needs. You can launches some interpreters like md in local mode and some
interpreters in yarn.
---
If your project is set u
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2336
This is minor issue and CI failure isn't related to this issue.
---
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 doe
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2329
> it's improve users' experiences
Sorry, I don't understand how it improve users' experience compared the
native spark yarn-cluster support. And does this approach mean all the
interpreter must
GitHub user jongyoul opened a pull request:
https://github.com/apache/zeppelin/pull/2336
[MINOR] Change `case` indent
### What is this PR for?
Changing style in case of 'case'. Zeppelin has two spaces as default
indentation but checkstyle has different style of 'case'.
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2329
It makes our logic much difficult, but it's improve users' experiences.
---
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 pro
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2329
In case of Spark, it has two yarn apps. I agree that it's a bit weird for
users. Otherwise, all other interpreters like python get better to launch it in
yarn cluster. I think it would be better
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2329
Do you mean there would be 2 yarn apps for launching one spark interpreter
? That looks a little weird to me. In this approach, we still launch remote
interpreter process in the zeppelin host, that
Github user guptarajat commented on the issue:
https://github.com/apache/zeppelin/pull/2323
"all these things are persisted and synchronized among web socket clients"
What does this mean?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2268
@Leemoonsoo
the separation discussed above in the comments
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2329
@zjffdu Thanks for quick review. I know spark already have yarn-cluster by
itself. In case of Zeppelin, SparkInterpreter will run yarn-client mode which
means driver is still in a process zeppeli
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2335
CI failed but irrelevant.
---
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 wish
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2333
I don't think it's good idea to include some interpreters as dependencies
onto zeppelin-zengine.
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
ClassloaderInterpreter could also be removed.
Jeff Zhang 于2017年5月10日周三 下午3:39写道：
>
> Hi moon
>
> Considering we use remote=true for interpreters now, and seems no user
> complain about that. So I think we could remove it completely. Agree with
> you that It could make code simpler, e.g. We don't
Hi moon
Considering we use remote=true for interpreters now, and seems no user
complain about that. So I think we could remove it completely. Agree with
you that It could make code simpler, e.g. We don't need to check to use
RemoteAngularObjectRegistry or AngularObjectRegistry.
https://github.com
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2272
@felixcheung toJson and fromJson is used in both zeppelin-interpreter and
zeppelin-zengine. I didn't find a proper place to put the common interface. And
actually the interface doesn't not eliminat
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2334
@FireArrow Could you help check the failed test ?
---
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 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2335
CI script can't parse `=` in the branch name.
- https://travis-ci.org/1ambda/zeppelin/builds/230940304
---
If your project is set up for it, you can reply to this email and have your
repl
GitHub user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/2335
[ZEPPELIN-2426] Improve graph setting DOMs
### What is this PR for?
Improve graph setting DOMs. Refer the attached screenshots.
### What type of PR is it?
[Improvement]
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2268
This PR adds two new field 'widget' and 'type' to each properties.
'widget' : input, checkbox, textarea, password
'type' : number, string, boolean, url, password
How about j
Jeff Zhang created ZEPPELIN-2529:
Summary: Create Base class for spark interpreter
Key: ZEPPELIN-2529
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2529
Project: Zeppelin
Issue Type: Im
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2323
Here's few feedbacks.
1. Table does not refreshed after height / width of paragraph change.
TBD
2. How about remove 'type' in column table option?
- Removed
-
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2268
As far as I Know, Zeppelin supports backward compatibility even for
interpreter settings.
- I think we need some migration code w/ documentation
---
If your project is set up for it, you
Github user VipinRathor commented on the issue:
https://github.com/apache/zeppelin/pull/2330
@prabhjyotsingh
Thanks for reviewing this.
> any specific reason for chaning ldapRealm =
org.apache.zeppelin.realm.LdapRealm to
ldapRealm=org.apache.zeppelin.realm.LdapRealm in a
Benjamin Vogan created ZEPPELIN-2528:
Summary: Add a password text input to the ZeppelinContext
Key: ZEPPELIN-2528
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2528
Project: Zeppelin
Jeff Zhang created ZEPPELIN-2527:
Summary: Cursor in editor is too thick
Key: ZEPPELIN-2527
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2527
Project: Zeppelin
Issue Type: Improvement
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2334
Right, we could do it in another 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 have this feature
enab
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2329
@jongyoul Thanks for the POC. One question about the cluster mode for spark
interpreter. It looks like you are creating a general yarn app support for all
the interpreters. But for spark, it has al
Github user tinkoff-dwh commented on the issue:
https://github.com/apache/zeppelin/pull/2268
@1ambda
i wrote about migration in upgrade.md. I'll see Notebook.java if the
documentation is not enough
---
If your project is set up for it, you can reply to this email and have your
r
Github user FireArrow commented on the issue:
https://github.com/apache/zeppelin/pull/2334
@zjffdu I agree. A also noticed there are plenty of sketchy
`synchrnoized`-blocks that doesn't really work as intended. But that's a
refactor for another PR
---
If your project is set up for i
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2334
LGTM, BTW for a long term solution, I think we should initialize
`ZeppelinContext` properly (including set `InterpreterContext`) in some base
class instead of initialize in each interpreter manuall
GitHub user FireArrow opened a pull request:
https://github.com/apache/zeppelin/pull/2334
[HOTFIX] Set interpreter context in zeppelin context from
SparkSqlInterpreter
### What is this PR for?
When running %sql paragraphs the query is dispatched to the cluster, but
when it's ti
Github user aspen01 commented on the issue:
https://github.com/apache/zeppelin/pull/1600
@EronWright Thank you for your feedback. When I first developed the
feature, I assumed that HDFS would not be used as default storage because
remote FS failure could affect zeppelin usage. So ext
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/2323
Tested and overall it's great improvement!
Here's few feedbacks.
1. Table does not refreshed after height / width of paragraph change.
2. How about remove 'type' in column t
Github user hayssams closed the pull request at:
https://github.com/apache/zeppelin/pull/1479
---
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
Github user hayssams commented on the issue:
https://github.com/apache/zeppelin/pull/1479
Replaced by PR #2333
---
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 hayssams opened a pull request:
https://github.com/apache/zeppelin/pull/2333
Notebook: (Web) HDFS as a backend storage (Read & Write Mode)
### What is this PR for ?
This PR replaces the PR-1479 by removing any hadoop dependency using
WEBHDFS as a communication protoc
Github user hayssams closed the pull request at:
https://github.com/apache/zeppelin/pull/2332
---
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
Pravin Dsilva created ZEPPELIN-2526:
---
Summary: NotebookSecurityRestApiTest fails with http 302 response
Key: ZEPPELIN-2526
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2526
Project: Zeppelin
GitHub user hayssams opened a pull request:
https://github.com/apache/zeppelin/pull/2332
WebHDFS Support
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ebiznext/incubator-zeppelin WEBHDFS
Alternatively you can review and apply
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2268
Sorry for the late review.
seems that it doesn't work with existing settings. Even can't start
Zeppelin instance.
```
Caused by: java.lang.ClassCastException: java.lang.String
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2300
even though CI is green in this PR, it seems that wasn't rebased recently.
As a result CI is failing only in the second travis JOB. @AlexanderShoshin
Could you check that?
---
If your project is
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2330
Also, I notice you have removed these two;
- ldapRealm.contextFactory=$ldapGroupContextFactory
- ldapRealm.contextFactory.systemAuthenticationMechanism=simple
any specific
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/2330
@VipinRathor any specific reason for chaning `ldapRealm =
org.apache.zeppelin.realm.LdapRealm` to
`ldapRealm=org.apache.zeppelin.realm.LdapRealm` in all the lines ?
---
If your project is
GitHub user VipinRathor opened a pull request:
https://github.com/apache/zeppelin/pull/2331
[minor] Replace 'SIMPLE' auth with 'simple' auth everywhere in documents
### What is this PR for?
This PR fixes a security hole in documentation when 'SIMPLE' authentication
mechanism is
GitHub user VipinRathor opened a pull request:
https://github.com/apache/zeppelin/pull/2330
[minor] Remove incorrect configuration and fix description of LdapRealm
### What is this PR for?
There is wrong description and incorrect configuration defined in comment
in LdapRealm.jav
Srikanth Kedari created ZEPPELIN-2525:
-
Summary: How to integrate zeppelin in java project
Key: ZEPPELIN-2525
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2525
Project: Zeppelin
Is
Github user vipul1409 commented on the issue:
https://github.com/apache/zeppelin/pull/2326
I checked Jenkins Job. The sub job failing is
https://travis-ci.org/vipul1409/zeppelin/jobs/230634157 It is failing because
of some git access issues. Any suggestions on what could be the issue?
49 matches
Mail list logo
