Github user jerryshao commented on the issue:
https://github.com/apache/zeppelin/pull/2459
CC @zjffdu , please help to review, thanks!
---
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 fea
GitHub user jerryshao opened a pull request:
https://github.com/apache/zeppelin/pull/2459
[ZEPPELIN-2716] Change the default value of zeppelin.livy.displayAppInfo to
true
### What is this PR for?
Since it is quite useful to expose the application info for user to monitor
a
Saisai Shao created ZEPPELIN-2716:
-
Summary: Change default of zeppelin.livy.displayAppInfo to true
Key: ZEPPELIN-2716
URL: https://issues.apache.org/jira/browse/ZEPPELIN-2716
Project: Zeppelin
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/2268
1. Please resolve the conflict.
2. Left a minor comment please check that.
3. Tested locally and works well with 6 newly added types.
4. Backward compatibility will be handed by the
`co
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2329
@zjffdu Thanks, Jeff for this kind review. I've addressed your comments and
will handle some suggestion, too.
---
If your project is set up for it, you can reply to this email and have your
repl
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2272
Thanks @jongyoul Will merge it 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 does not have t
GitHub user ebuildy opened a pull request:
https://github.com/apache/zeppelin/pull/2458
Zeppelin 2701
### What is this PR for?
From https://issues.apache.org/jira/browse/ZEPPELIN-2701, this set a HTML
title for notebook page, very good when you have multiple tabs opened.
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/2374
this is ready for more 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 your project does not have this feature
enab
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2272
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
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2329
What I meant is a kind of selenium test. And I saw it before and it will be
useful for testing yarn cluster manager itself. I'll try to add it.
---
If your project is set up for it, you can rep
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2329
Not sure what integration test you mean ? Do you mean selenium test ?
Actually `MiniYarnCluster` will launch a mini yarn cluster locally, and you can
submit jobs to this mini cluster. You can refer
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2329
Added simple descriptions :-)
---
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 an
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2329
@zjffdu Thanks for the quick reply. I'll add javadoc for describing it
simply. but concerning the unit test, this feature need integration test so I'm
not sure if unit test is useful or not. Do y
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/2329
@jongyoul Could you add more javadoc on these new classes and write a
simple summary about the design so that it is easier for other people to review
it ?
And I would suggest you to add un
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2440
I got it. I didn't recognize actual username is "anonymous" in production
case. If it can be, it will be a problem.
---
If your project is set up for it, you can reply to this email and have you
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2403
---
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/2418
@zjffdu Sounds good. I totally agree with your point of view. We need to
add pushing way from interpreter to server, and also abstract that layer
carefully to guess most of situation. This is a k
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2438
LGTM. Thanks for making test. Will merge it there's 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 proj
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2329
CI already has been passed:
https://travis-ci.org/jongyoul/zeppelin/builds/249265535
---
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 jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2329
Ready to review.
- Introduced `zeppelin-cluster/common` for cluster manager and its factory
class
- Added hadoop related setting in `dependecyManagement` of root `pom.xml`
---
If yo
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/2453
---
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/2454
---
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/2452
---
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 cfries commented on the issue:
https://github.com/apache/zeppelin/pull/2440
Hi @jongyoul : I don't understand your comment. You can have "anonymous"
when shirt is on. This is even in the shiro example (note: If I understood the
shiro example correctly: until the user is no
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/2440
I don't know the purpose of this PR. If username is anonymous, it means
shiro is off. What's your intention?
---
If your project is set up for it, you can reply to this email and have your
reply
25 matches
Mail list logo
