apache / apache/bookkeeper

Bookkeeper docker script does not support passing multiple ledger&journal files

Open
#2,168 0 comments 0 reactions 0 assignees View on GitHub
area/docker triage/week-4 type/bug
Dominant language
Java
Stars
2k
Forks
976
Avg merge
6d 15h
Merged PRs (30d)
7

Description

**BUG REPORT**

***Describe the bug***

The config [here](https://github.com/apache/bookkeeper/blob/master/conf/bk_server.conf#L368) says Bookkeeper supports multiple ledger files. However, it won't work properly if we specify something like `BK_ledgerDirectories=/tmp/bk-ledgers1,/tmp/bk-ledgers2` as the script [here](https://github.com/apache/bookkeeper/blob/master/docker/scripts/common.sh#L79) seems not supporting such format. Also, [here](https://github.com/apache/bookkeeper/blob/master/conf/bk_server.conf#L279) says it is preferable to use `journalDirectories`, but script above is still using `journalDirectory`.

***Expected behavior***

The docker script should support passing multiple ledger&journal files.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.