samza-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jake Maes <>
Subject Re: Review Request 45190: SAMZA-910 Fix expired request test in HostAwareContainerAllocator
Date Mon, 11 Apr 2016 18:45:03 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated April 11, 2016, 6:45 p.m.)

Review request for samza.

Repository: samza


SAMZA-910 Fix expired request test in HostAwareContainerAllocator

Summary of changes:
1. Remove the last sleep() from HostAwareContainerAllocator
2. Fix a silent failure in testRerequestOnAnyHostIfContainerStartFails by setting the neededContainers
to 1 before running the test.
3. Update MockContainerListener so assertion failures in other threads are thrown in the main
thread to fail the test. (no silent failures) This should help troubleshoot the tests if there
are any remaining issues.
4. Rename obscure hostnames to make it easier to reason about the tests.

Diffs (updated)

  samza-yarn/src/test/java/org/apache/samza/job/yarn/ b253f98f7258bb611e1ad6672f74b07ab7e20b70

  samza-yarn/src/test/java/org/apache/samza/job/yarn/ PRE-CREATION

  samza-yarn/src/test/java/org/apache/samza/job/yarn/util/ cb82cccf75b54cfbefd586700e8283cb41173833

  samza-yarn/src/test/java/org/apache/samza/job/yarn/util/ 879a7d0d06b087cfe0417f3fa5801b43ac7fc458

  samza-yarn/src/test/java/org/apache/samza/job/yarn/util/ 2f9669f8b7e77abb65b244ccd067ae7ab1f245c3



Ran build and check-all on both of my machines twice. I don't see any sporadic failures anymore.


Jake Maes

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message