This is an automated email from the ASF dual-hosted git repository.

reddycharan pushed a commit to branch branch-4.9
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git


The following commit(s) were added to refs/heads/branch-4.9 by this push:
     new ef7ba35  ISSUE #2075: Bookieshell lastmark command isn't functional, 
always returning 0-0
ef7ba35 is described below

commit ef7ba357ae73cbebd5d2c749531cc2aa766e0fc5
Author: karanmehta93 <[email protected]>
AuthorDate: Thu May 2 12:38:31 2019 -0700

    ISSUE #2075: Bookieshell lastmark command isn't functional, always 
returning 0-0
    
    Descriptions of the changes in this PR:
    
    
    
    ### Motivation
    
    `lastmark` shell command is to be used while debugging however currently it 
cannot be used since it returns incorrect value.
    
    ### Changes
    
    Updated `LedgerDirsManager` object initialization to use correct 
`ledgerDirs`.
    Master Issue: #2075
    
    
    
    
    Reviewers: Charan Reddy Guttapalem <[email protected]>, Sijie Guo 
<[email protected]>
    
    This closes #2076 from karanmehta93/ISSUE-2075, closes #2075
    
    (cherry picked from commit 707ae5c8536f967c0369f951d6178670c5dd031d)
    Signed-off-by: Charan Reddy Guttapalem <[email protected]>
---
 .../apache/bookkeeper/tools/cli/commands/bookie/LastMarkCommand.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/tools/cli/commands/bookie/LastMarkCommand.java
 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/tools/cli/commands/bookie/LastMarkCommand.java
index 61c1f92..f925e13 100644
--- 
a/bookkeeper-server/src/main/java/org/apache/bookkeeper/tools/cli/commands/bookie/LastMarkCommand.java
+++ 
b/bookkeeper-server/src/main/java/org/apache/bookkeeper/tools/cli/commands/bookie/LastMarkCommand.java
@@ -48,7 +48,7 @@ public class LastMarkCommand extends BookieCommand<CliFlags> {
     @Override
     public boolean apply(ServerConfiguration conf, CliFlags flags) {
         LedgerDirsManager dirsManager = new LedgerDirsManager(
-            conf, conf.getJournalDirs(),
+            conf, conf.getLedgerDirs(),
             new DiskChecker(conf.getDiskUsageThreshold(), 
conf.getDiskUsageWarnThreshold()));
         File[] journalDirs = conf.getJournalDirs();
 

Reply via email to