[jira] [Work logged] (GOBBLIN-1099) Handle orphaned Yarn containers in Gobblin-on-Yarn clusters
[ https://issues.apache.org/jira/browse/GOBBLIN-1099?focusedWorklogId=410741=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-410741 ] ASF GitHub Bot logged work on GOBBLIN-1099: --- Author: ASF GitHub Bot Created on: 27/Mar/20 00:35 Start Date: 27/Mar/20 00:35 Worklog Time Spent: 10m Work Description: codecov-io commented on issue #2940: GOBBLIN-1099: Handle orphaned Yarn containers in Gobblin-on-Yarn clus… URL: https://github.com/apache/incubator-gobblin/pull/2940#issuecomment-604751963 # [Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=h1) Report > Merging [#2940](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=desc) into [master](https://codecov.io/gh/apache/incubator-gobblin/commit/a63461257c3fcea8f4ff67087f8cb29be25d6baf=desc) will **decrease** coverage by `0.00%`. > The diff coverage is `41.57%`. [![Impacted file tree graph](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/graphs/tree.svg?width=650=150=pr=4MgURJ0bGc)](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=tree) ```diff @@ Coverage Diff @@ ## master#2940 +/- ## - Coverage 44.60% 44.60% -0.01% - Complexity 8980 8981 +1 Files 1936 1936 Lines 7323473296 +62 Branches 8083 8089 +6 + Hits 3266932695 +26 - Misses3751537550 +35 - Partials 3050 3051 +1 ``` | [Impacted Files](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=tree) | Coverage Δ | Complexity Δ | | |---|---|---|---| | [.../apache/gobblin/yarn/GobblinApplicationMaster.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi15YXJuL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3lhcm4vR29iYmxpbkFwcGxpY2F0aW9uTWFzdGVyLmphdmE=) | `17.80% <0.00%> (-0.25%)` | `3.00 <0.00> (ø)` | | | [...rg/apache/gobblin/yarn/GobblinYarnAppLauncher.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi15YXJuL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3lhcm4vR29iYmxpbllhcm5BcHBMYXVuY2hlci5qYXZh) | `20.66% <0.00%> (-0.51%)` | `8.00 <0.00> (ø)` | | | [...in/java/org/apache/gobblin/cluster/HelixUtils.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvSGVsaXhVdGlscy5qYXZh) | `34.12% <18.18%> (-4.14%)` | `14.00 <1.00> (ø)` | | | [...main/java/org/apache/gobblin/yarn/YarnService.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi15YXJuL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3lhcm4vWWFyblNlcnZpY2UuamF2YQ==) | `15.71% <22.22%> (+0.44%)` | `4.00 <0.00> (ø)` | | | [.../org/apache/gobblin/cluster/GobblinTaskRunner.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpblRhc2tSdW5uZXIuamF2YQ==) | `66.01% <75.00%> (+1.85%)` | `29.00 <4.00> (+2.00)` | | | [.../apache/gobblin/cluster/GobblinClusterManager.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpbkNsdXN0ZXJNYW5hZ2VyLmphdmE=) | `53.91% <100.00%> (ø)` | `26.00 <1.00> (ø)` | | | [...ava/org/apache/gobblin/fsm/FiniteStateMachine.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi11dGlsaXR5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2ZzbS9GaW5pdGVTdGF0ZU1hY2hpbmUuamF2YQ==) | `73.48% <0.00%> (-3.04%)` | `18.00% <0.00%> (-3.00%)` | | | [...lin/restli/throttling/ZookeeperLeaderElection.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi10aHJvdHRsaW5nLXNlcnZpY2UvZ29iYmxpbi10aHJvdHRsaW5nLXNlcnZpY2Utc2VydmVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3Jlc3RsaS90aHJvdHRsaW5nL1pvb2tlZXBlckxlYWRlckVsZWN0aW9uLmphdmE=) | `70.00% <0.00%> (-2.23%)` | `13.00% <0.00%> (ø%)` | | | ... and [3 more](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree-more) | | -- [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by
[GitHub] [incubator-gobblin] codecov-io edited a comment on issue #2940: GOBBLIN-1099: Handle orphaned Yarn containers in Gobblin-on-Yarn clus…
codecov-io edited a comment on issue #2940: GOBBLIN-1099: Handle orphaned Yarn containers in Gobblin-on-Yarn clus… URL: https://github.com/apache/incubator-gobblin/pull/2940#issuecomment-604751963 # [Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=h1) Report > Merging [#2940](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=desc) into [master](https://codecov.io/gh/apache/incubator-gobblin/commit/a63461257c3fcea8f4ff67087f8cb29be25d6baf=desc) will **decrease** coverage by `0.00%`. > The diff coverage is `41.57%`. [![Impacted file tree graph](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/graphs/tree.svg?width=650=150=pr=4MgURJ0bGc)](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=tree) ```diff @@ Coverage Diff @@ ## master#2940 +/- ## - Coverage 44.60% 44.60% -0.01% - Complexity 8980 8981 +1 Files 1936 1936 Lines 7323473296 +62 Branches 8083 8089 +6 + Hits 3266932695 +26 - Misses3751537550 +35 - Partials 3050 3051 +1 ``` | [Impacted Files](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=tree) | Coverage Δ | Complexity Δ | | |---|---|---|---| | [.../apache/gobblin/yarn/GobblinApplicationMaster.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi15YXJuL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3lhcm4vR29iYmxpbkFwcGxpY2F0aW9uTWFzdGVyLmphdmE=) | `17.80% <0.00%> (-0.25%)` | `3.00 <0.00> (ø)` | | | [...rg/apache/gobblin/yarn/GobblinYarnAppLauncher.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi15YXJuL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3lhcm4vR29iYmxpbllhcm5BcHBMYXVuY2hlci5qYXZh) | `20.66% <0.00%> (-0.51%)` | `8.00 <0.00> (ø)` | | | [...in/java/org/apache/gobblin/cluster/HelixUtils.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvSGVsaXhVdGlscy5qYXZh) | `34.12% <18.18%> (-4.14%)` | `14.00 <1.00> (ø)` | | | [...main/java/org/apache/gobblin/yarn/YarnService.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi15YXJuL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3lhcm4vWWFyblNlcnZpY2UuamF2YQ==) | `15.71% <22.22%> (+0.44%)` | `4.00 <0.00> (ø)` | | | [.../org/apache/gobblin/cluster/GobblinTaskRunner.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpblRhc2tSdW5uZXIuamF2YQ==) | `66.01% <75.00%> (+1.85%)` | `29.00 <4.00> (+2.00)` | | | [.../apache/gobblin/cluster/GobblinClusterManager.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpbkNsdXN0ZXJNYW5hZ2VyLmphdmE=) | `53.91% <100.00%> (ø)` | `26.00 <1.00> (ø)` | | | [...ava/org/apache/gobblin/fsm/FiniteStateMachine.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi11dGlsaXR5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2ZzbS9GaW5pdGVTdGF0ZU1hY2hpbmUuamF2YQ==) | `73.48% <0.00%> (-3.04%)` | `18.00% <0.00%> (-3.00%)` | | | [...lin/restli/throttling/ZookeeperLeaderElection.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi10aHJvdHRsaW5nLXNlcnZpY2UvZ29iYmxpbi10aHJvdHRsaW5nLXNlcnZpY2Utc2VydmVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3Jlc3RsaS90aHJvdHRsaW5nL1pvb2tlZXBlckxlYWRlckVsZWN0aW9uLmphdmE=) | `70.00% <0.00%> (-2.23%)` | `13.00% <0.00%> (ø%)` | | | ... and [3 more](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree-more) | | -- [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=footer). Last update [a634612...2915df6](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments). This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use
[jira] [Work logged] (GOBBLIN-1099) Handle orphaned Yarn containers in Gobblin-on-Yarn clusters
[ https://issues.apache.org/jira/browse/GOBBLIN-1099?focusedWorklogId=410737=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-410737 ] ASF GitHub Bot logged work on GOBBLIN-1099: --- Author: ASF GitHub Bot Created on: 27/Mar/20 00:19 Start Date: 27/Mar/20 00:19 Worklog Time Spent: 10m Work Description: codecov-io commented on issue #2940: GOBBLIN-1099: Handle orphaned Yarn containers in Gobblin-on-Yarn clus… URL: https://github.com/apache/incubator-gobblin/pull/2940#issuecomment-604751963 # [Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=h1) Report > Merging [#2940](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=desc) into [master](https://codecov.io/gh/apache/incubator-gobblin/commit/a63461257c3fcea8f4ff67087f8cb29be25d6baf=desc) will **decrease** coverage by `40.49%`. > The diff coverage is `0.00%`. [![Impacted file tree graph](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/graphs/tree.svg?width=650=150=pr=4MgURJ0bGc)](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=tree) ```diff @@ Coverage Diff @@ ## master #2940 +/- ## - Coverage 44.60% 4.11% -40.50% + Complexity 8980 750 -8230 Files 19361936 Lines 73234 73296 +62 Branches 80838089+6 - Hits 326693016-29653 - Misses37515 69960+32445 + Partials 3050 320 -2730 ``` | [Impacted Files](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=tree) | Coverage Δ | Complexity Δ | | |---|---|---|---| | [.../apache/gobblin/cluster/GobblinClusterManager.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpbkNsdXN0ZXJNYW5hZ2VyLmphdmE=) | `0.00% <0.00%> (-53.92%)` | `0.00 <0.00> (-26.00)` | | | [.../org/apache/gobblin/cluster/GobblinTaskRunner.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpblRhc2tSdW5uZXIuamF2YQ==) | `0.00% <0.00%> (-64.16%)` | `0.00 <0.00> (-27.00)` | | | [...in/java/org/apache/gobblin/cluster/HelixUtils.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvSGVsaXhVdGlscy5qYXZh) | `0.00% <0.00%> (-38.27%)` | `0.00 <0.00> (-14.00)` | | | [...n/compaction/action/CompactionWatermarkAction.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jb21wYWN0aW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NvbXBhY3Rpb24vYWN0aW9uL0NvbXBhY3Rpb25XYXRlcm1hcmtBY3Rpb24uamF2YQ==) | `0.00% <0.00%> (-77.05%)` | `0.00 <0.00> (-11.00)` | | | [...che/gobblin/salesforce/ResultChainingIterator.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1zYWxlc2ZvcmNlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NhbGVzZm9yY2UvUmVzdWx0Q2hhaW5pbmdJdGVyYXRvci5qYXZh) | `0.00% <0.00%> (ø)` | `0.00 <0.00> (ø)` | | | [.../apache/gobblin/yarn/GobblinApplicationMaster.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi15YXJuL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3lhcm4vR29iYmxpbkFwcGxpY2F0aW9uTWFzdGVyLmphdmE=) | `0.00% <0.00%> (-18.06%)` | `0.00 <0.00> (-3.00)` | | | [...rg/apache/gobblin/yarn/GobblinYarnAppLauncher.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi15YXJuL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3lhcm4vR29iYmxpbllhcm5BcHBMYXVuY2hlci5qYXZh) | `0.00% <0.00%> (-21.17%)` | `0.00 <0.00> (-8.00)` | | | [...main/java/org/apache/gobblin/yarn/YarnService.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi15YXJuL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3lhcm4vWWFyblNlcnZpY2UuamF2YQ==) | `0.00% <0.00%> (-15.28%)` | `0.00 <0.00> (-4.00)` | | | [...c/main/java/org/apache/gobblin/util/FileUtils.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi11dGlsaXR5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3V0aWwvRmlsZVV0aWxzLmphdmE=) | `0.00% <0.00%> (-100.00%)` | `0.00% <0.00%> (-4.00%)` | | |
[GitHub] [incubator-gobblin] codecov-io commented on issue #2940: GOBBLIN-1099: Handle orphaned Yarn containers in Gobblin-on-Yarn clus…
codecov-io commented on issue #2940: GOBBLIN-1099: Handle orphaned Yarn containers in Gobblin-on-Yarn clus… URL: https://github.com/apache/incubator-gobblin/pull/2940#issuecomment-604751963 # [Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=h1) Report > Merging [#2940](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=desc) into [master](https://codecov.io/gh/apache/incubator-gobblin/commit/a63461257c3fcea8f4ff67087f8cb29be25d6baf=desc) will **decrease** coverage by `40.49%`. > The diff coverage is `0.00%`. [![Impacted file tree graph](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/graphs/tree.svg?width=650=150=pr=4MgURJ0bGc)](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=tree) ```diff @@ Coverage Diff @@ ## master #2940 +/- ## - Coverage 44.60% 4.11% -40.50% + Complexity 8980 750 -8230 Files 19361936 Lines 73234 73296 +62 Branches 80838089+6 - Hits 326693016-29653 - Misses37515 69960+32445 + Partials 3050 320 -2730 ``` | [Impacted Files](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=tree) | Coverage Δ | Complexity Δ | | |---|---|---|---| | [.../apache/gobblin/cluster/GobblinClusterManager.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpbkNsdXN0ZXJNYW5hZ2VyLmphdmE=) | `0.00% <0.00%> (-53.92%)` | `0.00 <0.00> (-26.00)` | | | [.../org/apache/gobblin/cluster/GobblinTaskRunner.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpblRhc2tSdW5uZXIuamF2YQ==) | `0.00% <0.00%> (-64.16%)` | `0.00 <0.00> (-27.00)` | | | [...in/java/org/apache/gobblin/cluster/HelixUtils.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvSGVsaXhVdGlscy5qYXZh) | `0.00% <0.00%> (-38.27%)` | `0.00 <0.00> (-14.00)` | | | [...n/compaction/action/CompactionWatermarkAction.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jb21wYWN0aW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NvbXBhY3Rpb24vYWN0aW9uL0NvbXBhY3Rpb25XYXRlcm1hcmtBY3Rpb24uamF2YQ==) | `0.00% <0.00%> (-77.05%)` | `0.00 <0.00> (-11.00)` | | | [...che/gobblin/salesforce/ResultChainingIterator.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1zYWxlc2ZvcmNlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NhbGVzZm9yY2UvUmVzdWx0Q2hhaW5pbmdJdGVyYXRvci5qYXZh) | `0.00% <0.00%> (ø)` | `0.00 <0.00> (ø)` | | | [.../apache/gobblin/yarn/GobblinApplicationMaster.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi15YXJuL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3lhcm4vR29iYmxpbkFwcGxpY2F0aW9uTWFzdGVyLmphdmE=) | `0.00% <0.00%> (-18.06%)` | `0.00 <0.00> (-3.00)` | | | [...rg/apache/gobblin/yarn/GobblinYarnAppLauncher.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi15YXJuL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3lhcm4vR29iYmxpbllhcm5BcHBMYXVuY2hlci5qYXZh) | `0.00% <0.00%> (-21.17%)` | `0.00 <0.00> (-8.00)` | | | [...main/java/org/apache/gobblin/yarn/YarnService.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi15YXJuL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3lhcm4vWWFyblNlcnZpY2UuamF2YQ==) | `0.00% <0.00%> (-15.28%)` | `0.00 <0.00> (-4.00)` | | | [...c/main/java/org/apache/gobblin/util/FileUtils.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi11dGlsaXR5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3V0aWwvRmlsZVV0aWxzLmphdmE=) | `0.00% <0.00%> (-100.00%)` | `0.00% <0.00%> (-4.00%)` | | | [...n/java/org/apache/gobblin/fork/CopyableSchema.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree#diff-Z29iYmxpbi1jb3JlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2ZvcmsvQ29weWFibGVTY2hlbWEuamF2YQ==) | `0.00% <0.00%> (-100.00%)` | `0.00% <0.00%> (-2.00%)` | | | ... and [1150 more](https://codecov.io/gh/apache/incubator-gobblin/pull/2940/diff?src=pr=tree-more) | | -- [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2940?src=pr=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) >
[GitHub] [incubator-gobblin] sv2000 commented on issue #2940: GOBBLIN-1099: Handle orphaned Yarn containers in Gobblin-on-Yarn clus…
sv2000 commented on issue #2940: GOBBLIN-1099: Handle orphaned Yarn containers in Gobblin-on-Yarn clus… URL: https://github.com/apache/incubator-gobblin/pull/2940#issuecomment-604735451 @autumnust @htran1 please review. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[jira] [Work logged] (GOBBLIN-1099) Handle orphaned Yarn containers in Gobblin-on-Yarn clusters
[ https://issues.apache.org/jira/browse/GOBBLIN-1099?focusedWorklogId=410697=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-410697 ] ASF GitHub Bot logged work on GOBBLIN-1099: --- Author: ASF GitHub Bot Created on: 26/Mar/20 23:18 Start Date: 26/Mar/20 23:18 Worklog Time Spent: 10m Work Description: sv2000 commented on issue #2940: GOBBLIN-1099: Handle orphaned Yarn containers in Gobblin-on-Yarn clus… URL: https://github.com/apache/incubator-gobblin/pull/2940#issuecomment-604735451 @autumnust @htran1 please review. This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 410697) Time Spent: 20m (was: 10m) > Handle orphaned Yarn containers in Gobblin-on-Yarn clusters > --- > > Key: GOBBLIN-1099 > URL: https://issues.apache.org/jira/browse/GOBBLIN-1099 > Project: Apache Gobblin > Issue Type: Improvement > Components: gobblin-yarn >Affects Versions: 0.15.0 >Reporter: Sudarshan Vasudevan >Assignee: Abhishek Tiwari >Priority: Major > Fix For: 0.15.0 > > Time Spent: 20m > Remaining Estimate: 0h > > A Yarn application may leave behind orphaned containers, which can happen due > to lost node managers. The orphaned containers however can continue to run > (potentially forever) as participants in the Helix cluster. This can cause > the following problems for a Gobblin-on-Yarn application: > # Double publish of data and commit of state > # Task failures and partition starvation during application restarts, as > Helix may assign tasks to the orphaned containers which have a stale state > and configuration > # Container failures on application restarts due to Helix instance name > collisions with orphaned containers > -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (GOBBLIN-1099) Handle orphaned Yarn containers in Gobblin-on-Yarn clusters
[ https://issues.apache.org/jira/browse/GOBBLIN-1099?focusedWorklogId=410696=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-410696 ] ASF GitHub Bot logged work on GOBBLIN-1099: --- Author: ASF GitHub Bot Created on: 26/Mar/20 23:18 Start Date: 26/Mar/20 23:18 Worklog Time Spent: 10m Work Description: sv2000 commented on pull request #2940: GOBBLIN-1099: Handle orphaned Yarn containers in Gobblin-on-Yarn clus… URL: https://github.com/apache/incubator-gobblin/pull/2940 …ters Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below! ### JIRA - [x] My PR addresses the following [Gobblin JIRA](https://issues.apache.org/jira/browse/GOBBLIN/) issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR" - https://issues.apache.org/jira/browse/GOBBLIN-1099 ### Description - [x] Here are some details about my PR, including screenshots (if applicable): A Yarn application may leave behind orphaned containers, which can happen due to lost node managers. The orphaned containers however can continue to run (potentially forever) as participants in the Helix cluster. This can cause the following problems for a Gobblin-on-Yarn application: Double publish of data and commit of state Task failures and partition starvation during application restarts, as Helix may assign tasks to the orphaned containers which have a stale state and configuration Container failures on application restarts due to Helix instance name collisions with orphaned containers This PR incorporates the following changes to handle orphaned containers: 1. Disables live instances during Yarn application start up and shutdown in GobblinYarnAppLauncher. This ensures that any orphaned Helix instances are disabled from joining the cluster on restart and any tasks running on these instances are cancelled. 2. The GobblinApplicationMaster (inside YarnService) ensures that Helix instance name assigned to a newly allocated container is not a currently live instance to avoid instance name collisions. We also handle NM failures/restarts that result in Yarn RM "aborting" the container (i.e. container is deemed dead from Yarn's point of view, even though the container may physically be alive). In this case, we disable the instance to ensure it is fenced off from the Helix cluster. 3. Gobblin workers (i.e. GobblinYarnTaskRunner) retry in case of failure in joining a Helix cluster. The retry logic disconnects from the Helix cluster and drops the instance before reattempting to join the cluster. ### Tests - [x] My PR adds the following unit tests __OR__ does not need testing for this extremely good reason: Added unit test in GobblinTaskRunnerTest ### Commits - [x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "[How to write a good git commit message](http://chris.beams.io/posts/git-commit/)": 1. Subject is separated from body by a blank line 2. Subject is limited to 50 characters 3. Subject does not end with a period 4. Subject uses the imperative mood ("add", not "adding") 5. Body wraps at 72 characters 6. Body explains "what" and "why", not "how" This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 410696) Remaining Estimate: 0h Time Spent: 10m > Handle orphaned Yarn containers in Gobblin-on-Yarn clusters > --- > > Key: GOBBLIN-1099 > URL: https://issues.apache.org/jira/browse/GOBBLIN-1099 > Project: Apache Gobblin > Issue Type: Improvement > Components: gobblin-yarn >Affects Versions: 0.15.0 >Reporter: Sudarshan Vasudevan >Assignee: Abhishek Tiwari >Priority: Major > Fix For: 0.15.0 > > Time Spent: 10m > Remaining Estimate: 0h > > A Yarn application may leave behind orphaned containers, which can happen due > to lost node managers. The orphaned containers however can continue to run > (potentially forever) as participants in the Helix cluster. This can cause > the following problems for a Gobblin-on-Yarn application: > # Double publish of data and commit of state > # Task failures and
[GitHub] [incubator-gobblin] sv2000 opened a new pull request #2940: GOBBLIN-1099: Handle orphaned Yarn containers in Gobblin-on-Yarn clus…
sv2000 opened a new pull request #2940: GOBBLIN-1099: Handle orphaned Yarn containers in Gobblin-on-Yarn clus… URL: https://github.com/apache/incubator-gobblin/pull/2940 …ters Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below! ### JIRA - [x] My PR addresses the following [Gobblin JIRA](https://issues.apache.org/jira/browse/GOBBLIN/) issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR" - https://issues.apache.org/jira/browse/GOBBLIN-1099 ### Description - [x] Here are some details about my PR, including screenshots (if applicable): A Yarn application may leave behind orphaned containers, which can happen due to lost node managers. The orphaned containers however can continue to run (potentially forever) as participants in the Helix cluster. This can cause the following problems for a Gobblin-on-Yarn application: Double publish of data and commit of state Task failures and partition starvation during application restarts, as Helix may assign tasks to the orphaned containers which have a stale state and configuration Container failures on application restarts due to Helix instance name collisions with orphaned containers This PR incorporates the following changes to handle orphaned containers: 1. Disables live instances during Yarn application start up and shutdown in GobblinYarnAppLauncher. This ensures that any orphaned Helix instances are disabled from joining the cluster on restart and any tasks running on these instances are cancelled. 2. The GobblinApplicationMaster (inside YarnService) ensures that Helix instance name assigned to a newly allocated container is not a currently live instance to avoid instance name collisions. We also handle NM failures/restarts that result in Yarn RM "aborting" the container (i.e. container is deemed dead from Yarn's point of view, even though the container may physically be alive). In this case, we disable the instance to ensure it is fenced off from the Helix cluster. 3. Gobblin workers (i.e. GobblinYarnTaskRunner) retry in case of failure in joining a Helix cluster. The retry logic disconnects from the Helix cluster and drops the instance before reattempting to join the cluster. ### Tests - [x] My PR adds the following unit tests __OR__ does not need testing for this extremely good reason: Added unit test in GobblinTaskRunnerTest ### Commits - [x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "[How to write a good git commit message](http://chris.beams.io/posts/git-commit/)": 1. Subject is separated from body by a blank line 2. Subject is limited to 50 characters 3. Subject does not end with a period 4. Subject uses the imperative mood ("add", not "adding") 5. Body wraps at 72 characters 6. Body explains "what" and "why", not "how" This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[jira] [Created] (GOBBLIN-1099) Handle orphaned Yarn containers in Gobblin-on-Yarn clusters
Sudarshan Vasudevan created GOBBLIN-1099: Summary: Handle orphaned Yarn containers in Gobblin-on-Yarn clusters Key: GOBBLIN-1099 URL: https://issues.apache.org/jira/browse/GOBBLIN-1099 Project: Apache Gobblin Issue Type: Improvement Components: gobblin-yarn Affects Versions: 0.15.0 Reporter: Sudarshan Vasudevan Assignee: Abhishek Tiwari Fix For: 0.15.0 A Yarn application may leave behind orphaned containers, which can happen due to lost node managers. The orphaned containers however can continue to run (potentially forever) as participants in the Helix cluster. This can cause the following problems for a Gobblin-on-Yarn application: # Double publish of data and commit of state # Task failures and partition starvation during application restarts, as Helix may assign tasks to the orphaned containers which have a stale state and configuration # Container failures on application restarts due to Helix instance name collisions with orphaned containers -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Resolved] (GOBBLIN-1098) Remove commons-lang and slf4j from the orc-dep fat jar
[ https://issues.apache.org/jira/browse/GOBBLIN-1098?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Hung Tran resolved GOBBLIN-1098. Fix Version/s: 0.15.0 Resolution: Fixed Issue resolved by pull request #2939 [https://github.com/apache/incubator-gobblin/pull/2939] > Remove commons-lang and slf4j from the orc-dep fat jar > -- > > Key: GOBBLIN-1098 > URL: https://issues.apache.org/jira/browse/GOBBLIN-1098 > Project: Apache Gobblin > Issue Type: Improvement >Reporter: Hung Tran >Priority: Major > Fix For: 0.15.0 > > Time Spent: 20m > Remaining Estimate: 0h > > The commons-lang and slf4j are commonly used dependencies and are likely to > result in conflicts when included in a fat jar without shading. Remove these > dependencies from the orc-dep fat jar to avoid conflicts. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[jira] [Work logged] (GOBBLIN-1098) Remove commons-lang and slf4j from the orc-dep fat jar
[ https://issues.apache.org/jira/browse/GOBBLIN-1098?focusedWorklogId=410634=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-410634 ] ASF GitHub Bot logged work on GOBBLIN-1098: --- Author: ASF GitHub Bot Created on: 26/Mar/20 21:17 Start Date: 26/Mar/20 21:17 Worklog Time Spent: 10m Work Description: asfgit commented on pull request #2939: [GOBBLIN-1098] Remove commons-lang and slf4j from the orc-dep fat jar URL: https://github.com/apache/incubator-gobblin/pull/2939 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 410634) Time Spent: 20m (was: 10m) > Remove commons-lang and slf4j from the orc-dep fat jar > -- > > Key: GOBBLIN-1098 > URL: https://issues.apache.org/jira/browse/GOBBLIN-1098 > Project: Apache Gobblin > Issue Type: Improvement >Reporter: Hung Tran >Priority: Major > Time Spent: 20m > Remaining Estimate: 0h > > The commons-lang and slf4j are commonly used dependencies and are likely to > result in conflicts when included in a fat jar without shading. Remove these > dependencies from the orc-dep fat jar to avoid conflicts. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [incubator-gobblin] asfgit closed pull request #2939: [GOBBLIN-1098] Remove commons-lang and slf4j from the orc-dep fat jar
asfgit closed pull request #2939: [GOBBLIN-1098] Remove commons-lang and slf4j from the orc-dep fat jar URL: https://github.com/apache/incubator-gobblin/pull/2939 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[jira] [Work logged] (GOBBLIN-1098) Remove commons-lang and slf4j from the orc-dep fat jar
[ https://issues.apache.org/jira/browse/GOBBLIN-1098?focusedWorklogId=410582=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-410582 ] ASF GitHub Bot logged work on GOBBLIN-1098: --- Author: ASF GitHub Bot Created on: 26/Mar/20 20:18 Start Date: 26/Mar/20 20:18 Worklog Time Spent: 10m Work Description: htran1 commented on pull request #2939: [GOBBLIN-1098] Remove commons-lang and slf4j from the orc-dep fat jar URL: https://github.com/apache/incubator-gobblin/pull/2939 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below! ### JIRA - [X] My PR addresses the following [Gobblin JIRA](https://issues.apache.org/jira/browse/GOBBLIN/) issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR" - https://issues.apache.org/jira/browse/GOBBLIN-1098 ### Description - [X] Here are some details about my PR, including screenshots (if applicable): The commons-lang and slf4j are commonly used dependencies and are likely to result in conflicts when included in a fat jar without shading. Remove these dependencies from the orc-dep fat jar to avoid conflicts. ### Tests - [X] My PR adds the following unit tests __OR__ does not need testing for this extremely good reason: Built and tested with the new fat jar. ### Commits - [X] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "[How to write a good git commit message](http://chris.beams.io/posts/git-commit/)": 1. Subject is separated from body by a blank line 2. Subject is limited to 50 characters 3. Subject does not end with a period 4. Subject uses the imperative mood ("add", not "adding") 5. Body wraps at 72 characters 6. Body explains "what" and "why", not "how" This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking --- Worklog Id: (was: 410582) Remaining Estimate: 0h Time Spent: 10m > Remove commons-lang and slf4j from the orc-dep fat jar > -- > > Key: GOBBLIN-1098 > URL: https://issues.apache.org/jira/browse/GOBBLIN-1098 > Project: Apache Gobblin > Issue Type: Improvement >Reporter: Hung Tran >Priority: Major > Time Spent: 10m > Remaining Estimate: 0h > > The commons-lang and slf4j are commonly used dependencies and are likely to > result in conflicts when included in a fat jar without shading. Remove these > dependencies from the orc-dep fat jar to avoid conflicts. -- This message was sent by Atlassian Jira (v8.3.4#803005)
[GitHub] [incubator-gobblin] htran1 opened a new pull request #2939: [GOBBLIN-1098] Remove commons-lang and slf4j from the orc-dep fat jar
htran1 opened a new pull request #2939: [GOBBLIN-1098] Remove commons-lang and slf4j from the orc-dep fat jar URL: https://github.com/apache/incubator-gobblin/pull/2939 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below! ### JIRA - [X] My PR addresses the following [Gobblin JIRA](https://issues.apache.org/jira/browse/GOBBLIN/) issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR" - https://issues.apache.org/jira/browse/GOBBLIN-1098 ### Description - [X] Here are some details about my PR, including screenshots (if applicable): The commons-lang and slf4j are commonly used dependencies and are likely to result in conflicts when included in a fat jar without shading. Remove these dependencies from the orc-dep fat jar to avoid conflicts. ### Tests - [X] My PR adds the following unit tests __OR__ does not need testing for this extremely good reason: Built and tested with the new fat jar. ### Commits - [X] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "[How to write a good git commit message](http://chris.beams.io/posts/git-commit/)": 1. Subject is separated from body by a blank line 2. Subject is limited to 50 characters 3. Subject does not end with a period 4. Subject uses the imperative mood ("add", not "adding") 5. Body wraps at 72 characters 6. Body explains "what" and "why", not "how" This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
[jira] [Created] (GOBBLIN-1098) Remove commons-lang and slf4j from the orc-dep fat jar
Hung Tran created GOBBLIN-1098: -- Summary: Remove commons-lang and slf4j from the orc-dep fat jar Key: GOBBLIN-1098 URL: https://issues.apache.org/jira/browse/GOBBLIN-1098 Project: Apache Gobblin Issue Type: Improvement Reporter: Hung Tran The commons-lang and slf4j are commonly used dependencies and are likely to result in conflicts when included in a fat jar without shading. Remove these dependencies from the orc-dep fat jar to avoid conflicts. -- This message was sent by Atlassian Jira (v8.3.4#803005)