Closed
Conversation
### What is this PR for? jetty improvement jetty/jetty.project#3892 ### What type of PR is it? Improvement ### Todos * [ ] - Task ### What is the Jira issue? * ZEPPELIN-4733 ### How should this be tested? * https://travis-ci.org/github/ChowXu/zeppelin/builds/672386044 ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation?No Author: xuzhou <xuzhou@xiaohongshu.com> Closes #3725 from ChowXu/ZEPPELIN-4733 and squashes the following commits: e645878 [xuzhou] [ZEPPELIN-4733]Upgrade jetty 9.4.18.v20190429 to 9.4.27.v20200227 (cherry picked from commit aec6293) Signed-off-by: Alex Ott <alexott@apache.org>
…terpreterLauncher ### What is this PR for? This PR is to just merge process env into interpreter process env in InterpreterLauncher so that env defined in zeppelin-env.sh also apply to interpreters. e.g. `FLINK_HOME` specified in `zeppelin-env.sh` will apply on flink interpreter. But flink interpreter can override it. ### What type of PR is it? [Improvement] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4725 ### How should this be tested? * CI pass ### 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 <zjffdu@apache.org> Closes #3719 from zjffdu/ZEPPELIN-4725 and squashes the following commits: 33ecff4 [Jeff Zhang] address comment 41f1f87 [Jeff Zhang] [ZEPPELIN-4725]. Merge process env into interpreter process env in InterpreterLauncher (cherry picked from commit 0430161) Signed-off-by: Alex Ott <alexott@apache.org>
### What is this PR for? This commit declares all plugins in the `pluginManagement` section and unifies versions of all plugins - they are now declared as properties in the top-level `pom.xml`. All other `pom.xml` files shouldn't specify version explicitly. ### What type of PR is it? Improvement ### What is the Jira issue? * ZEPPELIN-4700 ### How should this be tested? * One build was clean: https://travis-ci.org/github/alexott/zeppelin/builds/668812389, but after removing the not necessary files from commit, some tests began to fail: https://travis-ci.org/github/alexott/zeppelin/builds/669129366 - not sure if it's a case of flaky tests (yes, it looks like it the case - after restart of failed builds - they are all green) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Alex Ott <alexott@gmail.com> Closes #3707 from alexott/ZEPPELIN-4700 and squashes the following commits: 6de1eb3 [Alex Ott] [ZEPPELIN-4700] Cleanup Maven warnings & unify plugin management (cherry picked from commit 8815eec) Signed-off-by: Alex Ott <alexott@apache.org>
…ment completion ### What is this PR for? Trivial PR which only return empty list instead of null. ### What type of PR is it? [Bug Fix] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4735 ### How should this be tested? * Manually 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: Jeff Zhang <zjffdu@apache.org> Closes #3726 from zjffdu/ZEPPELIN-4735 and squashes the following commits: 4efb2a9 [Jeff Zhang] address comment 1a534ac [Jeff Zhang] [ZEPPELIN-4735]. Fail to getCompletion when interpreter doesn't implement completion (cherry picked from commit ec2f03f) Signed-off-by: Alex Ott <alexott@apache.org>
### What is this PR for? The root cause is that when restarting interpreter, zeppelin will first cancel all jobs, while the cancelling paragraph thread in interpreter process may invoke thrift call on zeppelin server side. The stacktrace in jira description has one such example of flink interpreter. This PR fix this issue by canceling the paragraph in another thread. It is fine to cancel paragraph asynchronously ### What type of PR is it? [Bug Fix ] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4734 ### How should this be tested? * CI pass ### 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 <zjffdu@apache.org> Closes #3723 from zjffdu/ZEPPELIN-4734 and squashes the following commits: c0e8c80 [Jeff Zhang] [ZEPPELIN-4734]. Sometimes it is unable to restart interpreter (cherry picked from commit 8eaf864) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for?
This PR is to allow user to display streaming data in flink table api just like displaying streaming data in stream sql (%flink.ssql). I implement it in both scala and pyflink.
Here's one example in flink scala table api
```
val table = stenv.from("cdn_access_log")
.select("uuid, ip_to_province(client_ip) as province, response_size, request_time")
.groupBy("province")
.select( "province, count(uuid) as access_count, sum(response_size) as total_download, sum(response_size) * 1.0 / sum(request_time) as download_speed")
z.show(table, streamType="update")
```
### What type of PR is it?
[Feature ]
### Todos
* [ ] - Task
### What is the Jira issue?
* https://issues.apache.org/jira/browse/ZEPPELIN-4740
### How should this be tested?
Unit test is added and also verify it manually.
### 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 <zjffdu@apache.org>
Closes #3729 from zjffdu/ZEPPELIN-4740 and squashes the following commits:
bcd2597 [Jeff Zhang] [ZEPPELIN-4740]. Display streaming data in flink table api
(cherry picked from commit 80a3856)
Signed-off-by: Jeff Zhang <zjffdu@apache.org>
…nk interpreter ### What is this PR for? This PR would filter the temporary tables in show tables, only show the permanent tables. Otherwise it would confuse users if they see lots of temporary tables created by zeppelin. ### What type of PR is it? [ Improvement ] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4750 ### How should this be tested? * CI pass ### 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 <zjffdu@apache.org> Closes #3731 from zjffdu/ZEPPELIN-4750 and squashes the following commits: 85fe8b1 [Jeff Zhang] [ZEPPELIN-4750]. Don't display temporary tables in show tables of flink interpreter (cherry picked from commit 07a7f93) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
…mapping ### What is this PR for? When running the terminal interpreter in the notebook, the front end of the notebook needs to obtain the IP address of the server where the terminal interpreter is located to communicate. In a public cloud environment, the cloud host has an internal IP and an external access IP, and the interpreter runs in the cloud host. This will cause the notebook front end to be unable to connect to the terminal interpreter properly, resulting in the terminal interpreter being unusable. Solution: Set the mapping between internal IP and external IP in the terminal interpreter, and connect the front end of the notebook through the external IP of the terminal interpreter. ### What type of PR is it? [Improvement] ### What is the Jira issue? * https://jira.apache.org/jira/browse/ZEPPELIN-4747 ### 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: Xun Liu <liuxun@apache.org> Closes #3732 from liuxunorg/ZEPPELIN-4747 and squashes the following commits: 68f16ca [Xun Liu] Fixed c462c3e [Xun Liu] ZEPPELIN-4747. Terminal interpreter support internal and external IP mapping (cherry picked from commit faf082b) Signed-off-by: liuxun323 <liuxun@apache.org>
### What is this PR for? With this PR we are switching to the abstract class `ConfigStorage` to save/load Credentials. We have child classes of `ConfigStorage` which are able to save to disc or HDFS. NOTE: Cluster communication with changed user credentials is very simple. The `credential.json` content change is the only communication between cluster members. This leads to many rereads. ### What type of PR is it? Bug Fix ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4727 ### How should this be tested? * **Travis-CI**: https://travis-ci.org/github/Reamer/zeppelin/builds/673728600 ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Philipp Dallig <philipp.dallig@gmail.com> Closes #3718 from Reamer/credentials_storage and squashes the following commits: 6c8dd45 [Philipp Dallig] Use ConfigStage in Credentials and handle IOException in CredentialRestApi (cherry picked from commit c9d3f45) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? When Zeppelin is running on Kubernetes, SparkUI URL should be dynamically generated, while Kubernetes Service name for Spark interpreter Pod is generated on runtime. And Ingress controller or reverse-proxy route traffic to SparkUI. Problem is, depends on those Ingress or reverse proxy configuration, different SparkUI url format might be required. Currently, generated url format is hardcoded to "//<PORT>-<SERVICE_NAME>.<SERVICE_DOMAIN>". And letting user set 'zeppelin.spark.uiWebUrl' with static value doesn't help at all while url is decided on runtime. This PR accept [jinja template](https://jinja.palletsprojects.com/en/2.11.x/) string from 'zeppelin.spark.uiWebUrl' and bind 3 variables 'PORT', 'SERVICE_NAME', 'SERVICE_DOMAIN'. Therefore any URL pattern required by Ingress/Reverse-proxy can be specified. Each variable has values * PORT - spark ui port * SERVICE_NAME - [Service](https://kubernetes.io/docs/concepts/services-networking/service/) name for Spark Interpreter Pod. * SERVICE_DOMAIN - value of SERVICE_DOMAIN env variable. For example, when spark UI is running on port '4040', Service name for Spark interpreter pod is 'spark-wcoyqq', SERVICE_DOMAIN is my.domain.io, ``` https://port-{{PORT}}-{{SERVICE_NAME}}.{{SERVICE_DOMAIN}} ``` value on 'zeppelin.spark.uiWebUrl' property will generate Spark UI link with address ``` https://port-4040-spark-wcoyqq.mydomain.io ``` ### What type of PR is it? Improvement ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-4748 ### 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 <leemoonsoo@gmail.com> Closes #3728 from Leemoonsoo/ZEPPELIN-4748 and squashes the following commits: 0103c44 [Lee moon soo] update description of the property 42a51b6 [Lee moon soo] set classloader for jinjava to prevent "jinjava.javax.el.ELException: Class com.hubspot.jinjava.el.ExtendedSyntaxBuilder not found" c931253 [Lee moon soo] template rendering zeppelin.spark.uiWebUrl property value (cherry picked from commit d55c857) Signed-off-by: Lee moon soo <moon@apache.org>
…eter with K8s ### What is this PR for? This PullRequest fixes the impersonation issue when running Spark Zeppelin interpreter on K8s. A general user impersonation should not be necessary and possible on K8s, since the interpreter process runs unprivileged - without the right to change the user via ssh. ### What type of PR is it? * Improvement ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4722 ### How should this be tested? * **Travic-CI**: https://travis-ci.org/github/Reamer/zeppelin/builds/674723139 ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Philipp Dallig <philipp.dallig@gmail.com> Closes #3733 from Reamer/proxy_user and squashes the following commits: 8c2e58b [Philipp Dallig] proxy user in spark on k8s 3989c4e [Philipp Dallig] Some Cleanup (cherry picked from commit 85fc206) Signed-off-by: Lee moon soo <moon@apache.org>
### What is this PR for? fix non-compilable test: K8sRemoteInterpreterProcessTest.java ### What type of PR is it? Hot Fix Author: Alex Ott <alexott@apache.org> Closes #3734 from alexott/master and squashes the following commits: 5ca317a [Alex Ott] [hotfix] fix non-compilable test (cherry picked from commit 70c3a52) Signed-off-by: Alex Ott <alexott@apache.org>
### What is this PR for? Errors & warnings are produced from the checks that are assuming that Zeppelin is running on Linux (`getent` is present) & in the container environment (`/etc/passwd` is writable). This patch fixes following: - added check for presence of the `getent` - added explicit check that script is running in the container by checking `/proc/self/cgroup` - also removed `MaxPermSize` JVM flag that isn't supported in Java 8 ### What type of PR is it? Bug Fix ### What is the Jira issue? * ZEPPELIN-4757 ### How should this be tested? * https://travis-ci.org/github/alexott/zeppelin/builds/675356068 Author: Alex Ott <alexott@apache.org> Closes #3735 from alexott/ZEPPELIN-4757 and squashes the following commits: ee1475a [Alex Ott] [ZEPPELIN-4757] Fix startup script errors/warnings (cherry picked from commit b877e38) Signed-off-by: Alex Ott <alexott@apache.org>
### What is this PR for? This PR sorts documents returned by aggregation $graphLookup which used by MongoNotebookRepo to get the path of note. ### What type of PR is it? Bug Fix ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-4743 ### How should this be tested? 2 types of tests: * Unit test added * Create notes(more nested folders is better) in zeppelin web UI, then restart zeppelin or call /api/notebook-repositories/reload. After doing the above, check if the note path is correct. ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: archy.gu <archy.gu@maiscrm.com> Closes #3730 from zuiluoyin/ZEPPELIN-4743 and squashes the following commits: abb4c0d [archy.gu] [ZEPPELIN-4743] fix MongoNotebookRepo get unstable note path
…aragraph ### What is this PR for? This PR is to support setting savepoint for flink job and resume job from savepoint. It works for both flink sql and flink table api. ### What type of PR is it? [Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4717 ### How should this be tested? Unit test is added ### 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 <zjffdu@apache.org> Closes #3717 from zjffdu/ZEPPELIN-4717 and squashes the following commits: 4db897b [Jeff Zhang] [ZEPPELIN-4717]. Support savepoint for insert statement and non-sql paragraph (cherry picked from commit d340867) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? Render the graphs at the next tick to get the correct size ### What type of PR is it? [Bug Fix] ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-4755 ### How should this be tested? Repeatedly switch graph types and check their size ### 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: Hsuan Lee <hsuangm@gmail.com> Closes #3739 from hsuanxyz/fix/result-render and squashes the following commits: bf24d71 [Hsuan Lee] fix: result graph cannot be set to the correct size (cherry picked from commit 5cce538) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? Improve Java 8 support: - Remove obsolete Java options - add the check for minimum Java version (Java 8u151) to `zeppelin.sh` & `interpreter.sh` ### What type of PR is it? Refactoring ### What is the Jira issue? * ZEPPELIN-4758 ### How should this be tested? * https://travis-ci.org/github/alexott/zeppelin/builds/677566776 Author: Alex Ott <alexott@gmail.com> Closes #3738 from alexott/ZEPPELIN-4758 and squashes the following commits: e607ee6 [Alex Ott] [ZEPPELIN-4758] Additional cleanup for Java 8 support (cherry picked from commit b68248b) Signed-off-by: Alex Ott <alexott@apache.org>
### What is this PR for? Trivial PR which just remove minus sign in paragraph id. ### What type of PR is it? [ Improvement ] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4742 ### How should this be tested? * CI pass ### 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 <zjffdu@apache.org> Closes #3743 from zjffdu/ZEPPELIN-4742 and squashes the following commits: 8d025b3 [Jeff Zhang] [ZEPPELIN-4742]. Don't include minus sign in paragraph id (cherry picked from commit 75ff1d3) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? This PR just resolve the paragraph shaking issue. Before this PR, I will refresh paragraph output by 2 steps: step 1. clear previous output, step 2. update paragraph with new output. After this PR, I will just make it as just one step: update paragraph with new output. The main change is o method InterpreterOutput#clear. I add one flag to indicate whether it should send the update to frontend to refresh the output. ### What type of PR is it? [ Improvement ] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4759 ### How should this be tested? * CI pass ### Screenshots (if appropriate) * Before  * After  ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Jeff Zhang <zjffdu@apache.org> Closes #3741 from zjffdu/ZEPPELIN-4759 and squashes the following commits: b8787df [Jeff Zhang] [ZEPPELIN-4759]. Paragraph refreshing make the other paragraph shaking (cherry picked from commit ad33807) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? This PR fixes a small part of #3571, which introduced a SSL-connector regression. The port definition of the SSL connector has been deleted. https://github.com/apache/zeppelin/pull/3571/files#diff-3477d79f3012453f20c3727ec60338e9L337-L341 ### What type of PR is it? - Hot Fix ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4766 ### How should this be tested? * **Travis-CI:** https://travis-ci.org/github/Reamer/zeppelin/builds/677349664 ### 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: Philipp Dallig <philipp.dallig@gmail.com> Closes #3740 from Reamer/ssl_regression and squashes the following commits: 44125ce [Philipp Dallig] Fix SSL regression (cherry picked from commit f577f1e) Signed-off-by: Alex Ott <alexott@apache.org>
### What is this PR for? We should create a [k8s-deployment](https://kubernetes.io/docs/concepts/workloads/controllers/deployment/) for zeppelin-server. Changes in detail: - Switch from Pod to Deployment - Creation of a ServiceAccount `zeppelin-server` + update RoleBinding - Using a ConfigMap for all Environment variables. I think this ConfigMap give a better overview over all environment variables - Label change from `app` to [`app.kubernetes.io/name`](https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/#labels) ### What type of PR is it? - Improvement ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4770 ### How should this be tested? * Install on K8s with `kubectl apply -f k8s/zeppelin-server.yml` ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Philipp Dallig <philipp.dallig@gmail.com> Closes #3746 from Reamer/k8s_deployment and squashes the following commits: b8c158a [Philipp Dallig] Switch to K8s deployment (cherry picked from commit d252d0b) Signed-off-by: Lee moon soo <moon@apache.org>
### What is this PR for? This patch bumps versions of some components that were reported as having (potential?) vulnerabilities. It also unifies usage of different hadoop versions, etc., moving version configuration to a top-level pom.xml, instead of specification of them inside individual poms. This for example, also fixes #3702. ### What type of PR is it? Refactoring ### TODOs We'll need to followup on the other items, like: * Check dependencies in the Docker image ### What is the Jira issue? * ZEPPELIN-4695 ### How should this be tested? * https://travis-ci.org/github/alexott/zeppelin/builds/678099649 ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Alex Ott <alexott@apache.org> Closes #3745 from alexott/ZEPPELIN-4695 and squashes the following commits: 09e31f6 [Alex Ott] [ZEPPELIN-4695] Bump dependencies to fix CVEs (cherry picked from commit a1dc93d) Signed-off-by: Alex Ott <alexott@apache.org>
### What is this PR for? This PR is to support build real time dashboard via jdbc interpreter. User just need to specify refreshInterval to enable it. e.g. ``` %mysql(refreshInterval=2000) select * from flink_cdn.cdn_access_statistic; ``` ### What type of PR is it? [Feature ] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4764 ### How should this be tested? * Unit test is added, also manually 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: Jeff Zhang <zjffdu@apache.org> Closes #3742 from zjffdu/ZEPPELIN-4764 and squashes the following commits: d03f9fe [Jeff Zhang] [ZEPPELIN-4764]. Support real time dashboard for jdbc interpreter (cherry picked from commit 02749ab) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? Small typo fix ### What type of PR is it? Bug Fix ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4768 ### How should this be tested? * **Travis-CI**: https://travis-ci.org/github/Reamer/zeppelin/builds/677598691 ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Philipp Dallig <philipp.dallig@gmail.com> Closes #3744 from Reamer/typo_in_spark_user_impersonate and squashes the following commits: 7094ab0 [Philipp Dallig] Correct a typo in isUserImpersonateForSparkInterpreter (cherry picked from commit badb2b4) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? This PR is to fix the highlight issue. The root cause is in the magic parsing logic in frontend. This PR just move the magic logic from frontend to backend. So that we use the consistent parsing logic. ### What type of PR is it? [Bug Fix] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4775 ### How should this be tested? * CI pass ### Screenshots (if appropriate) Before  After  ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Jeff Zhang <zjffdu@apache.org> Closes #3748 from zjffdu/ZEPPELIN-4775 and squashes the following commits: a030427 [Jeff Zhang] [ZEPPELIN-4775]. Sometimes sql text is not highlighted (cherry picked from commit 28a0a6d) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? This PR uses crypto functions from the [bouncycastle project](https://www.bouncycastle.org/). The library was already used in `zeppelin-interpreter`. I don't see a license link in the [LICENSE](https://github.com/apache/zeppelin/blob/master/LICENSE). I hope someone can help me to correct this situation. EDIT: Thanks to Leemoonsoo for clarification ### What type of PR is it? - Feature ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4774 ### How should this be tested? * Travis-CI: https://travis-ci.org/github/Reamer/zeppelin/builds/679002184 ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? Yes Author: Philipp Dallig <philipp.dallig@gmail.com> Closes #3747 from Reamer/pem_certs and squashes the following commits: 6263e0b [Philipp Dallig] Create sslContextFactory with PEM files (cherry picked from commit 067aba8) Signed-off-by: Alex Ott <alexott@apache.org>
…streaming job ### What is this PR for? This PR is to remove duplicated flink job url. In ZEPPELIN-4759, I remove the refresh step in frontend, so it is not necessary to send job url again for each data refresh. So in this PR, I remove sendFlinkJobUrl in flink streaming job. ### What type of PR is it? [Bug Fix ] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4776 ### How should this be tested? * CI pass ### 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 <zjffdu@apache.org> Closes #3749 from zjffdu/ZEPPELIN-4776 and squashes the following commits: ff872ef [Jeff Zhang] [ZEPPELIN-4776]. The same flink job url is duplicated in fronted for streaming job (cherry picked from commit 564f166) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
… format ### What is this PR for? Without this PR, the bokeh output will is not in correct format (see below screenshot). This root cause is the output type is not correct. We should only use html when it is ir kernel for jupyter interpreter. ### What type of PR is it? [Bug Fix ] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4771 ### How should this be tested? * CI pass ### Screenshots (if appropriate) Before  After  ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Jeff Zhang <zjffdu@apache.org> Closes #3750 from zjffdu/ZEPPELIN-4771 and squashes the following commits: da6578e [Jeff Zhang] [ZEPPELIN-4771]. Bokeh output in IPythonInterpreter is not in correct format (cherry picked from commit 0a27eee) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
…-shaded"" This reverts commit 8e5e4b5.
…graphId) # What is this PR for? The root cause is that the interpreter context of python thread is null. This PR fix this issue by calling method `setInterpreterContextInPython` in python side. It fix this issue in both PySparkInterpreter & IPySparkInterpreter. ### What type of PR is it? [Bug Fix ] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5049 ### How should this be tested? * CI pass ### 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 <zjffdu@apache.org> Closes #3922 from zjffdu/ZEPPELIN-5049 and squashes the following commits: 84a196b [Jeff Zhang] [ZEPPELIN-5049]. NullPointerException when running z.run(noteId, paragraphId) (cherry picked from commit 8b04a94) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
…ch notebook ### What is this PR for? Use the interpreter group id as the spark app name if it is not set. So that user can easily figure out which yarn app is what he is looking for no matter it is note isolated or user isolated. ### What type of PR is it? [Improvement] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4409 ### How should this be tested? * CI pass ### 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 <zjffdu@apache.org> Closes #3927 from zjffdu/ZEPPELIN-4409 and squashes the following commits: d4b72a7 [Jeff Zhang] remove unnecessary code 28051f8 [Jeff Zhang] [ZEPPELIN-4409]. Set spark.app.name to know which user is running which notebook (cherry picked from commit e424808) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
…hen using hadoop filesystem ### What is this PR for? `ZeppelinConfiguration#getRelativeDir` actually get the absolute path intead of relative path, this PR just rename this method. ### What type of PR is it? [Improvement ] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4910 ### How should this be tested? * CI pass ### 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 <zjffdu@apache.org> Closes #3833 from zjffdu/ZEPPELIN-4910 and squashes the following commits: c3962b4 [Jeff Zhang] save ee86b2c [Jeff Zhang] [ZEPPELIN-4910]. Don convert relative path to local absolute path when using hadoop filesystem (cherry picked from commit 6a3e84b) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? There is a bug in the project: $intp variable is not defined for Spark/Scala 2.12 interpreter. It looks like this part of REPL in Spark 2.12 and in Spark 2.11 is the same, so I use the code from Spark/Scala 2.11 interpreter to fix it. ### What type of PR is it? Bug Fix ### What is the Jira issue? [ZEPPELIN-5081] ### How should this be tested? * Tested manually with master and Spark 2.4.2/Scala 2.12.8 Author: khud <vitaly.khudobakhshov@gmail.com> Closes #3935 from khud/fix_missing_intp and squashes the following commits: 301b7a4 [khud] Add test for $intp 142cf6f [khud] Add loopPostInit to bind $intp (cherry picked from commit fbf3019) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? This PR fix the explain statement issue in flink interpreter. For flink 1.10, 1.11, explain works in different approach, so we need to use shim to do that. This PR also add unit test to prevent regression in future. ### What type of PR is it? [Bug Fix] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5088 ### How should this be tested? * CI pass https://travis-ci.org/github/zjffdu/zeppelin/builds/734175742 ### 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 <zjffdu@apache.org> Closes #3936 from zjffdu/ZEPPELIN-5088 and squashes the following commits: 83ac341 [Jeff Zhang] [ZEPPELIN-5088]. Explain statement is broken in flink interpreter (cherry picked from commit d4c18c1) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? When trying to run a disabled paragraph, backend hangs because the paragraph is not started, but we are still waiting for it to finish. Instead, we should skip it just like empty paragraphs. Also, we need to change paragraph state to pending, so next state change to finished is broadcasted to client. ### What type of PR is it? Bug Fix ### Todos ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-5060 ### How should this be tested? ### Screenshots (if appropriate) ### Questions: Author: Timo Olkkonen <timo.p.olkkonen@gmail.com> Closes #3932 from olkkoti/ZEPPELIN-5060 and squashes the following commits: 9de27f0 [Timo Olkkonen] Update integration test: disabled paragraph should be FINISHED after run all. 921e3df [Timo Olkkonen] Set Paragraph status to Pending at start of execution, and skip if it is disabled.
…Spark Interpreter Scala REPL and fix the CI failure due to low Scala version ### What is this PR for? - fix the [CI failure](https://travis-ci.org/github/apache/zeppelin/builds/709913046) due to [PR-3852](#3852) ### What type of PR is it? [Bug Fix] ### Todos * [ ] - Task ### What is the Jira issue? * [ZEPPELIN-4962](https://issues.apache.org/jira/projects/ZEPPELIN/issues/ZEPPELIN-4962) ### How should this be tested? * CI test ### 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: xiejiajun <xiejiajun02@163.com> Author: xie-jia-jun <xiejiajun02@163.com> Author: JakeXie <xiejiajun02@163.com> Closes #3860 from xiejiajun/ZEPPELIN-4962 and squashes the following commits: 9128c9b [JakeXie] spark.repl.target docs update ad4c0e3 [xiejiajun] Clear irrelevant code a12d3a9 [xiejiajun] Support for manually specifying the Java version of Spark Interpreter Scala REPL and fix the CI failure due to low Scala version ab2b191 [xiejiajun] Merge branch 'master' of https://github.com/apache/zeppelin into apache-master 5569788 [xiejiajun] Merge branch 'master' of https://github.com/apache/zeppelin into apache-master 0a9af6c [xiejiajun] Merge branch 'master' of https://github.com/apache/zeppelin into apache-master be36b37 [xiejiajun] 合并Apache Master分支冲突解决 1335d55 [xiejiajun] Merge remote-tracking branch 'origin/master' fc59f57 [JakeXie] Merge pull request #4 from apache/master 9cc70fe [xiejiajun] Merge remote-tracking branch 'origin/master' 6ef9b23 [xie-jia-jun] Merge pull request #3 from apache/master 45af87a [xiejiajun] added timeout for getting Thrift client to avoid situations where the interpreter may not be restarted when the interpreter process exits unexpectedly f149c3b [xie-jia-jun] Merge pull request #1 from apache/master 5d4b645 [xie-jia-jun] Support OSSConfigStorage of Aliyun dbb6639 [xie-jia-jun] Add Aliyun OSS SDK bb47849 [xie-jia-jun] Support S3ConfigStorage of AWS (cherry picked from commit 185ffd4) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? This PR includes: - Migration from commons-httpclient:commons-httpclient to org.apache.httpcomponents:httpclient - Drop some unneeded interpreter dependencies ### What type of PR is it? - Improvement ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5090 ### How should this be tested? * Travis-CI: https://travis-ci.org/github/Reamer/zeppelin/builds/734928234 ### Questions: * Does the licenses files need update? Yes (done) * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Philipp Dallig <philipp.dallig@gmail.com> Closes #3938 from Reamer/http_components and squashes the following commits: 7829bc7 [Philipp Dallig] Update commons-httpclient to org.apache.httpcomponents:httpclient 8668485 [Philipp Dallig] Remove some unused libraries in Zeppelin Interpreter (cherry picked from commit 3ada8af) Signed-off-by: Philipp Dallig <philipp.dallig@gmail.com>
### What is this PR for? This PR updates from aether to maven resolver. ### What type of PR is it? - Improvement ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-4989 ### How should this be tested? * Travis-CI: https://travis-ci.org/github/Reamer/zeppelin/builds/729250923 ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Philipp Dallig <philipp.dallig@gmail.com> Closes #3875 from Reamer/maven_resolver and squashes the following commits: 9b7ec55 [Philipp Dallig] Update to Maven resolver 92ff397 [Philipp Dallig] Update download plugin (cherry picked from commit 56ac563) Signed-off-by: Philipp Dallig <philipp.dallig@gmail.com>
…presto interpreter ### What is this PR for? The root cause is that url is not an valid property of presto, so we remove that before connecting to presto. The null url cause the NPE. ### What type of PR is it? [Bug Fix ] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5042 ### How should this be tested? * Manually 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: Jeff Zhang <zjffdu@apache.org> Closes #3942 from zjffdu/ZEPPELIN-5042 and squashes the following commits: 4562788 [Jeff Zhang] [ZEPPELIN-5042]. NullPointerException is thrown for the 1st query on presto interpreter (cherry picked from commit 78ea11d) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? The root cause is that when there's only sub interpreter, zeppelin could not get the right editor setting based in interpreter name (because interpreter name is omitted by users). This PR fix this issue by just using method getDefaultInterpreterInfo to get the correct editor setting. ### What type of PR is it? [Bug Fix] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5094 ### How should this be tested? * Manually 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: Jeff Zhang <zjffdu@apache.org> Closes #3941 from zjffdu/ZEPPELIN-5094 and squashes the following commits: 256d6df [Jeff Zhang] [ZEPPELIN-5094]. Unable to get EditorSetting for jdbc interpreter (cherry picked from commit 80ca24c) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? There're 2 issues in yarn interpreter mode for flink interpreter: 1. Fail to submit yarn app when hive is enabled 2. Unable to create HiveCatalog in hadoop3 ### What type of PR is it? [Bug Fix ] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5093 ### How should this be tested? *CI pass https://travis-ci.org/github/zjffdu/zeppelin/builds/734950621 ### 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 <zjffdu@apache.org> Closes #3940 from zjffdu/ZEPPELIN-5093 and squashes the following commits: 847da21 [Jeff Zhang] [ZEPPELIN-5093]. yarn interpreter mode is broken for flink interpreter (cherry picked from commit c62eb87) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
…n.packages ### What is this PR for? This root cause of this issue is that we always create a temp folder for downloading artifacts, and the temp folder will change for each flink interpreter, that cause it would download artifacts every time, no cache is enabled. This PR is to use the local maven repository folder as the flink repository folder. So that it could reuse the downloaded artifacts. ### What type of PR is it? [Improvement] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5091 ### How should this be tested? * CI pass, manually 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: Jeff Zhang <zjffdu@apache.org> Closes #3939 from zjffdu/ZEPPELIN-5091 and squashes the following commits: 71014c6 [Jeff Zhang] [ZEPPELIN-5091]. No cache for downloading artifacts of flink.execution.packages (cherry picked from commit 6629db7) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? Included in this PR: - shutdown LifecylceManager and RM-Heartbeat - Shutdown the ParallelScheduler-Worker tasks - Use Initialization-on-demand_holder_idiom to create `SchedulerFactory` and `ExecutorFactory` singleton - Shutdown all scheduler tasks, when destroying the `SchedulerFactory` - Soft shutdown of ExecutorService with utility class `ExecutorUtil`, which makes visible which thread is not shut down nicely ### What type of PR is it? - Improvement ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5089 ### How should this be tested? * Travis-CI: https://travis-ci.org/github/Reamer/zeppelin/builds/735636051 ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Philipp Dallig <philipp.dallig@gmail.com> Closes #3937 from Reamer/executor_services and squashes the following commits: 394363d [Philipp Dallig] Use ScheduledExecutorService to schedule RM-Heartbeat 457e981 [Philipp Dallig] final corrections f76b708 [Philipp Dallig] Use ExecutorFactory in TimeoutLifecycleManager 81bbf65 [Philipp Dallig] Add scheduledExecutors to ExecutorFactory ef8100a [Philipp Dallig] Soft shutdown of executor threads bc0f358 [Philipp Dallig] stop all sub scheduler 2be1f02 [Philipp Dallig] Use Initialization-on-demand holder idiom for static singleton f1e40e2 [Philipp Dallig] style changes (cherry picked from commit 4fceaf5) Signed-off-by: Philipp Dallig <philipp.dallig@gmail.com>
### What is this PR for? If a paragraph fails on something other than exception in Zeppelin, notebook will continue running paragraphs after it. This is unintuitive and regression from previous version. To fix it, return "false" from running a paragraph, if it the actual user code fails. Also, after a failing "run all" operation, we need to broadcast paragraph states to client, so they wont stay in pending state after a failed run. ### What type of PR is it? Bug Fix ### Todos ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-5077 ### How should this be tested? ### Screenshots (if appropriate) ### Questions: Author: Timo Olkkonen <timo.p.olkkonen@gmail.com> Closes #3933 from olkkoti/ZEPPELIN-5077 and squashes the following commits: 5c0f9be [Timo Olkkonen] Stop runAllParagraphs if paragraph result code is ERROR. aa4d079 [Timo Olkkonen] Add comment on why we need to broadcast paragraph states after failed runall. b9ae304 [Timo Olkkonen] Add a test to see that paragraph after a failing one is not run in runAll. 5e5a595 [Timo Olkkonen] If running all paragraphs fails, broadcast paragraphs states to client.
### What is this PR for? As described in the linked issue, there has been a bug where, when a user creates or updates a note, that user's note list is published to all connected users, instead of their own note lists. This has the security problem that it leaks information about a user's note list to other users who may not have permission to see it. This PR fixes that bug by querying each connected user's note list and publishing that instead. ### What type of PR is it? Bug Fix ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-5061 ### How should this be tested? Manual testing: * Configure a list of users using a `shiro.ini` file. Example: ``` [users] # List of users with their password allowed to access Zeppelin. # To use a different strategy (LDAP / Database / ...) check the shiro doc at http://shiro.apache.org/configuration.html#Configuration-INISections admin = password1, admin user1 = password2, role1 user2 = password3, role1 ``` * Turn off public notebooks in `zeppelin-site.xml` by setting the property `zeppelin.notebook.public` to `false` * Boot Zeppelin server on this PR * Log in as user1 * In a separate browser, log in as user2 * As user1, create a note Previous result: user1's note appears on user2's dashboard. Refreshing user2's browser will cause the note to disappear from their dashboard. Fixed expectations: - user1's note does not appear on user2's dashboard. - the display of any notes created by user1 is not affected when user2 creates a note - when `zeppelin.notebook.public` is set to `true`, user1's note appears as expected on user1's dashboard Author: David Golightly <david.golightly@leapyear.io> Closes #3926 from david-golightly-leapyear/dg/notebook-sharing-fix and squashes the following commits: 175285e [David Golightly] address feedback 4f00231 [David Golightly] get variable outside of loop 9930618 [David Golightly] remove unused import e9926e6 [David Golightly] fix note list broadcast
…r for a long time ### What is this PR for? The root cause is that by default FlinkILoop would create tmp folder for storing jars, but tmp folder may be delete by system. So this PR would always create this folder if it doesn't exist. ### What type of PR is it? [Improvement ] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5097 ### How should this be tested? * CI pass https://travis-ci.org/github/zjffdu/zeppelin/builds/736268130 ### 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 <zjffdu@apache.org> Closes #3945 from zjffdu/ZEPPELIN-5097 and squashes the following commits: 14de2ab [Jeff Zhang] [ZEPPELIN-5097]. FileNotFoundException after running flink interpreter for a long time (cherry picked from commit 39a9ec9) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? This PR includes: - Use `java.util.regex.Pattern` instead of `com.sun.org.apache.xerces.internal.impl.xpath.regex.RegularExpression` which is an internal proprietary API - style changes in `zeppelin-jupyter/src/test/java/org/apache/zeppelin/jupyter/nbformat/JupyterUtilTest.java` - add a simple test for `getNbformat` ### What type of PR is it? - Refactoring ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5098 ### How should this be tested? * Travis-CI: https://travis-ci.org/github/Reamer/zeppelin/builds/734957587 ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Philipp Dallig <philipp.dallig@gmail.com> Closes #3944 from Reamer/Jupyter and squashes the following commits: 1f10310 [Philipp Dallig] use java.util.regex.Pattern instead of com.sun.org.apache.xerces.internal.impl.xpath.regex.RegularExpression aa22f67 [Philipp Dallig] some style changes 4820e8b [Philipp Dallig] some cleanup (cherry picked from commit fee1e29) Signed-off-by: Philipp Dallig <philipp.dallig@gmail.com>
…rovement ### What is this PR for? This PR includes: - an error message from K8s launcher to help users understand why the Pod won't start - give the option to disable the timeout handling during the initial pod lifecycle phase `pending` ### What type of PR is it? - Improvement ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5096 ### How should this be tested? * Travis-CI: https://travis-ci.org/github/Reamer/zeppelin/builds/735737408 ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: Philipp Dallig <philipp.dallig@gmail.com> Closes #3943 from Reamer/k8s_start and squashes the following commits: 2c8b4a2 [Philipp Dallig] Add option to disable timeout handling when starting interpreter pods a68ce2e [Philipp Dallig] Print a nice ErrorMessage (cherry picked from commit d39d1c9) Signed-off-by: Philipp Dallig <philipp.dallig@gmail.com>
### What is this PR for? The root cause is that in flink 1.10 there's only temporary view, but in 1.11 flink introduce permanent view, so we should always use ddl to drop view instead of calling `tbenv.dropTemporaryView` ### What type of PR is it? [Bug Fix] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5101 ### How should this be tested? * Unit test is added https://travis-ci.org/github/zjffdu/zeppelin/builds/737948009 ### 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 <zjffdu@apache.org> Closes #3949 from zjffdu/ZEPPELIN-5101 and squashes the following commits: 64f4d21 [Jeff Zhang] [ZEPPELIN-5001]. Drop view doesn't work in flink interpreter (cherry picked from commit b1aac73) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? In this PR, we only print the error message when it is `AnalysisException`, this would make the error message more clear for users. ### What type of PR is it? [Improvement] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5102 ### How should this be tested? * Manually 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: Jeff Zhang <zjffdu@apache.org> Closes #3951 from zjffdu/ZEPPELIN-5102 and squashes the following commits: a912b21 [Jeff Zhang] [ZEPPELIN-5102]. Simplify spark sql error message (cherry picked from commit fa14699) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? In this PR, I would only print exception message when it is SQLException, otherwise would print the full stacktrace, because in this case it is most likely due to jdbc interpreter's internal error. ### What type of PR is it? [Improvement] ### Todos * [ ] - Task ### What is the Jira issue? * https://issues.apache.org/jira/browse/ZEPPELIN-5103 ### How should this be tested? * CI pass ### 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 <zjffdu@apache.org> Closes #3952 from zjffdu/ZEPPELIN-5103 and squashes the following commits: 4001e1b [Jeff Zhang] [ZEPPELIN-5103]. Simply jdbc interpreter error message (cherry picked from commit 6981c4d) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
### What is this PR for? Provide a configuration option that will allow to use path based access for S3 Notebook repo ### What type of PR is it? Improvement ### Todos * [x] - Task ### What is the Jira issue? https://issues.apache.org/jira/browse/ZEPPELIN-5105 ### How should this be tested? Start minio cluster without MINIO_DOMAIN. Configure minio s3 endpoint in zeppelin for notebook repo.Enable the configuration option, save a notebook in zeppelin, and confirm in the minio that the related file was saved. ### Questions: Does the licenses files need update? No Is there breaking changes for older versions? No. Does this needs documentation? I added mentions of the new option in existing documentation. Thank you! Author: jenil.shah <jenil.shah@ishisystems.com> Closes #3953 from jenil-shah135/master and squashes the following commits: b46820b [jenil.shah] S3 Service Bucket Access Using Path Style (cherry picked from commit 70ea189) Signed-off-by: Jeff Zhang <zjffdu@apache.org>
Contributor
|
I am closing this PR because it makes no sense to me. |
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
20 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.
NOT FOUND CLASS IN zeppelin-0.9.0-preview2
org.apache.zeppelin.interpreter.jupyter.proto.CancelRequest
org.apache.zeppelin.interpreter.jupyter.proto.* All Class is not found in source