Task #657
closed
Synchronization transform to DTOs
100%
Description
- SyncActionLog,
- SyncConfig,
- SyncItemLog,
- SyncLog.
to DTOs. Check tests and FE.
Related issues
Updated by Ondřej Kopr over 6 years ago
- % Done changed from 0 to 60
Refactor synchronization to DTO was completed. All tests are passed. Now i will start with FE testing.
Updated by Ondřej Kopr over 6 years ago
- refactor itemlogs to dto: https://github.com/bcvsolutions/CzechIdMng/commit/e2ababc24519231aa67ea820161523a950dd9488
- refactor other entities, service, rest and etc: https://github.com/bcvsolutions/CzechIdMng/commit/1a31a093520b7261477155aee5c40fb483bea9db
- reafctor tests: https://github.com/bcvsolutions/CzechIdMng/commit/0d6c9d8b2673eec1115fc9d370b00c1e8bcf243d
- resolve problem with duplicate item logs: https://github.com/bcvsolutions/CzechIdMng/commit/8b4f0a0b108479fbf27b566d9dbc1bfffd35c525
Updated by Ondřej Kopr over 6 years ago
- % Done changed from 60 to 80
Finished with refactor synchronization to DTO and tests.
commits:- fix redundant save https://github.com/bcvsolutions/CzechIdMng/commit/ea0fe619547aa24001e742e562e11ac8e8e67327
- remove excerpt https://github.com/bcvsolutions/CzechIdMng/commit/057a0345e317b8d238b76149666a987b2c925922
- fix problem with FE https://github.com/bcvsolutions/CzechIdMng/commit/4dad93d7493dfbe21ab72993b85d62a8615bfb56
from FE check different behavior. Now i test synchronization with lots of data.
Updated by Ondřej Kopr over 6 years ago
On FE agenda was small bug with mapping.
commit: https://github.com/bcvsolutions/CzechIdMng/commit/717e68d7561077420374f3aabcb00be44df3e48d
Updated by Ondřej Kopr over 6 years ago
- % Done changed from 80 to 90
I did tests with big data:
identities synchronization, username (transform script with FOR cycle), first name, last name, description and email.
Synchronization start about 16:30 after night it was synchronized ~164.000 identities - 9:00 (16,5 hours). Memory usage: 6.7GB (when synchronization start it was about 6GB).
Synchronization still continuing.
Before start synchronization i was explored bug (after transform mapping to dtos), system copy not working.
Updated by Ondřej Kopr over 6 years ago
another test result:
first 100 of identities - 0:21 min
last 100 of identities - 2:40 min
slowdown is about 120.000 records, 100.000 records lasting 5h
Updated by Ondřej Kopr over 6 years ago
- Status changed from In Progress to Needs feedback
- Assignee changed from Ondřej Kopr to Vít Švanda
fix issue with transform script checkbox and duplicate copy
commit: https://github.com/bcvsolutions/CzechIdMng/commit/791dc41903cc02249cfdca04eb4695208ec2ccfe
also add test for duplicate system check synchronization
commit: https://github.com/bcvsolutions/CzechIdMng/commit/0763dee8fc27f45d4b80a67e44ed3ecd0ca9e702
please Vitek could you make a review? Thank you.
Updated by Ondřej Kopr over 6 years ago
- Related to Task #663: Schema transform to DTO added
Updated by Ondřej Kopr over 6 years ago
- Related to Task #663: Schema transform to DTO added
Updated by Ondřej Kopr over 6 years ago
- Related to Task #675: Role system transform to DTO added
Updated by Ondřej Kopr over 6 years ago
- Related to Task #676: AccAccount transform to DTO added
Updated by Ondřej Kopr over 6 years ago
- Related to Task #685: Transfrom SysSystem to DTO added
Updated by Ondřej Kopr over 6 years ago
- Related to Task #694: Provisioning transform to DTO added
Updated by Vít Švanda over 6 years ago
- Status changed from Needs feedback to Resolved
- Assignee changed from Vít Švanda to Ondřej Kopr
- % Done changed from 90 to 100
I did review, everythings works fine. I did few minor improvements (in commit https://github.com/bcvsolutions/CzechIdMng/commit/579a0985f1a1f667f6b332659e97b1399e5e7973).
Thanks for that.
Updated by Radek Tomiška over 6 years ago
- Status changed from Resolved to Closed