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

Some play with suspendable transactions etc #152

Conversation

@kennethshackleton
Copy link
Member Author

kennethshackleton commented Jul 18, 2021

Screenshot 2021-07-18 at 09 37 34

@kennethshackleton kennethshackleton force-pushed the suspend-transactions branch 2 times, most recently from 485ed0a to 5564746 Compare July 18, 2021 08:46
@codecov
Copy link

codecov bot commented Jul 18, 2021

Codecov Report

Merging #152 (0924162) into main (3608e83) will decrease coverage by 0.33%.
The diff coverage is 87.50%.

❗ Current head 0924162 differs from pull request most recent head 85f9f10. Consider uploading reports for the commit 85f9f10 to get more accurate results
Impacted file tree graph

@@             Coverage Diff              @@
##               main     #152      +/-   ##
============================================
- Coverage     97.17%   96.84%   -0.34%     
+ Complexity      366      365       -1     
============================================
  Files            49       49              
  Lines          1877     1931      +54     
  Branches        257      261       +4     
============================================
+ Hits           1824     1870      +46     
- Misses           30       35       +5     
- Partials         23       26       +3     
Impacted Files Coverage Δ
...rg/selekt/android/support/SupportSQLiteDatabase.kt 100.00% <ø> (ø)
...lin/com/bloomberg/selekt/android/SQLiteDatabase.kt 85.59% <80.00%> (-0.25%) ⬇️
.../src/main/kotlin/com/bloomberg/selekt/Databases.kt 95.56% <87.17%> (-1.55%) ⬇️
...b/src/main/kotlin/com/bloomberg/selekt/Sessions.kt 98.24% <90.00%> (-1.76%) ⬇️
...rc/main/kotlin/com/bloomberg/selekt/pools/Mutex.kt 86.20% <0.00%> (-1.73%) ⬇️

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 3608e83...85f9f10. Read the comment docs.

@kennethshackleton kennethshackleton force-pushed the suspend-transactions branch 2 times, most recently from ecd777c to 10d6ae2 Compare July 18, 2021 19:09
@kennethshackleton kennethshackleton changed the title Intrinsic approach to suspendable transactions Initial suspendable transactions Jul 18, 2021
@kennethshackleton kennethshackleton added the question Further information is requested label Jul 18, 2021
@kennethshackleton kennethshackleton force-pushed the suspend-transactions branch 20 times, most recently from deff627 to bd94683 Compare July 19, 2021 18:11
@kennethshackleton kennethshackleton force-pushed the suspend-transactions branch 10 times, most recently from 005bb7b to c7af979 Compare July 21, 2021 19:53
@kennethshackleton kennethshackleton deleted the suspend-transactions branch July 22, 2021 08:15
@kennethshackleton kennethshackleton changed the title Initial suspendable transactions Failed play with suspendable transactions etc Jul 25, 2021
@kennethshackleton kennethshackleton changed the title Failed play with suspendable transactions etc Some play with suspendable transactions etc Jul 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant