search

Alluxio / Community

New Contributor Tasks for Alluxio
20 stars 38 forks source link

Fix logging message template in RocksStore #615

Closed dbw9580 closed 2 years ago

dbw9580 commented 2 years ago

https://github.com/Alluxio/alluxio/blob/99942ae847738c8ac4a2bace2228e4c117b5314c/core/server/master/src/main/java/alluxio/master/metastore/rocks/RocksStore.java#L105 https://github.com/Alluxio/alluxio/blob/99942ae847738c8ac4a2bace2228e4c117b5314c/core/server/master/src/main/java/alluxio/master/metastore/rocks/RocksStore.java#L221

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

Change 

LOG.info("Closed store at %s", mDbPath);

to 

LOG.info("Closed store at {}", mDbPath);
dbw9580 commented 2 years ago

@LuQQiu Please add labels for this task. It's new contributor level.