From 044a99ddd7b787eeee0d07ea0252d759dbbdf3af Mon Sep 17 00:00:00 2001
From: Alex Goodman <agoodm@users.noreply.github.com>
Date: Tue, 4 Oct 2016 17:46:30 -0700
Subject: [PATCH] Ensure that registered hooks are applied after call to open()

---
 .../interpreter/remote/RemoteInterpreterServer.java       | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java
index 2f3b13bb1a..3850c7daad 100644
--- a/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java
+++ b/zeppelin-interpreter/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreterServer.java
@@ -431,6 +431,14 @@ public class RemoteInterpreterServer
       try {
         InterpreterContext.set(context);
         
+        // Open the interpreter instance prior to calling interpret().
+        // This is necessary because the earliest we can register a hook
+        // is from within the open() method.
+        LazyOpenInterpreter lazy = (LazyOpenInterpreter) interpreter;
+        if (!lazy.isOpen()) {
+          lazy.open();
+        }
+        
         // Add hooks to script from registry.
         // Global scope first, followed by notebook scope
         processInterpreterHooks(null);