Project

General

Profile

Actions
Copy link

Task #628

closed

Improve LOG's agenda

Added by Ondřej Kopr over 7 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Ondřej Kopr
Category:
Logging
Target version:
Diamond (7.4.0)
Start date:
08/04/2017
Due date:
% Done:

100%

Estimated time:
Owner:

Description

  • Add index to column formatted message - (create new column with index and copy current date into newly created),
  • create new LRT, this LRT will remove old logs (days param),
  • create new filed that will be constains raw errors (now is error only in table).

Related issues

Related to IdStory Identity Manager - Task #574: New agenda for LOGsClosedOndřej Kopr07/13/2017

Actions
Actions
Copy link
 #1

Updated by Ondřej Kopr over 7 years ago

  • Related to Task #574: New agenda for LOGs added
Actions
Copy link
 #2

Updated by Ondřej Kopr over 7 years ago

  • % Done changed from 0 to 60
  • added new button to exception log table with show copyable content (modal + script area) - this raw source contains all rows,
  • added new LRT (RemoveOldLogsTaskExecutor) (change sevices IdmLoggingEventService and IdmLoggingEventExceptionService to Read Write),
Actions
Copy link
 #3

Updated by Ondřej Kopr over 7 years ago

  • Status changed from New to In Progress
Actions
Copy link
 #4

Updated by Ondřej Kopr over 7 years ago

  • Status changed from In Progress to Needs feedback
  • Assignee changed from Ondřej Kopr to Radek Tomiška
  • % Done changed from 60 to 90

add index into formatted_message table logging_event.

commits:
https://github.com/bcvsolutions/CzechIdMng/commit/04f7e1af13d668f16a24706a8283d3fec4c328b5
https://github.com/bcvsolutions/CzechIdMng/commit/09aebd194e2b9bbc2d6f4747f291bd13837bef6f

Radek could you make a review? Attention in this ticket is new flyway script. Create index on formatted_message can take up about 20 second ~200.000 rows.

Actions
Copy link
 #5

Updated by Ondřej Kopr over 7 years ago

  • Assignee changed from Radek Tomiška to Ondřej Kopr

+ implemented pageable to LTR.

Actions
Copy link
 #6

Updated by Ondřej Kopr over 7 years ago

  • Assignee changed from Ondřej Kopr to Radek Tomiška
Actions
Copy link
 #7

Updated by Radek Tomiška over 7 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

I did test and review:
- I've removed change script with index - index was to large and exceeds default postgresql index size limit (8kb). We will find different solution in future, if will be needed.
- code looks nice, i moved LRT into audit package.

I merged it into develop. Thx!

Actions
Copy link

Also available in: Atom PDF