Opened 9 years ago
Closed 9 years ago
#2408 closed defect (done)
Fix exception handling in Hive client
| Reported by: | ascheibe | Owned by: | ascheibe |
|---|---|---|---|
| Priority: | medium | Milestone: | HeuristicLab 3.3.12 |
| Component: | Hive.Client | Version: | 3.3.11 |
| Keywords: | Cc: |
Description
The Hive client globally catches all unobserved exceptions from tasks and sets them observed. This should be removed as it may hides problems and transforms "normal" exceptions to Hive exceptions.
Change History (6)
comment:1 Changed 9 years ago by ascheibe
- Status changed from new to accepted
comment:2 Changed 9 years ago by ascheibe
comment:3 Changed 9 years ago by ascheibe
- Owner changed from ascheibe to mkommend
- Status changed from accepted to reviewing
comment:4 follow-up: ↓ 6 Changed 9 years ago by mkommend
- Owner changed from mkommend to ascheibe
- Status changed from reviewing to readytorelease
r12579 looks good, though I don't know what side effects in the Hive this changeset has.
comment:5 Changed 9 years ago by ascheibe
comment:6 in reply to: ↑ 4 Changed 9 years ago by ascheibe
- Resolution set to done
- Status changed from readytorelease to closed
Replying to mkommend:
r12579 looks good, though I don't know what side effects in the Hive this changeset has.
It is only used in the Hive Job Manager, so no other parts of Hive are effected. I think that the idea was to prevent HL from crashing when downloading jobs where the plugins required for loading the job were missing. But I tested it and this is not the case any more, so I think there should be no problems.
Note: See
TracTickets for help on using
tickets.
r12579 removed exception handling code from Hive client