RED-9123: Improve performance of re-analysis (Spike) #450

Merged

maverick.studer merged 0 commits from RED-9123 into master 2024-10-08 09:24:20 +02:00

Conversation 16 Commits - Files Changed -

maverick.studer commented 2024-06-27 15:53:07 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

• async loading of all files
• improved performance of contains/containedBy

Closes RED-9123

* async loading of all files * improved performance of contains/containedBy Closes RED-9123

maverick.studer commented 2024-06-27 15:53:07 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

assigned to @maverick.studer

assigned to @maverick.studer

dominique.eiflaender1 (Migrated from gitlab.knecon.com) approved these changes 2024-06-27 15:53:07 +02:00

maverick.studer commented 2024-06-27 15:57:50 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

added 18 commits

• ee0cfb4f...c869aab1 - 16 commits from branch master
• bcddfc0c - Merge branch 'master' into RED-9123
• 6b801922 - RED-9123: Improve performance of re-analysis (Spike)

Compare with previous version

added 18 commits <ul><li>ee0cfb4f...c869aab1 - 16 commits from branch <code>master</code></li><li>bcddfc0c - Merge branch &#39;master&#39; into RED-9123</li><li>6b801922 - RED-9123: Improve performance of re-analysis (Spike)</li></ul> [Compare with previous version](/redactmanager/redaction-service/-/merge_requests/450/diffs?diff_id=46951&start_sha=ee0cfb4fc2342553cf91280c562f68470fee473f)

kilian.schuettler1 commented 2024-06-27 16:51:40 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

Why not for reanalyze as well?

Also I feel like the fetching of data code is now large enough to warrant its own service class.

Why not for reanalyze as well? Also I feel like the fetching of data code is now large enough to warrant its own service class.

maverick.studer commented 2024-07-04 12:48:49 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

restored source branch RED-9123

restored source branch `RED-9123`

maverick.studer commented 2024-07-17 12:14:25 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

added 6 commits

• 03571697 - RED-9123: Improve performance of re-analysis (Spike)
• d40ffcdb - RED-9123: Improve performance of re-analysis (Spike)
• a4165608 - RED-9123: Improve performance of re-analysis (Spike)
• e6a79237 - RED-9123: Improve performance of re-analysis (Spike)
• 90b01144 - RED-9123: Improve performance of re-analysis (Spike)
• e4648646 - RED-9123: Improve performance of re-analysis (Spike)

Compare with previous version

added 6 commits <ul><li>03571697 - RED-9123: Improve performance of re-analysis (Spike)</li><li>d40ffcdb - RED-9123: Improve performance of re-analysis (Spike)</li><li>a4165608 - RED-9123: Improve performance of re-analysis (Spike)</li><li>e6a79237 - RED-9123: Improve performance of re-analysis (Spike)</li><li>90b01144 - RED-9123: Improve performance of re-analysis (Spike)</li><li>e4648646 - RED-9123: Improve performance of re-analysis (Spike)</li></ul> [Compare with previous version](/redactmanager/redaction-service/-/merge_requests/450/diffs?diff_id=48540&start_sha=9abfb0d7f5f61017efd2a1cac8de17b898d168e7)

maverick.studer commented 2024-07-18 12:55:22 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

mentioned in commit 3e75f3e5e7

mentioned in commit 3e75f3e5e785978665f465426131ece4012f8cfc

maverick.studer commented 2024-09-10 14:53:52 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

added 80 commits

• e4648646...3c165070 - 79 commits from branch master
• bd7ada84 - RED-9123: post rebase fixup

Compare with previous version

added 80 commits <ul><li>e4648646...3c165070 - 79 commits from branch <code>master</code></li><li>bd7ada84 - RED-9123: post rebase fixup</li></ul> [Compare with previous version](/redactmanager/redaction-service/-/merge_requests/450/diffs?diff_id=51693&start_sha=e464864642a24f483610ffbbdc38b2081987c371)

maverick.studer commented 2024-09-10 16:39:00 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

added 1 commit

• 95f835e8 - RED-9123: added fallback logic when no protobuf files are present

Compare with previous version

added 1 commit <ul><li>95f835e8 - RED-9123: added fallback logic when no protobuf files are present</li></ul> [Compare with previous version](/redactmanager/redaction-service/-/merge_requests/450/diffs?diff_id=51744&start_sha=bd7ada8444ac48f6adfe074ccdb7ca5f7c458a8e)

maverick.studer commented 2024-10-07 14:18:27 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

added 23 commits

• 95f835e8...725e6c1e - 18 commits from branch master
• 6493ad82 - RED-9123: list instead of dict for positions in position data
• 1a87fc91 - Merge branch 'master' into RED-9123
• 6cf5602f - Merge branch 'master' into RED-9123
• ffe99b25 - RED-9123: post merge fixup, changed tests after layout-parser-version update
• 6c15063c - RED-9123: update dependencies

Compare with previous version

added 23 commits <ul><li>95f835e8...725e6c1e - 18 commits from branch <code>master</code></li><li>6493ad82 - RED-9123: list instead of dict for positions in position data</li><li>1a87fc91 - Merge branch &#39;master&#39; into RED-9123</li><li>6cf5602f - Merge branch &#39;master&#39; into RED-9123</li><li>ffe99b25 - RED-9123: post merge fixup, changed tests after layout-parser-version update</li><li>6c15063c - RED-9123: update dependencies</li></ul> [Compare with previous version](/redactmanager/redaction-service/-/merge_requests/450/diffs?diff_id=52911&start_sha=95f835e8563cd33b69a16859a77748bbc445886c)

maverick.studer commented 2024-10-07 14:19:18 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

resolved all threads

resolved all threads

maverick.studer commented 2024-10-07 14:21:48 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

added 5 commits

• 6c15063c...daed4e07 - 4 commits from branch master
• 0c8d1771 - Merge branch 'master' into RED-9123

Compare with previous version

added 5 commits <ul><li>6c15063c...daed4e07 - 4 commits from branch <code>master</code></li><li>0c8d1771 - Merge branch &#39;master&#39; into RED-9123</li></ul> [Compare with previous version](/redactmanager/redaction-service/-/merge_requests/450/diffs?diff_id=52912&start_sha=6c15063c324eacd57a2479d52dcb8b61a8115c08)

maverick.studer (Migrated from gitlab.knecon.com) scheduled this pull request to auto merge when all checks succeed 2024-10-07 14:22:10 +02:00

maverick.studer commented 2024-10-07 14:35:41 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

added 1 commit

• cd28d0f5 - RED-9123: post-merge fixup

Compare with previous version

added 1 commit <ul><li>cd28d0f5 - RED-9123: post-merge fixup</li></ul> [Compare with previous version](/redactmanager/redaction-service/-/merge_requests/450/diffs?diff_id=52916&start_sha=0c8d17717f1d3390aab7e9473cd839a9bf3c8804)

christoph.schabert commented 2024-10-07 14:48:22 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

SonarQube Code Analysis

Quality Gate failed

Failed conditions
56 New issues
1 Security Hotspot
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

## SonarQube Code Analysis ## Quality Gate failed Failed conditions [](https://sonarqube.knecon.com/project/issues?id=redaction-service%3Aredaction-service-server-v1&pullRequest=450&resolved=false) [56 New issues](https://sonarqube.knecon.com/project/issues?id=redaction-service%3Aredaction-service-server-v1&pullRequest=450&resolved=false) [](https://sonarqube.knecon.com/security_hotspots?id=redaction-service%3Aredaction-service-server-v1&pullRequest=450) [1 Security Hotspot](https://sonarqube.knecon.com/security_hotspots?id=redaction-service%3Aredaction-service-server-v1&pullRequest=450) [](https://sonarqube.knecon.com/component_measures?id=redaction-service%3Aredaction-service-server-v1&pullRequest=450&metric=new_coverage&view=list) [0.0% Coverage on New Code](https://sonarqube.knecon.com/component_measures?id=redaction-service%3Aredaction-service-server-v1&pullRequest=450&metric=new_coverage&view=list) (required ≥ 80%) [See analysis details on SonarQube](https://sonarqube.knecon.com/dashboard?id=redaction-service%3Aredaction-service-server-v1&pullRequest=450) ##  Catch issues before they fail your Quality Gate with our IDE extension [](https://www.sonarsource.com/products/sonarlint/features/connected-mode/?referrer=pull-request) [SonarLint](https://www.sonarsource.com/products/sonarlint/features/connected-mode/?referrer=pull-request)

maverick.studer (Migrated from gitlab.knecon.com) scheduled this pull request to auto merge when all checks succeed 2024-10-07 15:21:32 +02:00

dominique.eiflaender1 commented 2024-10-08 09:24:18 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

approved this merge request

approved this merge request

maverick.studer (Migrated from gitlab.knecon.com) merged commit 0aea884da2 into master 2024-10-08 09:24:20 +02:00

maverick.studer commented 2024-10-08 09:24:21 +02:00 (Migrated from gitlab.knecon.com)

Copy Link

Copy Source

mentioned in commit 0aea884da2

mentioned in commit 0aea884da221b51445d0994d3c4006d311a6d2a8

Sign in to join this conversation.

Reviewers

No Reviewers

dominique.eiflaender1

Labels

Clear labels

renovate

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

Marmelator (Kilian Schüttler)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Marmelator/redaction-service#450