Merging #6
Merged
Merged
Conversation
### What is this PR for? Fix for https://issues.apache.org/jira/browse/ZEPPELIN-602 "elasticsearch throws ArrayIndexOutOfBoundsException for interpreting an empty paragraph" ### What type of PR is it? Bug Fix ### Todos * [X] - Code : check cmd parameter ### Is there a relevant Jira issue? ZEPPELIN-602 ### How should this be tested? Start elasticsearch interpreter with an empty paragraph ### 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: Bruno Bonnin <bbonnin@gmail.com> Closes #646 from bbonnin/master and squashes the following commits: 05993d0 [Bruno Bonnin] Update unit tests and complete fix 378bf52 [Bruno Bonnin] Update ElasticsearchInterpreter.java 7651647 [Bruno Bonnin] ZEPPELIN-602 elasticsearch throws ArrayIndexOutOfBoundsException for interpreting an empty paragraph
### What is this PR for? Output from interpreter is displayed after completion of paragraph execution. It'll be useful if output can be streamed to front-end during execution. Previous work #593 injects InterpreterOutput stream object to Interpreter. This PR is based on #593 and stream the data from InterpreterOutput to front-end. This implementation only streams output is %text. Other output type (%html, %angular, %table) is not streamed to the front end. While this PR keeps backward compatibility, Interpreter who want to use this feature will need to modify code to write output into `InterpreterOutput` instead of return with `InterpreterResult`. This PR includes modification of SparkInterpreter to use InterpreterOutput. ### What type of PR is it? Feature ### Todos ### Is there a relevant Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-554 ### How should this be tested? Run such code using Spark interpreter ``` (1 to 10).foreach{ i=> Thread.sleep(1000) println("Hello " + i) } ``` ### 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: Lee moon soo <moon@apache.org> Closes #611 from Leemoonsoo/output_stream_frontend and squashes the following commits: 53e2bb4 [Lee moon soo] Not persist on every append dedae0d [Lee moon soo] Remove debug lines 8251fb4 [Lee moon soo] Fix syntax and style 9c9c8fd [Lee moon soo] update test 18215a3 [Lee moon soo] fix style f7e6a4d [Lee moon soo] Fix syntax error d29cfbf [Lee moon soo] workaround jshint 07b3e1a [Lee moon soo] Handle clear output correctly bc6262e [Lee moon soo] Make PysparkInterpreter stream output 6d9cc51 [Lee moon soo] Pass InterpreterOutput to SparkILoop b68180e [Lee moon soo] Add InterpreterOutput on spark interpreter unitest 626ad48 [Lee moon soo] Add license header 846015b [Lee moon soo] Update scalding 37d6920 [Lee moon soo] Handle display system directive correctly e278e84 [Lee moon soo] Clear output correctly 479b836 [Lee moon soo] Add test c01df62 [Lee moon soo] Connect Spark interpreter Console.out to outputstream 8a1223f [Lee moon soo] Handle update output correctly e7a9b37 [Lee moon soo] Delayed persist 2060c1e [Lee moon soo] Clear before render text 786c978 [Lee moon soo] update paragraph object after witing to outputstream 258ff38 [Lee moon soo] Barely working 6f607f7 [Lee moon soo] Add newline listener 89d9798 [Lee moon soo] Render text output line by line a42e4ff [Lee moon soo] Update test fb5e7b5 [Lee moon soo] Update test 0f60b54 [Lee moon soo] Update test a07d7db [Lee moon soo] Implement InterpreterResult.toString 1f419b6 [Lee moon soo] Add InterpreterOutput c91f498 [Lee moon soo] prepend interpreteroutputstream to interpreter result
### What is this PR for? This PR provides HOTFIX for master branch build error introduced by #611 ### What type of PR is it? Hot Fix ### Todos * [x] - Fix unittest ### Is there a relevant 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: Lee moon soo <moon@apache.org> Closes #663 from Leemoonsoo/hotfix_ZEPPELIN-554 and squashes the following commits: a468b2e [Lee moon soo] Update unit test
### What is this PR for? Zeppelin has already provided a useful utility for reviewing PR ( #513 ). So I added a **Testing a Pull Request** section to `CONTRIBUTING.md` for whom to review and test PR quickly. ### What type of PR is it? Documentation ### Todos * [x] - Add a **Testing a Pull Request** section to CONTRIBUTING.md ### Is there a relevant Jira issue? [ZEPPELIN-484](https://issues.apache.org/jira/browse/ZEPPELIN-484) and PR #513 ### How should this be tested? ### 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: Ryu Ah young <fbdkdud93@hanmail.net> Closes #656 from AhyoungRyu/add_pr_review_information and squashes the following commits: 5f370d5 [Ryu Ah young] Fix some grammar errors 3781550 [Ryu Ah young] Delete 'like' 9ced186 [Ryu Ah young] Add some information about how we can review and test PR
Due to the documentation of interpreters written by various authors, styles of documentations are vary and has broken tags and inconsistent spaces. This fix cleans those little glitches to make documentation more readable and look cleaner to users. 1) Clean up unnecessary or wrong HTML tags ex: \<hr/\> after titles 2) Fix too wide spaces between paragraph and tables 3) Remove unnecessary spaces at end of paragraph and lines Author: Jesang Yoon <yoonjs2@kanizsalab.com> Closes #648 from yoonjs2/master and squashes the following commits: eba0315 [Jesang Yoon] Merge remote-tracking branch 'upstream/master' db8b4da [Jesang Yoon] Merge remote-tracking branch 'upstream/master' 781954b [Jesang Yoon] Interpreter documentation merge with commit #578 af55811 [Jesang Yoon] Merge remote-tracking branch 'origin/master' 079480f [Jesang Yoon] Merge remote-tracking branch 'origin/master' 5f0a6e0 [Jesang Yoon] Merge remote-tracking branch 'origin/master' 4d1503a [Jesang Yoon] Merge remote-tracking branch 'origin/master' 5b091e4 [Jesang Yoon] Fix wrong HTML tags, indention and space between paragraph and tables. Remove unnecessary spaces. 5665dcf [Jesang Yoon] Fix wrong HTML tags, indention and space between paragraph and tables. Remove unnecessary spaces.
ramaswamydevarajan
added a commit
that referenced
this pull request
Jan 21, 2016
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 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.
No description provided.