This is an automated email from the ASF dual-hosted git repository.
sijie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git
The following commit(s) were added to refs/heads/master by this push:
new a512b73 BP-25: MovingChecksumToProto--Refactor the checksum part of
bookkeeper
a512b73 is described below
commit a512b737160e7d52ed46964018c51302b2408967
Author: Arvin <[email protected]>
AuthorDate: Wed Jan 24 11:03:34 2018 -0800
BP-25: MovingChecksumToProto--Refactor the checksum part of bookkeeper
Descriptions of the changes in this PR:
Current the checksum implementation is in client module while the checksum
semantic is more close to protocol. Moreover, moving the checksum
implementation to protocol will avoid server module's dependency to client
module when doing checksum in server side.
Master Issue: #1007
Author: Arvin <[email protected]>
Reviewers: Enrico Olivelli <[email protected]>, Sijie Guo
<[email protected]>, Venkateswararao Jujjuri <[email protected]>
This closes #1008 from ArvinDevel/BP_25_MovingChecksumToProto
---
site/bps/BP-25-MovingChecksumToProto.md | 27 +++++++++++++++++++++++++++
site/community/bookkeeper_proposals.md | 2 +-
2 files changed, 28 insertions(+), 1 deletion(-)
diff --git a/site/bps/BP-25-MovingChecksumToProto.md
b/site/bps/BP-25-MovingChecksumToProto.md
new file mode 100644
index 0000000..09f5019
--- /dev/null
+++ b/site/bps/BP-25-MovingChecksumToProto.md
@@ -0,0 +1,27 @@
+---
+title: "BP-25: MovingChecksumToProto--Refactor the checksum part of bookkeeper"
+issue: https://github.com/apache/bookkeeper/issues/1007
+state: "Accepted"
+release: "N/A"
+---
+
+### Motivation
+
+Current the checksum implementation is in client module while the checksum
semantic is more close to protocol. Moreover, moving the checksum
implementation to protocol will avoid server module's dependency to client
module when doing checksum in server side.
+
+### Public Interfaces
+
+An internal refactor not affecting public interfaces.
+
+### Proposed Changes
+
+Move the DigestManager and related classes to proto module
+
+### Compatibility, Deprecation, and Migration Plan
+N/A
+
+### Test Plan
+The original all tests should work as before.
+
+### Rejected Alternatives
+N/A
\ No newline at end of file
diff --git a/site/community/bookkeeper_proposals.md
b/site/community/bookkeeper_proposals.md
index a918f9a..2802c65 100644
--- a/site/community/bookkeeper_proposals.md
+++ b/site/community/bookkeeper_proposals.md
@@ -85,7 +85,7 @@ using Google Doc.
This section lists all the _bookkeeper proposals_ made to BookKeeper.
-*Next Proposal Number: 23*
+*Next Proposal Number: 26*
### Inprogress
--
To stop receiving notification emails like this one, please contact
[email protected].
