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

refactor: Remove the 3rd mode #456

Merged
merged 5 commits into from
Sep 1, 2023
Merged

refactor: Remove the 3rd mode #456

merged 5 commits into from
Sep 1, 2023

Conversation

bigboss2063
Copy link
Member

No description provided.

* refactor: remove all unit tests of 3rd mode

* refactor: remove all usage of 3rd mode

* refactor: remove all methods and objects about the 3rd mode

* refactor: remove all files about 3rd mode and remove in memory db

* style: remove useless code
* fix: fix incorrect use of error

* style: remove useless space
@codecov
Copy link

codecov bot commented Sep 1, 2023

Codecov Report

Merging #456 (84b63fa) into master (15267dd) will increase coverage by 5.97%.
The diff coverage is 74.87%.

@@            Coverage Diff             @@
##           master     #456      +/-   ##
==========================================
+ Coverage   66.62%   72.60%   +5.97%     
==========================================
  Files          34       31       -3     
  Lines        5897     4187    -1710     
==========================================
- Hits         3929     3040     -889     
+ Misses       1614      881     -733     
+ Partials      354      266      -88     
Files Changed Coverage Δ
btree.go 92.00% <ø> (ø)
index.go 67.74% <ø> (ø)
list.go 73.60% <ø> (ø)
options.go 78.66% <ø> (ø)
utils.go 60.00% <ø> (-4.00%) ⬇️
tx_list.go 72.15% <38.46%> (ø)
tx_btree.go 72.22% <72.22%> (ø)
db.go 67.40% <82.14%> (+2.50%) ⬆️
tx.go 80.31% <88.23%> (+10.46%) ⬆️
merge.go 70.17% <100.00%> (-0.52%) ⬇️
... and 3 more

... and 2 files with indirect coverage changes

# Conflicts:
#	db.go
#	merge.go
@bigboss2063 bigboss2063 merged commit 758ba2c into nutsdb:master Sep 1, 2023
6 checks passed
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.

None yet

1 participant