GitHub user sudheer0553 opened a pull request:
https://github.com/apache/zeppelin/pull/2654
[ZEPPELIN-3016] How to Setup Zeppelin User Impersonation
### What is this PR for?
Documentation and how to setup for the zeppelin User Impersonation
### What type of PR is it?
[ Improvement | Documentation ]
### Todos
* [ ] - Task
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-3016
### How should this be tested?
* First time? Setup Travis CI as described on
https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
* Strongly recommended: add automated unit tests for any new or changed
behavior
* Outline any manual steps to test the PR here.
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update?
* Is there breaking changes for older versions?
* Does this needs documentation?
Author: Venkata Sudheer Kumar M ([email protected])
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/zeppelin master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/2654.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2654
----
commit 6c4ccf331e7e5aa3f17cef40d340c29930887413
Author: Prabhjyot Singh <[email protected]>
Date: 2017-04-30T20:41:47Z
[hotfix] Improve Session #2302 fix
### What is this PR for?
After the merge of #2302 breaks authentication.
### What type of PR is it?
[Hot Fix]
### Todos
* [x] - Fix selenium logout issue (flaky test)
### Questions:
* Does the licenses files need update?
* Is there breaking changes for older versions?
* Does this needs documentation?
Author: Prabhjyot Singh <[email protected]>
Closes #2304 from prabhjyotsingh/hotfix/ImproveSession and squashes the
following commits:
94ba9e774 [Prabhjyot Singh] check if modal window is present, and then
close it
f993b59fa [Prabhjyot Singh] improve session
commit 2a06292c1fcc64ea4e503a29212a179f93699b07
Author: Jeff Zhang <[email protected]>
Date: 2017-05-02T10:03:11Z
[HOTFIX]. Minor doc fix for back-end-angular.md
### What is this PR for?
The example in the doc would fail due to typo, straightforward fix.
### What type of PR is it?
[Hot Fix]
### Todos
* [ ] - Task
### How should this be tested?
Outline the steps to test the PR here.
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: Jeff Zhang <[email protected]>
Closes #2307 from zjffdu/doc_hotfix and squashes the following commits:
133fe72 [Jeff Zhang] [HOTFIX]. Minor doc fix for back-end-angular.md
commit 73ae291b0553789fbf00980aa8f283d8570e9e1b
Author: Elek, Márton <[email protected]>
Date: 2017-04-28T12:46:10Z
[ZEPPELIN-2468] Enable websocket without Origin if allowed.origins is *
Change-Id: Iaad10a69983036e84b766a22fbc32113b926b60d
### What is this PR for?
With ZEPPELIN-2288 we restored the check of the Origin field for websocket
requests.
Unfortunately the current implementation will deny the request if the
Origin HTTP header is empty, even if the zeppelin.server.allowed.origins is *
This patch enables websocket requests without Origin in the HTTP header if
the zeppelin.server.allowed.origins=*. This fixes the work behind a restrictive
reverse proxy (or behind Apache Knox)
### What type of PR is it?
Bug Fix
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2468
### How should this be tested?
It could be tested with curl as described in ZEPPELIN-2288, but I added
additional unit test, so the change has been covered on unit test level.
### Screenshots (if appropriate)
N/A
### Questions:
* Does the licenses files need update? NO
* Is there breaking changes for older versions? NO
* Does this needs documentation? NO
Author: Elek, Márton <[email protected]>
Closes #2299 from elek/ZEPPELIN-2468 and squashes the following commits:
d95bb41 [Elek, Márton] [ZEPPELIN-2468] Enable websocket without Origin if
allowed.origins is *
commit e4f399f1b021d763271eb436410d179719e5ac03
Author: Michael Ressler <[email protected]>
Date: 2017-05-02T14:50:03Z
Some grammar updates.
### What is this PR for?
Just noticed some grammar inconsistencies that I thought I could clean up
while getting myself started on Zeppelin. Thanks for a great project!
### What type of PR is it?
Documentation
Author: Michael Ressler <[email protected]>
Closes #2308 from mressler/master and squashes the following commits:
aa39597 [Michael Ressler] Some grammar updates.
commit 82c92d110ccbdd986c58e5600a62f7cd44aedfb1
Author: Renjith Kamath <[email protected]>
Date: 2017-04-26T16:54:01Z
ZEPPELIN-2459 Zeppelin Usability Improvement for new paragraph
### What is this PR for?
Move cursor to second line for new paragraphs
### What type of PR is it?
Improvement
### Todos
* [ ] - Task
### What is the Jira issue?
ZEPPELIN-2459
### How should this be tested?
For a new paragraph he initial cursor position should be on the second line.
### Screenshots (if appropriate)
**Before**
<img width="324" alt="screen shot 2017-04-26 at 10 27 48 pm"
src="https://cloud.githubusercontent.com/assets/2031306/25446579/9dcdeb2a-2acf-11e7-883d-e0ccf2ef834d.png";>
**After**
<img width="394" alt="screen shot 2017-04-26 at 10 27 33 pm"
src="https://cloud.githubusercontent.com/assets/2031306/25446593/a66289da-2acf-11e7-8d8e-2029339f0133.png";>
### Questions:
* Does the licenses files need update? n/a
* Is there breaking changes for older versions? n/a
* Does this needs documentation? n/a
Author: Renjith Kamath <[email protected]>
Closes #2291 from r-kamath/ZEPPELIN-2459 and squashes the following commits:
4c6525d1c [Renjith Kamath] ZEPPELIN-2459 Zeppelin Usability Improvement for
new paragraph
commit 62aec9fbdc3921f319b9901b7f7bd9c4c9e0f4fc
Author: krishna-pandey <[email protected]>
Date: 2017-04-27T10:44:33Z
[ZEPPELIN-2461] Masking Jetty Server version with User-configurable
parameter
### What is this PR for?
Security conscious organisations does not want to reveal the Application
Server name and version to prevent Script-kiddies from finding the information
easily when fingerprinting the Application. The exact version number can tell
an Attacker if the current Application Server is patched for or vulnerable to
certain publicly known CVE associated to it.
### What type of PR is it?
[Improvement | Feature]
### What is the Jira issue?
* [ZEPPELIN-2461](https://issues.apache.org/jira/browse/ZEPPELIN-2461)
### How should this be tested?
Providing a value in zeppelin-site.xml will replace the actual Jetty server
version found in HTTP Header with provided value. E.g.
- edit zeppelin-site.xml and add a property `zeppelin.server.jetty.name`
and with value say `TOMCAT`
- restart the server
- open the app in browser then observe the Response Headers for the key
"Server" this should now reflect "TOMCAT"
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? N/A
* Is there breaking changes for older versions? N/A
* Does this needs documentation? N/A
Author: krishna-pandey <[email protected]>
Closes #2293 from krishna-pandey/ZEPPELIN-2461 and squashes the following
commits:
b071f7ad0 [krishna-pandey] Set App Server name to config value
commit 08a9814033e216eb2d340c49d171f3b66c19eb61
Author: Prabhjyot Singh <[email protected]>
Date: 2017-04-29T18:45:19Z
[Zeppelin 2367] Hive JDBC proxy user option should be available even
without kerberos
### What is this PR for?
Hive JDBC proxy user option should be available generically.
### What type of PR is it?
[Improvement]
### What is the Jira issue?
* [Zeppelin 2367](https://issues.apache.org/jira/browse/ZEPPELIN-2367)
### How should this be tested?
Enable Shiro authentication and set `zeppelin.jdbc.auth.type` as `SIMPLE`
in the interpreter setting, and observe the connection string for the Hive.
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need an update? N/A
* Is there breaking changes for older versions? N/A
* Does this needs documentation? Yes
Author: Prabhjyot Singh <[email protected]>
Closes #2229 from prabhjyotsingh/ZEPPELIN-2367 and squashes the following
commits:
84b5e55b2 [Prabhjyot Singh] add logger.warn for hive and impersonation
45c90a8e2 [Prabhjyot Singh] improve doc
9fee9d2a9 [Prabhjyot Singh] replace hive with generic method
a348e969a [Prabhjyot Singh] revert
"zeppelin.jdbc.auth.kerberos.proxy.enable" behaviour
e2bdbb2ad [Prabhjyot Singh] include e as inner exception
c180f5ce2 [Prabhjyot Singh] Merge remote-tracking branch 'origin/master'
into ZEPPELIN-2367
1802b453f [Prabhjyot Singh] remove hive string from logger
513987a28 [Prabhjyot Singh] apply genric logic to appendProxyUserToURL
3fa2b1e98 [Prabhjyot Singh] change name to appendProxyUserToURL
a75167415 [Prabhjyot Singh] Merge remote-tracking branch 'origin/master'
into ZEPPELIN-2367
4c382eefa [Prabhjyot Singh] log user details as well
d51e770b2 [Prabhjyot Singh] add doc in jdbc.md
01b18b9d2 [Prabhjyot Singh] add doc (reverted from commit
ee8a6b524c481210486761032cb1f5fd6266bb54)
40489c89d [Prabhjyot Singh] Merge remote-tracking branch 'origin/master'
into ZEPPELIN-2367
ee8a6b524 [Prabhjyot Singh] add doc
8999d93ae [Prabhjyot Singh] ZEPPELIN-2367: Hive JDBC proxy user option
should be avail even without kerberos
commit 92e3abc9d4a6a95da8440c6a4985234190741aae
Author: Jeff Zhang <[email protected]>
Date: 2017-05-03T12:42:34Z
ZEPPELIN-2486. AngularElem's onChange is only invoked once
### What is this PR for?
`ng-change` can not capture event, so that means `$event.timeStamp` is
undefined. https://github.com/angular/angular.js/issues/6370
This cause AngularElem's onChange is only invoked once. This PR use another
approach to update the angularObject.
### What type of PR is it?
[Bug Fix]
### Todos
* [ ] - Task
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-2486
### How should this be tested?
Tested manually
### Screenshots (if appropriate)
Before
After
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: Jeff Zhang <[email protected]>
Closes #2312 from zjffdu/ZEPPELIN-2486 and squashes the following commits:
4039d6b [Jeff Zhang] ZEPPELIN-2486. AngularElem's onChange is only invoked
once
commit a231cf5ddc17ea8a849315be6e0c2e6c0e08b7fc
Author: CloverHearts <[email protected]>
Date: 2017-04-28T03:01:54Z
[ZEPPELIN-2452] Can not handle message in revision mode on web.
### What is this PR for?
Requests interpreter editor mode information in revision mode.
However, no information is needed in revision mode.
Produces misleading logs with misleading requests to the server.
```
ERROR [2017-04-26 18:07:48,718] ({qtp1273765644-60}
NotebookServer.java[onMessage]:383) - Can't handle message
java.lang.NullPointerException
ERROR [2017-04-26 18:07:49,081] ({qtp1273765644-109}
NotebookServer.java[onMessage]:383) - Can't handle message
java.lang.NullPointerException
ERROR [2017-04-26 18:07:49,083] ({qtp1273765644-14}
NotebookServer.java[onMessage]:383) - Can't handle message
java.lang.NullPointerException
```
This log is requested as many as the number of paragraphs contained in the
note.
### What type of PR is it?
Bug fix
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2452
### How should this be tested?
1. Create a git-repo based note. (Supported by default)
2. Modify the paragraph and commit.
3. Go to the committed revision.
4. Reload the page.
5. Check the server logs.
```
NotebookServer.java[onMessage]:383) - Can't handle message
java.lang.NullPointerException
```
### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no
Author: CloverHearts <[email protected]>
Closes #2289 from
cloverhearts/fix/incorrectcall-geteditorsetting-on-revisionmode and squashes
the following commits:
4b19fb0d [CloverHearts] fix indent
8c77f769 [CloverHearts] fix call geteditorsetting
commit bb72b3dea818fdeb3069e497db838d9dbcfd060d
Author: CloverHearts <[email protected]>
Date: 2017-04-28T05:02:48Z
[ZEPPELIN-2452] block update paragraph event on revision mode
### What is this PR for?
In revision mode using git-repository, by default all paragraphs must
remain unmodifiable.
However, we are currently performing an incorrect update.
### What type of PR is it?
Bug Fix
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2452
### How should this be tested?
1. create note and modify to paragraph and version control commit on web
2. modify to paragraph and execute
3. open your 2 browser - one browser is HEAD, and other browser move to
before commit
4. insert paragarph or modify and execute.
check update
### Screenshots (if appropriate)
#### problem
#### fixed (this pr)
### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no
Author: CloverHearts <[email protected]>
Closes #2288 from cloverhearts/fix/invalidupdateonrevision and squashes the
following commits:
8fe72c9c [CloverHearts] add block event on revision mode
19e74799 [CloverHearts] block update paragraph event on revision mode
commit 6db9929dd3caf957e41344b2cbc65eb515fde615
Author: Jongyoul Lee <[email protected]>
Date: 2017-04-26T17:01:23Z
[HOTFIX] Remove adduser while creating note
### What is this PR for?
Adding user paragraph while creating paragraph makes default paragraph and
user paragraph have same reference. It breaks personalized mode works well.
### What type of PR is it?
[Bug Fix | Hot Fix]
### Todos
* [x] - Remove `addUser` from `createParagraph`
* [x] - Add test case for it
### What is the Jira issue?
N/A
### How should this be tested?
N/A
### Screenshots (if appropriate)
N/A
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: Jongyoul Lee <[email protected]>
Closes #2292 from jongyoul/hotfix/remove-wrong-adduser-and-add-testcase and
squashes the following commits:
6ed521a4 [Jongyoul Lee] Removed adduser while creating note Added test to
guarantee paragraph.getUserParagraph is different from paragraph itself
commit ae05740b1852a4251b66b9559486b62a85af5dd9
Author: Benoy Antony <[email protected]>
Date: 2017-05-02T22:17:44Z
[ZEPPELIN-2484] Fix NullPointerException in check for empty last paragraph
### What is this PR for?
Prevent NullPointerException during check to determine whether a new
paragraph needs to added.
The fix is to switch order of null check and trim operation so that null
check is performed before trim()
### What type of PR is it?
Bug Fix
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2484
### How should this be tested?
This can be tested with a hive interpreter
Create a note and add a paragraph with some query in it.
Run all paragraphs.
A new paragraph is automatically added.
Run all paragraphs again. A NullPointerException is logged in the logs.
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: Benoy Antony <[email protected]>
Closes #2310 from benoyantony/ZEPPELIN-2484 and squashes the following
commits:
1cc2f8b [Benoy Antony] ZEPPELIN-2484 do a Null check before calling trim on
paragraph's text
commit 8e96d8bd7b6e1a01df004800c526d316725a2b62
Author: Khalid Huseynov <[email protected]>
Date: 2017-05-02T09:45:11Z
[ZEPPELIN-2479] Fix incorrect tooltip view inside of version control menu
### What is this PR for?
This is to fix incorrect appearance of tooltip inside version control menu
### What type of PR is it?
Improvement
### Todos
* [x] - fix/remove tooltip
### What is the Jira issue?
[ZEPPELIN-2479](https://issues.apache.org/jira/browse/ZEPPELIN-2479)
### How should this be tested?
go to `version control` menu in notebook action bar of note
### Screenshots (if appropriate)
before:
<img width="689" alt="screen shot 2017-05-02 at 3 32 23 pm"
src="https://cloud.githubusercontent.com/assets/1642088/25606467/62f0901c-2f4c-11e7-95ef-c22f1df410be.png";>
after:
### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no
Author: Khalid Huseynov <[email protected]>
Closes #2306 from khalidhuseynov/fix/ZEPPELIN-2479 and squashes the
following commits:
ac4e651 [Khalid Huseynov] add css class
commit d9c4a5f0b6c3355753b50f466199cbe551cbd89a
Author: Jongyoul Lee <[email protected]>
Date: 2017-05-05T17:28:31Z
[MINOR] Move remoteinterpreter into zengine
### What is this PR for?
RemoteInterpreter is only used in the server side then zeppelin-interpreter
doesn't have to include this class. Moving this class helps to reduce
interpreter binary size and change RemoteInterpreter without adding more
dependencies if we want
### What type of PR is it?
[Refactoring]
### Todos
* [x] - Move RemoteInterpreter and related files out of
zeppelin-interpreter module
### What is the Jira issue?
N/A
### How should this be tested?
N/A
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: Jongyoul Lee <[email protected]>
Closes #2320 from jongyoul/minor/move-remoteinterpreter-into-zengine and
squashes the following commits:
80979913c [Jongyoul Lee] Removed author tag
e1425dfa8 [Jongyoul Lee] Adopted DummyInterpreter
99c093229 [Jongyoul Lee] Made DummyInterpreter
5ac8dfbbd [Jongyoul Lee] Moved RemoteInterpreterServer to
zeppelin-interpreter
0a881c1b3 [Jongyoul Lee] Removed unused package imported Removed
unnecessary classes imported
b7e0b9436 [Jongyoul Lee] moved some files related remote interpreter and
fix some minor things
7e8721592 [Jongyoul Lee] move some files of remote packages from
zeppelin-interpreter to zeppelin-zengine
commit e902801773c5b955be841d8275ea50def4082ae9
Author: 1ambda <[email protected]>
Date: 2017-05-04T07:29:13Z
[ZEPPELIN-2492] Use single Dockerfile for each tag
### What is this PR for?
Use single `Dockerfile` for each release since [apache infra uses tag
pushes](https://issues.apache.org/jira/browse/INFRA-12781) to build an image.
- https://issues.apache.org/jira/browse/INFRA-12781
After release process finishes, dockerhub will build using the pushed tag.
### What type of PR is it?
[Improvement]
### What is the Jira issue?
[ZEPPELIN-2492](https://issues.apache.org/jira/browse/ZEPPELIN-2492)
### How should this be tested?
1. `./dev/change_zeppelin_version.sh 0.8.0-SNAPSHOT 0.7.1`
2. Check that version is properly set: `vi
scrtips/docker/zeppelin/bin/Dockerfile`
3. Build docker image `cd scripts/docker/zeppelin/bin; docker build -t
zeppelin:0.7.1 ./`
4. Run the image: `docker run -p 8080:8080 --rm --name zeppelin
zeppelin:0.7.1`
### Screenshots (if appropriate)
NONE
### Questions:
* Does the licenses files need update? - NO
* Is there breaking changes for older versions? - NO
* Does this needs documentation? - NO
Author: 1ambda <[email protected]>
Closes #2318 from 1ambda/ZEPPELIN-2492/use-single-dockerfile-for-each-tag
and squashes the following commits:
483bec3 [1ambda] docs: Update README for Dockerfile
5826c8c [1ambda] fix: Use single dockerfile for tag push
commit fb80bf7dd9cfe6ca5ff649d249a09c03f09ca166
Author: Christian Fries <[email protected]>
Date: 2017-04-30T19:44:12Z
[ZEPPELIN-2382] Fixed windows compatibility of interpreter binding.
### What is this PR for?
Fixing ZEPPELIN-2382:
A fresh install of Zeppelin 0.7.1 failed to start on a Windows due to the
use of
[getPosixFilePermissions](https://docs.oracle.com/javase/7/docs/api/java/nio/file/Files.html#getPosixFilePermissions(java.nio.file.Path,%20java.nio.file.LinkOption...))
This method fails on Windows with an UnsupportedOperationException -
however, recovering (that is ignoring the exception) appears to have no
negative effects. (For a discussion on this related to a different project see,
e.g., https://github.com/GoogleCloudPlatform/google-cloud-intellij/issues/937 )
### What type of PR is it?
[Bug Fix]
### Todos
* [x] - Have the binary distribution checked by the reporter of
[ZEPPELIN-2382]
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2382
### How should this be tested?
Check if clean distribution runs on Windows. Important: Ensure that
HADOOP_HOME points to a folder containing bin/winutils.exe since this pull
request does not include a fix for
https://issues.apache.org/jira/browse/ZEPPELIN-2438 (which is part of a
separate pull request).
### Screenshots (if appropriate)
N/A
### Questions:
* Does the licenses files need update?
No
* Is there breaking changes for older versions?
No
* Does this needs documentation?
No
Author: Christian Fries <[email protected]>
Closes #2305 from cfries/windows-compatiblity-interpreter-binding and
squashes the following commits:
e3b5738 [Christian Fries] Added graceful exception handling if posix
permission is not available
096fed6 [Christian Fries] Merge branch 'master' into apache/master
3635ed8 [Christian Fries] [ZEPPELIN-2375]: Avoid modification of CLASSPATH
variable.
commit 15ecbb49db17f3938ca0b4d584193354ff243429
Author: Alexander Shoshin <[email protected]>
Date: 2017-05-03T08:45:15Z
[ZEPPELIN-2467] Restoring second travis job functionality
### What is this PR for?
Second travis job didn't work correctly. It suppose to test all core unit
tests plus integration test (except for spark related test) but it was doing
nothing.
It was because of the exclamation mark in **-Dtest** property:
```
-Dtest='!ZeppelinSparkClusterTest,!org.apache.zeppelin.spark.*'
```
which is not supported by maven-surefire-plugin of version 2.17 (which is
used in Zeppelin). Exclamation mark is supported started from 2.19 but still
does not work properly.
I've added plugin **exclude** configuratin instead of **-Dtest** property.
After travis job was restored I had found that not all core tests were
working properly. I have excluded them from the travis job for now and created
jira issues:
https://issues.apache.org/jira/browse/ZEPPELIN-2469
https://issues.apache.org/jira/browse/ZEPPELIN-2470
https://issues.apache.org/jira/browse/ZEPPELIN-2471
https://issues.apache.org/jira/browse/ZEPPELIN-2473
### What type of PR is it?
[Bug Fix]
### What is the Jira issue?
[ZEPPELIN-2467](https://issues.apache.org/jira/browse/ZEPPELIN-2467)
### Questions:
* Does the licenses files need update? **no**
* Is there breaking changes for older versions? **no**
* Does this needs documentation? **no**
Author: Alexander Shoshin <[email protected]>
Closes #2300 from AlexanderShoshin/ZEPPELIN-2467 and squashes the following
commits:
78771e9 [Alexander Shoshin] made a comment about excluded tests
dfa332f [Alexander Shoshin] changed -Dtest flag to -Dtests.to.exclude and
excluded unstable tests
0448c4a [Alexander Shoshin] added ability to exclude some unit tests using
command line
commit 246c3d0264c9ed8b10ce47f4ea3599d2aebbdc47
Author: Thomas Grant <[email protected]>
Date: 2017-05-04T13:47:15Z
ZEPPELIN-2268. Adding png and jpg support for helium module imports.
### What is this PR for?
To allow helium modules to import CSS that contains references to png or
jpg images.
### What type of PR is it?
Improvement
### Todos
* [ ] - Task
### What is the Jira issue?
* [ZEPPELIN-2268]
### How should this be tested?
The volume-leaflet helium module
[https://github.com/volumeint/helium-volume-leaflet] imports the Leaflet css
file. That css file references a couple of png files. Before this change, the
helium module would fail to enable, after the change it succeeds.
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No.
* Is there breaking changes for older versions? No.
* Does this needs documentation? No.
Author: Thomas Grant <[email protected]>
Closes #2313 from volumeint/master and squashes the following commits:
672b789 [Thomas Grant] Merge branch 'master' of
https://github.com/apache/zeppelin
c736dbe [Thomas Grant] Merge branch 'master' of
https://github.com/apache/zeppelin
19500d5 [Thomas Grant] ZEPPELIN-2268. Adding png and jpg support for helium
module imports.
commit 28ef8aa90865a77b5aade4700bb945e6c0de9c3e
Author: Prabhjyot Singh <[email protected]>
Date: 2017-05-04T05:42:36Z
[minor] Fix JDBC doc after #2229
### What is this PR for?
Fix JDBC doc after #2229.
### What type of PR is it?
[Bug Fix]
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update?
* Is there breaking changes for older versions?
* Does this needs documentation?
Author: Prabhjyot Singh <[email protected]>
Closes #2314 from prabhjyotsingh/minor/jdbc-doc-zeppelin-2367 and squashes
the following commits:
e54a3a2b7 [Prabhjyot Singh] @AhyoungRyu review comments
0f396ac64 [Prabhjyot Singh] fix doc for zeppelin-2367
commit e5e3a88cd434880af6319392a61ef0571f8319d6
Author: David Ersson <[email protected]>
Date: 2017-05-10T16:03:19Z
[HOTFIX] Set interpreter context in zeppelin context from
SparkSqlInterpreter
### What is this PR for?
When running `%sql` paragraphs sometimes a NullPointer is thrown instead of
presenting the result.
This was traced back to
`sparkInterpreter.getZeppelinContext().setInterpreterContext(context);` not
being called as expected from SparkSqlInterpreter. Since the SparkInterpreter
is used from the SparkSqlInterpreter, if a `%spark` paragraph is executed
before a `%sql` paragraph it works as expected (as SparkInterpreter sets the
interpreter context in the zeppelin context).
### What type of PR is it?
[Hot Fix]
### Todos
### What is the Jira issue?
Hotfix
### How should this be tested?
Start the spark context by executing
```
%sql
select 1
```
Expected result is a table with 1 in.
Before this patch a NullPointer will be thrown.
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: David Ersson <[email protected]>
Closes #2334 from FireArrow/sqlinterpreter_hotfix and squashes the
following commits:
3d2b0a9 [David Ersson] Set interpreter context in zeppelin context
commit 4668da159fbd2b2243fbb3fedd55f2dc42beb64d
Author: Jongyoul Lee <[email protected]>
Date: 2017-05-10T06:22:57Z
[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'.
### What type of PR is it?
[Bug Fix]
### Todos
* [x] - Change style and adopt it
### What is the Jira issue?
N/A
### How should this be tested?
N/A
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: Jongyoul Lee <[email protected]>
Closes #2336 from jongyoul/minor/change-case-indent and squashes the
following commits:
26788c5f [Jongyoul Lee] Adopted new style
f95418f7 [Jongyoul Lee] Changed case indentation to 2
7d404a17 [Jongyoul Lee] Reformatted xml
commit 8194a5e0af0e1b926b27ea9e0d12bdf9d0e43b4b
Author: Jongyoul Lee <[email protected]>
Date: 2017-05-12T09:15:19Z
Revert "[ZEPPELIN-2467] Restoring second travis job functionality"
This reverts commit 15ecbb49db17f3938ca0b4d584193354ff243429.
commit 8056bc9b6298634464e0a1ab6b74040acac0584c
Author: soralee <[email protected]>
Date: 2017-04-24T04:47:02Z
[ZEPPELIN-2388] Improve the keyboard shortcuts dialog
## What is this PR for?
This PR is for Improving the keyboard shortcuts dialog form and here is
what i improve.
- change to table shape
- add scrollbar
- fixed that text is misaligned when resizing windows
## What type of PR is it?
[Bug fix | Improvement]
## What is the Jira issue?
* [ZEPPELIN-2388; Prettify Keyboard Shortcut
Dialog](https://issues.apache.org/jira/browse/ZEPPELIN-2388)
## How should this be tested?
1. Click `Keyboard Shortcuts' icon in a paragraph.
2. Check `Keyboard Shortcuts` shape.
3. Check resizing windows.
## Screenshots (if appropriate)
### [Before - 1. default]
### [Before - 2. text is misaligned when resizing windows]
### [Before - 3. not scrollbar]
### [After - 1. default]
### [After - 2. resizing windows (my image is a little broken.)]
### Last improvement dialog shape
### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No
Author: soralee <[email protected]>
Closes #2274 from soralee/ZEPPELIN-2388_shortcut_key and squashes the
following commits:
2428787 [soralee] remove separator and modify tight space
fdcf7f2 [soralee] modify as comments
6d0cee2 [soralee] improve the keyboard shortcuts dialog
commit ed2422a0961fe2c9671b70ab81707c200b969ab0
Author: 1ambda <[email protected]>
Date: 2017-05-10T01:47:15Z
[ZEPPELIN-2520] DON'T set bottom margin for markdown table
### What is this PR for?
DON'T set bottom margin for the markdown table.
#### Implementation Details
that's because `.paragraph table` overwrites `.markdown-body table`
### What type of PR is it?
[Bug Fix]
### Todos
* [x] - Bump version up from 2.4.0 to 2.6.0
* [x] - Update `bin_license/LICENSE`
* [x] - Import it using commonjs
* [x] - Fix paragraph.css
### What is the Jira issue?
[ZEPPELIN-2520](https://issues.apache.org/jira/browse/ZEPPELIN-2520)
### How should this be tested?
1. Build: `mvn clean package -DskipTests;`
2. Execute a paragraph using this text.
```md
Colons can be used to align columns.
| Tables | Are | Cool |
| ------------- |:-------------:| -----:|
| col 3 is | right-aligned | $1600 |
| col 2 is | centered | $12 |
| zebra stripes | are neat | $1 |
There must be at least 3 dashes separating each header cell.
The outer pipes (|) are optional, and you don't need to make the
raw Markdown line up prettily. You can also use inline Markdown.
Markdown | Less | Pretty
--- | --- | ---
*Still* | `renders` | **nicely**
1 | 2 | 3
```
### Screenshots (if appropriate)
#### Before
#### After
### Questions:
* Does the licenses files need update?
* Is there breaking changes for older versions?
* Does this needs documentation?
Author: 1ambda <[email protected]>
Closes #2327 from 1ambda/ZEPPELIN-2520/bottom-margin-for-md-table and
squashes the following commits:
2c54b66 [1ambda] fix: use css loader for test
49ea902 [1ambda] fix: DON'T set bottom-margin for markdown table
03bf9a3 [1ambda] fix: Use github-markdown-css 2.6.0 from npm
commit 5fde8c21d67ccba2d939387777bf9a7d28920838
Author: Vipin Rathor <[email protected]>
Date: 2017-05-10T09:49:48Z
[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.java. Since the documentation at
(http://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/security/shiroauthentication.html)
does not mention LdapRealm, the information should be correct in the code
comments at the least.
### What type of PR is it?
Documentation
### What is the Jira issue?
N/A
### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no
Author: Vipin Rathor <[email protected]>
Closes #2330 from VipinRathor/fix-LdapRealm-description and squashes the
following commits:
73148e3b9 [Vipin Rathor] [minor] Remove incorrect configuration and fix
description of LdapRealm
commit 95980c7ac0d3b53bb3068bfed630b5afd61cadaa
Author: Vipin Rathor <[email protected]>
Date: 2017-05-10T10:25:51Z
[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 defined in Shiro configuration
(http://zeppelin.apache.org/docs/0.8.0-SNAPSHOT/security/shiroauthentication.html).
With that, user can log in without entering his/her password. Zeppelin
documentation should recommend the correct value which is 'simple'.
### What type of PR is it?
Documentation
### What is the Jira issue?
N/A
### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? yes
Author: Vipin Rathor <[email protected]>
Closes #2331 from VipinRathor/fix-SIMPLE-auth-doc and squashes the
following commits:
a1f0e48 [Vipin Rathor] [minor] Replace 'SIMPLE' auth with 'simple' auth
everywhere in documents
commit d37684403af045b343b1008977bcf6deef06e1d5
Author: Benoy Antony <[email protected]>
Date: 2017-05-04T04:26:58Z
[ZEPPELIN-2463] Avoid Locking interpreterSettings during Notebook deletion
### What is this PR for?
Deletion of a notebook requires locking interpreterSettings. If the
deletion is delayed , then lock is not released. At that point, we cannot run
any notebook because everything is waiting to lock interpreterSettings.
Looking at the code, there is no reason to lock the InterpreterSettings
object in InterpreterSettingManager.removeNoteInterpreterSettingBinding.
Similarly in InterpreterSettingManager.getNoteInterpreterSettingBinding
only interpreterSettingBinding is accessed and its already a thread safe
object. So we can remove synchronization on InterpreterSettings
### What type of PR is it?
Bug Fix
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2463
### How should this be tested?
Being a concurrency issue, it is difficult to test.
Please see the comments in the jira to see the issue experienced on a
production zeppelin server.
### Questions:
* Does the licenses files need update? NO
* Is there breaking changes for older versions? NO
* Does this needs documentation? NO
Author: Benoy Antony <[email protected]>
Closes #2296 from benoyantony/ZEPPELIN-2463 and squashes the following
commits:
6f28d74 [Benoy Antony] replace check and remove with remove
0e0bb60 [Benoy Antony] fix checkstyle warnings
67f1514 [Benoy Antony] ZEPPELIN-2463 Avoid Locking interpreterSettings
during Notebook deletion
commit e857070f3bd71f1a9e0413b8c0eaebba1f6f7772
Author: 1ambda <[email protected]>
Date: 2017-05-09T06:46:31Z
[ZEPPELIN-2516] Set offset, tolerance for navbar, actionbar hiding
### What is this PR for?
Set offset, tolerance for navbar, actionbar hiding. See the attached
screenshots.
### What type of PR is it?
[Bug Fix]
### Todos
NONE
### What is the Jira issue?
[ZEPPELIN-2516](https://issues.apache.org/jira/browse/ZEPPELIN-2516)
### How should this be tested?
Refer the screenshots.
### Screenshots (if appropriate)
#### Before
#### After
### Questions:
* Does the licenses files need update? - NO
* Is there breaking changes for older versions? - NO
* Does this needs documentation? - NO
Author: 1ambda <[email protected]>
Closes #2325 from 1ambda/ZEPPELIN-2516/set-offset-tolerance and squashes
the following commits:
5d9317c [1ambda] fix: Set offset, tolerance
commit d92d14aae8678bba8b01d19abc8bbf875585e0ea
Author: 1ambda <[email protected]>
Date: 2017-05-04T04:36:57Z
[ZEPPELIN-2491] fix: Bump spell package version too
### What is this PR for?
[ZEPPELIN-2491] fix: Bump spell package version too
### What type of PR is it?
[Bug Fix]
### Todos
NONE
### What is the Jira issue?
[ZEPPELIN-2491](https://issues.apache.org/jira/browse/ZEPPELIN-2491)
### How should this be tested?
1. ` ./dev/change_zeppelin_version.sh 0.8.0-SNAPSHOT 0.8.0`
2. Check `/zeppelin-web/src/app/spell/package.json`
### Screenshots (if appropriate)
NONE
### Questions:
* Does the licenses files need update? - NO
* Is there breaking changes for older versions? - NO
* Does this needs documentation? - NO
Author: 1ambda <[email protected]>
Closes #2315 from 1ambda/ZEPPELIN-2491/bump-spell-package-version-too and
squashes the following commits:
20a8fae [1ambda] fix: Bump spell package version too
commit 164db0794fea1cf289068de71af2b62ae6811cfd
Author: 1ambda <[email protected]>
Date: 2017-05-08T16:35:38Z
[ZEPPELIN-2512] Prevent slow note, interpreter search. (master, branch-0.7)
### What is this PR for?
Use debounce in search input to avoid the slow search.
See more:
### What type of PR is it?
[Improvement]
### Todos
NONE
### What is the Jira issue?
[ZEPPELIN-2512](https://issues.apache.org/jira/browse/ZEPPELIN-2512)
### How should this be tested?
1. Create many notes with different names. (50+)
2. Search using note name filter in home and navbar.
3. Do the same thing for interpreters in the interpreter page
### Screenshots (if appropriate)
NONE
### Questions:
* Does the licenses files need update? - NO
* Is there breaking changes for older versions? - NO
* Does this needs documentation? - NO
Author: 1ambda <[email protected]>
Closes #2324 from 1ambda/ZEPPELIN-2512/add-debounce-for-search-inputs and
squashes the following commits:
20292fb [1ambda] feat: Add debounce for interpreter search input
b792ee5 [1ambda] feat: Add debounce for note name filter
----
---
![https://cloud.githubusercontent.com/assets/2031306/25446579/9dcdeb2a-2acf-11e7-883d-e0ccf2ef834d.png"](https://cloud.githubusercontent.com/assets/2031306/25446579/9dcdeb2a-2acf-11e7-883d-e0ccf2ef834d.png"){kind=link}
![https://cloud.githubusercontent.com/assets/2031306/25446593/a66289da-2acf-11e7-8d8e-2029339f0133.png"](https://cloud.githubusercontent.com/assets/2031306/25446593/a66289da-2acf-11e7-8d8e-2029339f0133.png"){kind=link}
![https://cloud.githubusercontent.com/assets/1642088/25606467/62f0901c-2f4c-11e7-95ef-c22f1df410be.png"](https://cloud.githubusercontent.com/assets/1642088/25606467/62f0901c-2f4c-11e7-95ef-c22f1df410be.png"){kind=link}
