-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
OneOffJobBootstrap doesn't support parameters #1430
Comments
Yes, we are aware of this issue. We will support it soon. |
What is the current stable version? |
In my opinion, OneOffJob is a type of Bootstrap and SimpleJob or FlowJob are types of ElasticJob, they are different concepts. Parameters can be provided by JobConfiguration and distinguish by ShardingContext. Just create a custom implementation of JobItemExecutor and you can get JobConfiguration and ShardingContext. There is no need to change the API of OneOffJobBootstrap. |
Yes, keep the current api, and will support parameters in JobConfiguration. |
Master branch is more preferred. No unresolved bugs. We are talking about the extension of OneOffJob. |
according to OCP , Yes,keep the current api
…------------------ 原始邮件 ------------------
发件人: "Tboy"<[email protected]>;
发送时间: 2020年9月2日(星期三) 上午10:19
收件人: "apache/shardingsphere-elasticjob"<[email protected]>;
抄送: "努力努力❌努力"<[email protected]>; "Comment"<[email protected]>;
主题: Re: [apache/shardingsphere-elasticjob] OneOffJobBootstrap doesn't support parameters (#1430)
In my opinion, OneOffJob is a type of Bootstrap and SimpleJob or FlowJob are types of ElasticJob, they are different concepts. Parameters can be provided by JobConfiguration and distinguish by ShardingContext. Just create a custom implementation of JobItemExecutor and you can get JobConfiguration and ShardingContext. There is no need to change the API of OneOffJobBootstrap.
What do you think? @Technoboy- @specYyt
Yes, keep the current api, and will support parameters in JobConfiguration.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
After talking about the details, we think it's not a common scenario, and will not support recently. Thanks for your reporting. |
Hello!
I think OneOffJob is different with SimpleJob or FlowJob.
SimpleJob focus on loop but oneoffjob focus on dispatch. So oneOffJob should have parameters.
Looking forward to your reply!
The text was updated successfully, but these errors were encountered: