[
https://issues.apache.org/jira/browse/BOOKKEEPER-82?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13147187#comment-13147187
]
[email protected] commented on BOOKKEEPER-82:
---------------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2745/#review3130
-----------------------------------------------------------
Good job, Sijie. I only have a few minor comments. One important general
comment I have is that we need documentation for this feature. Ideally we
commit a patch for BOOKKEEPER-82 that includes doc changes. Otherwise, I'd like
to have a blocker jira to make sure that we don't have a release without the
doc changes.
http://svn.apache.org/repos/asf/zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Bookie.java
<https://reviews.apache.org/r/2745/#comment6943>
I think we can remove this todo, right?
http://svn.apache.org/repos/asf/zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Bookie.java
<https://reviews.apache.org/r/2745/#comment6941>
Typo on journal.
http://svn.apache.org/repos/asf/zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java
<https://reviews.apache.org/r/2745/#comment6942>
If there are corrupted entries, it is true that we can fetch copies from
other bookies, but I don't think we have a mechanism for doing it yet, so it
sounds safer to keep throwing an exception here for now, no?
- fpj
On 2011-11-07 17:46:11, Sijie Guo wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/2745/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2011-11-07 17:46:11)
bq.
bq.
bq. Review request for bookkeeper.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. implement journal rolling
bq.
bq. 1) remove journals whose id are less than marked journal id. since they
journals' data and index has been persisted to disk.
bq. 2) replay entires started from marked position in marked journal to speed
up bookie start up.
bq.
bq.
bq. This addresses bug BOOKKEEPER-82.
bq. http://issues.apache.org/jira/browse/BOOKKEEPER-82
bq.
bq.
bq. Diffs
bq. -----
bq.
bq.
http://svn.apache.org/repos/asf/zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Bookie.java
1198776
bq.
http://svn.apache.org/repos/asf/zookeeper/bookkeeper/trunk/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java
1198776
bq.
http://svn.apache.org/repos/asf/zookeeper/bookkeeper/trunk/bookkeeper-server/src/test/java/org/apache/bookkeeper/test/BookieJournalRollingTest.java
PRE-CREATION
bq.
bq. Diff: https://reviews.apache.org/r/2745/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq.
bq. Thanks,
bq.
bq. Sijie
bq.
bq.
> support journal rolling
> -----------------------
>
> Key: BOOKKEEPER-82
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-82
> Project: Bookkeeper
> Issue Type: Improvement
> Components: bookkeeper-server
> Affects Versions: 4.0.0
> Reporter: Sijie Guo
> Assignee: Sijie Guo
> Fix For: 4.0.0
>
> Attachments: bookkeeper-82.patch, bookkeeper-82.patch_v2
>
>
> now bookkeeper is writing a single journal file, so the journal file has no
> chance to be garbage collected and the disk space keeps growing.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
