{"push_id":2774554557,"size":39,"distinct_size":39,"ref":"refs/heads/master","head":"cbf22284834e52f718c8dbf29afd35d40ba39db8","before":"0f9a1a80c76054c322e2d1768cf24a3e4bd49496","commits":[{"sha":"c195b5609bb4dbb9fd037381ea58d1650cee92bf","author":{"name":"Jongyoul Lee","email":"580258f4dc0f7b7fba6752c3e796a6cb1bdd91fd@gmail.com"},"message":"[ZEPPELIN-3593] Change LuceneSearch's directory to file system from memory\n\n### What is this PR for?\nReducing a usage of memory by moving Lucene search's directory to the file system\n\n### What type of PR is it?\n[Improvement]\n\n### Todos\n* [x] - Replace `RamDirectory` to `MMapDirectory`\n\n### What is the Jira issue?\n* https://issues.apache.org/jira/browse/ZEPPELIN-3593\n\n### How should this be tested?\nIt doesn't affect any current behaviors. Test search service\n\n### Screenshots (if appropriate)\n\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation? No\n\nAuthor: Jongyoul Lee <jongyoul@gmail.com>\n\nCloses #3058 from jongyoul/ZEPPELIN-3593 and squashes the following commits:\n\ncdae583c4 [Jongyoul Lee] Add a logic to check if directoryPath is null or not\n5a383c4ab [Jongyoul Lee] Add `RamDirectory` to use search service in case where disk is not available\n3b76cc1c0 [Jongyoul Lee] Set a default temporary directory to `java.io.tmpdir` Clean up javadoc\n1e415e00e [Jongyoul Lee] Add deletion logic for the temporary directory\nd26e64f88 [Jongyoul Lee] Replace `RamDirectory` to `MMapDirectory` provided by Lucene","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/c195b5609bb4dbb9fd037381ea58d1650cee92bf"},{"sha":"453be73abb19eaf2448d1a63347afb8ca5911e1a","author":{"name":"Jongyoul Lee","email":"580258f4dc0f7b7fba6752c3e796a6cb1bdd91fd@gmail.com"},"message":"[HOTFIX] Change Spark download url for spark 1.6.x\n\n### What is this PR for?\nSpark 1.6 was removed from CDN. We need to download it from apache archive.\n\n### What type of PR is it?\n[Hot Fix]\n\n### Todos\n* [x] - Change the location for downloading Spark 1.6.x\n\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation? No\n\nAuthor: Jongyoul Lee <jongyoul@gmail.com>\n\nCloses #3075 from jongyoul/hotfix/change-spark-download-location and squashes the following commits:\n\n87bfd2abf [Jongyoul Lee] Removed comments Reduce changes\na51d5b93b [Jongyoul Lee] Change download url for spark 1.6.x\n9f63f8358 [Jongyoul Lee] Change download url for spark 1.6.x","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/453be73abb19eaf2448d1a63347afb8ca5911e1a"},{"sha":"ec141669518042350dc410d1178a772c2a815ed7","author":{"name":"Jongyoul Lee","email":"580258f4dc0f7b7fba6752c3e796a6cb1bdd91fd@gmail.com"},"message":"[ZEPPELIN-3627] Remove starting server option for some functional tests\n\n### What is this PR for?\nRemoving `AbstractTestRestApi` dependency to reduce testing time\n\n### What type of PR is it?\n[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]\n\n### Todos\n* [x] - Remove `AbstractTestRestApi`\n\n### What is the Jira issue?\n* https://issues.apache.org/jira/browse/ZEPPELIN-3627\n\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation? No\n\nAuthor: Jongyoul Lee <jongyoul@gmail.com>\n\nCloses #3076 from jongyoul/ZEPPELIN-3627 and squashes the following commits:\n\n505580062 [Jongyoul Lee] Revert commented flaky tests\nac7d827b4 [Jongyoul Lee] Change flaky test\n0ab6d66b0 [Jongyoul Lee] Disable flaky tests\ne99522dde [Jongyoul Lee] Remove `AbstractRestApiTest` from `CredentialRestApiTest`","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/ec141669518042350dc410d1178a772c2a815ed7"},{"sha":"a66bff06f8cc2bd7511f12a175a9118aa4ed451a","author":{"name":"karth295","email":"df97c5b83c4624acefbeefd859ef1310313a14ce@uw.edu"},"message":"Fix minor formatting issues in storage.md\n\nAuthor: karth295 <karth295@uw.edu>\n\nCloses #3026 from karth295/patch-1 and squashes the following commits:\n\n7f23774e6 [karth295] Fix minor formatting issues in storage.md","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/a66bff06f8cc2bd7511f12a175a9118aa4ed451a"},{"sha":"50356437d3a742fb21b1da74762902bcdb7fb9c8","author":{"name":"Jeff Zhang","email":"ced9a24e2e494bb60341a9f26a61a78e7a11abad@apache.org"},"message":"[HOTFIX] Fix flaky travis build due to corrupted jar\n\n(cherry picked from commit 98b926ae2e422e3b327faacdb396eff29070b44b)","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/50356437d3a742fb21b1da74762902bcdb7fb9c8"},{"sha":"cf3127561c4212e9299ffcaffc95b37cd7fe9781","author":{"name":"Khalid Huseynov","email":"8560a0d4bf3157d27d05065beddf9fbc99dfcbfb@gmail.com"},"message":"[zeppelin-3625] Pandasql interpreter fails to query over python interpreter dataframe\n\n### What is this PR for?\nfix pandasql query over dataframe for `master`. the reason is that it can't load `bootstrap_sql.py` from resources because path of path\n\n### What type of PR is it?\n[Bug Fix]\n\n### Todos\n* [x] - fix path to use relative\n* [x] - enable tests\n\n### What is the Jira issue?\nhttps://issues.apache.org/jira/browse/ZEPPELIN-3625\n\n### How should this be tested?\nfollow the steps in issue\n\n### Screenshots (if appropriate)\nbefore:\n<img width=\"1268\" alt=\"screen shot 2018-07-15 at 7 08 24 pm\" src=\"https://user-images.githubusercontent.com/1642088/42732816-822682f0-8862-11e8-9d55-dd7d00b09b48.png\">\n\nafter:\n<img width=\"1239\" alt=\"screen shot 2018-07-15 at 7 03 59 pm\" src=\"https://user-images.githubusercontent.com/1642088/42732771-e712f848-8861-11e8-9ab9-eeb479082d1d.png\">\n\n### Questions:\n* Does the licenses files need update?\n* Is there breaking changes for older versions?\n* Does this needs documentation?\n\nAuthor: Khalid Huseynov <khalidhnv@gmail.com>\n\nCloses #3071 from khalidhuseynov/fix-master/ZEPPELIN-3625 and squashes the following commits:\n\n33aad7589 [Khalid Huseynov] fix matplotlib test\nf94139b33 [Khalid Huseynov] enable all python tests\nfd5685552 [Khalid Huseynov] remove unnecessary test\nbde2316c7 [Khalid Huseynov] add test\n8f9541411 [Khalid Huseynov] fix test setup not to interpret empty line\n3a561d247 [Khalid Huseynov] fix from absolute to relative path in resources","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/cf3127561c4212e9299ffcaffc95b37cd7fe9781"},{"sha":"2e3043e774769592841db54e6952049e6936749c","author":{"name":"oxygen311","email":"0f712447ea18e18e97321220d17b3e0aa499f352@mail.ru"},"message":"[ZEPPELIN-3580] Adds rename note method to rest api\n\n### What is this PR for?\nAdds rename note method to rest api\n\n### What type of PR is it?\nImprovement\n\n### What is the Jira issue?\nissue on Jira: https://issues.apache.org/jira/browse/ZEPPELIN-3580\n\n### Questions:\n- Does the licenses files need update? No\n- Is there breaking changes for older versions? No\n- Does this needs documentation? No\n\nAuthor: oxygen311 <alex39-09@mail.ru>\n\nCloses #3049 from oxygen311/ZEPPELIN-3585 and squashes the following commits:\n\n62d081c9d [oxygen311] Merge branch 'ZEPPELIN-3585' of https://github.com/TinkoffCreditSystems/zeppelin into ZEPPELIN-3585\n57f6d2905 [oxygen311] Fix excess symbol bug\nca1b0b397 [oxygen311] Fix codestyle\nbbbe26134 [oxygen311] Merge branch 'master' of https://github.com/apache/zeppelin into ZEPPELIN-3585\n4df7c6dba [oxygen311] Add rename method to rest api\nad880ed31 [oxygen311] Add rename method to rest api","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/2e3043e774769592841db54e6952049e6936749c"},{"sha":"1af1d83955fa2b4abfc6bc8d8dc29b9b8cb205ff","author":{"name":"Sanjay Dasgupta","email":"1bfdc00ef185e58081407f5dbdc13c91d69c6b6e@gmail.com"},"message":"[ZEPPELIN-2605] Import notebook has 1MB size limit but bigger notebooks can be saved without error or warning\n\n### What is this PR for?\nNotebooks of any size can be exported and saved, but the import facility only accepts notebooks up to 1Mb in size. Allowing notebooks bigger than 1Mb to be saved silently (without any warning or error notification) is therefore a dangerous trap for users.\n\nIt has been argued that notebooks bigger than 1Mb are rare and unusual, but a saved notebook also contains the displayed output - and can contain large volumes of tabular data, and even images from matplotlib output. Large notebooks can be common when zeppelin is used for presentation of visual analysis.\n\nThis PR adds a popup dialog which is displayed when the user attempts to export a notebook that has a size exceeding 1Mb. A screenshot of the popup is shown below.\n\n![popup-dialog](https://user-images.githubusercontent.com/477015/41522941-cb2c5c7c-72f5-11e8-83e6-b3f6f29ee834.png)\n\n### What type of PR is it?\n[Bug Fix]\n\n### Todos\n\n### What is the Jira issue?\nhttps://issues.apache.org/jira/browse/ZEPPELIN-2605\n\n### How should this be tested?\nCI pass\nManual testing. A notebook that is small when outputs are cleared, but becomes bigger than the limit when all cells are run is included (big-output-tester.json).\n\n[big-output-tester.json.zip](https://github.com/apache/zeppelin/files/2110281/big-output-tester.json.zip)\n\n### Screenshots (if appropriate)\nSee above.\n\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation? No\n\nAuthor: Sanjay Dasgupta <sanjay.dasgupta@gmail.com>\n\nCloses #3027 from sanjaydasgupta/z2605-import-notebook-has-1mb-limit and squashes the following commits:\n\nd8f778668 [Sanjay Dasgupta] z2605-import-notebook-has-1mb-limit: Initial content","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/1af1d83955fa2b4abfc6bc8d8dc29b9b8cb205ff"},{"sha":"a290ccb0be70f6d11f39d7aa35f24208ae033553","author":{"name":"egorklimov","email":"345a1c620d546d873f44733c0e78cf4800564d59@gmail.com"},"message":"[ZEPPELIN-3636] Add timeout for s3 amazon bucket endpoint\n\n### What is this PR for?\nIf there is no connection to amazonaws, app will wait for 2 minutes during setup:\n```\nINFO [2018-07-13 14:45:19,644] (Helium.java[loadConf]:103) - Add helium local registry /opt/zeppelin/product/helium\nINFO [2018-07-13 14:45:19,645] (Helium.java[loadConf]:100) - Add helium online registry https://s3.amazonaws.com/helium-package/helium.json\nERROR [2018-07-13 14:47:27,098] (HeliumOnlineRegistry.java[getAll]:80) - Connect to s3.amazonaws.com:443 [s3.amazonaws.com/54.231.120.10] failed: Connection timed out (Connection timed out)\nINFO [2018-07-13 14:47:28,104] ( ContextHandler.java[doStart]:744) - Started o.e.j.w.WebAppContext161479c6\n```\n\nEven if Amazon S3 notebook storage wasn't configured Helium goes to zeppelin.notebook.s3.endpoint.\n\nIt would be nice if we could set timeout for s3 bucket if we know that there is no connection to amazonaws.\n\n### What type of PR is it?\nImprovement\n\n### What is the Jira issue?\nissue on Jira https://issues.apache.org/jira/browse/ZEPPELIN-3636\n\n### How should this be tested?\n* Log after setting timeout property to 20000 in zeppelin-site.xml:\n```\nINFO [2018-07-18 19:35:55,514] ({main} Helium.java[loadConf]:103) - Add helium local registry /home/egklimov/IdeaProjects/zeppelin/helium\nINFO [2018-07-18 19:35:55,514] ({main} Helium.java[loadConf]:100) - Add helium online registry https://s3.amazonaws.com/helium-package/helium.json\nERROR [2018-07-18 19:36:15,690] ({main} HeliumOnlineRegistry.java[getAll]:91) - Connect to s3.amazonaws.com:443 [s3.amazonaws.com/192.168.65.17] failed: connect timed out\nINFO [2018-07-18 19:36:16,823] ({main} ContextHandler.java[doStart]:744) - Started o.e.j.w.WebAppContext6107227e\n```\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation?\nYes, information about Zeppelin Properties updated\n\nAuthor: egorklimov <klim.electronicmail@gmail.com>\n\nCloses #3082 from egorklimov/ZEPPELIN-3636 and squashes the following commits:\n\n00fd306a1 [egorklimov] Socket timeout added\n78d1fdd3c [egorklimov] Code refactored\n28286bb03 [egorklimov] Timeout test added\n9770a4793 [egorklimov] zeppelin-site.xml.template updated\n0c9821610 [egorklimov] Docs updated\n7d2c4d178 [egorklimov] ZeppelinConfiguration property zeppelin.notebook.s3.timeout added","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/a290ccb0be70f6d11f39d7aa35f24208ae033553"},{"sha":"716dfd1c54d1112fd5f6b4339bdd59784e867351","author":{"name":"Jongyoul Lee","email":"580258f4dc0f7b7fba6752c3e796a6cb1bdd91fd@gmail.com"},"message":"[HOTFIX] Fix checkstyle\n\n### What is this PR for?\nFixing style issue\n\n### What type of PR is it?\n[Hot Fix]\n\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation? No\n\nAuthor: Jongyoul Lee <jongyoul@gmail.com>\n\nCloses #3086 from jongyoul/hotfix/fix-checkstyle and squashes the following commits:\n\n488166a75 [Jongyoul Lee] Fixed style\n6b17a592d [Jongyoul Lee] Fix style","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/716dfd1c54d1112fd5f6b4339bdd59784e867351"},{"sha":"4467ac01dfe4a39c8d191d2266d7e772ab35a1a9","author":{"name":"Jongyoul Lee","email":"580258f4dc0f7b7fba6752c3e796a6cb1bdd91fd@gmail.com"},"message":"Revert \"[ZEPPELIN-3593] Change LuceneSearch's directory to file system from memory\"\n\nThis reverts commit c195b5609bb4dbb9fd037381ea58d1650cee92bf.","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/4467ac01dfe4a39c8d191d2266d7e772ab35a1a9"},{"sha":"30aa87948f4b92e95a695e95b94e8157fa3af938","author":{"name":"Jeff Zhang","email":"ced9a24e2e494bb60341a9f26a61a78e7a11abad@apache.org"},"message":"ZEPPELIN-3637. Allow custom spark UI url for new spark interpreter\n\n### What is this PR for?\nTrivia fix to allow custom spark UI url for new spark interpreter\n\n### What type of PR is it?\n[Bug Fix]\n\n### Todos\n* [ ] - Task\n\n### What is the Jira issue?\n* https://issues.apache.org/jira/browse/ZEPPELIN-3637\n\n### How should this be tested?\n* CI pass\n\n### Screenshots (if appropriate)\n\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation? No\n\nAuthor: Jeff Zhang <zjffdu@apache.org>\n\nCloses #3083 from zjffdu/ZEPPELIN-3637 and squashes the following commits:\n\n9789e975e [Jeff Zhang] ZEPPELIN-3637. Allow custom spark UI url for new spark interpreter","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/30aa87948f4b92e95a695e95b94e8157fa3af938"},{"sha":"2d31f2200da5eb1a95cbc3d2c5fa3768f99a5760","author":{"name":"Jeff Zhang","email":"ced9a24e2e494bb60341a9f26a61a78e7a11abad@apache.org"},"message":"ZEPPELIN-3566. Add imageWidth as paragraph level properties in SparkRInterpreter\n\n### What is this PR for?\nUpdate SparkRInterpreter to put imageWidth as paragraph properties.\n\n### What type of PR is it?\n[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]\n\n### Todos\n* [ ] - Task\n\n### What is the Jira issue?\n* https://issues.apache.org/jira/browse/ZEPPELIN-3666\n\n### How should this be tested?\n* CI pass\n\n### Screenshots (if appropriate)\n\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation? No\n\nAuthor: Jeff Zhang <zjffdu@apache.org>\n\nCloses #3080 from zjffdu/ZEPPELIN-3566 and squashes the following commits:\n\nb682e9e11 [Jeff Zhang] ZEPPELIN-3566. Add imageWidth as paragraph level properties in SparkRInterpreter","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/2d31f2200da5eb1a95cbc3d2c5fa3768f99a5760"},{"sha":"e6af8f10e20ba7d140f9dd4db4160f46d8f0c6c7","author":{"name":"Jeff Zhang","email":"ced9a24e2e494bb60341a9f26a61a78e7a11abad@apache.org"},"message":"ZEPPELIN-3587. Interpret paragarph text as whole code statement instead of breaking into lines\n\n### What is this PR for?\n\nJust to make this kind of code to work.\n\n```\nsc\n.range(1,10\n.sum()\n```\n\n### What type of PR is it?\n[ Improvement ]\n\n### Todos\n* [ ] - Task\n\n### What is the Jira issue?\n* https://issues.apache.org/jira/browse/ZEPPELIN-3587\n\n### How should this be tested?\n* CI pass\n\n### Screenshots (if appropriate)\n\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation? No\n\nAuthor: Jeff Zhang <zjffdu@apache.org>\n\nCloses #3055 from zjffdu/ZEPPELIN-3587 and squashes the following commits:\n\n0d6c24ad0 [Jeff Zhang] ZEPPELIN-3587. Interpret paragarph text as whole code statement instead of breaking into lines","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/e6af8f10e20ba7d140f9dd4db4160f46d8f0c6c7"},{"sha":"480e6471eef9f5ffebea7b2d4f6410ced9ddfecc","author":{"name":"Jeff Zhang","email":"ced9a24e2e494bb60341a9f26a61a78e7a11abad@apache.org"},"message":"ZEPPELIN-3635. Drop support for spark before 1.6\n\n### What is this PR for?\nThis PR is to remove legacy code to drop support for spark before 1.6\n\n### What type of PR is it?\n[Refactoring]\n\n### Todos\n* [ ] - Task\n\n### What is the Jira issue?\n* https://issues.apache.org/jira/browse/ZEPPELIN-3635\n\n### How should this be tested?\n* Ci pass\n\n### Screenshots (if appropriate)\n\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation? No\n\nAuthor: Jeff Zhang <zjffdu@apache.org>\n\nCloses #3079 from zjffdu/ZEPPELIN-3635 and squashes the following commits:\n\nc8427072c [Jeff Zhang] ZEPPELIN-3635. Drop support for spark before 1.6","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/480e6471eef9f5ffebea7b2d4f6410ced9ddfecc"},{"sha":"1d493abb490c7b5bf3de28f42546239827180384","author":{"name":"Jongyoul Lee","email":"580258f4dc0f7b7fba6752c3e796a6cb1bdd91fd@gmail.com"},"message":"[ZEPPELIN-3593] Add an option to use disk while indexing notes for search\n\n### What is this PR for?\nAdding a new option to enable using disk while running search service\n\n### What type of PR is it?\n[Improvement]\n\n### What is the Jira issue?\n* https://issues.apache.org/jira/browse/ZEPPELIN-3593\n\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation? No\n\nAuthor: Jongyoul Lee <jongyoul@gmail.com>\n\nCloses #3089 from jongyoul/ZEPPELIN-3593 and squashes the following commits:\n\n4e61070fd [Jongyoul Lee] Remove `throws` signature\n9179b4aee [Jongyoul Lee] Change exception type from IOException to RuntimeException\nd63a60ad3 [Jongyoul Lee] Add option to enable to use disk index feature\n6f445b788 [Jongyoul Lee] Add a logic to check if directoryPath is null or not\n3676c41fd [Jongyoul Lee] Add `RamDirectory` to use search service in case where disk is not available\nb5aa5dfb6 [Jongyoul Lee] Set a default temporary directory to `java.io.tmpdir` Clean up javadoc\n605e9ab29 [Jongyoul Lee] Add deletion logic for the temporary directory\nf818accd9 [Jongyoul Lee] Replace `RamDirectory` to `MMapDirectory` provided by Lucene","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/1d493abb490c7b5bf3de28f42546239827180384"},{"sha":"e6b6c0077fff3dc9275a4a9fb4a191bea8a2f9ff","author":{"name":"Jeff Zhang","email":"ced9a24e2e494bb60341a9f26a61a78e7a11abad@apache.org"},"message":"[MINOR] Fix classpath issue in interpreter.sh\n\n### What is this PR for?\nMinor fix for classpath in interpreter.sh\n\n### What type of PR is it?\n[Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]\n\n### Todos\n* [ ] - Task\n\n### What is the Jira issue?\n* Open an issue on Jira https://issues.apache.org/jira/browse/ZEPPELIN/\n* Put link here, and add [ZEPPELIN-*Jira number*] in PR title, eg. [ZEPPELIN-533]\n\n### How should this be tested?\n* First time? Setup Travis CI as described on https://zeppelin.apache.org/contribution/contributions.html#continuous-integration\n* Strongly recommended: add automated unit tests for any new or changed behavior\n* Outline any manual steps to test the PR here.\n\n### Screenshots (if appropriate)\n\n### Questions:\n* Does the licenses files need update?\n* Is there breaking changes for older versions?\n* Does this needs documentation?\n\nAuthor: Jeff Zhang <zjffdu@apache.org>\n\nCloses #3085 from zjffdu/minor_fix and squashes the following commits:\n\n71cf8389b [Jeff Zhang] [MINOR] Fix classpath issue in interpreter.sh","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/e6b6c0077fff3dc9275a4a9fb4a191bea8a2f9ff"},{"sha":"0dac535fcd964c7f76a4a5af8b958e7ac396da74","author":{"name":"Jeff Zhang","email":"ced9a24e2e494bb60341a9f26a61a78e7a11abad@apache.org"},"message":"ZEPPELIN-3621. Refactor getInterpreterInTheSameSession of Interpreter to reduce code duplication\n\n### What is this PR for?\nThis PR is trying to refactor method getInterpreterInTheSameSession in Interpreter, so that each interpreter don't need to reimplement its own and can reduce many code duplication.\n\n### What type of PR is it?\n[Refactoring]\n\n### Todos\n* [ ] - Task\n\n### What is the Jira issue?\n* https://issues.apache.org/jira/browse/ZEPPELIN-3621\n\n### How should this be tested?\n* CI pass\n\n### Screenshots (if appropriate)\n\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation? No\n\nAuthor: Jeff Zhang <zjffdu@apache.org>\n\nCloses #3069 from zjffdu/ZEPPELIN-3621 and squashes the following commits:\n\n06a6cc624 [Jeff Zhang] ZEPPELIN-3621. refactor getInterpreterInTheSameSession of Interpreter to reduce code duplication","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/0dac535fcd964c7f76a4a5af8b958e7ac396da74"},{"sha":"d49dcc251d26fb65f5b254ba31fb4fbd98891474","author":{"name":"Jeff Zhang","email":"ced9a24e2e494bb60341a9f26a61a78e7a11abad@apache.org"},"message":"[HOTFIX] Enable some commented test","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/d49dcc251d26fb65f5b254ba31fb4fbd98891474"},{"sha":"8769d41ad568155d67b5013c5e4ebe453e19190e","author":{"name":"Alex Bush","email":"87acdcb62e6d96d90e254645c158acc13c3a3d28@coxauto.co.uk"},"message":"[ZEPPELIN-3656] Fix for completion with Livy interpreter\n\n### What is this PR for?\nFix for NullPointerException when using code completion in the Livy Interpreter when Shared Interpreter is enabled.\n\n### What type of PR is it?\nBug Fix\n\n### What is the Jira issue?\n[ZEPPELIN-3656](https://issues.apache.org/jira/browse/ZEPPELIN-3656)\n\n### How should this be tested?\nRun Livy Interpreter in an environment where Shared Interpreter is enabled and attempt to trigger code completions.\n\nA unit test has been added to test the completion API when using a shared interpreter. Before applying the fix this test was failing with the same NPE.\n\n### Screenshots (if appropriate)\n\n### Questions:\n* Does the licenses files need update? No\n* Is there breaking changes for older versions? No\n* Does this needs documentation? No\n\nAuthor: Alex Bush <alex.bush@coxauto.co.uk>\n\nCloses #3094 from alexjbush/ZEPPELIN-3656 and squashes the following commits:\n\n4f4a875bf [Alex Bush] Correct completion test\ncd0c2133f [Alex Bush] Force a failing test\n71562e0f1 [Alex Bush] Test IT for Changes\n32aa7c567 [Alex Bush] [ZEPPELIN-3656] Fix for completion with Livy interpreter","distinct":true,"url":"https://api.github.com/repos/sameer79/zeppelin/commits/8769d41ad568155d67b5013c5e4ebe453e19190e"}]}
{"actor":{"display_login":"sameer79"}}