[GitHub] sijie commented on issue #1094: BP-27 (part 1): New BookKeeper CLI Skeleton

2018-02-16 Thread GitBox
sijie commented on issue #1094: BP-27 (part 1): New BookKeeper CLI Skeleton URL: https://github.com/apache/bookkeeper/pull/1094#issuecomment-366300264 all CI are green except "integration tests". since this PR is about a new CLI tool, it is irrelative to integration tests. going to ignore

[GitHub] sijie commented on issue #1094: BP-27 (part 1): New BookKeeper CLI Skeleton

2018-02-16 Thread GitBox
sijie commented on issue #1094: BP-27 (part 1): New BookKeeper CLI Skeleton URL: https://github.com/apache/bookkeeper/pull/1094#issuecomment-366300303 IGNORE CI This is an automated message from the Apache Git Service. To

[GitHub] sijie commented on issue #1094: BP-27 (part 1): New BookKeeper CLI Skeleton

2018-02-16 Thread GitBox
sijie commented on issue #1094: BP-27 (part 1): New BookKeeper CLI Skeleton URL: https://github.com/apache/bookkeeper/pull/1094#issuecomment-366291019 retrigger the travis CI This is an automated message from the Apache Git

[GitHub] sijie commented on issue #1094: BP-27 (part 1): New BookKeeper CLI Skeleton

2018-02-15 Thread GitBox
sijie commented on issue #1094: BP-27 (part 1): New BookKeeper CLI Skeleton URL: https://github.com/apache/bookkeeper/pull/1094#issuecomment-366139229 @reddycharan good point about "-". Created an issue #1153 for finalizing the new bookkeeper-cli name.

[GitHub] sijie commented on issue #1094: BP-27 (part 1): New BookKeeper CLI Skeleton

2018-02-15 Thread GitBox
sijie commented on issue #1094: BP-27 (part 1): New BookKeeper CLI Skeleton URL: https://github.com/apache/bookkeeper/pull/1094#issuecomment-366109515 > People make all kind of mistakes with usage of '-' in the script file name/command name. what are the mistakes of using "-" in

[GitHub] sijie commented on issue #1094: BP-27 (part 1): New BookKeeper CLI Skeleton

2018-02-14 Thread GitBox
sijie commented on issue #1094: BP-27 (part 1): New BookKeeper CLI Skeleton URL: https://github.com/apache/bookkeeper/pull/1094#issuecomment-365808654 @reddycharan updated this PR to use "bookkeeper-cli" instead of "bookie-shell" in the command output, to avoid confusion.