Skip to content
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.

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

consolidate DotNetty to use single threadpool #4676

Conversation

Aaronontheweb
Copy link
Member

part of #4636

@Aaronontheweb
Copy link
Member Author

Before

ProcessorCount:                    16
ClockSpeed:                        0 MHZ
Actor Count:                       32
Messages sent/received per client: 200000  (2e5)
Is Server GC:                      True

Num clients, Total [msg], Msgs/sec, Total [ms]
         1,  200000,      1093,  183115.33
         5, 1000000,    193536,    5167.89
        10, 2000000,    193611,   10330.10
        15, 3000000,    185564,   16167.11
        20, 4000000,    184690,   21658.68
        25, 5000000,    185069,   27017.80
        30, 6000000,    186661,   32144.23

After

ProcessorCount:                    16           
ClockSpeed:                        0 MHZ        
Actor Count:                       32           
Messages sent/received per client: 200000  (2e5)
Is Server GC:                      True         
                                                
Num clients, Total [msg], Msgs/sec, Total [ms]  
         1,  200000,      1466,  136443.40      
         5, 1000000,    183319,    5455.14      
        10, 2000000,    184485,   10841.54      
        15, 3000000,    173371,   17304.35      
        20, 4000000,    177581,   22525.02      
        25, 5000000,    176635,   28307.63      
        30, 6000000,    177138,   33872.07      

Looks like a marginal loss in performance to me, but I'll need to compare this to the idle CPU numbers I've been measuring.

@Aaronontheweb
Copy link
Member Author

This change does not appear to have helped on the idle CPU or performance front. Going to close it.

@Aaronontheweb Aaronontheweb deleted the dotnetty/consolidate-threadpool branch December 17, 2020 23:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant