pulsar-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] merlimat commented on issue #3572: Configure BK memory sizes automatically from JVM parameters
Date Thu, 14 Feb 2019 14:20:43 GMT
merlimat commented on issue #3572: Configure BK memory sizes automatically from JVM parameters
URL: https://github.com/apache/pulsar/pull/3572#issuecomment-463644150
 
 
   The problem is that DbLedgerStorage is failing when the config key is present but empty:

   
   ```
   14:50:13.375 [main] ERROR org.apache.bookkeeper.server.Main - Failed to build bookie server
   org.apache.commons.configuration.ConversionException: 'dbStorage_writeCacheMaxSizeMb' doesn't
map to a Long object
   	at org.apache.commons.configuration.AbstractConfiguration.getLong(AbstractConfiguration.java:884)
~[commons-configuration-commons-configuration-1.6.jar:1.6]
   	at org.apache.commons.configuration.AbstractConfiguration.getLong(AbstractConfiguration.java:865)
~[commons-configuration-commons-configuration-1.6.jar:1.6]
   	at org.apache.bookkeeper.bookie.storage.ldb.DbLedgerStorage.initialize(DbLedgerStorage.java:97)
~[org.apache.bookkeeper-bookkeeper-server-4.9.0.jar:4.9.0]
   	at org.apache.bookkeeper.bookie.Bookie.<init>(Bookie.java:790) ~[org.apache.bookkeeper-bookkeeper-server-4.9.0.jar:4.9.0]
   	at org.apache.bookkeeper.proto.BookieServer.newBookie(BookieServer.java:137) ~[org.apache.bookkeeper-bookkeeper-server-4.9.0.jar:4.9.0]
   	at org.apache.bookkeeper.proto.BookieServer.<init>(BookieServer.java:106) ~[org.apache.bookkeeper-bookkeeper-server-4.9.0.jar:4.9.0]
   	at org.apache.bookkeeper.server.service.BookieService.<init>(BookieService.java:43)
~[org.apache.bookkeeper-bookkeeper-server-4.9.0.jar:4.9.0]
   	at org.apache.bookkeeper.server.Main.buildBookieServer(Main.java:301) ~[org.apache.bookkeeper-bookkeeper-server-4.9.0.jar:4.9.0]
   	at org.apache.bookkeeper.server.Main.doMain(Main.java:221) [org.apache.bookkeeper-bookkeeper-server-4.9.0.jar:4.9.0]
   	at org.apache.bookkeeper.server.Main.main(Main.java:203) [org.apache.bookkeeper-bookkeeper-server-4.9.0.jar:4.9.0]
   	at org.apache.bookkeeper.proto.BookieServer.main(BookieServer.java:314) [org.apache.bookkeeper-bookkeeper-server-4.9.0.jar:4.9.0]
   Caused by: org.apache.commons.configuration.ConversionException: Could not convert  to
java.lang.Long
   	at org.apache.commons.configuration.PropertyConverter.toNumber(PropertyConverter.java:413)
~[commons-configuration-commons-configuration-1.6.jar:1.6]
   	at org.apache.commons.configuration.PropertyConverter.toLong(PropertyConverter.java:280)
~[commons-configuration-commons-configuration-1.6.jar:1.6]
   	at org.apache.commons.configuration.AbstractConfiguration.getLong(AbstractConfiguration.java:880)
~[commons-configuration-commons-configuration-1.6.jar:1.6]
   	... 10 more
   ```
   
   We want the key to be present (and empty) in the config file so that we can easily override
in Docker environment.
   
   Moving this to 2.3.1 since it will require a small change in BK side 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message