Project

General

Profile

Feature #11782

upgrade PatternEngine.main() to enable running in multi-threaded mode

Added by Greg Shah 29 days ago. Updated 2 days ago.

Status:
Review
Priority:
Normal
Assignee:
Octavian Adrian Gavril
Target version:
-
Start date:
Due date:
% Done:

100%

billable:
No
vendor_id:
GCD
case_num:
version_reported:
version_resolved:
reviewer:
Greg Shah
production:
No
env_name:
topics:

History

#1 Updated by Octavian Adrian Gavril 13 days ago

  • Status changed from New to WIP
  • Assignee set to Octavian Adrian Gavril

I'm looking into this one.

#2 Updated by Octavian Adrian Gavril 12 days ago

  • % Done changed from 0 to 100
  • Status changed from WIP to Review

Created task branch 11782a.

I've committed the changes that enable multi-threaded mode for PatternEngine.main(), SearchTrees.main() and ReportApi.runSearch(). These can be found in 11782/16749.

I've tested both PatternEngine.main and SearchTrees.main by running standalone commands in hotel_gui. I tested ReportApi.runSearch by running searches in FWD Analytics. The improvements are noticeable.

Greg, please review.

#3 Updated by Octavian Adrian Gavril 12 days ago

  • reviewer Greg Shah added

#4 Updated by Octavian Adrian Gavril 6 days ago

Greg, would it be OK if I included the fix mentioned in #7143-1994 in 11782a?

#5 Updated by Greg Shah 6 days ago

Yes

#6 Updated by Octavian Adrian Gavril 6 days ago

Committed the fix documented in #7143-1994 in 11782a/16750.

#7 Updated by Octavian Adrian Gavril 2 days ago

I've committed the changes discussed in #11818-66 in 11782a/16751. Now we emit the warnings for missing multiThreadedAccess flag only at the most verbose debug level.

Greg, please review the changes.

Also available in: Atom PDF