Project

General

Profile

Actions
Copy link

Task #1090

closed

Add processed items for AbstractLongRunningTaskExecutor

Added by Ondřej Kopr almost 6 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Ondřej Kopr
Category:
Long running task
Target version:
Jade (8.1.0)
Start date:
05/09/2018
Due date:
% Done:

100%

Estimated time:
Owner:

Description

Log item to processed after is executed from AbstractLongRunningTaskExecutor (similar behavior as in AbstractSchedulableStatefulExecutor).

New record will be created for every processed item. (only create, remove items from this queue isn't needed)

Actions
Copy link
 #1

Updated by Ondřej Kopr almost 6 years ago

  • Status changed from New to Needs feedback
  • Assignee changed from Ondřej Kopr to Radek Tomiška
  • % Done changed from 0 to 90

I moved method logItemProcessed from SchedulableStatefulExecutorto LongRunningTaskExecutor. Old behavior works well.

For create new log from AbstractLongRunningTaskExecutor is necessary add method logItemProcessed(dto, result); to process method.

Tests are included.

Commit: https://github.com/bcvsolutions/CzechIdMng/commit/983b543ba143ca9cfee206cbf28d49160143cdee (branch okopr/1090-precessed-items-lrt) (I already merged this branch into ticket #1081)

Please Radek could you made a review of this changes? Thank you. Only potential problem that I see is with removed method. Please look at this and give me feedback with this.

Actions
Copy link
 #2

Updated by Radek Tomiška almost 6 years ago

  • Status changed from Needs feedback to Closed
  • Assignee changed from Radek Tomiška to Ondřej Kopr
  • % Done changed from 90 to 100

Thx for this feature, it works nice! I did test and review and added modal details to long running task items and queue.

Merged into develop.

Actions
Copy link

Also available in: Atom PDF