-
Notifications
You must be signed in to change notification settings - Fork 895
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
Add new api resumeCompaction and suspendCompaction #3509
Conversation
5d32cea
to
2efd7db
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job.
We need to standardize naming for the API to not have two names for the same functionality.
Also needs documentation, see https://bookkeeper.apache.org/docs/admin/http
bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/InterleavedLedgerStorage.java
Outdated
Show resolved
Hide resolved
bookkeeper-http/http-server/src/main/java/org/apache/bookkeeper/http/HttpRouter.java
Outdated
Show resolved
Hide resolved
62bdf76
to
c07f653
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@dlg99 PTAL again, thx |
api docs added |
rerun failure checks |
bookkeeper-http/http-server/src/main/java/org/apache/bookkeeper/http/HttpRouter.java
Outdated
Show resolved
Hide resolved
09db963
to
3a4c407
Compare
…and suspend compaction through api
9da6feb
to
4201ec7
Compare
@hangc0276 PTAL again @dlg99 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
…and suspend compaction through api (apache#3509) ### Motivation Compaction would bring high disk io util, It would better allowing to control suspending and resuming compaction through api ### Changes Add two apis to control resuming and suspending compaction. `/api/v1/bookie/gc/suspend_compaction` `/api/v1/bookie/gc/resume_compaction` (cherry picked from commit e169cbd)
…and suspend compaction through api (apache#3509) ### Motivation Compaction would bring high disk io util, It would better allowing to control suspending and resuming compaction through api ### Changes Add two apis to control resuming and suspending compaction. `/api/v1/bookie/gc/suspend_compaction` `/api/v1/bookie/gc/resume_compaction`
Motivation
Compaction would bring high disk io util, It would better allowing to control suspending and resuming compaction through api
Changes
Add two apis to control resuming and suspending compaction.
/api/v1/bookie/gc/suspend_compaction
/api/v1/bookie/gc/resume_compaction