Ole8700 / openhab

Automatically exported from code.google.com/p/openhab
GNU General Public License v3.0
1 stars 0 forks source link

ConcurrentModificationException at ExecGenericBindingProvider #260

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

Regular startup

What is the expected output? What do you see instead?

Works ok but exception is present in log

What version of the product are you using? On what operating system?

1.2.0/Ubuntu 12.10/Java 1.6.0_45

Please provide any additional information below.

Logs

11:40:51.018 INFO  o.o.u.w.i.s.WebAppServlet[:110]- Stopped Classic UI
11:41:11.690 INFO  o.o.c.internal.CoreActivator[:92]- openHAB runtime has been 
started (v1.2.0).
11:41:19.034 INFO  o.o.m.c.i.f.FolderObserver[:163]- Loaded file 'default.items'
11:41:19.042 ERROR o.o.c.s.AbstractActiveService[:170]- Error while executing 
background thread Exec Refresh Service
java.util.ConcurrentModificationException: null
        at java.util.WeakHashMap$HashIterator.nextEntry(WeakHashMap.java:762)
        at java.util.WeakHashMap$KeyIterator.next(WeakHashMap.java:795)
        at org.openhab.binding.exec.internal.ExecGenericBindingProvider.getInBindingItemNames(ExecGenericBindingProvider.java:329)
        at org.openhab.binding.exec.internal.ExecBinding.execute(ExecBinding.java:123)
        at org.openhab.core.binding.AbstractActiveBinding$BindingActiveService.execute(AbstractActiveBinding.java:168)
        at org.openhab.core.service.AbstractActiveService$RefreshThread.run(AbstractActiveService.java:168)
11:41:21.828 INFO  o.o.i.s.i.DiscoveryServiceImpl[:92]- mDNS service has been 
started
11:41:24.523 INFO  o.o.i.r.i.RESTApplication[:158]- Started REST API at /rest
11:41:29.298 INFO  o.o.m.c.i.f.FolderObserver[:163]- Loaded file 
'default.sitemap'
11:41:29.442 INFO  o.o.m.c.i.f.FolderObserver[:163]- Loaded file 'rrd4j.persist'
11:41:32.280 INFO  o.o.u.w.i.s.WebAppServlet[:99]- Started Classic UI at 
/openhab.app
11:41:40.284 INFO  o.o.m.c.i.f.FolderObserver[:163]- Loaded file 'db4o.persist'
11:41:40.346 INFO  o.o.m.c.i.f.FolderObserver[:163]- Loaded file 'cosm.persist'
11:41:40.380 INFO  o.o.m.c.i.f.FolderObserver[:163]- Loaded file 'sense.persist'
11:41:50.970 INFO  o.o.m.c.i.f.FolderObserver[:163]- Loaded file 'default.rules'

Original issue reported on code.google.com by esch...@gmail.com on 27 Apr 2013 at 7:49

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago

Original comment by teichsta on 13 Aug 2013 at 3:56

GoogleCodeExporter commented 9 years ago
I'd say this one is fixed, because of 
https://code.google.com/p/openhab/source/detail?r=6ada85efd2c7#

Although I do have one remark:

    protected Map<String, BindingConfig> bindingConfigs = new ConcurrentHashMap<String, BindingConfig>(new WeakHashMap<String, BindingConfig>());

The WeakHashMap is not necessary, because ConcurentHashMap is a container, 
instead of a wrapper like Collections.synchronizedMap.

Original comment by jwsp...@gmail.com on 20 Sep 2013 at 6:20

GoogleCodeExporter commented 9 years ago

Original comment by teichsta on 5 Nov 2013 at 10:47

GoogleCodeExporter commented 9 years ago
This issue has been migrated to Github. If this issue id is greater than103 its 
id has been preserved on Github. You can open your issue by calling the URL 
https://github.com/openhab/openhab/issues/<issueid>. Issues with ids less or 
equal 103 new ids were created.

Original comment by teichsta on 17 Nov 2013 at 8:08

GoogleCodeExporter commented 9 years ago
see above!

Issue has been migrated to Github and should be discussed there.

Original comment by teichsta on 21 Nov 2013 at 1:51