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:13:16 UTC

[GitHub] [pulsar] lhotari removed a comment on pull request #10670: [Issue 10669] Add Pulsar_GC_LOG when start broker with bin/pulsar

lhotari removed a comment 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