Ticket #2272 (s3c_implementation_ok bug) — at Version 7
Model performance
| Reported by: | mira | Owned by: | mira |
|---|---|---|---|
| Priority: | major | Milestone: | |
| Component: | BASE_RESOURCE_MODEL | Version: | 2.0 |
| Keywords: | Cc: | ||
| Category: | unknown | Effort: | |
| Importance: | Ticket_group: | ||
| Estimated Number of Hours: | 0 | Add Hours to Ticket: | 0 |
| Billable?: | yes | Total Hours: | 0 |
| Analysis_owners: | mira | Design_owners: | mira |
| Imp._owners: | mira | Test_owners: | |
| Analysis_reviewers: | meddle | Changelog: | |
| Design_reviewers: | meddle | Imp._reviewers: | meddle, deyan |
| Test_reviewers: | Analysis_score: | 3 | |
| Design_score: | 3 | Imp._score: | 4 |
| Test_score: | 0 |
Description (last modified by meddle) (diff)
The model classes (resource-accesses and changers) work too slow because they are always parsing their resource refs. '
Change History
comment:2 Changed 15 years ago by mira
- Status changed from s1b_analysis_finished to s2a_design_started
comment:3 Changed 15 years ago by mira
- Status changed from s2a_design_started to s2b_design_finished
Maybe the best thing to do here is to make the ResourceRef keep ImmList<String> instead of the whole String location. However, this would cost too much efford. This problem could be fixed with memorization.
The DelegatingResourceAccess should memorize its root key in the big model.
The SubResourceChanger should have its root key calculated when it is constructed.
The Key equals and hashCode methods should use directly the getParts() instead of the getId() method.
comment:5 Changed 15 years ago by mira
- Owner set to mira
- Status changed from s2c_design_ok to s3a_implementation_started
comment:6 Changed 15 years ago by mira
- Status changed from s3a_implementation_started to s3b_implementation_finished
There is a patch file attached in this ticket.
comment:7 Changed 15 years ago by meddle
- Status changed from s3b_implementation_finished to s3c_implementation_ok
- Analysis_reviewers set to meddle
- Imp._reviewers set to meddle, deyan
- Design_reviewers set to meddle
- Imp._score changed from 0 to 4
- Description modified (diff)
Merged into the trunk at [8532].
4p
Note: See
TracTickets for help on using
tickets.
