Opened 2 years ago

Closed 23 months 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 2 years ago by ascheibe

  • Status changed from new to accepted

comment:2 Changed 2 years ago by ascheibe

r12579 removed exception handling code from Hive client

comment:3 Changed 2 years ago by ascheibe

  • Owner changed from ascheibe to mkommend
  • Status changed from accepted to reviewing

comment:4 follow-up: Changed 23 months 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 23 months ago by ascheibe

r12665 merged r12579 into stable

comment:6 in reply to: ↑ 4 Changed 23 months 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.