Skip to content

Added executeBlocking() on refresh calls and adjusted instance count/pool size #235

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
gmsdelmundo merged 62 commits into from
Feb 19, 2025
Merged

Added executeBlocking() on refresh calls and adjusted instance count/pool size #235

gmsdelmundo merged 62 commits into from
Feb 19, 2025

Conversation

@gmsdelmundo
Copy link
Contributor

@gmsdelmundo gmsdelmundo commented Feb 17, 2025 •
edited
Loading

Changes

  • Set Vertx instance count to 3 and worker pool size to 1000
  • Added parallelized execution for blocking /refresh calls

Note for reviewers

  • Main.java and config file changes are responsible for allowing configuration of Vertx instance count and worker pool size
  • Test file changes are renaming to fit our usual test name format and some other folder regrouping fixes
  • Other files are all related to executeBlocking() fix and refactoring

Evidence
Load test for 3 instances and 1000 worker threads:

gmsdelmundo and others added 30 commits February 7, 2025 17:44
@gmsdelmundo
Increased Core vertx instances from 1 to 8
9ae60df
@github-actions
[CI Pipeline] Released Snapshot version: 2.24.11-alpha-83-SNAPSHOT
d7db2b0
@gmsdelmundo
Reduced instance count to 4
cee7486
@gmsdelmundo
Merge branch 'gdm-UID2-4660-core-load-testing' of github.com:IABTechL…
aa7f6bb 
…ab/uid2-core into gdm-UID2-4660-core-load-testing
@github-actions
[CI Pipeline] Released Snapshot version: 2.24.12-alpha-84-SNAPSHOT
40acf2a
@gmsdelmundo
Reduced instance count to 2
0f14706
@github-actions
[CI Pipeline] Released Snapshot version: 2.24.13-alpha-85-SNAPSHOT
643c30c
@gmsdelmundo
Increased instance count to 6
476a835
@gmsdelmundo
Merge branch 'gdm-UID2-4660-core-load-testing' of github.com:IABTechL…
57c0040 
…ab/uid2-core into gdm-UID2-4660-core-load-testing
@github-actions
[CI Pipeline] Released Snapshot version: 2.24.14-alpha-87-SNAPSHOT
25c1113
@gmsdelmundo
Merged main
6060848
@github-actions
[CI Pipeline] Released Snapshot version: 2.25.15-alpha-88-SNAPSHOT
855e7c6
@gmsdelmundo
Updated .trivyignore for testing
f8a22c5
@gmsdelmundo
Merge branch 'gdm-UID2-4660-core-load-testing' of github.com:IABTechL…
d45f978 
…ab/uid2-core into gdm-UID2-4660-core-load-testing
@github-actions
[CI Pipeline] Released Snapshot version: 2.25.16-alpha-89-SNAPSHOT
fec44f1
@gmsdelmundo
Updated Vertx to 4.5.13 to fix CVE-2025-24970
779bff7
@github-actions
[CI Pipeline] Released Snapshot version: 2.25.17-alpha-90-SNAPSHOT
61be2d6
@gmsdelmundo
Refactored metadata providers
e545f1c
@gmsdelmundo
Added executeBlocking to handle refresh functions
ce46e30
@gmsdelmundo
Merge branch 'gdm-UID2-4660-core-load-testing' of github.com:IABTechL…
18dff90 
…ab/uid2-core into gdm-UID2-4660-core-load-testing
@gmsdelmundo
Reduced core verticle instance count to 3
58af135
@gmsdelmundo
Fixed missing parameter
cfb86fa
@gmsdelmundo
Fixed incorrect function call
39f5795
@gmsdelmundo
Fixed routing context handling with 403 error
be4a087
@github-actions
[CI Pipeline] Released Snapshot version: 2.25.18-alpha-91-SNAPSHOT
78a66bc
@gmsdelmundo
Added ignored vulnerabilities
28cc177
@github-actions
[CI Pipeline] Released Snapshot version: 2.25.19-alpha-92-SNAPSHOT
5b95747
@gmsdelmundo
Increased Vertx instance count to 5
3f38d24
@gmsdelmundo
Merge branch 'gdm-UID2-4660-core-load-testing' of github.com:IABTechL…
6513934 
…ab/uid2-core into gdm-UID2-4660-core-load-testing
@github-actions
[CI Pipeline] Released Snapshot version: 2.25.20-alpha-93-SNAPSHOT
a2a908e
@gmsdelmundo gmsdelmundo force-pushed the gdm-UID2-4660-core-load-testing branch from 0a81353 to af5725c Compare February 17, 2025 06:47
@gmsdelmundo gmsdelmundo force-pushed the gdm-UID2-4660-core-load-testing branch from af5725c to 64f998f Compare February 17, 2025 06:47
vishalegbert-ttd
vishalegbert-ttd reviewed Feb 17, 2025
View reviewed changes
aulme
aulme reviewed Feb 18, 2025
View reviewed changes
aulme
aulme reviewed Feb 18, 2025
View reviewed changes
@gmsdelmundo gmsdelmundo merged commit 1d32099 into main Feb 19, 2025
4 checks passed
@gmsdelmundo gmsdelmundo deleted the gdm-UID2-4660-core-load-testing branch February 19, 2025 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vishalegbert-ttd vishalegbert-ttd vishalegbert-ttd left review comments

+1 more reviewer

@aulme aulme aulme approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@gmsdelmundo @aulme @vishalegbert-ttd