[bookkeeper] branch branch-4.10 updated: 2279 fix NPE when exec readjournal command

[si...@apache.org]

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

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


The following commit(s) were added to refs/heads/branch-4.10 by this push:
     new 7662819  2279 fix NPE when exec readjournal command
7662819 is described below

commit 7662819ebb1c7cfdf81165d5301bd47344edc6db
Author: Aloys <18...@163.com>
AuthorDate: Thu Mar 19 07:51:46 2020 +0800

    2279 fix NPE when exec readjournal command
    
    Descriptions of the changes in this PR:
    This PR fix NullPointerException when exec readjournal command
    See issue #2279
    
    
    Reviewers: Enrico Olivelli <eo...@gmail.com>, Sijie Guo <None>
    
    This closes #2280 from aloyszhang/read-jounral
    
    (cherry picked from commit bd3d3b5473f1b8d90d681d03a128c9332c80425d)
    Signed-off-by: Sijie Guo <si...@apache.org>
---
 .../src/main/java/org/apache/bookkeeper/bookie/BookieShell.java         | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieShell.java b/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieShell.java
index 28ef757..d7f5575 100644
--- a/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieShell.java
+++ b/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieShell.java
@@ -1013,7 +1013,7 @@ public class BookieShell implements Tool {
 
         ReadJournalCmd() {
             super(CMD_READJOURNAL);
-            rjOpts.addOption("dir", false, "Journal directory (needed if more than one journal configured)");
+            rjOpts.addOption("dir", true, "Journal directory (needed if more than one journal configured)");
             rjOpts.addOption("m", "msg", false, "Print message body");
         }