ivankelly commented on a change in pull request #1850: Ship git info in jar and
print at boot
URL: https://github.com/apache/incubator-pulsar/pull/1850#discussion_r191717201
##########
File path:
pulsar-broker/src/main/java/org/apache/pulsar/broker/PulsarService.java
##########
@@ -308,6 +308,13 @@ public ServiceConfiguration getConfiguration() {
public void start() throws PulsarServerException {
mutex.lock();
+ LOG.info("Starting Pulsar Broker service; version: '{}'", (
brokerVersion != null ? brokerVersion : "unknown" ) );
+ LOG.info("Git Revision {}",
PulsarBrokerVersionStringUtils.getGitSha());
+ LOG.info("Built by {} on {} at {}",
+ PulsarBrokerVersionStringUtils.getBuildUser(),
+ PulsarBrokerVersionStringUtils.getBuildHost(),
Review comment:
Personally, I think the information is useful. If we get a log from a user,
and we don't recognise the username, then it's pretty clear they've built it
themselves, so we should be careful about assuming things about the code. I can
remove it if you feel strongly about this though.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
