You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@gobblin.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2019/07/23 17:36:00 UTC

[jira] [Work logged] (GOBBLIN-817) Implement a workaround for Helix Workflow being stuck in STOPPING state

     [ https://issues.apache.org/jira/browse/GOBBLIN-817?focusedWorklogId=281186&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-281186 ]

ASF GitHub Bot logged work on GOBBLIN-817:
------------------------------------------

                Author: ASF GitHub Bot
            Created on: 23/Jul/19 17:35
            Start Date: 23/Jul/19 17:35
    Worklog Time Spent: 10m 
      Work Description: codecov-io commented on issue #2681: GOBBLIN-817: Implement a workaround for Helix Workflow being stuck in…
URL: https://github.com/apache/incubator-gobblin/pull/2681#issuecomment-514309719
 
 
   # [Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2681?src=pr&el=h1) Report
   > Merging [#2681](https://codecov.io/gh/apache/incubator-gobblin/pull/2681?src=pr&el=desc) into [master](https://codecov.io/gh/apache/incubator-gobblin/commit/bee393352880aad5d9edb05fcc2e5f5f4e5f4811?src=pr&el=desc) will **decrease** coverage by `<.01%`.
   > The diff coverage is `61.11%`.
   
   [![Impacted file tree graph](https://codecov.io/gh/apache/incubator-gobblin/pull/2681/graphs/tree.svg?width=650&token=4MgURJ0bGc&height=150&src=pr)](https://codecov.io/gh/apache/incubator-gobblin/pull/2681?src=pr&el=tree)
   
   ```diff
   @@             Coverage Diff              @@
   ##             master    #2681      +/-   ##
   ============================================
   - Coverage      44.8%   44.79%   -0.01%     
   + Complexity     8686     8685       -1     
   ============================================
     Files          1878     1878              
     Lines         70045    70060      +15     
     Branches       7701     7702       +1     
   ============================================
   + Hits          31382    31384       +2     
   - Misses        35759    35771      +12     
   - Partials       2904     2905       +1
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/incubator-gobblin/pull/2681?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | [...bblin/cluster/GobblinClusterConfigurationKeys.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2681/diff?src=pr&el=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpbkNsdXN0ZXJDb25maWd1cmF0aW9uS2V5cy5qYXZh) | `0% <ø> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | [...er/GobblinHelixDistributeJobExecutionLauncher.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2681/diff?src=pr&el=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpbkhlbGl4RGlzdHJpYnV0ZUpvYkV4ZWN1dGlvbkxhdW5jaGVyLmphdmE=) | `57.93% <100%> (+1.02%)` | `10 <0> (ø)` | :arrow_down: |
   | [...pache/gobblin/cluster/GobblinHelixJobLauncher.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2681/diff?src=pr&el=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpbkhlbGl4Sm9iTGF1bmNoZXIuamF2YQ==) | `81.53% <100%> (+0.28%)` | `26 <0> (ø)` | :arrow_down: |
   | [...in/java/org/apache/gobblin/cluster/HelixUtils.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2681/diff?src=pr&el=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvSGVsaXhVdGlscy5qYXZh) | `32.71% <30%> (-9.13%)` | `11 <0> (-2)` | |
   | [...org/apache/gobblin/publisher/DataPublisherKey.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2681/diff?src=pr&el=tree#diff-Z29iYmxpbi1jb3JlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3B1Ymxpc2hlci9EYXRhUHVibGlzaGVyS2V5LmphdmE=) | `53.33% <0%> (-13.34%)` | `5% <0%> (-1%)` | |
   | [...lin/elasticsearch/writer/FutureCallbackHolder.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2681/diff?src=pr&el=tree#diff-Z29iYmxpbi1tb2R1bGVzL2dvYmJsaW4tZWxhc3RpY3NlYXJjaC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZ29iYmxpbi9lbGFzdGljc2VhcmNoL3dyaXRlci9GdXR1cmVDYWxsYmFja0hvbGRlci5qYXZh) | `61.42% <0%> (-1.43%)` | `4% <0%> (ø)` | |
   | [.../apache/gobblin/runtime/api/JobExecutionState.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2681/diff?src=pr&el=tree#diff-Z29iYmxpbi1ydW50aW1lL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3J1bnRpbWUvYXBpL0pvYkV4ZWN1dGlvblN0YXRlLmphdmE=) | `79.43% <0%> (-0.94%)` | `24% <0%> (ø)` | |
   | [.../org/apache/gobblin/cluster/GobblinTaskRunner.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2681/diff?src=pr&el=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpblRhc2tSdW5uZXIuamF2YQ==) | `65.72% <0%> (+0.93%)` | `29% <0%> (ø)` | :arrow_down: |
   | [.../limiter/RedirectAwareRestClientRequestSender.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2681/diff?src=pr&el=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi10aHJvdHRsaW5nLXNlcnZpY2UvZ29iYmxpbi10aHJvdHRsaW5nLXNlcnZpY2UtY2xpZW50L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3V0aWwvbGltaXRlci9SZWRpcmVjdEF3YXJlUmVzdENsaWVudFJlcXVlc3RTZW5kZXIuamF2YQ==) | `87.2% <0%> (+2.32%)` | `18% <0%> (+1%)` | :arrow_up: |
   | ... and [2 more](https://codecov.io/gh/apache/incubator-gobblin/pull/2681/diff?src=pr&el=tree-more) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2681?src=pr&el=continue).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2681?src=pr&el=footer). Last update [bee3933...5103293](https://codecov.io/gh/apache/incubator-gobblin/pull/2681?src=pr&el=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 the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Issue Time Tracking
-------------------

    Worklog Id:     (was: 281186)
    Time Spent: 20m  (was: 10m)

> Implement a workaround for Helix Workflow being stuck in STOPPING state
> -----------------------------------------------------------------------
>
>                 Key: GOBBLIN-817
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-817
>             Project: Apache Gobblin
>          Issue Type: Improvement
>          Components: gobblin-service
>    Affects Versions: 0.15.0
>            Reporter: Sudarshan Vasudevan
>            Assignee: Abhishek Tiwari
>            Priority: Major
>             Fix For: 0.15.0
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Helix task cancellation can cause a workflow to be indefinitely stuck in the STOPPING state, if any of the tasks in the workflow ignores the cancel request. The Helix bug is tracked in [HELIX-1933|https://jira01.corp.linkedin.com:8443/browse/HELIX-1933]. This temporary workaround deletes the Helix workflow after a predefined time interval in the STOPPING state.  



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)