Skip to content
This repository has been archived by the owner on Nov 5, 2022. It is now read-only.

Memory leak #2

Merged
merged 2 commits into from
Feb 20, 2015
Merged

Memory leak #2

merged 2 commits into from
Feb 20, 2015

Conversation

jglick
Copy link
Member

@jglick jglick commented Feb 18, 2015

Found that Caller.Info can be holding onto script classes even after the script has finished. @reviewbybees esp. @kohsuke

@jglick jglick mentioned this pull request Feb 18, 2015
8 tasks
@stephenc
Copy link
Member

👍

1 similar comment
@tfennelly
Copy link
Member

👍

@jglick jglick merged commit 4c6f76f into cloudbees:master Feb 20, 2015
jglick added a commit that referenced this pull request Feb 20, 2015
@jglick jglick deleted the memory-leak branch February 20, 2015 13:52
@jglick
Copy link
Member Author

jglick commented Feb 20, 2015

Apparently caused

java.lang.NullPointerException
    at com.cloudbees.groovy.cps.impl.Caller.isAsynchronous(Caller.java:50)
    at com.cloudbees.groovy.cps.impl.Caller$isAsynchronous.call(Unknown Source)
    at com.cloudbees.groovy.cps.CpsDefaultGroovyMethods.each(CpsDefaultGroovyMethods.groovy:24)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:606)
    at org.codehaus.groovy.runtime.metaclass.ReflectionMetaMethod.invoke(ReflectionMetaMethod.java:51)
    at org.codehaus.groovy.runtime.metaclass.NewInstanceMetaMethod.invoke(NewInstanceMetaMethod.java:54)
    at org.codehaus.groovy.runtime.callsite.PojoMetaMethodSite$PojoMetaMethodSiteNoUnwrapNoCoerce.invoke(PojoMetaMethodSite.java:271)
    at org.codehaus.groovy.runtime.callsite.PojoMetaMethodSite.call(PojoMetaMethodSite.java:53)
    at org.codehaus.groovy.runtime.callsite.AbstractCallSite.call(AbstractCallSite.java:116)
    at jenkins.model.InterruptedBuildAction.summary$_run_closure1.doCall(summary.groovy:6)

which I am investigating.

kohsuke added a commit that referenced this pull request Jul 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants