#211 closed defect (done)
Full operator-graphs are stored multiple times in the database even though most of it is static anyway
Reported by: | gkronber | Owned by: | gkronber |
---|---|---|---|
Priority: | medium | Milestone: | HeuristicLab 3.3.0 |
Component: | CEDMA.Core | Version: | 3.1 |
Keywords: | Cc: |
Description
Change History (17)
comment:1 Changed 16 years ago by gkronber
- Status changed from new to assigned
comment:2 Changed 16 years ago by gkronber
- Summary changed from Full operator-graphs are stored multiple times in the database even most of it is static anyway to Full operator-graphs are stored multiple times in the database even though most of it is static anyway
comment:3 Changed 16 years ago by gkronber
comment:4 Changed 16 years ago by gkronber
created a new type of operator library that is connected to the CEDMA db and a view for it with r417
comment:5 Changed 16 years ago by gkronber
added class OperatorLink that allows to create CombinedOperators in the DatabaseOperatorLibrary which use other operators from the library (not via a copy but via a link) with r418
comment:6 Changed 16 years ago by gkronber
- Resolution set to fixed
- Status changed from assigned to closed
worked on patching the operator-graph of an agent before execution with r419
comment:7 Changed 16 years ago by gkronber
- Resolution fixed deleted
- Status changed from closed to reopened
ticket was closed unintentionally in a hurry
comment:8 Changed 16 years ago by gkronber
- Implemented downloading of linked operators from the database in AgentList (to allow view/edit of the whole operator graph)
- Implemented downloading and patching of linked operators in RunScheduler to prepare the operator-graph for execution (OperatorLink operator can't be executed)
with r420
(there is some code duplication because the patching is very similar in AgentList DatabaseOperatorLibrary and RunScheduler. This needs some more work)
comment:9 Changed 16 years ago by gkronber
- Status changed from reopened to new
comment:10 Changed 16 years ago by gkronber
- Status changed from new to assigned
comment:11 Changed 16 years ago by gkronber
worked on this issue with r491:493
Patching of all operator links works correctly in test runs. However now all operator-links are patched on the first sweep. This means that sub-agents are already completely expanded (again fully expanded operator-graphs are stored multiple times in the DB unnecessarily). This issue needs to be fixed next.
comment:12 Changed 16 years ago by gkronber
fixed the bug in the run scheduler that caused full expansion of all operator links with r497.
However this solution is just barely working for the test example. We probably need a better solution for this.
comment:13 Changed 16 years ago by gkronber
There is still a bug where an operator-link is not patched before a view is created. Either make the operator-link patch itself or make sure all links are patched before the root operator is displayed.
comment:14 Changed 16 years ago by gkronber
- Resolution set to fixed
- Status changed from assigned to closed
r513: Fixed the problem that sometimes an operator link was not patched before a view was created (which resulted in a NullException) by adding a property for the source database in OperatorLink! and updating this property recursively whenever an operator is loaded from the CEDMA DB.
The method for recursively setting the property is in static class OperatorLinkPatcher because the code is needed in Agent, DatabaseOperatorLibrary and OperatorLink.
Closing this ticket because with the help of OperatorLinks it's easy to define the static parts of operator-graphs separately in the server side operator library and link them into the algorithm.
comment:15 Changed 16 years ago by swagner
- Milestone changed from 3.1 to Iteration 0
Milestone 3.1 deleted
comment:16 Changed 14 years ago by swagner
- Milestone changed from Iteration 0 to Current
Milestone Iteration 0 deleted
comment:11 Changed 14 years ago by swagner
- Milestone changed from Current to HeuristicLab 3.3.0
Milestone Current deleted
added a table for operators in the CEDMA database with r416