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

[java] Deprecate max-threads flag. Add an alternate flag in the distributor for new session thread pool size. #10995

Merged
merged 3 commits into from
Oct 17, 2022

Conversation

pujagani
Copy link
Contributor

@pujagani pujagani commented Sep 1, 2022

Thanks for contributing to Selenium!
A PR well described will help maintainers to quickly review and merge it

Before submitting your PR, please check our contributing guidelines.
Avoid large PRs, help reviewers by making them as simple and short as possible.

Description

Related to #10332.

Motivation and Context

The "max-threads" flag was intended to be used in the Distributor and this was mainly used in Grid 3. However, in Grid 4, the flag's value is not actually being used. So "max-threads" is deprecated and will be removed minimum 2 versions later. As an alternative, a new flag is added to the distributor whose value is used in the code base.
The new session creation thread pool size affects the Distributor's ability to create sessions parallel. In most cases, the default thread pool size should be sufficient. But in a large grid, it might be beneficial for the user to set its value as required. Thus, making the thread pool size configurable.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I have read the contributing document.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@pujagani pujagani changed the title [java] Make new session creation threadpool size configurable [java] Deprecate max-threads flags. Add an alternate flag in the distributor for new session thread pool size. Sep 2, 2022
@pujagani pujagani changed the title [java] Deprecate max-threads flags. Add an alternate flag in the distributor for new session thread pool size. [java] Deprecate max-threads flag. Add an alternate flag in the distributor for new session thread pool size. Sep 2, 2022
@codecov-commenter
Copy link

codecov-commenter commented Sep 2, 2022

Codecov Report

❗ No coverage uploaded for pull request base (trunk@02b23e0). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##             trunk   #10995   +/-   ##
========================================
  Coverage         ?   52.28%           
========================================
  Files            ?       81           
  Lines            ?     5499           
  Branches         ?      198           
========================================
  Hits             ?     2875           
  Misses           ?     2426           
  Partials         ?      198           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Sep 2, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Copy link
Member

@diemol diemol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @pujagani!

@diemol diemol merged commit 01bf373 into SeleniumHQ:trunk Oct 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants