Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added TransactionRetentionOnFailureTest #55

Merged
merged 1 commit into from
Aug 16, 2019
Merged

Conversation

kptfh
Copy link
Collaborator

@kptfh kptfh commented Aug 16, 2019

Added TransactionRetentionOnFailureTest that checks whether transaction:

  • removed on failed lock;
  • retains on failed mutation

…on: - removed on failed lock; - retains on failed mutation
@codecov-io
Copy link

Codecov Report

Merging #55 into develop will increase coverage by 0.13%.
The diff coverage is 85.71%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop      #55      +/-   ##
=============================================
+ Coverage      86.68%   86.81%   +0.13%     
- Complexity       218      219       +1     
=============================================
  Files             23       23              
  Lines            766      766              
  Branches          68       68              
=============================================
+ Hits             664      665       +1     
  Misses            67       67              
+ Partials          35       34       -1
Impacted Files Coverage Δ Complexity Δ
...nusgraph/aerospike/operations/BasicOperations.java 97.61% <ø> (ø) 21 <0> (ø) ⬇️
...ph/aerospike/operations/BasicMutateOperations.java 100% <ø> (ø) 15 <0> (ø) ⬇️
...raph/aerospike/operations/BasicLockOperations.java 85.1% <100%> (ø) 37 <0> (ø) ⬇️
.../aerospike/transaction/WriteAheadLogCompleter.java 78.65% <80%> (ø) 15 <0> (ø) ⬇️
...ka/janusgraph/aerospike/AerospikeStoreManager.java 83.33% <0%> (+1.66%) 17% <0%> (+1%) ⬆️

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 bcea5c5...0c98601. Read the comment docs.

@kptfh kptfh merged commit fafb107 into develop Aug 16, 2019
@kptfh kptfh deleted the feature/test-added branch April 21, 2020 07:42
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.

2 participants