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

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

                Author: ASF GitHub Bot
            Created on: 02/Nov/23 19:16
            Start Date: 02/Nov/23 19:16
    Worklog Time Spent: 10m 
      Work Description: codecov-commenter commented on PR #3819:
URL: https://github.com/apache/gobblin/pull/3819#issuecomment-1791404337

   ## 
[Codecov](https://app.codecov.io/gh/apache/gobblin/pull/3819?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 Report
   > Merging 
[#3819](https://app.codecov.io/gh/apache/gobblin/pull/3819?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 (0327e06) into 
[master](https://app.codecov.io/gh/apache/gobblin/commit/c865b6a8e4628f9602671151a1a33c8a27dacdef?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 (c865b6a) will **decrease** coverage by `0.15%`.
   > The diff coverage is `50.00%`.
   
   ```diff
   @@             Coverage Diff              @@
   ##             master    #3819      +/-   ##
   ============================================
   - Coverage     47.66%   47.52%   -0.15%     
   + Complexity    11058    11021      -37     
   ============================================
     Files          2156     2156              
     Lines         85328    85332       +4     
     Branches       9485     9485              
   ============================================
   - Hits          40671    40553     -118     
   - Misses        40966    41082     +116     
   - Partials       3691     3697       +6     
   ```
   
   
   | 
[Files](https://app.codecov.io/gh/apache/gobblin/pull/3819?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | Coverage Δ | |
   |---|---|---|
   | 
[.../java/org/apache/gobblin/runtime/api/FlowSpec.java](https://app.codecov.io/gh/apache/gobblin/pull/3819?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-Z29iYmxpbi1ydW50aW1lL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3J1bnRpbWUvYXBpL0Zsb3dTcGVjLmphdmE=)
 | `43.75% <100.00%> (+5.21%)` | :arrow_up: |
   | 
[...ervice/monitoring/DagActionStoreChangeMonitor.java](https://app.codecov.io/gh/apache/gobblin/pull/3819?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-Z29iYmxpbi1zZXJ2aWNlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NlcnZpY2UvbW9uaXRvcmluZy9EYWdBY3Rpb25TdG9yZUNoYW5nZU1vbml0b3IuamF2YQ==)
 | `0.00% <0.00%> (ø)` | |
   
   ... and [20 files with indirect coverage 
changes](https://app.codecov.io/gh/apache/gobblin/pull/3819/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   
   :mega: We’re building smart automated test selection to slash your CI/CD 
build times. [Learn 
more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   




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

    Worklog Id:     (was: 888502)
    Time Spent: 40m  (was: 0.5h)

> Use Consist flowExecutionId Across Participants
> -----------------------------------------------
>
>                 Key: GOBBLIN-1948
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-1948
>             Project: Apache Gobblin
>          Issue Type: New Feature
>          Components: gobblin-service
>            Reporter: Urmi Mustafi
>            Assignee: Abhishek Tiwari
>            Priority: Major
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> A consistent flowExecutionId is used during lease arbitration between hosts 
> and that id is available to the DagActionStoreChangeMonitor but not used when 
> passing launch events to the Orchestrator for recompilation and eventual 
> execution. The unintended consequence of this is that a different 
> flowExecutionId will be used across each participant when the flow is 
> recompiled before passing to the DagManager. This messes up the job status of 
> the most recent flow execution of each flow as it appears there are N flow 
> execution Ids, 1 for each of the N hosts. Only one will be executed but the 
> other N-1 are stuck in "compiling" job status state.  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to