You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We are using Deployment Options to create Worker Model specific options.
We are using Vertx Runner to bootstrap the MainVerticle which uses Vertx Run Command to do so.
The issue is the following : the RunCommand uses setWorker(boolean) [see below] to try to setup the worker model, but in the deployment options method the check seems to be incorrect as it is comparing the boolean (worker) parameter to thread model enum value which will always be false and hence result in Event Model being chosen.
I can see this has been removed from the Master.
I am currently using 4.5.3 version but the same issue is also in 4.5.1.
Use Case:
We are using Deployment Options to create Worker Model specific options.
We are using Vertx Runner to bootstrap the MainVerticle which uses Vertx Run Command to do so.
The issue is the following : the RunCommand uses setWorker(boolean) [see below] to try to setup the worker model, but in the deployment options method the check seems to be incorrect as it is comparing the boolean (worker) parameter to thread model enum value which will always be false and hence result in Event Model being chosen.
I can see this has been removed from the Master.
I am currently using 4.5.3 version but the same issue is also in 4.5.1.
Please can you have a look.
Thank you
Code reference (branch 4.5.3) :
https://github.com/eclipse-vertx/vert.x/blob/ce654726ce4c7642143258c78b3f8992be99e3c7/src/main/java/io/vertx/core/DeploymentOptions.java#L165C10-L165C25
https://github.com/eclipse-vertx/vert.x/blob/ce654726ce4c7642143258c78b3f8992be99e3c7/src/main/java/io/vertx/core/impl/launcher/commands/RunCommand.java#L258C51-L258C57
The text was updated successfully, but these errors were encountered: