Bug #464
Performance Bottleneck: expensive construction of Scope objects
| Status: | Resolved | Start date: | 08/01/2011 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | J. Moringen | % Done: | 100% | |
| Category: | C++ | |||
| Target version: | 0.4 |
Description
Scope construction consumes more than 60% time within spread::ReceiverTask::notifyHandler
(much more than actual deserialization of Protocolbuffer-content)
see attached profiler output
callgrind.out.protobuf200hz - Profiler output: open with kcachegrind <callgrind.out.protobuf200hz> (2.72 MB)
event_creation.cc 
- test program
(254 Bytes)
Makefile - for building test program (211 Bytes)
Related issues
Associated revisions
Use ad-hoc parser in Scope constructor in src/rsb/Scope.{h,cpp}
fixes #464
fixes #464
- src/rsb/Scope.{h,cpp}: removed regex-based validation and
Boost.Algorithm-based splitting; replaced with ad-hoc one-pass
iteration + validation + component extraction
a bit more optimization for parsing scopes
refs #464
History
#1 Updated by J. Moringen over 12 years ago
- Status changed from New to In Progress
- Assignee set to J. Moringen
- Target version set to 0.4
#2 Updated by J. Moringen over 12 years ago
- Status changed from In Progress to Resolved
- % Done changed from 0 to 100
Applied in changeset r2199.
#3 Updated by J. Moringen over 12 years ago
- File event_creation.cc added
- File Makefile added
- Status changed from Resolved to Feedback
The attached test program gives the following results:
before r2199:
/tmp $ time ./event_creation # old 3.211 secs
after r2199:
/tmp $ time ./event_creation 0.502 secs
#4 Updated by M. Rolf over 12 years ago
works fine.
#5 Updated by J. Moringen over 12 years ago
- Status changed from Feedback to Resolved