Skip to content

SAMZA-2304: Existing container locality mapping is incorrect when building job model - #1141

Merged
cameronlee314 merged 1 commit into
apache:masterfrom
cameronlee314:too_many_containers
Aug 20, 2019
Merged

cameronlee314 merged 1 commit into
apache:masterfrom
cameronlee314:too_many_containers

Conversation

@cameronlee314

Copy link
Copy Markdown
Contributor

Testing done (verification done through log messages on the AM):

  1. Used samza-hello-samza to verify that the correct number of containers is used when using GroupByContainerIds as the task name grouper, which depends on the container mapping.
  2. Verified that preferred host got set when host affinity is enabled.

@cameronlee314
cameronlee314 merged commit 83467a0 into apache:master Aug 20, 2019
@cameronlee314
cameronlee314 deleted the too_many_containers branch October 4, 2019 21:47
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