Addressed feedback on PR

Author: spmallette

gremlin-console/src/main/groovy/org/apache/tinkerpop/gremlin/console/GremlinGroovysh.groovy

@@ -25,10 +25,8 @@ import org.apache.groovy.groovysh.ParseCode
 import org.apache.groovy.groovysh.Parser
 import org.apache.groovy.groovysh.Parsing
 import org.apache.groovy.groovysh.util.CommandArgumentParser
-import org.apache.groovy.groovysh.util.ScriptVariableAnalyzer
 import org.apache.tinkerpop.gremlin.console.commands.GremlinSetCommand
 import org.codehaus.groovy.control.CompilerConfiguration
-import org.codehaus.groovy.control.MultipleCompilationErrorsException
 import org.codehaus.groovy.control.customizers.ASTTransformationCustomizer
 import org.codehaus.groovy.tools.shell.IO
 
@@ -43,7 +41,7 @@ class GremlinGroovysh extends Groovysh {
     private final static CompilerConfiguration compilerConfig = new CompilerConfiguration(CompilerConfiguration.DEFAULT) {{
         addCompilationCustomizers(new ASTTransformationCustomizer(ThreadInterrupt.class))
     }}
-    private Parsing remoteParser = new RemoteParser()
+    private Parsing remoteParser = new LocalSafeParser()
 
     public GremlinGroovysh(final Mediator mediator, final IO io) {
         super(io, compilerConfig)

gremlin-console/src/main/groovy/org/apache/tinkerpop/gremlin/console/RemoteParser.groovy renamed to gremlin-console/src/main/groovy/org/apache/tinkerpop/gremlin/console/LocalSafeParser.groovy

@@ -31,13 +31,13 @@ import org.codehaus.groovy.control.messages.SyntaxErrorMessage
  * the Groovy compiler to try to compile the script and analyzes any compilation errors to determine if the script is
  * incomplete. It does not evaluate the script, which prevents local execution and potential local failure.
  */
-class RemoteParser implements Parsing {
+class LocalSafeParser implements Parsing {
 
     // Try to compile the script
-    def config = new CompilerConfiguration()
+    private final CompilerConfiguration config = new CompilerConfiguration()
 
     // Create a custom GroovyClassLoader that doesn't write class files
-    def gcl = new GroovyClassLoader(this.class.classLoader, config) {
+    private final GroovyClassLoader gcl = new GroovyClassLoader(this.class.classLoader, config) {
         @Override
         public Class defineClass(String name, byte[] bytes) {
             // Skip writing to disk, just define the class in memory

gremlin-console/src/test/groovy/org/apache/tinkerpop/gremlin/console/RemoteParserTest.groovy renamed to gremlin-console/src/test/groovy/org/apache/tinkerpop/gremlin/console/LocalSafeParserTest.groovy

@@ -25,18 +25,17 @@ import org.junit.Test
 
 import static org.junit.Assert.assertEquals
 import static org.junit.Assert.assertNotNull
-import static org.junit.Assert.assertTrue
 
 /**
- * Unit tests for {@link RemoteParser}.
+ * Unit tests for {@link LocalSafeParser}.
  */
-class RemoteParserTest {
+class LocalSafeParserTest {
 
-    private RemoteParser parser
+    private LocalSafeParser parser
 
     @Before
     void setUp() {
-        parser = new RemoteParser()
+        parser = new LocalSafeParser()
     }
 
     @Test