Skip to content
This repository was archived by the owner on May 25, 2021. It is now read-only.

Update to use new purge API #22

Open
wants to merge 1 commit into
base: COUCHDB-3326-clustered-purge
Choose a base branch
from
Open

Update to use new purge API #22

wants to merge 1 commit into from

Conversation

@jiangphcn
Copy link

@jiangphcn jiangphcn commented Mar 20, 2017 •
edited
Loading

COUCHDB-3326

@jiangphcn jiangphcn changed the title Return value change COUCHDB 3326 clustered purge Mar 20, 2017
rnewson
rnewson reviewed Mar 20, 2017
View reviewed changes
src/couch_index_updater.erl
StateAcc
end,
{ok, NewStateAcc} = couch_db:fold_purged_docs(
Db,
Copy link
Member

@rnewson rnewson Mar 20, 2017

Choose a reason for hiding this comment

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

odd whitespace change here, check your editor settings?

Copy link
Author

@jiangphcn jiangphcn Mar 20, 2017

Choose a reason for hiding this comment

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

Thanks, Bob. The format issue was fixed here intentionally to use same indent mode.

@jiangphcn jiangphcn force-pushed the COUCHDB-3326-clustered-purge branch from 6fbc319 to 0bad010 Compare March 29, 2017 04:23
@jiangphcn jiangphcn changed the title COUCHDB 3326 clustered purge Update to use new purge API Mar 29, 2017
@jiangphcn jiangphcn force-pushed the COUCHDB-3326-clustered-purge branch from 0bad010 to 329958e Compare March 29, 2017 05:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@rnewson rnewson rnewson left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jiangphcn @rnewson