Add force layout visualization in paragraphs for SQL queries #1044
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 5 commits
June 18, 2016 14:47
…of interpreter is being used ### What is this PR for? Cron job can fail when notebook uses multiple types of paragraphs. Problem reported here http://apache-zeppelin-users-incubating-mailing-list.75479.x6.nabble.com/Cron-job-fails-to-run-a-paragraph-that-runs-correctly-manually-tt2265.html ### What type of PR is it? Bug Fix ### Todos * [x] - Fix * [x] - Unittest ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-1015 ### How should this be tested? Create two paragraphs in the notebook First takes longer than second (last) paragraph. First paragraph and second paragraph should use different interpreter. If cron schedule the notebook with 'auto-restart interpreter on cron execution' checked. Then interpreters will be restarted when second paragraph finished, but first paragraph is still running. That may cause abort of first paragraph run. ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no Author: Lee moon soo <[email protected]> Closes #1019 from Leemoonsoo/ZEPPELIN-1015 and squashes the following commits: ccee60a [Lee moon soo] update unittest 9ad4cbb [Lee moon soo] Fix problem by waiting all paragraphs in note be finished
### What is this PR for? Remove .Rhistory from git file tracking Author: Mina Lee <[email protected]> Closes #1036 from minahlee/ignore_rhistory and squashes the following commits: d3278e7 [Mina Lee] Add .Rhistory file to .gitignore
### What is this PR for? This PR is to make available information on the currently authenticated user(subject) in the Notebook storage level. It can be used for multiple purposes (ACL, login to third parties, etc.) once storage layer is user aware. It basically updates NotebookRepo api with additional user info ### What type of PR is it? Improvement ### Todos * [x] - add authInfo(subject) to list() * [x] - add authInfo(subject) to get() * [x] - add authInfo(subject) to save() * [x] - add authInfo(subject) to remove() ### What is the Jira issue? ### How should this be tested? storage layer should be working as before with and without authenticated user, green CI ### Screenshots (if appropriate) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? not breaking, but some api changes * Does this needs documentation? No Author: Khalid Huseynov <[email protected]> Closes #998 from khalidhuseynov/update-api/pass-auth-info-to-repo and squashes the following commits: b91e78a [Khalid Huseynov] propagate subject to versioning api f3bae60 [Khalid Huseynov] fix checkstyle bb57eae [Khalid Huseynov] address changes for job management pr after rebasing 8b48577 [Khalid Huseynov] fix test after master merge a69d04f [Khalid Huseynov] add descriptions to NotebookRepo interface 0bf40b3 [Khalid Huseynov] propagate changes to tests aa95537 [Khalid Huseynov] propagate changes upstream a0ebd14 [Khalid Huseynov] add subject to checkpoint signature of all repos 50ba14a [Khalid Huseynov] add subject to NotebookRepo api bd0a290 [Khalid Huseynov] propagate changes to tests 91426af [Khalid Huseynov] propagate changes with remove upstream 232d1af [Khalid Huseynov] propagate remove(noteId, subject) to all repo signatures d0d7b98 [Khalid Huseynov] add subject to repo remove api 21f189d [Khalid Huseynov] propagate changes to save into tests b5f88e2 [Khalid Huseynov] propagate subject changes upstream b7b007a [Khalid Huseynov] add subject to all repo signatures 0a4a8d2 [Khalid Huseynov] add subject to repo save api 8fdaed3 [Khalid Huseynov] propagate changes to tests 3ea544d [Khalid Huseynov] propagate changes up to Notebook 91fc500 [Khalid Huseynov] modify all repo signatures with get(noteId, subject) fb93e22 [Khalid Huseynov] add subject to repo api for get 7d964c7 [Khalid Huseynov] propagate list(subject) to tests 8e7d8bd [Khalid Huseynov] propagate changes to list up f66dc1b [Khalid Huseynov] modify all repo signatureswith list(subject) 1c29bee [Khalid Huseynov] change api for repo list
### What is this PR for? This PR is for blocking unnecessary zeppelin-web build errors. ``` [ERROR] npm WARN package.json zeppelin-web0.0.0 No license field. [ERROR] bower perfect-scrollbar extra-resolution Unnecessary resolution: perfect-scrollbar#~0.5.4 ``` ### What type of PR is it? Improvement ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-1014 ### How should this be tested? You can run ```mvn clean package -DskipTests``` in the zeppelin-web folder. ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no Author: astroshim <[email protected]> Closes #1018 from astroshim/ZEPPELIN-1014 and squashes the following commits: 991ef8c [astroshim] fix zeppelin-web build error.
### What is this PR for? Most of other projects have **Quick Start** or **Getting Started** page for the beginner. Currently, Zeppelin also has [Zeppelin Install](https://zeppelin.apache.org/docs/0.6.0-SNAPSHOT/install/install.html) which is similar with those kind of instruction page. But it has only contents that explain just installation and configuration. So I updated this page to **Quick Start** so that it can include step by step guide for the beginners. ### What type of PR is it? Improvement & Documentation ### Todos * [x] - Add each title link to the head of documentation * [x] - Add more information about Zeppelin installation * [x] - Reorder contents ### What is the Jira issue? [ZEPPELIN-998](https://issues.apache.org/jira/browse/ZEPPELIN-998) ### How should this be tested? See the attached screenshot images ### Screenshots (if appropriate) <img width="809" alt="screen shot 2016-06-14 at 2 59 47 pm" src="https://cloud.githubusercontent.com/assets/10060731/16061227/d88abe56-3240-11e6-845c-e37a9975aceb.png"> <img width="796" alt="screen shot 2016-06-14 at 3 00 01 pm" src="https://cloud.githubusercontent.com/assets/10060731/16061229/daca8b06-3240-11e6-821b-7d118b7b3e09.png"> <img width="789" alt="screen shot 2016-06-14 at 3 00 15 pm" src="https://cloud.githubusercontent.com/assets/10060731/16061234/dd12072c-3240-11e6-9a0a-cf1e320fd879.png"> <img width="785" alt="screen shot 2016-06-14 at 3 00 27 pm" src="https://cloud.githubusercontent.com/assets/10060731/16061238/df7f904c-3240-11e6-83e0-73f3688c0815.png"> <img width="789" alt="screen shot 2016-06-14 at 3 00 38 pm" src="https://cloud.githubusercontent.com/assets/10060731/16061242/e1ad1402-3240-11e6-81ba-2e7125cec98e.png"> ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: AhyoungRyu <[email protected]> Closes #1010 from AhyoungRyu/ZEPPELIN-998 and squashes the following commits: b2b1aa8 [AhyoungRyu] Change some sentences as @bzz suggested 12da298 [AhyoungRyu] Fix pointing link in index.md af50576 [AhyoungRyu] Address @bzz feedback af68a32 [AhyoungRyu] Extend install.md -> Quick Start
### What is this PR for? Switch to FIFO scheduler as in current implementation `.interpret()` is not thread-safe and so in parallel one 'Run All' fails some paragraphs with NPE in logs ### What type of PR is it? Bug Fix | Improvement ### How should this be tested? 'Run All' passes without NPE in logs i.e on this [Zeppelin notebook for python](https://www.zeppelinhub.com/viewer/notebooks/aHR0cHM6Ly9yYXcuZ2l0aHVidXNlcmNvbnRlbnQuY29tL2J6ei9pbmN1YmF0b3ItemVwcGVsaW4vMTkyZjU3YjZjMGZkMjc4NzgwZDI3NDAzMGY1YmJlOTZlZThkNzdiYi9ub3RlYm9vay8yQlFBMzVDSlovbm90ZS5qc29u) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Alexander Bezzubov <[email protected]> Closes #1033 from bzz/fix/python-run-all and squashes the following commits: 72e9d62 [Alexander Bezzubov] Python: switch to FIFO scheduler
### What is this PR for? It adds basic [python matplotlib example notebook](https://www.zeppelinhub.com/viewer/notebooks/aHR0cHM6Ly9yYXcuZ2l0aHVidXNlcmNvbnRlbnQuY29tL2J6ei9pbmN1YmF0b3ItemVwcGVsaW4vMTkyZjU3YjZjMGZkMjc4NzgwZDI3NDAzMGY1YmJlOTZlZThkNzdiYi9ub3RlYm9vay8yQlFBMzVDSlovbm90ZS5qc29u). ### What type of PR is it? Improvement | Documentation ### What is the Jira issue? [ZEPPELIN-1027](https://issues.apache.org/jira/browse/ZEPPELIN-1027) ### How should this be tested? New [zeppelin notebook](https://www.zeppelinhub.com/viewer/notebooks/aHR0cHM6Ly9yYXcuZ2l0aHVidXNlcmNvbnRlbnQuY29tL2J6ei9pbmN1YmF0b3ItemVwcGVsaW4vMTkyZjU3YjZjMGZkMjc4NzgwZDI3NDAzMGY1YmJlOTZlZThkNzdiYi9ub3RlYm9vay8yQlFBMzVDSlovbm90ZS5qc29u) shows up in the list. ### 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: Alexander Bezzubov <[email protected]> Closes #1032 from bzz/python/add-example-notebook and squashes the following commits: 192f57b [Alexander Bezzubov] Python: add basic matplotlib example notebook
…execution ### What is this PR for? Changed scheduler from FIFO to Parallels in JdbcInterpreter. This is a default behaviour of HiveInterpreter. When we merge all JDBC-like interpreter into JDBC, we need to change default behaviour of JdbcInterpreter. ### What type of PR is it? [Feature] ### Todos * [x] - Changed scheduler ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-995 ### How should this be tested? You can run multiple queries simultaneously. ### 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 #1005 from jongyoul/ZEPPELIN-995 and squashes the following commits: af360fa [Jongyoul Lee] Added option to choose which scheduler we use 3bda988 [Jongyoul Lee] Changed scheduler from FIFO to Parallels in JdbcInterpreter
### What is this PR for? The suggestion list under note permissions for readers, writers & owners should hide on-click anywhere on the notebook. ### What type of PR is it? [Bug Fix] ### Todos ### What is the Jira issue? [ZEPPELIN-1020](https://issues.apache.org/jira/browse/ZEPPELIN-1020) ### How should this be tested? 1.Click on any field for setting permission for owners, readers & writers. 2.The suggestion list is displayed. 3.Now, click on any part of the notebook the suggestion list hides. ### Screenshots (if appropriate) **BEFORE:** http://g.recordit.co/7Esaq245Tp.gif **AFTER** http://g.recordit.co/Rfgn5dONPz.gif ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: suvam97 <[email protected]> Author: System Administrator <[email protected]> Closes #1027 from suvam97/ZEPPELIN-1020 and squashes the following commits: 29870cd [suvam97] changed $ to angular.element c9ef56c [suvam97] add angular.element in place of $ 4372f49 [System Administrator] remove suggestion list on click
### What is this PR for? Enable selenium test with SafariDriver. ### What type of PR is it? [Bug Fix] ### Todos * [x] - Upgrade guava version ### What is the Jira issue? N/A ### How should this be tested? 1. Install selenium driver for Safari 1. Make sure that you don't have firefox installed 1. Start Zeppelin without this patch 1. run a selenium test. `TEST_SELENIUM="true" mvn package -DfailIfNoTests=false -pl 'zeppelin-interpreter,zeppelin-zengine,zeppelin-server' -Dtest=ParagraphActionsIT` 1. Got error 1. Repeat after patch 1. Success ### Screenshots (if appropriate) Got error like: ``` java.lang.NoSuchMethodError: com.google.common.util.concurrent.MoreExecutors.directExecutor()Ljava/util/concurrent/Executor; at org.openqa.selenium.safari.WebSocketConnection.send(WebSocketConnection.java:139) at org.openqa.selenium.safari.SafariDriverCommandExecutor.execute(SafariDriverCommandExecutor.java:188) at org.openqa.selenium.remote.RemoteWebDriver.execute(RemoteWebDriver.java:620) at org.openqa.selenium.remote.RemoteWebDriver.startSession(RemoteWebDriver.java:247) at org.openqa.selenium.remote.RemoteWebDriver.<init>(RemoteWebDriver.java:129) at org.openqa.selenium.remote.RemoteWebDriver.<init>(RemoteWebDriver.java:142) at org.openqa.selenium.safari.SafariDriver.<init>(SafariDriver.java:62) at org.openqa.selenium.safari.SafariDriver.<init>(SafariDriver.java:42) at org.apache.zeppelin.WebDriverManager.getWebDriver(WebDriverManager.java:103) at org.apache.zeppelin.integration.ParagraphActionsIT.startUp(ParagraphActionsIT.java:51) ``` ### 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 #1035 from jongyoul/minor-upgrade-guava-for-selenium-with-safaridriver and squashes the following commits: 6f93278 [Jongyoul Lee] Changed guava version to 19.0
### What is this PR for? Automatically adds %pyspark in the code area after paragraph created, if previous paragraph's typing is %pyspark. ### What type of PR is it? New Feature ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-707 ### How should this be tested? unit test Run-time checking ### Screenshots (if appropriate) - default interpreter  - while run paragraph, go to line end  Author: Minwoo Kang <[email protected]> Closes #806 from mwkang/ZEPPELIN-707 and squashes the following commits: 3652dbb [Minwoo Kang] [ZEPPELIN-707]Add test case a159903 [Minwoo Kang] [ZEPPELIN-707]Modify return empty string when repl name is blank 720ffba [Minwoo Kang] [ZEPPELIN-707]Add note.setLastReplName in Rest API 4c0904b [Minwoo Kang] [ZEPPELIN-707]Resolved conflicts 0fd122c [Minwoo Kang] [ZEPPELIN-707]Modify checking nullable list location 85ee14b [Minwoo Kang] [ZEPPELIN-707]Fix test in error 334b15f [Minwoo Kang] [ZEPPELIN-707]Remove OccupiedInterpreter class. Add lastReplName in Note.java f3ab5ea [Minwoo Kang] [ZEPPELIN-707]Fix cannot find symbol error 532e98e [Minwoo Kang] [ZEPPELIN-707]Fix cannot find symbol error 1484d1c [Minwoo Kang] [ZEPPELIN-707]Change cursor of new paragraph go at the end of the line 2366709 [Minwoo Kang] [ZEPPELIN-707]Use ConcurrentHashMap instead of HashMap caeb9a2 [Minwoo Kang] [ZEPPELIN-707]Use ConcurrentHashMap instead of HashMap 854e96b [Minwoo Kang] [ZEPPELIN-707]Fix 'paragraphId' used out of scope. 3c7485d [Minwoo Kang] [ZEPPELIN-707]Fix go to line end 7dd3f68 [Minwoo Kang] [ZEPPELIN-707]Make %.. is default interpreter 27f3925 [Minwoo Kang] [ZEPPELIN-707]Fixed ParagraphActionsIT test 011f4e3 [Minwoo Kang] [ZEPPELIN-707]Fixed interpreter parsing problems 83003f5 [Minwoo Kang] [ZEPPELIN-707]Fixed test f1e59b2 [Minwoo Kang] [ZEPPELIN-707]Automatically adds %.. of previous paragraph's typing
### What is this PR for? Fix selenium test error described in [ZEPPELIN-1009](https://issues.apache.org/jira/browse/ZEPPELIN-1009) ### What type of PR is it? Hot Fix ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-1009 ### How should this be tested? Outline the steps to test the PR here. ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no Author: Lee moon soo <[email protected]> Closes #1034 from Leemoonsoo/ZEPPELIN-1009 and squashes the following commits: 9a048af [Lee moon soo] restore changes 13e15f1 [Lee moon soo] print browser log 63e8682 [Lee moon soo] sleep ec03834 [Lee moon soo] try trusty f05d741 [Lee moon soo] set firefox addon ver c33d9ff [Lee moon soo] try different xpath 430e273 [Lee moon soo] restore f145da5 [Lee moon soo] Take some screenshots b4cab5f [Lee moon soo] Remove debug mesg 1c1b2b8 [Lee moon soo] Increase resolution b2edf6f [Lee moon soo] change xvfb screen resolution 41d9875 [Lee moon soo] more msg 4d27bf7 [Lee moon soo] More mesg 1dd6e50 [Lee moon soo] add more debug msg 54d58b6 [Lee moon soo] escape quote fb3e9f6 [Lee moon soo] Add a paragraph infront to prevent main menu cover angular element 667578c [Lee moon soo] add debug messages da38256 [Lee moon soo] update getParagraphXPath() 55e78ec [Lee moon soo] trigger ci 13dee87 [Lee moon soo] fix
### What is this PR for? This PR is for fixing typo of the interpreter menu description. ### What type of PR is it? Bug Fix ### 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: astroshim <[email protected]> Closes #1037 from astroshim/feat/typoInterpreters and squashes the following commits: 97f1fb7 [astroshim] space bind/unbind too. 64f1f60 [astroshim] fix typo description of interpreter menu.
### What is this PR for? This PR is for fixing [this issue](https://issues.apache.org/jira/browse/ZEPPELIN-1033) ### What type of PR is it? Bug Fix ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-1033 ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no Author: astroshim <hsshim.nflabs.com> Author: root <[email protected]> Closes #1048 from astroshim/ZEPPELIN-1033 and squashes the following commits: eb80103 [astroshim] remove importing Ini and spaces 8eeb1bf [astroshim] update version of shiro config module. 5a2b268 [astroshim] add shiro ini to SecurityManager. 13a6139 [astroshim] add shiro-config-core 6c99111 [root] add initSecurityManager method.
### What is this PR for? dec31d6 After updating the code, and test cases fail. The code was modified to add code to the test. The interpreter automatically name is added, and therefore empty para graph is not generated. ``` testCreateNewButton(org.apache.zeppelin.integration.ParagraphActionsIT) Time elapsed: 11.069 sec <<< FAILURE! java.lang.AssertionError: Paragraph is created above Expected: "" but: was "%spark " at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20) at org.junit.Assert.assertThat(Assert.java:865) at org.junit.rules.ErrorCollector$1.call(ErrorCollector.java:65) at org.junit.rules.ErrorCollector.checkSucceeds(ErrorCollector.java:78) at org.junit.rules.ErrorCollector.checkThat(ErrorCollector.java:63) at org.apache.zeppelin.integration.ParagraphActionsIT.testCreateNewButton(ParagraphActionsIT.java:95) testCreateNewButton(org.apache.zeppelin.integration.ParagraphActionsIT) Time elapsed: 11.069 sec <<< FAILURE! java.lang.AssertionError: Paragraph is created below Expected: "" but: was "%spark " at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20) at org.junit.Assert.assertThat(Assert.java:865) at org.junit.rules.ErrorCollector$1.call(ErrorCollector.java:65) at org.junit.rules.ErrorCollector.checkSucceeds(ErrorCollector.java:78) at org.junit.rules.ErrorCollector.checkThat(ErrorCollector.java:63) at org.apache.zeppelin.integration.ParagraphActionsIT.testCreateNewButton(ParagraphActionsIT.java:105) ``` ### What type of PR is it? Bug Fix ### Todos * [x] - Updated Test case (createNewButton - ParagraphActionsIT.java) ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-1039 ### 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 #1053 from cloverhearts/fixed/paragraphActionsIT-AutomaticallyintpTag and squashes the following commits: bbff034 [CloverHearts] modifed paragraphActionIT Test case - automatically interpreter tag.
Author
|
@corneadoug Have you seen this new PR? let me know if everything's good |
### What is this PR for? Small ui improvements - Change keyboard shortcut icon - Modify commit container style - Change table asc/desc icon - Change interpreter create button font ### What type of PR is it? Improvement ### Screenshots (if appropriate) **Before** <img width="165" alt="screen shot 2016-06-13 at 12 55 41 pm" src="https://cloud.githubusercontent.com/assets/8503346/16021450/aebc14dc-3167-11e6-91d2-32fa8cd607af.png"> **After** <img width="164" alt="screen shot 2016-06-13 at 12 55 54 pm" src="https://cloud.githubusercontent.com/assets/8503346/16021451/b159c86a-3167-11e6-9311-b59cfffb3f6b.png"> **Before** <img width="254" alt="screen shot 2016-06-13 at 1 07 15 pm" src="https://cloud.githubusercontent.com/assets/8503346/16021480/d23661e2-3167-11e6-90f9-44340d5efe8f.png"> **After** <img width="252" alt="screen shot 2016-06-13 at 1 07 26 pm" src="https://cloud.githubusercontent.com/assets/8503346/16021482/d4cf92ac-3167-11e6-95a0-1eb28d0ed325.png"> **Before** <img width="82" alt="screen shot 2016-06-18 at 11 08 20 am" src="https://cloud.githubusercontent.com/assets/8503346/16172808/03295cec-3545-11e6-953a-c2c9a48ca841.png"> **After** <img width="82" alt="screen shot 2016-06-21 at 2 17 38 am" src="https://cloud.githubusercontent.com/assets/8503346/16224348/86c3f73e-3756-11e6-8b50-a4654d738d18.png"> **Before** <img width="383" alt="screen shot 2016-06-14 at 4 43 58 pm" src="https://cloud.githubusercontent.com/assets/8503346/16063627/60bb32de-324f-11e6-88a3-318575dcafc2.png"> **After** <img width="384" alt="screen shot 2016-06-14 at 4 43 20 pm" src="https://cloud.githubusercontent.com/assets/8503346/16063628/653316ba-324f-11e6-92ca-b559b62e234c.png"> **Before** <img width="113" alt="screen shot 2016-06-14 at 9 38 05 pm" src="https://cloud.githubusercontent.com/assets/8503346/16068142/54f5f6d6-3278-11e6-977a-1902a274c1f1.png"> **After** <img width="105" alt="screen shot 2016-06-14 at 9 37 34 pm" src="https://cloud.githubusercontent.com/assets/8503346/16068144/59344c70-3278-11e6-8038-ff3775cc432a.png"> ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? Screenshot needs to be updated Author: Mina Lee <[email protected]> Closes #1006 from minahlee/minor/ui_improve and squashes the following commits: dc6dbe8 [Mina Lee] Change interpreter create button font 72ff26e [Mina Lee] Change table asc desc icon bbca1ba [Mina Lee] Refine version control action UI 68619f8 [Mina Lee] Change keyboard shortcut icon
### What is this PR for? In the current implementation JDBC interpret only catches SQLException and ClassNotFoundException. However, there can be a case for example NullPointerException or for that matter anything else can be thrown. ### What type of PR is it? [Improvement] ### Todos * [x] - increase more logs, and return appropriate error to front-end ### What is the Jira issue? * Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/ * Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. [ZEPPELIN-533] ### How should this be tested? As an example for hive2. #### Properties |name| value| | ------------- |:-------------:| |default.driver| org.apache.hive.jdbc.HiveDriver| |default.url |jdbc:hive2://hive-machine:2181/;serviceDiscoveryMode=zooKeeper;zooKeeperNamespace=hiveserver2| #### Dependencies |artifact |exclude| | ------------- |:-------------:| |org.apache.hive:hive-jdbc:1.2.1 || |org.apache.hadoop:hadoop-common:2.7.2|| With the above setting used on an older driver for hive2 it shows exception in log, but on the front-end paragraph shows error, and no logs. ``` ERROR [2016-06-19 14:47:46,186] ({pool-2-thread-2} Job.java[run]:182) - Job failed java.lang.NullPointerException at org.apache.thrift.transport.TSocket.open(TSocket.java:168) at org.apache.thrift.transport.TSaslTransport.open(TSaslTransport.java:248) at org.apache.thrift.transport.TSaslClientTransport.open(TSaslClientTransport.java:37) at org.apache.hive.jdbc.HiveConnection.openTransport(HiveConnection.java:190) at org.apache.hive.jdbc.HiveConnection.<init>(HiveConnection.java:163) at org.apache.hive.jdbc.HiveDriver.connect(HiveDriver.java:105) at java.sql.DriverManager.getConnection(DriverManager.java:571) at java.sql.DriverManager.getConnection(DriverManager.java:187) at org.apache.zeppelin.jdbc.JDBCInterpreter.getConnection(JDBCInterpreter.java:218) at org.apache.zeppelin.jdbc.JDBCInterpreter.getStatement(JDBCInterpreter.java:229) at org.apache.zeppelin.jdbc.JDBCInterpreter.executeSql(JDBCInterpreter.java:288) at org.apache.zeppelin.jdbc.JDBCInterpreter.interpret(JDBCInterpreter.java:393) at org.apache.zeppelin.interpreter.LazyOpenInterpreter.interpret(LazyOpenInterpreter.java:94) at org.apache.zeppelin.interpreter.remote.RemoteInterpreterServer$InterpretJob.jobRun(RemoteInterpreterServer.java:341) at org.apache.zeppelin.scheduler.Job.run(Job.java:176) at org.apache.zeppelin.scheduler.FIFOScheduler$1.run(FIFOScheduler.java:139) at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471) at java.util.concurrent.FutureTask.run(FutureTask.java:262) at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178) at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292) at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615) at java.lang.Thread.run(Thread.java:745) ``` ### Screenshots (if appropriate) Before: <img width="1436" alt="screen shot 2016-06-20 at 5 11 59 pm" src="https://cloud.githubusercontent.com/assets/674497/16192988/20ce0312-370a-11e6-967e-281e99dbb18f.png"> After: <img width="1435" alt="screen shot 2016-06-20 at 5 10 26 pm" src="https://cloud.githubusercontent.com/assets/674497/16192984/1121d222-370a-11e6-8da9-43adc80c9c15.png"> ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no Author: Prabhjyot Singh <[email protected]> Closes #1051 from prabhjyotsingh/ZEPPELIN-1036 and squashes the following commits: 595ebc6 [Prabhjyot Singh] have e.getStackTrace() in all cases. 2e8dc0c [Prabhjyot Singh] increase more logs, and return appropriate error to front-end
### What is this PR for? This PR is to automate release publish to maven repository. We used to use maven-deploy-plugin and maven-release-plugin for release but somehow it didn't work well with Zeppelin so 0.5.5 and 0.5.6 haven't been published to maven repository. Publishing release to maven repository will eventually help zeppelin to reduce binary package size by leading users to use Dynamic interpreter loading(#908). Originally below modules were skipped for maven release - all interpreters(except spark) - zeppelin-display - zeppelin-server - zeppelin-distribution on the other hand this pr will skip only: - zeppelin-distribution ### What type of PR is it? Infra ### Todos - [x] Include SparkR/R interpreter in release - [x] Create common_release.sh to remove build configuration duplication - [x] Check curl networking failure ### What is the Jira issue? [ZEPPELIN-986](https://issues.apache.org/jira/browse/ZEPPELIN-986) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? Yes, https://cwiki.apache.org/confluence/display/ZEPPELIN/Preparing+Zeppelin+Release will be updated accordingly once this pr is merged. Author: Mina Lee <[email protected]> Closes #994 from minahlee/ZEPPELIN-986 and squashes the following commits: b0e8e67 [Mina Lee] Revert "Add geode, scalding profile in maven artifact build" cd4cbcd [Mina Lee] curl failure check c0ea07c [Mina Lee] Fix wrong indentation a88bc1d [Mina Lee] Add geode, scalding profile in maven artifact build 2cced61 [Mina Lee] Add r to binary package and maven build 903bc12 [Mina Lee] Move duplicate code to common_release.sh a3eb676 [Mina Lee] Include zeppelin-server module in publish artifiact 48d338f [Mina Lee] Rollback mistakenly removed plugin aafaf42 [Mina Lee] Follow google shell style guide 30dcc65 [Mina Lee] remove deploy plugin from pom since custom script will be used instead for deploy cd1f08c [Mina Lee] Refactor create release script e764f5f [Mina Lee] Add maven publish release script
### What is this PR for? This PR is fixing import/clone notebook with error result. This PR adds test based on #933. > Note: This issue is one of the [blockers](https://issues.apache.org/jira/browse/ZEPPELIN-889) of 0.6.0 release so should be merged into branch-0.6 before release. ### What type of PR is it? Bug Fix ### What is the Jira issue? [ZEPPELIN-905](https://issues.apache.org/jira/browse/ZEPPELIN-905) ### How should this be tested? When you try to import or clone notebook with error result, it should work. ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Kevin Kim <[email protected]> Author: Mina Lee <[email protected]> Closes #1043 from minahlee/ZEPPELIN-905 and squashes the following commits: 69b8c02 [Mina Lee] Add test for clone notebook with String type result e7af919 [Kevin Kim] stylish code 7bf5d01 [Kevin Kim] log info -> warn, add message d4f6699 [Kevin Kim] log exception 32949bc [Kevin Kim] trigger CI build 803e08a [Kevin Kim] revert implementation c13293f [Kevin Kim] fix test, better implementation 1e45a9e [Kevin Kim] [ZEPPELIN-905] add test a4188be [Kevin Kim] [ZEPPELIN-905] fix failed notebook import bug
### What is this PR for? Hotfix for compile error of master, branch-0.6 after merging #1043 Author: Mina Lee <[email protected]> Closes #1070 from minahlee/hotfix/compile_error and squashes the following commits: b0d5a6b [Mina Lee] Fix compile error after merging #1043
### What is this PR for? This PR is applying new interpreter register mechanism to python interpreter. ### What type of PR is it? Improvement ### What is the Jira issue? [ZEPPELIN-1045](https://issues.apache.org/jira/browse/ZEPPELIN-1045) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Mina Lee <[email protected]> Closes #1063 from minahlee/ZEPPELIN-1045 and squashes the following commits: 66b8f73 [Mina Lee] Add zeppelin.python.maxResult property to python interpreter 5013890 [Mina Lee] Apply new mechanism to PythonInterpreter
### What is this PR for? Display Pandas DataFrame using Zeppelin's Table Display system. ### What type of PR is it? Feature ### Todos * [x] fix NPE in logs on empty paragraph execution * [x] matplotlib: refactor `zeppelin_show(plt)` -> `z.show(plt)` * [x] pandas: support `z.show(df)` * [x] update docs ### What is the Jira issue? [ZEPPELIN-1048](https://issues.apache.org/jira/browse/ZEPPELIN-1048) ### How should this be tested? "Zeppelin Tutorial: Python - matplotlib basic" should work, and ```python import pandas as pd rates = pd.read_csv("bank.csv", sep=";") z.show(rates) ``` ### Screenshots (if appropriate)  ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? Yes Author: Alexander Bezzubov <[email protected]> Closes #1067 from bzz/python/pandas-support and squashes the following commits: 3b1ad36 [Alexander Bezzubov] Python: update docs to reffer new API ee6668b [Alexander Bezzubov] Python: update docs, add Pandas integration 71be418 [Alexander Bezzubov] Python: limit 1000 for table display system on DataFrame 52e787d [Alexander Bezzubov] Python: pandas DataFrame using Table display system bc91b86 [Alexander Bezzubov] Python: skip interpreting empty paragraphs a7248cd [Alexander Bezzubov] Python: draft of pandas support 15646a1 [Alexander Bezzubov] Python: refactoring to z.show()
### What is this PR for? Apply margin-bottom: 20px on `box` class by removing `home` class ### What type of PR is it? Bug Fix ### What is the Jira issue? [ZEPPELIN-1082](https://issues.apache.org/jira/browse/ZEPPELIN-1082) ### Screenshots (if appropriate) **Before** <img width="1280" alt="screen shot 2016-06-28 at 11 56 16 pm" src="https://cloud.githubusercontent.com/assets/8503346/16443219/554778b6-3d8c-11e6-884f-8a7024e5d3d1.png"> **After** <img width="1280" alt="screen shot 2016-06-28 at 11 56 48 pm" src="https://cloud.githubusercontent.com/assets/8503346/16443239/6142ccc4-3d8c-11e6-914f-87f523bf8275.png"> ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Mina Lee <[email protected]> Closes #1106 from minahlee/ZEPPELIN-1082 and squashes the following commits: fcace59 [Mina Lee] Remove home class to make space between boxes
### What is this PR for? Small refactoring to Shell interpreter. Add missing unit test and initial doc. ### What type of PR is it? Improvement, Documentation, Refactoring ### Todos * [ ] - Task ### What is the Jira issue? * Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/ * Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. [ZEPPELIN-533] ### 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: fvaleri <[email protected]> Closes #1087 from fvaleri/shell-intepreter-test and squashes the following commits: 56bb663 [fvaleri] re-trigger CI e4b25ef [fvaleri] re-trigger CI 9b9487b [fvaleri] re-trigger CI 03d073f [fvaleri] remove unused imports 7c42733 [fvaleri] Add new interpreter registration mechanism c069074 [fvaleri] rebase
### What is this PR for? LivySparkSQLInterpreter should work in FIFO with LivySparkInterpreter just like SparkSqlInterpreter works with SparkInterpreter ### What type of PR is it? [Hot Fix] ### Todos * [x] - LivySparkSQLInterpreter should work in FIFO just like SparkSqlInterpreter * [x] - add in property file zeppelin.livy.concurrentSQL ### What is the Jira issue? * [Zeppelin-1090](https://issues.apache.org/jira/browse/ZEPPELIN-1090) ### How should this be tested? In a notebook create 2 paragraph make content of first as ``` %livy Thread.sleep(10000) ``` and other as ``` %livy.sql show tables ``` The second paragraph should not get executed before first. ### 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: Prabhjyot Singh <[email protected]> Closes #1109 from prabhjyotsingh/ZEPPELIN-1090 and squashes the following commits: a2f863e [Prabhjyot Singh] check for both dead and error cases fe2ec72 [Prabhjyot Singh] LivySparkSQLInterpreter should work in FIFO just like SparkSqlInterpreter 2f1c64c [Prabhjyot Singh] add in property file zeppelin.livy.concurrentSQL
### What is this PR for? Removing redundant codes between `NoteInterpreterLoader` and `InterpreterFactory`, reducing the cost to add new features, and making refactoring on `InterpreterFactory` easy ### What type of PR is it? [Refactoring] ### Todos * [x] - Remove `NoteInterpreterLoader` and move the functionality into `InterpreterFactory` ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-1075 ### How should this be tested? Must pass all tests ### 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 #1102 from jongyoul/ZEPPELIN-1075 and squashes the following commits: d9816f1 [Jongyoul Lee] Fixed related codes a2d8104 [Jongyoul Lee] Fixed some style and removed unused codes 28bf520 [Jongyoul Lee] Fixed style 600de98 [Jongyoul Lee] Removed NoteInterpreterLoader Fixed some tests 536059b [Jongyoul Lee] Duplicated all method in NoteInterpreterLoader to InterpreterFactory
### What is this PR for? Every time user clicks on "Run all paragraphs" button system keeps appending an empty paragraph. Ideally content of paragraph should be checked before adding any empty paragraph. This started happening after [ZEPPELIN-707](https://issues.apache.org/jira/browse/ZEPPELIN-707) was merged. ### What type of PR is it? [Bug Fix | Improvement] ### Todos * [x] - add more condition before calling note.addParagraph() ### What is the Jira issue? * [ZEPPELIN-1094](https://issues.apache.org/jira/browse/ZEPPELIN-1094) ### Screenshots (if appropriate) Before  After  ### 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: Prabhjyot Singh <[email protected]> Closes #1111 from prabhjyotsingh/ZEPPELIN-1094 and squashes the following commits: 7919648 [Prabhjyot Singh] add more condition before calling note.addParagraph()
Author
|
Hi @bzz, is there is issue with my PR? Please let me know if there is something I can do to help get it done. Thanks 😄 |
### What is this PR for? Running ZeppelinServer.main normally. It will enable user to debug ZeppelinServer easily. ### What type of PR is it? [Bug Fix] ### Todos * [x] - Fix configuration values ### What is the Jira issue? N/A ### How should this be tested? - mvn clean package -DskipTests - Debug ZeppelinServer in your IDE ### 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 #1112 from jongyoul/minor-enable-launch-z-debugger-mode and squashes the following commits: 375a95b [Jongyoul Lee] Fixed to pass the tests eb1f733 [Jongyoul Lee] Fixed to pass the tests 161ffce [Jongyoul Lee] Removed unused import d24b854 [Jongyoul Lee] Set some configuration to pass `mvn verify` 921a293 [Jongyoul Lee] Changed the default value of some configurations
### What is this PR for?
This PR implements pluggable Application Framework.
Here's summary of the change.
**Applicaiton**
Base class of user application. User application should extends `org.apache.zeppelin.helium.Application`.
It looks like
```java
public abstract class Application {
/**
* When application loaded
* params context Application context
*/
public Application(ApplicationContext context) {
this.context = context;
}
/**
* This method can be invoked multiple times before unload(),
* Invoked just after application loaded or when paragraph output is updated
* param args Required resources from paragraph
*/
public abstract void run(ResourceSet args)
throws ApplicationException, IOException;
/**
* this method is invoked just before application is removed
*/
public abstract void unload() throws ApplicationException;
}
```
**HeliumPackage**
`org.apache.zeppelin.helium.HeliumPackage` keeps information of application package.
Package informations includes
```
{
type : "APPLICATION", // type of this package
name : "zeppelin.app", // application name. [organizationName].[appName] format
description : "App description", // description
artifact : "groupId:artifactId:version", // artifact name for maven repository
className : "org.apache.zeppelin.helium.MyApp", // application class that extends org.apache.zeppelin.helium.Application
resources : [[]], // required resource
icon : '<i class="fa fa-clock-o"></i>' // icon
}
```
`resources` field defines what kind of data this application requires, from ResourcePool.
"resourceName" and ":className" works in the array.
inner array combines each resourceName, :className with 'AND'
outer array combines inner array with 'OR'.
For example,
```
resources : [
[ "name1", ":java.util.Date"],
[ "name1", "name2"]
]
```
Then Zeppelin searches ResourcePool, first for resource name "name1" and resource type "java.util.Date" from resourcePool then resource name "name1" and "name2".
Once resources are found, they'll be passed to `Application.run()` method
**Package Registry**
`org.apache.zeppelin.helium.HeliumRegistry` provides list of available HeliumPackages.
Currently `HeliumLocalRegistry` is implemented and it provides list by reading HeliumPackage json files under ./helium directory in the file system. Later there will be some class like `HeliumOnlineRegistry' which reads available package from a community managed online central registry.
**Development mode**
`org.apache.zeppelin.interpreter.dev.*` package is provided for development support.
Developer can run their app inside of their IDE, that connects to running Zeppelin instance and display output.
**Application Suggestion**
Once paragraph becomes FINISHED status after run, Zeppelin searches resources from ResourcesPool that belongs to current paragraph. And compare all HeliumPackage if these resources satisfies their requirements ('resources' field). If there're available app, Helium button will be displayed.
User can see and select available application for this paragraph.
**Application selection**
After application is loaded, application icon will be displayed. So user can select and switch between apps.
if built-in visualization is available, application icon will be displayed next to built-in visualizations
**Experimental**
While this is new feature and API and specification may change, i'd suggest mark this feature as a experimental for a while.
### What type of PR is it?
Feature
### Todos
* [x] - Helium Application framework
* [x] - App launcher
* [x] - App display and selectors
* [x] - Package registry
* [x] - Development mode
* [x] - Make CI green
* [x] - Improve comment in source codes
* [x] - Documentation
* [x] - Examples
* [x] - Mark experimental
* [ ] - Review
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-732
### How should this be tested?
Will be updated with examples
### Screenshots (if appropriate)
### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? yes
Author: Lee moon soo <[email protected]>
Closes #836 from Leemoonsoo/ZEPPELIN-732-up and squashes the following commits:
3577777 [Lee moon soo] Address Ahyoung's comment
0665380 [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
83b529c [Lee moon soo] Update unittest
e1b0e79 [Lee moon soo] Update unittest
6785441 [Lee moon soo] interpreter order changed
3249ddb [Lee moon soo] Remove incubating
4030fff [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
4e98cfe [Lee moon soo] Update DependencyResolver
c714a19 [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
d558591 [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
0467077 [Lee moon soo] update test
d43ab57 [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
f999f71 [Lee moon soo] Update getParagraphXpath
eaa68c6 [Lee moon soo] print processes
5b5f6c8 [Lee moon soo] trigger ci
1bdea70 [Lee moon soo] Print error message in the test
562f083 [Lee moon soo] CLose interpreter after test
385dab3 [Lee moon soo] Fix xpath in the SparkParagraphIt test
ac963ff [Lee moon soo] Remove unnecessary stacktrace and provide more meaningful message
644add4 [Lee moon soo] Takecare csv/tsv download button position after merge
6a5a19e [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
5bef173 [Lee moon soo] Update NoteTest
e54d633 [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
88dbc71 [Lee moon soo] change style
6f1343f [Lee moon soo] add license header
4571781 [Lee moon soo] double quote the keys in json
710487b [Lee moon soo] fix syntax error
6c5019e [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
3619afd [Lee moon soo] Exclude test data file from rat check
b6811a6 [Lee moon soo] set SPARK_HOME to find bin/interpreter.sh
9110897 [Lee moon soo] arrange buttons
14efb6c [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
76de11c [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
bff2217 [Lee moon soo] Fix SparkParagraphIT
e60cd8e [Lee moon soo] Add docs
7f174e3 [Lee moon soo] Add more comments
0671dfd [Lee moon soo] Fix compile error
bfcdf34 [Lee moon soo] Fix style
1dc4409 [Lee moon soo] Remove incubating from example pom.xml
e36d2fb [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
e0a2046 [Lee moon soo] Add annotation to api
93fa766 [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
99dd0b0 [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
e89177e [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
f782fb0 [Lee moon soo] clean pom.xml file
b974eb1 [Lee moon soo] Add horizontal bar visualization example
5225551 [Lee moon soo] let instead of var
b068065 [Lee moon soo] Replace . to _
b6e4141 [Lee moon soo] helper for printing javascript and inject $z
8186daf [Lee moon soo] Update Application Status from RemoteInterpreterProcess event
f07ada1 [Lee moon soo] Construct classpath correctly
7451479 [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
ab1de03 [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
51fc113 [Lee moon soo] Include examples in CI build
83eba98 [Lee moon soo] UnloadApp on interpreter restart
3b891de [Lee moon soo] Provide resource pool for testing
0730ece [Lee moon soo] Add license into package json
620f79f [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
01a1646 [Lee moon soo] Add example
1f1a3b5 [Lee moon soo] Fix test
864bea1 [Lee moon soo] Merge branch 'master' into ZEPPELIN-732-up
c30f53c [Lee moon soo] null check
03be3a1 [Lee moon soo] Pass required resource to run() method
fade3c1 [Lee moon soo] Handle output update in angular mode
0e4d81c [Lee moon soo] Remove unnecessary log
ec2fdea [Lee moon soo] Match classname correctly
b4ff52f [Lee moon soo] Put last value of scala repl into resource pool
f2ab95d [Lee moon soo] Prevent unnecessary output update
71f814d [Lee moon soo] Better way to find resource dir for InterpreterOutput watcher
024d7fc [Lee moon soo] Dev mode
98f3872 [Lee moon soo] Managed interpreter process and Running interpreter process
b47ca74 [Lee moon soo] Fix tests
5503f9c [Lee moon soo] Improved
be3a1fa [Lee moon soo] Add license header
412480a [Lee moon soo] Fix style
16f6887 [Lee moon soo] Angular object update for helium app
6223cd4 [Lee moon soo] App output display
bd0f467 [Lee moon soo] Style
94b490d [Lee moon soo] initial rest api impl
134bbe6 [Lee moon soo] Change HeliumRegistry constructor argument type
7aeb64a [Lee moon soo] Unload app on paragraph / note removal as well as interpreter unbind
4eaeea7 [Lee moon soo] sync -> async api
9f5c493 [Lee moon soo] Application output
b239f1b [Lee moon soo] Helium application factory
b891b98 [Lee moon soo] HeliumRegistry
568ee54 [Lee moon soo] ApplicationLoader
7424af2 [Lee moon soo] Remove resource on note / paragraph removal
Contributor
|
@davidtzoor Can you rebase? |
…mes. ### What is this PR for? This PR fixes the problem that does not show the text result in the paragraph sometimes. ### What type of PR is it? Bug Fix ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-1078 ### How should this be tested? Try run python code constantly like screenshot. ### Screenshots (if appropriate) - before  ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no Author: astroshim <[email protected]> Closes #1104 from astroshim/ZEPPELIN-1078 and squashes the following commits: e0044db [astroshim] rebase 189c5eb [astroshim] remove resultRefreshed from TEXT type only ec44166 [astroshim] remove resultRefreshed from TEXT type only edf2397 [astroshim] Merge branch 'master' into ZEPPELIN-1078 441357c [astroshim] remove resultRefreshed value.
…json ### What is this PR for? Fixing NPE while initializing interpreters from interpreter-setting.json ### What type of PR is it? [Bug Fix | Hot Fix] ### Todos * [x] - Fix NPE ### What is the Jira issue? N/A ### How should this be tested? 1. delete `properties` in one of `interpreter-setting.json` 1. Build and run 1. Got error 1. Apply this PR 1. Build and run 1. Run it successfully ### 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 #1128 from jongyoul/hotfix/interpreter-properties-is-null and squashes the following commits: 1dcf562 [Jongyoul Lee] Fixed NPE when there's no properties field in interpreter-setting.json
Author
|
@corneadoug do you want me to rebase to the master on my fork? |
Contributor
|
@davidtzoor Right, there has been some changes that brings conflicts |
Member
|
RIght, in you fork you need to pull the master from apache git and then rebase this branch on top of it, resolving the conflicts. |
…pendencyLoading ### What is this PR for? Catch Exception in case dialog is removed from dom to fix flaky test of `ZeppelinIT. testSparkInterpreterDependencyLoading` ### What type of PR is it? Hot Fix ### What is the Jira issue? [ZEPPELIN-1110](https://issues.apache.org/jira/browse/ZEPPELIN-1110) ### Questions: * Does the licenses files need update? no * Is there breaking changes for older versions? no * Does this needs documentation? no Author: Mina Lee <[email protected]> Closes #1129 from minahlee/ZEPPELIN-1110 and squashes the following commits: c681410 [Mina Lee] Retrigger CI 69da3c1 [Mina Lee] Retrigger CI 21435be [Mina Lee] Retrigger CI 8cd7065 [Mina Lee] catch StaleElementReferenceException in case dialog is removed from dom
Author
|
@bzz I fixed the conflict |
Contributor
|
@davidtzoor It seems many unrelated commits are included in this PR after you rebased. Could you check again? |
Author
|
@AhyoungRyu I just did a pull from the zeppelin repo to my fork, rebase and resolved the conflicts. |
Author
|
I think these are all the PR that were accepted since my initial pull request. |
Contributor
|
@davidtzoor I'm not sure what you missed, your local master branch seems quite outdated. Zeppelin follows fork & pull model as a source control workflow. 1. In your forked local repository2. set upstream and update local master3. rebase your branch
|
Author
|
I am closing this PR and creating a new one (again). sorry for all the mess |
Contributor
|
@davidtzoor Never mind. Please feel free to do that :) |
Author
|
@AhyoungRyu I created a new PR (#1135) and closing this one. I hope this time it will work. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
18 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What is this PR for?
Add force layout (graph) visualization to the existing set of visualizations in Zeppelin Web notp.
What type of PR is it?
[Feature]
What is the Jira issue?
[ZEPPELIN-1011]
How should this be tested?
The testes are incorporated within the existing test suite.
Screenshot
Questions:
No.
No.
We can add some documentation explaining about the different fields that need to be selected in order to generate the force layout component.