Skip to content

Commit

Permalink
[Client]Allow to override PULSAR_MEM settings via PULSAR_EXTRA_OPS (a…
Browse files Browse the repository at this point in the history
…pache#13381)

### Motivation

`pulsar-client` runs `conf/pulsar_tools_env.sh` to set `PULSAR_EXTRA_OPTS`. If `PULSAR_EXTRA_OPTS` are given in the command line and contain `-Xmx`, it will be overridden by `PULSAR_MEM`.

### Modifications

Exchange the order of applying env variables. First go the hardcoded PULSAR_MEM settings, then potentially the user settings.
  • Loading branch information
kijanowski authored Dec 23, 2021
1 parent 6c7dcc0 commit c9d8ecf
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion conf/pulsar_tools_env.sh
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ PULSAR_MEM="-Xmx128m -XX:MaxDirectMemorySize=128m"
PULSAR_GC=" -client "

# Extra options to be passed to the jvm
PULSAR_EXTRA_OPTS="${PULSAR_EXTRA_OPTS} ${PULSAR_MEM} ${PULSAR_GC} ${PULSAR_GC_LOG} -Dio.netty.leakDetectionLevel=disabled"
PULSAR_EXTRA_OPTS="${PULSAR_MEM} ${PULSAR_GC} ${PULSAR_GC_LOG} -Dio.netty.leakDetectionLevel=disabled ${PULSAR_EXTRA_OPTS}"

# Add extra paths to the bookkeeper classpath
# PULSAR_EXTRA_CLASSPATH=
Expand Down

0 comments on commit c9d8ecf

Please sign in to comment.