[KAFKA-10092] Remove unnecessary enum modifier in NioEchoServer

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 2.8.0
• Labels: newbie
• Assignee: Andrey Falko
• Reporter: Jakob Homan
• Created: 03/Jun/20 20:47
• Updated: 26/Oct/20 05:08
• Resolved: 26/Oct/20 05:08

Description

n.b. This is a newbie ticket designed to be an introduction to contributing for the assignee.

In NioEchoServer the enum has its constructor declared as private, which is redundant. We can remove this.

public class NioEchoServer extends Thread {
    public enum MetricType {
        TOTAL, RATE, AVG, MAX;

        private final String metricNameSuffix;

        private MetricType() {
            metricNameSuffix = "-" + name().toLowerCase(Locale.ROOT);
        }}} 

Issue Links

• links to: GitHub Pull Request #8794
• links to: GitHub Pull Request #9195