search

Alluxio / Community

New Contributor Tasks for Alluxio
20 stars 38 forks source link

Fix logging message template in AlluxioMasterProcess #617

Closed dbw9580 closed 2 years ago

dbw9580 commented 2 years ago

The logging message template in AlluxioMasterProcess should use {} as a placeholder, instead of %s.

Change this line of code:

https://github.com/Alluxio/alluxio/blob/e0bd13efcf98c79ef7d7848a22a48fe0408c1680/core/server/master/src/main/java/alluxio/master/AlluxioMasterProcess.java#L194

    LOG.info("Starting all masters as: %s.", (isLeader) ? "leader" : "follower");

to

    LOG.info("Starting all masters as: {}.", (isLeader) ? "leader" : "follower");
xwzbupt commented 2 years ago

/assign @xwzbupt

dbw9580 commented 2 years ago

Fixed by https://github.com/Alluxio/alluxio/pull/14683