Skip to content

Commit

Permalink
fix: Better management of memory consumption
Browse files Browse the repository at this point in the history
  • Loading branch information
aelamrani committed Oct 25, 2019
1 parent dd8babb commit 5f20682
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
2 changes: 1 addition & 1 deletion src/main/java/io/gravitee/policy/groovy/GroovyPolicy.java
Expand Up @@ -221,7 +221,7 @@ private Class<?> getOrCreate(String script) throws CompilationFailedException {
String key = Sha1.sha1(script);
return sources.computeIfAbsent(key, s -> {
GroovyCodeSource gcs = new GroovyCodeSource(script, key, GroovyShell.DEFAULT_CODE_BASE);
return GROOVY_SHELL.getClassLoader().parseClass(gcs, true);
return GROOVY_SHELL.getClassLoader().parseClass(gcs, false);
});
}

Expand Down
6 changes: 0 additions & 6 deletions src/test/java/io/gravitee/policy/groovy/GroovyPolicyTest.java
Expand Up @@ -43,16 +43,12 @@ public class GroovyPolicyTest {

@Mock
private ExecutionContext executionContext;

@Mock
private Request request;

@Mock
private Response response;

@Mock
private PolicyChain policyChain;

@Mock
private GroovyPolicyConfiguration configuration;

Expand Down Expand Up @@ -149,6 +145,4 @@ private String readInputStreamToString(InputStream stream, Charset defaultCharse

return builder.toString();
}


}

0 comments on commit 5f20682

Please sign in to comment.