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 (sudheer0...@gmail.com) 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 <prabhjyotsi...@gmail.com> 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 <prabhjyotsi...@gmail.com> 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 <zjf...@apache.org> 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 <zjf...@apache.org> 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 <e...@users.noreply.github.com> 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 <e...@users.noreply.github.com> 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 <mress...@diamondkinetics.com> 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 <mress...@diamondkinetics.com> Closes #2308 from mressler/master and squashes the following commits: aa39597 [Michael Ressler] Some grammar updates. commit 82c92d110ccbdd986c58e5600a62f7cd44aedfb1 Author: Renjith Kamath <renjith.kam...@gmail.com> 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 <renjith.kam...@gmail.com> 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 <krish.pande...@gmail.com> 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 <krish.pande...@gmail.com> 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 <prabhjyotsi...@gmail.com> 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 <prabhjyotsi...@gmail.com> 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 <zjf...@apache.org> 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 ![zeppelin_before](https://cloud.githubusercontent.com/assets/164491/25662178/227ec2ec-3046-11e7-9852-9e041d008698.gif) After ![zeppelin_after](https://cloud.githubusercontent.com/assets/164491/25662177/2108e898-3046-11e7-816f-7685480df83e.gif) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Jeff Zhang <zjf...@apache.org> 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 <cloverhearts...@gmail.com> 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 <cloverhearts...@gmail.com> 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 <cloverhearts...@gmail.com> 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 ![incorrectrevisionupdate](https://cloud.githubusercontent.com/assets/10525473/25425492/1a0ebcce-2aa7-11e7-9a06-cfc84a1c1fe3.gif) #### fixed (this pr) ![correctrevisionupdate](https://cloud.githubusercontent.com/assets/10525473/25425498/1dce1bfc-2aa7-11e7-816c-c25a64963475.gif) ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no Author: CloverHearts <cloverhearts...@gmail.com> 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 <jongy...@gmail.com> 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 <jongy...@gmail.com> 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 <be...@apache.org> 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 <be...@apache.org> 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 <khalid...@gmail.com> 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: ![screen shot 2017-05-02 at 6 55 24 pm](https://cloud.githubusercontent.com/assets/1642088/25613839/aac216a4-2f6a-11e7-8cab-a77ddb48382b.png) ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no Author: Khalid Huseynov <khalid...@gmail.com> Closes #2306 from khalidhuseynov/fix/ZEPPELIN-2479 and squashes the following commits: ac4e651 [Khalid Huseynov] add css class commit d9c4a5f0b6c3355753b50f466199cbe551cbd89a Author: Jongyoul Lee <jongy...@gmail.com> 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 <jongy...@gmail.com> 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 <1am...@gmail.com> 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 <1am...@gmail.com> 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 <em...@christian-fries.de> 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 <em...@christian-fries.de> 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 <alexander_shos...@epam.com> 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 <alexander_shos...@epam.com> 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 <tgr...@volumeintegration.com> 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 <tgr...@volumeintegration.com> 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 <prabhjyotsi...@gmail.com> 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 <prabhjyotsi...@gmail.com> 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 <ders...@kth.se> 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 <ders...@kth.se> 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 <jongy...@gmail.com> 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 <jongy...@gmail.com> 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 <jongy...@apache.org> Date: 2017-05-12T09:15:19Z Revert "[ZEPPELIN-2467] Restoring second travis job functionality" This reverts commit 15ecbb49db17f3938ca0b4d584193354ff243429. commit 8056bc9b6298634464e0a1ab6b74040acac0584c Author: soralee <sora0...@zepl.com> 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] ![old_keyboard_shortcut](https://cloud.githubusercontent.com/assets/8110458/25274009/412a3f68-26c9-11e7-82ac-07835a04de83.png) ### [Before - 2. text is misaligned when resizing windows] ![broken_shortcuts](https://cloud.githubusercontent.com/assets/8110458/25274381/c27c75d0-26ca-11e7-9262-4955bc4fcd42.png) ### [Before - 3. not scrollbar] ![need_scrollbar](https://cloud.githubusercontent.com/assets/8110458/25274313/76b4097e-26ca-11e7-8f11-599629c6c2b3.gif) ### [After - 1. default] ![new_keboard_shortcuts](https://cloud.githubusercontent.com/assets/8110458/25274041/5919f2bc-26c9-11e7-82a9-bd25526b118b.gif) ### [After - 2. resizing windows (my image is a little broken.)] ![resizing_shortcut](https://cloud.githubusercontent.com/assets/8110458/25274044/5c4e7494-26c9-11e7-9f23-2dbf2c9cf40f.gif) ### Last improvement dialog shape ![image](https://cloud.githubusercontent.com/assets/8110458/25366441/9a70432c-29aa-11e7-9b07-8a6b7cf9cc8a.png) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: soralee <sora0...@zepl.com> 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 <1am...@gmail.com> 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` ![image](https://cloud.githubusercontent.com/assets/4968473/25874159/542adc12-354c-11e7-93fd-7bdb65766b63.png) ### 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 ![image](https://cloud.githubusercontent.com/assets/4968473/25874112/14d03a08-354c-11e7-8abb-8a270cd0b35b.png) #### After ![image](https://cloud.githubusercontent.com/assets/4968473/25874100/025c030c-354c-11e7-9c66-c0c3874304a9.png) ### Questions: * Does the licenses files need update? * Is there breaking changes for older versions? * Does this needs documentation? Author: 1ambda <1am...@gmail.com> 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 <v.rat...@gmail.com> 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 <v.rat...@gmail.com> 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 <v.rat...@gmail.com> 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 <v.rat...@gmail.com> 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 <be...@apache.org> 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 <be...@apache.org> 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 <1am...@gmail.com> 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 ![2516_before](https://cloud.githubusercontent.com/assets/4968473/25854795/4221f2e8-350c-11e7-8ce9-447f2f77d489.gif) #### After ![2516_after](https://cloud.githubusercontent.com/assets/4968473/25854802/476af2b8-350c-11e7-8fea-8e1d94fcafc4.gif) ### Questions: * Does the licenses files need update? - NO * Is there breaking changes for older versions? - NO * Does this needs documentation? - NO Author: 1ambda <1am...@gmail.com> Closes #2325 from 1ambda/ZEPPELIN-2516/set-offset-tolerance and squashes the following commits: 5d9317c [1ambda] fix: Set offset, tolerance commit d92d14aae8678bba8b01d19abc8bbf875585e0ea Author: 1ambda <1am...@gmail.com> 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 <1am...@gmail.com> 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 <1am...@gmail.com> 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: ![image](https://cloud.githubusercontent.com/assets/4968473/25814653/3be8890e-3459-11e7-9be2-6a440ab41861.png) ### 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 <1am...@gmail.com> 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 ---- ---