You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/05/25 20:11:04 UTC
[GitHub] [pulsar] lhotari commented on pull request #10670: [Issue 10669] Add Pulsar_GC_LOG when start broker with bin/pulsar
lhotari commented on pull request #10670:
URL: https://github.com/apache/pulsar/pull/10670#issuecomment-848226916
Now that we are running on Java 11 for the Docker image in master branch, there's a required change related to `PULSAR_GC_LOG` . It's not currently compatible with Java 11. The logic in `conf/pulsar_env.sh` to set PULSAR_GC_LOG currently handles only Java 8:
https://github.com/apache/pulsar/blob/8d3c0df33e6589bf1ab8cc427409c7d222509237/conf/pulsar_env.sh#L50-L58
there's also similar logic in `bin/function-localrunner` so it probably makes sense to fix that too:
https://github.com/apache/pulsar/blob/c6158e3d3e0e215e0bade91a557dc4bdb63cc297/bin/function-localrunner#L31-L39
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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