10/15/2018 4:49:55 PM
(#1974)
|
10/15/2018 3:18:11 PM
to a separate class
* Fix typos in comments
* Apply save actions on ExecutorManager
* Delete unused field lastCleanerThreadCheckTime
* Rename ExecutingManagerUpdaterThread to RunningExecutionsUpdaterThread
* Extract ExecutionFinalizer & RunningExecutions from ExecutorManager
This allowed removing a circular dependency between ExecutorManager & RunningExecutionsUpdaterThread
* Inject ExecutionFinalizer to ExecutorManager
* Inject RunningExecutionsUpdaterThread to ExecutorManager
* Removed unused constructor arg of ExecutorManager
* Extract ActiveExecutors from ExecutorManager
* Add TODO comment about refreshing executors
* Fix injection to ActiveExecutors
* Added copyright & javadocs to ActiveExecutors
+ Refactored to clearly distinguish between multi-executor mode vs. single executor
* Added copyright & javadocs to ExecutionFinalizer
+ Added @Nullable annotation
* Extracted class RunningExecutionsUpdater from RunningExecutionsUpdaterThread
+ Added copyright & javadocs to both
* Simplified RunningExecutions
+ Added copyright & javadocs
* Added copyright & javadocs to ExecutorManagerUpdaterStage
* Improve ExecutorManagerTest speed
+ Apply save actions
* Remove a confusing TODO comment
|
|
|
|
|
9/27/2018 6:25:12 PM
Include reasons for finalizing an execution
- Improve status check: alert with error email if the flow status is anything else but SUCCEEDED
- Get rid of e.printStackTrace(), use logger instead
- Fix MailCreator argument name 'reasons' & remove unused args in interface
|
9/24/2018 3:58:10 PM
3.59.0
(#1963)
This reverts commit b70ee97.
Reportal has to work with internal version of presto JDBC instead of OSS version.
|
9/19/2018 10:17:33 PM
3.58.0
jdbc is required by presto reportal jobtype.
|
9/19/2018 9:46:24 PM
summary (#1958)
* Conditional workflow UI support - add condition in job info and flow summary.
* address comment
|
|
|
|
|
