Opened 18 months ago
Last modified 16 months ago
#3018 reviewing defect
Interdependence between mutliple HiveJobManagerViews
Reported by: | fholzing | Owned by: | jkarder |
---|---|---|---|
Priority: | lowest | Milestone: | HeuristicLab 3.3.17 |
Component: | Hive.Client | Version: | trunk |
Keywords: | Cc: |
Description
Closing a HiveJobManagerView while having multiple HiveJobManagerViews instantiated empties the Jobs of all HiveJobManagerViews.
Change History (6)
comment:1 Changed 17 months ago by jkarder
- Status changed from new to accepted
comment:2 Changed 17 months ago by jkarder
- Owner changed from jkarder to mkommend
- Status changed from accepted to reviewing
comment:3 Changed 17 months ago by jkarder
- Milestone changed from HeuristicLab 3.3.x Backlog to HeuristicLab 3.3.17
comment:4 Changed 17 months ago by jkarder
r17222: fixed condition
comment:5 follow-up: ↓ 6 Changed 16 months ago by mkommend
- Owner changed from mkommend to jkarder
r17267: Refactored condition for clean up of hive client.
The usage of singletons for hive client is highly frowned upon! Without the used instance pattern most of the clean up logic and coupling issues would not be present.
comment:6 in reply to: ↑ 5 Changed 16 months ago by jkarder
Replying to mkommend:
r17267: Refactored condition for clean up of hive client.
The usage of singletons for hive client is highly frowned upon! Without the used instance pattern most of the clean up logic and coupling issues would not be present.
Thanks for improving readability. Using singletons vs. using multiple service instances is certainly something we should discuss when a new version of Hive will be implemented.
r17221: changed HiveJobManagerView.OnClosed() to only clear the HiveClient if no other view is displaying it