Fix typo#1312
Closed
kw2542 wants to merge 18 commits into
Closed
Conversation
Design: https://cwiki.apache.org/confluence/display/SAMZA/SEP-23%3A+Simplify+Job+Runner Changes: 1. Simplify CommandLine by removing unnecessary arguments. API Changes: 1. config-loader-factory removed in CommandLine 2. config-loader-properties removed in CommandLine Upgrade Instruction: 1. Usage of --config-factory and --config-loader-factory will be switched to --config job.config.loader.factory 2. Usage of --config-path and --config-loader-properties will be switched to --config job.config.loader.properties Usage Instruction --config-factory=org.apache.samza.config.factories.PropertiesConfigFactory --config-path=file:///location/file.txt and --config-loader-factory=org.apache.samza.config.loaders.PropertiesConfigLoaderFactory --config-loader-properties path=file:///location/file.txt will be updated to --config job.config.loader.factory=org.apache.samza.config.loaders.PropertiesConfigLoaderFactory --config job.config.loader.properties.path=file:///location/file.txt Tests 1. Unit tests 2. Test against Hello Samza example with deploy/samza/bin/run-app.sh --config job.config.loader.factory=org.apache.samza.config.loaders.PropertiesConfigLoaderFactory --config job.loader.properties.path=$PWD/deploy/samza/config/wikipedia-feed.properties
Design: https://cwiki.apache.org/confluence/display/SAMZA/SEP-23%3A+Simplify+Job+Runner Changes: 1. Add app.main.class to ApplicationConfig 2. Add app.main.args to ApplicationConfig API Changes: N/A. This is part of a series PRs, detailed information will be provided in the last/main PR. Upgrade Instructions: N/A. This is part of a series PRs, detailed information will be provided in the last/main PR. Usage Instructions: N/A. This is part of a series PRs, detailed information will be provided in the last/main PR. Tests: N/A.
Design: https://cwiki.apache.org/confluence/display/SAMZA/SEP-23%3A+Simplify+Job+Runner Changes: 1. Invoke app.main.class with app.main.args when present instead of invoking ClusterBasedJobCoordinator#run API Changes: N/A Upgrade Instructions: For beam jobs, beam's main class and pipeline options need to passed with app.main.class and app.main.args configs. Usage Instructions: Take https://github.com/apache/samza-beam-examples as an example: 1. run-beam-yarn.sh#line 36 will be updated to exec $(dirname $0)/run-class.sh $1 --config app.main.class=$1 --config app.main.args="--runner=org.apache.beam.runners.samza.SamzaRunner --configOverride=\"$override\" \"${@:2}\"" Tests: N/A.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
getAppMainArgs & getAppMainClass are reversed.