Experimental Physics and
Industrial Control System

Shuei YAMADA <[email protected]> · Thu, 24 Aug 2017 10:30:30 +0900

Dear Kay,

> Kunal added a lock to ApplianceValueIterator, see https://github.com/ControlSystemStudio/cs-studio/commit/d902628f7272ca9b3d392a4d8636533799d5738d
> Check that your sources include this lock.
> If so, maybe you need to add/modify the locking in the ApplianceOptimizedValueIterator similar to what Kunal did in the ApplianceValueIterator.

I checked thru my source code and confirmed that the lock Kunal added
is already included. It seems that I don't have to modify
ApplianceOptimizedValueIterator because
ApplianceOptimizedValueIterator::fetchDataInternal() calls
ApplianceValueIterator::fetchDataInternal() to fetch data.

> You added synch. in the ArchiveFetchJob, but that is the generic part which interfaces to all types of archive readers, and those readers are expected to support multithreading.

This is exactly the reason why I "said quick and dirty workaround".
And it is better to retrieve all requested data than disabling
multi-threading, until the problem really get fixed.

best regards,
Shuei
-- 
                 ----------------------------------------------------
                 Shuei Yamada,
                 High Energy Accelerator Research Organization (KEK)
                 mailto:[email protected]
                 ----------------------------------------------------

Experimental Physics and Industrial Control System

Experimental Physics and
Industrial Control System