Skip to content

Conversation

@mykhailo-kuchma
Copy link
Contributor

TaskContext should not trigger cancellation after request was
completed. This could happen when the execution function keeps
the last owning reference to the object that cancels task context
in destruction.

Resolves: OLPSUP-10456

Signed-off-by: Mykhailo Kuchma ext-mykhailo.kuchma@here.com

TaskContext should not trigger cancellation after request was
completed. This could happen when the execution function keeps
the last owning reference to the object that cancels task context
in destruction.

Resolves: OLPSUP-10456

Signed-off-by: Mykhailo Kuchma <ext-mykhailo.kuchma@here.com>
@codecov
Copy link

codecov bot commented May 12, 2020

Codecov Report

Merging #833 into master will increase coverage by 0.0%.
The diff coverage is 100.0%.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #833   +/-   ##
======================================
  Coverage    78.7%   78.7%           
======================================
  Files         292     292           
  Lines        9944    9945    +1     
======================================
+ Hits         7823    7826    +3     
+ Misses       2121    2119    -2     
Impacted Files Coverage Δ
...cpp-sdk-core/include/olp/core/client/TaskContext.h 100.0% <100.0%> (ø)
...re/include/olp/core/client/CancellationContext.inl 92.6% <0.0%> (+7.4%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f8437a8...3944e1a. Read the comment docs.

@mykhailo-kuchma mykhailo-kuchma merged commit 311adc8 into master May 13, 2020
@mykhailo-kuchma mykhailo-kuchma deleted the bug/OLPSUP-10456 branch May 13, 2020 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants