You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by zj...@apache.org on 2020/10/09 03:44:28 UTC
[zeppelin] branch master updated: Revert "Revert "[ZEPPELIN-4681].
Shade thrift in zeppelin-interpreter-shaded""
This is an automated email from the ASF dual-hosted git repository.
zjffdu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zeppelin.git
The following commit(s) were added to refs/heads/master by this push:
new efcc156 Revert "Revert "[ZEPPELIN-4681]. Shade thrift in zeppelin-interpreter-shaded""
efcc156 is described below
commit efcc1564f149d9fc744263163ffa38c1f73f4df8
Author: Jeff Zhang <zj...@apache.org>
AuthorDate: Wed Oct 7 16:02:22 2020 +0800
Revert "Revert "[ZEPPELIN-4681]. Shade thrift in zeppelin-interpreter-shaded""
This reverts commit 8e5e4b5e47a01e90a39551ad0a62267175476268.
---
.../java/org/apache/zeppelin/groovy/GObject.java | 5 ++---
zeppelin-interpreter-shaded/pom.xml | 2 --
.../zeppelin/interpreter/ZeppelinContext.java | 24 +++++++++-------------
3 files changed, 12 insertions(+), 19 deletions(-)
diff --git a/groovy/src/main/java/org/apache/zeppelin/groovy/GObject.java b/groovy/src/main/java/org/apache/zeppelin/groovy/GObject.java
index e7f27c3..2b4038b 100644
--- a/groovy/src/main/java/org/apache/zeppelin/groovy/GObject.java
+++ b/groovy/src/main/java/org/apache/zeppelin/groovy/GObject.java
@@ -18,7 +18,6 @@ package org.apache.zeppelin.groovy;
import groovy.lang.Closure;
import groovy.xml.MarkupBuilder;
-import org.apache.thrift.TException;
import org.apache.zeppelin.annotation.ZeppelinApi;
import org.apache.zeppelin.display.AngularObject;
import org.apache.zeppelin.display.AngularObjectRegistry;
@@ -240,7 +239,7 @@ public class GObject extends groovy.lang.GroovyObjectSupport {
}
@SuppressWarnings("unchecked")
- public void angularBind(String name, Object o, String noteId) throws TException {
+ public void angularBind(String name, Object o, String noteId) {
z.angularBind(name, o, noteId);
}
@@ -251,7 +250,7 @@ public class GObject extends groovy.lang.GroovyObjectSupport {
* @param name name of the variable
* @param o value
*/
- public void angularBind(String name, Object o) throws TException {
+ public void angularBind(String name, Object o) {
angularBind(name, o, interpreterContext.getNoteId());
}
diff --git a/zeppelin-interpreter-shaded/pom.xml b/zeppelin-interpreter-shaded/pom.xml
index a6f4ddc..f00a496 100644
--- a/zeppelin-interpreter-shaded/pom.xml
+++ b/zeppelin-interpreter-shaded/pom.xml
@@ -103,8 +103,6 @@
<exclude>org/apache/zeppelin/**/*</exclude>
<exclude>org/apache/hadoop/*</exclude>
<exclude>org/apache/hadoop/**</exclude>
- <exclude>org/apache/thrift/*</exclude>
- <exclude>org/apache/thrift/**/*</exclude>
<exclude>org/slf4j/*</exclude>
<exclude>org/slf4j/**/*</exclude>
<exclude>org/apache/commons/logging/*</exclude>
diff --git a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/ZeppelinContext.java b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/ZeppelinContext.java
index e678414..b50c54e 100644
--- a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/ZeppelinContext.java
+++ b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/ZeppelinContext.java
@@ -17,7 +17,6 @@
package org.apache.zeppelin.interpreter;
-import org.apache.thrift.TException;
import org.apache.zeppelin.annotation.Experimental;
import org.apache.zeppelin.annotation.ZeppelinApi;
import org.apache.zeppelin.display.AngularObject;
@@ -673,10 +672,9 @@ public abstract class ZeppelinContext {
*
* @param name name of the variable
* @param o value
- * @throws TException
*/
@ZeppelinApi
- public void angularBind(String name, Object o) throws TException {
+ public void angularBind(String name, Object o) {
angularBind(name, o, interpreterContext.getNoteId());
}
@@ -688,7 +686,7 @@ public abstract class ZeppelinContext {
* @param o value
*/
@Deprecated
- public void angularBindGlobal(String name, Object o) throws TException {
+ public void angularBindGlobal(String name, Object o) {
angularBind(name, o, (String) null);
}
@@ -701,7 +699,7 @@ public abstract class ZeppelinContext {
* @param watcher watcher of the variable
*/
@ZeppelinApi
- public void angularBind(String name, Object o, AngularObjectWatcher watcher) throws TException {
+ public void angularBind(String name, Object o, AngularObjectWatcher watcher) {
angularBind(name, o, interpreterContext.getNoteId(), watcher);
}
@@ -714,8 +712,7 @@ public abstract class ZeppelinContext {
* @param watcher watcher of the variable
*/
@Deprecated
- public void angularBindGlobal(String name, Object o, AngularObjectWatcher watcher)
- throws TException {
+ public void angularBindGlobal(String name, Object o, AngularObjectWatcher watcher) {
angularBind(name, o, null, watcher);
}
@@ -791,7 +788,7 @@ public abstract class ZeppelinContext {
* @param name
*/
@ZeppelinApi
- public void angularUnbind(String name) throws TException {
+ public void angularUnbind(String name) {
String noteId = interpreterContext.getNoteId();
angularUnbind(name, noteId);
}
@@ -802,7 +799,7 @@ public abstract class ZeppelinContext {
* @param name
*/
@Deprecated
- public void angularUnbindGlobal(String name) throws TException {
+ public void angularUnbindGlobal(String name) {
angularUnbind(name, null);
}
@@ -814,7 +811,7 @@ public abstract class ZeppelinContext {
* @param o value
* @param noteId
*/
- public void angularBind(String name, Object o, String noteId) throws TException {
+ public void angularBind(String name, Object o, String noteId) {
AngularObjectRegistry registry = interpreterContext.getAngularObjectRegistry();
if (registry.get(name, noteId, null) == null) {
@@ -833,7 +830,7 @@ public abstract class ZeppelinContext {
* @param noteId
* @param paragraphId
*/
- public void angularBind(String name, Object o, String noteId, String paragraphId) throws TException {
+ public void angularBind(String name, Object o, String noteId, String paragraphId) {
AngularObjectRegistry registry = interpreterContext.getAngularObjectRegistry();
if (registry.get(name, noteId, paragraphId) == null) {
@@ -852,8 +849,7 @@ public abstract class ZeppelinContext {
* @param o value
* @param watcher watcher of the variable
*/
- private void angularBind(String name, Object o, String noteId, AngularObjectWatcher watcher)
- throws TException {
+ private void angularBind(String name, Object o, String noteId, AngularObjectWatcher watcher) {
AngularObjectRegistry registry = interpreterContext.getAngularObjectRegistry();
if (registry.get(name, noteId, null) == null) {
@@ -908,7 +904,7 @@ public abstract class ZeppelinContext {
*
* @param name
*/
- private void angularUnbind(String name, String noteId) throws TException {
+ private void angularUnbind(String name, String noteId) {
AngularObjectRegistry registry = interpreterContext.getAngularObjectRegistry();
registry.remove(name, noteId, null);
}