[
https://issues.apache.org/jira/browse/GOBBLIN-963?focusedWorklogId=346194&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-346194
]
ASF GitHub Bot logged work on GOBBLIN-963:
------------------------------------------
Author: ASF GitHub Bot
Created on: 19/Nov/19 19:19
Start Date: 19/Nov/19 19:19
Worklog Time Spent: 10m
Work Description: codecov-io commented on issue #2818: [GOBBLIN-963]
Remove duplicated copies of TaskContext/TaskState when constructing
TaskIFaceWrapper
URL:
https://github.com/apache/incubator-gobblin/pull/2818#issuecomment-555301108
#
[Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2818?src=pr&el=h1)
Report
> Merging
[#2818](https://codecov.io/gh/apache/incubator-gobblin/pull/2818?src=pr&el=desc)
into
[master](https://codecov.io/gh/apache/incubator-gobblin/commit/e24bc26a28434bfee9ea4926e1b7d59901e63ec8?src=pr&el=desc)
will **decrease** coverage by `<.01%`.
> The diff coverage is `77.77%`.
[](https://codecov.io/gh/apache/incubator-gobblin/pull/2818?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #2818 +/- ##
============================================
- Coverage 45.56% 45.56% -0.01%
+ Complexity 8954 8952 -2
============================================
Files 1900 1900
Lines 71213 71214 +1
Branches 7856 7856
============================================
- Hits 32451 32448 -3
+ Misses 35780 35779 -1
- Partials 2982 2987 +5
```
| [Impacted
Files](https://codecov.io/gh/apache/incubator-gobblin/pull/2818?src=pr&el=tree)
| Coverage Δ | Complexity Δ | |
|---|---|---|---|
|
[...pache/gobblin/runtime/GobblinMultiTaskAttempt.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2818/diff?src=pr&el=tree#diff-Z29iYmxpbi1ydW50aW1lL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3J1bnRpbWUvR29iYmxpbk11bHRpVGFza0F0dGVtcHQuamF2YQ==)
| `56.3% <77.77%> (ø)` | `27 <0> (ø)` | :arrow_down: |
|
[...lin/util/filesystem/FileSystemInstrumentation.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2818/diff?src=pr&el=tree#diff-Z29iYmxpbi11dGlsaXR5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3V0aWwvZmlsZXN5c3RlbS9GaWxlU3lzdGVtSW5zdHJ1bWVudGF0aW9uLmphdmE=)
| `92.85% <0%> (-7.15%)` | `4% <0%> (ø)` | |
|
[...ava/org/apache/gobblin/fsm/FiniteStateMachine.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2818/diff?src=pr&el=tree#diff-Z29iYmxpbi11dGlsaXR5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2ZzbS9GaW5pdGVTdGF0ZU1hY2hpbmUuamF2YQ==)
| `73.48% <0%> (-3.04%)` | `18% <0%> (-3%)` | |
|
[...lin/elasticsearch/writer/FutureCallbackHolder.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2818/diff?src=pr&el=tree#diff-Z29iYmxpbi1tb2R1bGVzL2dvYmJsaW4tZWxhc3RpY3NlYXJjaC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvZ29iYmxpbi9lbGFzdGljc2VhcmNoL3dyaXRlci9GdXR1cmVDYWxsYmFja0hvbGRlci5qYXZh)
| `61.42% <0%> (-1.43%)` | `4% <0%> (ø)` | |
|
[.../org/apache/gobblin/cluster/GobblinTaskRunner.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2818/diff?src=pr&el=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvR29iYmxpblRhc2tSdW5uZXIuamF2YQ==)
| `64.35% <0%> (-0.47%)` | `27% <0%> (-1%)` | |
|
[...va/org/apache/gobblin/metrics/event/EventName.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2818/diff?src=pr&el=tree#diff-Z29iYmxpbi1tZXRyaWNzLWxpYnMvZ29iYmxpbi1tZXRyaWNzLWJhc2Uvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2dvYmJsaW4vbWV0cmljcy9ldmVudC9FdmVudE5hbWUuamF2YQ==)
| `0% <0%> (ø)` | `0% <0%> (ø)` | :arrow_down: |
|
[...main/java/org/apache/gobblin/util/HadoopUtils.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2818/diff?src=pr&el=tree#diff-Z29iYmxpbi11dGlsaXR5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3V0aWwvSGFkb29wVXRpbHMuamF2YQ==)
| `30.53% <0%> (+0.33%)` | `24% <0%> (ø)` | :arrow_down: |
|
[...e/gobblin/runtime/locks/ZookeeperBasedJobLock.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2818/diff?src=pr&el=tree#diff-Z29iYmxpbi1ydW50aW1lL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3J1bnRpbWUvbG9ja3MvWm9va2VlcGVyQmFzZWRKb2JMb2NrLmphdmE=)
| `64.44% <0%> (+1.11%)` | `16% <0%> (+1%)` | :arrow_up: |
|
[...lin/restli/throttling/ZookeeperLeaderElection.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2818/diff?src=pr&el=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi10aHJvdHRsaW5nLXNlcnZpY2UvZ29iYmxpbi10aHJvdHRsaW5nLXNlcnZpY2Utc2VydmVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3Jlc3RsaS90aHJvdHRsaW5nL1pvb2tlZXBlckxlYWRlckVsZWN0aW9uLmphdmE=)
| `72.22% <0%> (+2.22%)` | `13% <0%> (ø)` | :arrow_down: |
------
[Continue to review full report at
Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2818?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/2818?src=pr&el=footer).
Last update
[e24bc26...ccf5c45](https://codecov.io/gh/apache/incubator-gobblin/pull/2818?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:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 346194)
Time Spent: 1h (was: 50m)
> Inconsistent TaskContext/TaskState created for TaskIFaceWrapper in
> GobblinMultiTaskAttempt
> ------------------------------------------------------------------------------------------
>
> Key: GOBBLIN-963
> URL: https://issues.apache.org/jira/browse/GOBBLIN-963
> Project: Apache Gobblin
> Issue Type: Bug
> Reporter: Chen Guo
> Priority: Major
> Time Spent: 1h
> Remaining Estimate: 0h
>
> Currently in GobblinMultiTaskAttempt, when we call createTaskRunnable, we
> rely on a TaskFactory to create a task and return a wrapper for that task.
> However, when we create the wrapper, we initialized the TaskContext twice.
> # For the task creation
> # For the task wrapper creation
> These two TaskContexts, which generate two copies of TaskState, will cause
> inconsistency in the following operation. The task state update within the
> task won't be reflected in TaskWrapper.
>
> Specifically, this is the buggy line of code
> {code:java}
> return new TaskIFaceWrapper(taskFactoryOpt.get().createTask(new
> TaskContext(workUnitState)),
> new TaskContext(workUnitState), countDownLatch, this.taskStateTracker);
> {code}
> Ideally, we should create the TaskContext once, and reuse the same
> TaskContext.
>
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
