Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1668
Sounds reasonable. CI is green, merging to master then.
---
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 th
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1700
Looks good to me, merging to master as a hotfix based on user feedback in
[this thread](http://markmail.org/message/4z6hxxf3eem2wgp2), if there is no
further discussion.
---
If your project is set u
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1696
@bzz That's because invalid `_tools/checkstyle.xml`. I'v just updated sorry
for distraction :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1689
CI fails \w
```
[ERROR] Failed to execute goal
com.googlecode.maven-download-plugin:download-maven-plugin:1.3.0:wget
(download-pyspark-files) on project zeppelin-spark-dependencies_2.11:
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1668
@bzz I think this can be merged before #1339. I guess It takes time more
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1696
Spark 2.0 profile fails with very stragne
```
[ERROR] Failed to execute goal
org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check
(checkstyle-fail-build) on project zeppelin: Faile
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1441
change name for item on select box.
Undefined to Interpreter is not set
---
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 cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/1441
---
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 featur
GitHub user cloverhearts reopened a pull request:
https://github.com/apache/zeppelin/pull/1441
[ ZEPPELIN-967 ] Implement search bar in job manager
### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1441
@Leemoonsoo
Thank you for your review a lot.
First, the select box's interpreter shows only the first interpreter,
because it is information about the default interpreter's setti
GitHub user khalidhuseynov opened a pull request:
https://github.com/apache/zeppelin/pull/1700
[DOCS] Update Shiro configuration file with new package names
### What is this PR for?
Since most of authentication related realm classes has moved from
`org.apache.zeppelin.server` pa
GitHub user astroshim opened a pull request:
https://github.com/apache/zeppelin/pull/1699
[ZEPPELIN-1724] fix conda version
### What is this PR for?
Because `conda run` command removed since version `4.0.9`,
PythonCondaInterpreter not working after the `conda-4.0.9`.
This PR
Github user anthonycorbacho commented on the issue:
https://github.com/apache/zeppelin/pull/1681
@khalidhuseynov thanks for the review, i addressed it.
Merging if no more discussion
---
If your project is set up for it, you can reply to this email and have your
reply appear o
I think during refactoring LdapGroupRealm has moved into different package,
so could you try in your shiro config with:
ldapRealm = org.apache.zeppelin.realm.LdapGroupRealm
On Tue, Nov 29, 2016 at 2:33 AM, Ruslan Dautkhanov
wrote:
> + dev list
>
> Could somebody please let me know if shiro-LDAP
GitHub user prabhjyotsingh opened a pull request:
https://github.com/apache/zeppelin/pull/1698
Downloading dependency/changing version for an interpreter results in error
sometimes
### What is this PR for?
At times on downloading dependency/changing version for an interpreter
r
GitHub user anthonycorbacho opened a pull request:
https://github.com/apache/zeppelin/pull/1697
[ZEPPELIN-1352] Refactor Notebook repo versioning to return Revision Empty
instead of NULL
### What is this PR for?
In Zeppelin notebookRepo, versioning (checkpoint, get, revisionHist
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1658
Thanks @agoodm @khalidhuseynov for the review!
Merge to master if there're no more comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1679
Looks good to me.
CI is green now, so will merge it to master if there is no further
discussion.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1339
Short summary and small thought about #1399
1. Using symlink like `local-spark/master` would be safe i think. It
enables user replace his local spark without renaming directories. Currentl
Kshitij Badani created ZEPPELIN-1726:
Summary: Downloading dependency/changing version for an
interpreter results in error sometimes
Key: ZEPPELIN-1726
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1726
Kshitij Badani created ZEPPELIN-1725:
Summary: Downloading dependency/changing version for an
interpreter results in error sometimes
Key: ZEPPELIN-1725
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1725
Shim HyungSung created ZEPPELIN-1724:
Summary: conda run command removed in 4.1.0
Key: ZEPPELIN-1724
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1724
Project: Zeppelin
Issue Type:
GitHub user zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1612
ZEPPELIN-1607. Refactor Livy Interpreter to adapt scope mode
### What is this PR for?
ZEPPELIN-1606 will make scoped mode as the default interpreter mode of
livy. This PR is to refactor livy
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1612
---
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 AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1690
@1ambda Sounds great thanks! 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 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/1696
CI test for 1689
This is a comparison group for #1689
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/1ambda/zeppelin ci-test-for-1689
Altern
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1690
I separated plugin link with setting guide @AhyoungRyu :)
---
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 ha
GitHub user prabhjyotsingh reopened a pull request:
https://github.com/apache/zeppelin/pull/1672
ZEPPELIN-1701: User impersonation not work with SPARK_HOME set.
### What is this PR for?
User impersonation of front end user does not work with SPARK_HOME set.
### What type
Github user prabhjyotsingh closed the pull request at:
https://github.com/apache/zeppelin/pull/1672
---
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 feat
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1672
@zjffdu did you try it again, as this is working for me, or if you can let
me know your environment I can try on the same.
---
If your project is set up for it, you can reply to this emai
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1693
Looks great to me, thank you @1ambda
Merging to master if there is no further discussion.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user 1ambda commented on the issue:
https://github.com/apache/zeppelin/pull/1668
rebased to keep sync with master (+ to see CI is green as well)
---
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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1686
@soralee Tested again with `ctrl + option(alt) + l` and it works well. LGTM
ð
---
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 bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1689
Looks great to me, thank you @1ambda !
May be, as soon as CI is green let's try to squash some commits, before
merging it in - 37 may be a bit too much to be useful in merge commit message
su
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1684
---
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 AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1685
Merge into master if there are no more 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 does not have th
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1658
tried locally and works as expected, 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 thi
Github user matthewLiem commented on the issue:
https://github.com/apache/zeppelin/pull/1617
@randerzander @Leemoonsoo
Thanks for this. Do you know of a workaround we could use in the meantime?
---
If your project is set up for it, you can reply to this email and have your
repl
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@tae-jun Great work! Take your time and feel free to ping me again when
you're ready.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1606
@giaosudau Thanks! ð I appreciate it ð
---
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 tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
CI failure is because of me. I will fix it :)
---
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 giaosudau commented on the issue:
https://github.com/apache/zeppelin/pull/1606
@tae-jun https://issues.apache.org/jira/browse/ZEPPELIN-1723
---
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
Chanh Le created ZEPPELIN-1723:
--
Summary: Math formula support library path error
Key: ZEPPELIN-1723
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1723
Project: Zeppelin
Issue Type: Bug
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1660
@Leemoonsoo Yes, I haven't check the front-side. I'll fix it
---
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 n
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1660
@jongyoul I think key requirement of personalized mode is two or more
different user want to keep their own dynamic form input and the output of
paragraph. How can this be done without let user
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1660
@Leemoonsoo I've not tested the graph. If you mean a dynamic change of
graph, I'll check it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1660
@Leemoonsoo As we discussed before, the first version of personalized mode
is to avoid changing note simultaneously while two user see a same note. I
think this version fits it. one user want to
Github user agoodm commented on the issue:
https://github.com/apache/zeppelin/pull/1658
Very nice @Leemoonsoo ð
I have tested this with the matplotlib tutorial notebook just to see if
[ZEPPELIN-1638](https://issues.apache.org/jira/browse/ZEPPELIN-1638) is also
resolved by
Lee moon soo created ZEPPELIN-1722:
--
Summary: Modernize visualization/transformation using ES6 class
Key: ZEPPELIN-1722
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1722
Project: Zeppelin
Lee moon soo created ZEPPELIN-1721:
--
Summary: Create / manage settings inside of each visualization.
Key: ZEPPELIN-1721
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1721
Project: Zeppelin
GitHub user PetroKostyuk opened a pull request:
https://github.com/apache/zeppelin/pull/1695
Zeppelin 1283
### What is this PR for?
It's adding option to specify column in scatterPlot, that will be displayed
in label
### What type of PR is it?
Feature
Github user prabhjyotsingh commented on the issue:
https://github.com/apache/zeppelin/pull/1694
Haven't tried it yet, but IMO we should have a check at backned for the
same (both API and websocket).
---
If your project is set up for it, you can reply to this email and have your
reply
Github user prabhu-pictink commented on the issue:
https://github.com/apache/zeppelin/pull/1694
Haven't tried it yet, but I think we should have a check at backned for the
same (both API and websocket).
---
If your project is set up for it, you can reply to this email and have your
r
Github user epahomov commented on the issue:
https://github.com/apache/zeppelin/pull/1688
Yeaaa, I love it!
---
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,
Github user krskoondra commented on the issue:
https://github.com/apache/zeppelin/pull/1688
OK, @Leemoonsoo, the button is now in the dropdown menu of the paragraph
and new keyboard shortcut ctrl+shift+c is added.
---
If your project is set up for it, you can reply to this email and
Github user tae-jun commented on the issue:
https://github.com/apache/zeppelin/pull/1630
@AhyoungRyu Sorry for my late response! But it's due to tests :)
I reproduced the error you mentioned and you were right!
Furthermore, chasing a reason of the error, I notice that
+ dev list
Could somebody please let me know if shiro-LDAP is known to be broken in
master?
So I will stop my attempts to work with 0.7.
[org.apache.zeppelin.server.LdapGroupRealm] for object named 'ldapRealm'.
Please ensure you've specified the fully qualified class name correctly.
at or
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1617
@randerzander Thanks for the contribution.
LGTM and Merge to master if there're no more discussions.
---
If your project is set up for it, you can reply to this email and have your
rep
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1637
@cloverhearts Thanks for the great contribution.
LGTM and merge to master if there're no more discussions.
---
If your project is set up for it, you can reply to this email and have your
re
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1637
@Leemoonsoo Sorry, I think I misunderstood your answer. All edits are
complete.
Thank you for your reply.
---
If your project is set up for it, you can reply to this email and have your
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1637
Sorry, I think I misunderstood your answer. All edits are complete.
Thank you for your reply.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1685
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
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1685
@astroshim Tested on CentOS 7 again and faced the issue you described.
Also I checked the other files that contains `import scala.tools.nsc.*`
(e.g.
[FlinkInterpreter.java](https://github.
Github user cloverhearts commented on the issue:
https://github.com/apache/zeppelin/pull/1637
ci retry
---
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 i
Github user cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/1637
---
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 featur
GitHub user cloverhearts reopened a pull request:
https://github.com/apache/zeppelin/pull/1637
[ZEPPELIN-1665] Z.run with external note executable and access resource for
zeppelin in each interpreter
### What is this PR for?
Currently, the z.run command is restricted.
Only p
GitHub user AhyoungRyu reopened a pull request:
https://github.com/apache/zeppelin/pull/1694
[ZEPPELIN-1718] Prevent anonymous user to set note permission / interpreter
owner
### What is this PR for?
Currently anonymous user can set note permission / interpreter's owner like
be
Github user AhyoungRyu closed the pull request at:
https://github.com/apache/zeppelin/pull/1694
---
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
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1647
Although it changes user experience, it is more consistent behavior
considering other type of dynamic form behavior (input, checkbox).
Regarding the test, while other types of dynamic f
Lee moon soo created ZEPPELIN-1720:
--
Summary: Make tests to ensure behavior of dynamic form
Key: ZEPPELIN-1720
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1720
Project: Zeppelin
Issu
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1688
Thanks for updatding description!
Agree with @Leemoonsoo. And FYI, you can refer #1686 for assigning keyboard
shortcut :D
---
If your project is set up for it, you can reply to this email
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1684
Merge into master if there are no more 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 does not have th
GitHub user AhyoungRyu opened a pull request:
https://github.com/apache/zeppelin/pull/1694
[ZEPPELIN-1718] Prevent anonymous user to set note permission / interpreter
owner
### What is this PR for?
Currently anonymous user can set note permission / interpreter's owner like
belo
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1688
Top right corner of each paragraph was pretty crowded in early days of
Zeppelin.
Run, Move up, Move down, Delete, Show title, Show codes, Hides output, and
so on.
Because it was to
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1637
I still can see some unnecessary changes on the import code block in many
files.
Could you manage them unchanged? I have listed files in previous comment.
---
If your project is set up for
Github user Leemoonsoo commented on the issue:
https://github.com/apache/zeppelin/pull/1686
Ctrl+Alt+L sounds good 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 this feature
enabled
Github user khalidhuseynov commented on the issue:
https://github.com/apache/zeppelin/pull/1681
@anthonycorbacho tested and works as expected. may check comment for log,
LGTM!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Abhijit Bhole created ZEPPELIN-1719:
---
Summary: Error parsing %table when first cell in row is html
Key: ZEPPELIN-1719
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1719
Project: Zeppelin
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1632
I think it is mostly the test output that pollutes the CI logs...
So, if I read TravisCI logs right - with 2 new profiles `Elapsed time 31
min`, and on `master` i.e
[yesterday](https://travi
Ahyoung Ryu created ZEPPELIN-1718:
-
Summary: Should prevent to set permission by anonymous user
Key: ZEPPELIN-1718
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1718
Project: Zeppelin
I
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1689
Thank you @1ambda !
I think it's a great idea to have checkstyle off the most of profiles, but
add a new one with just checkstyle.
After CI finishes, let's post the CI time statistics in P
Github user bzz commented on the issue:
https://github.com/apache/zeppelin/pull/1690
Looks great to me, thank you for taking care of both, Eclipse and IDEA
---
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 d
Github user vgmartinez commented on the issue:
https://github.com/apache/zeppelin/pull/918
Hi @Leemoonsoo,
I have refactored the code to be able to add more channels of
notifications, e.g slack, SNS (in the next PR)...
Please take a look...
thks
---
If your project
Jeff Zhang created ZEPPELIN-1717:
Summary: Weird output for SparkR
Key: ZEPPELIN-1717
URL: https://issues.apache.org/jira/browse/ZEPPELIN-1717
Project: Zeppelin
Issue Type: Bug
Affects Ve
Github user zjffdu commented on the issue:
https://github.com/apache/zeppelin/pull/1691
@Leemoonsoo Please help 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
enabled an
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1693
Thanks @1ambda for the contribution :smile:
I tested on Ubuntu and it works well as expected
```
---
T E S T S
-
Github user soralee commented on the issue:
https://github.com/apache/zeppelin/pull/1686
Thanks for review @AhyoungRyu @Leemoonsoo, and could you check this PR
again, please?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
GitHub user 1ambda opened a pull request:
https://github.com/apache/zeppelin/pull/1693
[ZEPPELIN-1695] chore: Set xml plugin phase to verify (minor)
### What is this PR for?
Changed the phase of xml validation plugin from validate to verify so that
developers can use `mvn t
GitHub user zjffdu reopened a pull request:
https://github.com/apache/zeppelin/pull/1691
ZEPPELIN-1716. Error of interpreter not found is not propagated to frâ¦
### What is this PR for?
It is introduced in ZEPPELIN-1399, we call
`InterpreterFactory.getInterpreter` before callin
Github user zjffdu closed the pull request at:
https://github.com/apache/zeppelin/pull/1691
---
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 cloverhearts reopened a pull request:
https://github.com/apache/zeppelin/pull/1692
[ZEPPELIN-1710 - WIP] Ace Editor upgrade
### What is this PR for?
The current version of the ace editor is too old.
You should also fix a bug that improperly typed non-English typin
Github user cloverhearts closed the pull request at:
https://github.com/apache/zeppelin/pull/1692
---
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 featur
92 matches
Mail list logo