lahodaj commented on PR #7382: URL: https://github.com/apache/netbeans/pull/7382#issuecomment-2147803368
> I saw this exception twice in CI logs of this PR: https://github.com/apache/netbeans/actions/runs/9366885368/attempts/1 https://github.com/apache/netbeans/actions/runs/9319212924/attempts/1 > > I don't remember this being a common failure. (CV tests on linux typically don't need retries) edit: looking further into this, this seems to be cross platform, I see the same exception on win, mac and linux and I can't find it in CI runs of the master branch. > > @lahodaj do you think this relates to the changes made here? > > ``` > NbModuleSuite has been started with failOnMessage(OFF) and failOnException(INFO). The following failures have been captured: > [org.openide.util.RequestProcessor] THREAD: org.netbeans.modules.java.file.launcher.queries.MultiSourceRootProvider MSG: Error in RequestProcessor org.netbeans.modules.java.file.launcher.queries.MultiSourceRootProvider$AttributeBasedClassPathImplementation$$Lambda$428/0x0000000400a262c8 > org.openide.util.RequestProcessor$FastItem: java.lang.NullPointerException: Cannot invoke "java.util.concurrent.atomic.AtomicReference.get()" because "this.processorModulePath" is null > Caused by: java.lang.NullPointerException: Cannot invoke "java.util.concurrent.atomic.AtomicReference.get()" because "this.processorModulePath" is null > org.netbeans.modules.java.source.indexing.APTUtils.validatePaths(APTUtils.java:348) > org.netbeans.modules.java.source.indexing.APTUtils.verifyAttributes(APTUtils.java:452) > org.netbeans.modules.java.source.indexing.APTUtils.stateChanged(APTUtils.java:307) > org.netbeans.modules.java.source.indexing.APTUtils.propertyChange(APTUtils.java:316) > java.desktop/java.beans.PropertyChangeSupport.fire(PropertyChangeSupport.java:343) > java.desktop/java.beans.PropertyChangeSupport.firePropertyChange(PropertyChangeSupport.java:335) > org.netbeans.api.java.classpath.ClassPath.firePropertyChange(ClassPath.java:691) > org.netbeans.api.java.classpath.ClassPath$SPIListener.propertyChange(ClassPath.java:1328) > org.openide.util.WeakListenerImpl$PropertyChange.propertyChange(WeakListenerImpl.java:189) > java.desktop/java.beans.PropertyChangeSupport.fire(PropertyChangeSupport.java:343) > java.desktop/java.beans.PropertyChangeSupport.firePropertyChange(PropertyChangeSupport.java:335) > java.desktop/java.beans.PropertyChangeSupport.firePropertyChange(PropertyChangeSupport.java:268) > org.netbeans.modules.java.file.launcher.queries.MultiSourceRootProvider$AttributeBasedClassPathImplementation.doUpdateDelegates(MultiSourceRootProvider.java:371) > org.openide.util.RequestProcessor$Task.run(RequestProcessor.java:1420) > org.netbeans.modules.openide.util.GlobalLookup.execute(GlobalLookup.java:45) > org.openide.util.lookup.Lookups.executeWith(Lookups.java:287) > org.openide.util.RequestProcessor$Processor.run(RequestProcessor.java:2035) > ``` Hmm, I think there's surely a latent bug in `APTUtils` - it starts to listen before the fields are initialized, so if the even arrives quickly, it may cause issues. The thing that might have change here is that now the event might realistically arrive. I've tried to improve the listening here: https://github.com/apache/netbeans/pull/7382/commits/3e47c08121e60aa7a972a7b791b8fd9a6b387a7e Let's see how that goes. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
