-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Added ability to specify runtime for localrunner #7681
Conversation
@@ -618,6 +618,8 @@ protected void validateFunctionConfigs(FunctionConfig functionConfig) { | |||
protected Integer DEPRECATED_instanceIdOffset = null; | |||
@Parameter(names = "--instance-id-offset", description = "Start the instanceIds from this offset") | |||
protected Integer instanceIdOffset = 0; | |||
@Parameter(names = "--runtime", description = "either THREAD or PROCESS. Only applies for Java functions") | |||
protected String runtime = "THREAD"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think it makes sense to default to thread runtime especially when users can run functions in python which must be in process runtime. The default should just be null. In LocalRunner, if Java is used the default is thread runtime anyways.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed
* Added ability to specify runtime for localrunner * Address feedback Co-authored-by: Sanjeev Kulkarni <sanjeevk@splunk.com>
* Added ability to specify runtime for localrunner * Address feedback Co-authored-by: Sanjeev Kulkarni <sanjeevk@splunk.com>
* Added ability to specify runtime for localrunner * Address feedback Co-authored-by: Sanjeev Kulkarni <sanjeevk@splunk.com>
(If this PR fixes a github issue, please add
Fixes #<xyz>
.)Fixes #
(or if this PR is one task of a github issue, please add
Master Issue: #<xyz>
to link to the master issue.)Master Issue: #
Motivation
Currently there is no way to control whether to use Process or Thread runtime for localrun in functions. This pr adds that support
Modifications
Describe the modifications you've done.
Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)
Does this pull request potentially affect one of the following parts:
If
yes
was chosen, please highlight the changesDocumentation