The MetricGroup interface contains methods to create groups and metrics using an int as a name. The original intention was to allow pattern like |group.addGroup("subtaskIndex").addGroup(0)| , but this didn't really work out, with |addGroup(String, String)|  serving this use case much better.

Metric methods accept an int mostly for consistency, but there's no good use-case for it.

These methods also offer hardly any convenience since all they do is save potential users from using |String.valueOf| on one argument. That's doesn't seem valuable enough for something that doubles the size of the interface.

I propose to remove said method.


https://cwiki.apache.org/confluence/display/FLINK/FLIP-341%3A+Remove+MetricGroup+methods+accepting+an+int+as+a+name

Reply via email to