Skip to content
This repository has been archived by the owner on Dec 1, 2023. It is now read-only.

expose versions field to support pruning #2

Closed
wants to merge 1 commit into from
Closed

expose versions field to support pruning #2

wants to merge 1 commit into from

Conversation

ackratos
Copy link
Contributor

@HaoyangLiu
Copy link
Contributor

Now deleteVersionsFrom has a defect. Suppose you want to prune version from 100 to 10000, if one of the versions in this range has been deleted already. Then this delete operation will be aborted.
I have fixed this issue on irisnet iavl
Does our application need to handle this situation?

@darren-liu
Copy link

Now deleteVersionsFrom has a defect. Suppose you want to prune version from 100 to 10000, if one of the versions in this range has been deleted already. Then this delete operation will be aborted.
I have fixed this issue on irisnet iavl
Does our application need to handle this situation?
@HaoyangLiu , can you please cherry-pick the change here?

@HaoyangLiu
Copy link
Contributor

HaoyangLiu commented Mar 26, 2019 •
edited
Loading

OK, I have cherried pick this change and the releated PR is #5

@unclezoro unclezoro closed this Dec 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@darren-liu darren-liu darren-liu approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ackratos @HaoyangLiu @darren-liu @unclezoro