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

Runtime: Add delete-if-exists check for the no longer used indexer.sqlite file… #5531

Merged
merged 3 commits into from
Jul 10, 2023

Conversation

gmalouf
Copy link
Contributor

@gmalouf gmalouf commented Jul 7, 2023

Summary

Add delete-if-exists check for the no longer used indexer.sqlite file in genesis data directory.

Test Plan

Ran node locally with indexer.sqlite file present, not present, and initial condition with data directory not created yet.

@gmalouf gmalouf changed the title [Runtime] Add delete-if-exists check for the no longer used indexer.sqlite file… Runtime: Add delete-if-exists check for the no longer used indexer.sqlite file… Jul 7, 2023
@codecov
Copy link

codecov bot commented Jul 7, 2023

Codecov Report

Merging #5531 (9bc0036) into master (3e80027) will decrease coverage by 0.73%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #5531      +/-   ##
==========================================
- Coverage   55.78%   55.06%   -0.73%     
==========================================
  Files         446      446              
  Lines       63262    63272      +10     
==========================================
- Hits        35293    34841     -452     
- Misses      25590    26042     +452     
- Partials     2379     2389      +10     
Impacted Files Coverage Δ
cmd/algod/main.go 0.00% <0.00%> (ø)

... and 55 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@gmalouf gmalouf marked this pull request as ready for review July 7, 2023 19:10
winder
winder previously approved these changes Jul 7, 2023
Copy link
Contributor

@winder winder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One typo, one nit, otherwise LGTM

cmd/algod/main.go Outdated Show resolved Hide resolved
cmd/algod/main.go Outdated Show resolved Hide resolved
cmd/algod/main.go Outdated Show resolved Hide resolved
cmd/algod/main.go Outdated Show resolved Hide resolved
@onetechnical
Copy link
Contributor

Still not sure if this is a good idea to bake in, or if we should just let legacy users manage with a release note. Seems like it would affect a small number of users.

@gmalouf gmalouf requested a review from onetechnical July 10, 2023 16:28
@gmalouf gmalouf merged commit 217f791 into algorand:master Jul 10, 2023
@gmalouf gmalouf deleted the delete-indexer-sqlite branch July 10, 2023 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants