5/29/2018 7:15:43 PM
methods never throw ServletException.
|
5/29/2018 6:31:15 PM
(#1773)
Original error message is misleading -
1. it doesn't clarify the error is from web server or executor.
2. executor service sometimes rejects new flow submission because it's being shut down after executor's shutdown api is called, but the message still indicates the reason as exceeding the thread pool limit.
|
5/28/2018 12:42:30 AM
which is > 6 months old.
4.7 is the latest version as of today.
Updated using
`./gradlew wrapper --gradle-version 4.6`
Remove the gradle version in the wrapper task config since this information
is included in the checkedin gradle-warpper.properties already.
See
https://docs.gradle.org/current/userguide/gradle_wrapper.html
https://gradle.org/releases/
|
5/26/2018 4:01:46 PM
test ExecutorServiceUtilsTest
Failure on Travis:
azkaban.utils.ExecutorServiceUtilsTest > force_shutdown_after_timeout FAILED java.lang.AssertionError: Expecting: <11L> to be less than: <10L> at azkaban.utils.ExecutorServiceUtilsTest.force_shutdown_after_timeout(ExecutorServiceUtilsTest.java:60)
azkaban.utils.ExecutorServiceUtilsTest
Cause:
When the test host is busy, the execution time may be greater.
Fix:
Add buffer and a comment to explain the test.
* Assert that the test thread is indeed interrupted
The previous test will continue to succeed even if the task is not terminated.
|
5/25/2018 10:09:22 PM
logger.error call
|
5/25/2018 7:35:16 PM
(#1770)
|
5/22/2018 12:29:58 PM
Refactor DagService shutdown method and add tests
Part of #1486
Create a new utility class to encapsulate the generic shutdown logic to
make it easier to test and allow better code reuse.
Change the shutdown API to take a Duration type instead of a long. This is more precise and flexible. Also reduced the shutdown timeout from 60 seconds to 10 seconds.
After this change, the dag package will have 100% line
coverage.
|
5/21/2018 9:26:09 PM
line coverage, the logical coverage is more
limited. This change improves that.
|
5/21/2018 9:09:49 PM
state such as killing won't have
any effect.
With this change, the DAG class has 100% code coverage.
Also make ImmutableSet in the Status enum more strongly typed.
|
5/17/2018 8:14:35 PM
failed. (#1764)
With this test, the Node class has 100% code coverage.
|
