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 af24652  Enable storeSystemTimeAsLedgerUnderreplicatedMarkTime
af24652 is described below

commit af246529b0728087b1c65a50b919abe7d4b5103f
Author: Charan Reddy Guttapalem <[email protected]>
AuthorDate: Mon Sep 24 00:18:21 2018 -0700

    Enable storeSystemTimeAsLedgerUnderreplicatedMarkTime
    
    
    
    Descriptions of the changes in this PR:
    
    - Enable the Auditor to use system time as underreplicated ledger mark time.
    If this is enabled, Auditor will write a ctime field into the
    underreplicated ledger znode.
    
    
https://github.com/apache/bookkeeper/commit/f4094c4992d5b22630fa633085a00a9152b87ffe
 introduced feature to Record ctime for underreplicated ledger mark time.
    
    Author: Sijie Guo <[email protected]>
    Author: Qi Wang <[email protected]>
    Author: cguttapalem <[email protected]>
    Author: Charan Reddy Guttapalem <[email protected]>
    
    Reviewers: Sijie Guo <[email protected]>, Enrico Olivelli 
<[email protected]>
    
    This closes #1692 from reddycharan/enablectime
---
 .../src/main/java/org/apache/bookkeeper/conf/AbstractConfiguration.java | 2 +-
 conf/bk_server.conf                                                     | 2 +-
 site/_data/config/bk_server.yaml                                        | 1 +
 3 files changed, 3 insertions(+), 2 deletions(-)

diff --git 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/AbstractConfiguration.java
 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/AbstractConfiguration.java
index 666b377..a3c47ab 100644
--- 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/AbstractConfiguration.java
+++ 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/AbstractConfiguration.java
@@ -851,7 +851,7 @@ public abstract class AbstractConfiguration<T extends 
AbstractConfiguration>
      *         underreplicated ledger mark time.
      */
     public boolean getStoreSystemTimeAsLedgerUnderreplicatedMarkTime() {
-        return 
getBoolean(STORE_SYSTEMTIME_AS_LEDGER_UNDERREPLICATED_MARK_TIME, false);
+        return 
getBoolean(STORE_SYSTEMTIME_AS_LEDGER_UNDERREPLICATED_MARK_TIME, true);
     }
 
     /**
diff --git a/conf/bk_server.conf b/conf/bk_server.conf
index 93c12a1..97404fb 100755
--- a/conf/bk_server.conf
+++ b/conf/bk_server.conf
@@ -911,7 +911,7 @@ zkEnableSecurity=false
 
 # Enable the Auditor to use system time as underreplicated ledger mark time.
 # If this is enabled, Auditor will write a ctime field into the 
underreplicated ledger znode.
-# storeSystemTimeAsLedgerUnderreplicatedMarkTime=false
+# storeSystemTimeAsLedgerUnderreplicatedMarkTime=true
 
 #############################################################################
 ## Replication Worker settings
diff --git a/site/_data/config/bk_server.yaml b/site/_data/config/bk_server.yaml
index 991d6f1..b60bfbf 100644
--- a/site/_data/config/bk_server.yaml
+++ b/site/_data/config/bk_server.yaml
@@ -648,6 +648,7 @@ groups:
     default: 0
   - param: storeSystemTimeAsLedgerUnderreplicatedMarkTime
     description: Enable the Auditor to use system time as underreplicated 
ledger mark time. If this is enabled, Auditor will write a ctime field into the 
underreplicated ledger znode.
+    default: true
 
 - name: AutoRecovery replication worker settings
   params:

Reply via email to