Skip to content

SAMZA-2377: Batch mode should be computed based on generated configs#1215

Merged
xinyuiscool merged 1 commit intoapache:masterfrom
xinyuiscool:SAMZA-2377
Nov 12, 2019
Merged

SAMZA-2377: Batch mode should be computed based on generated configs#1215
xinyuiscool merged 1 commit intoapache:masterfrom
xinyuiscool:SAMZA-2377

Conversation

@xinyuiscool
Copy link
Copy Markdown
Contributor

Currently the batch mode in job planner is computed based on the userconfig. This should be changed after the Samza InputDescriptor change. We should compute that based on the generated configs now.

Also add the logic to remove host affinity config for batch mode.

@xinyuiscool
Copy link
Copy Markdown
Contributor Author

@mynameborat : pls help take a look.

Copy link
Copy Markdown
Contributor

@mynameborat mynameborat left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@xinyuiscool xinyuiscool merged commit ba02b46 into apache:master Nov 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants