narendly commented on a change in pull request #1199:
URL: https://github.com/apache/helix/pull/1199#discussion_r463906344
##########
File path: helix-core/src/main/java/org/apache/helix/InstanceType.java
##########
@@ -22,7 +22,7 @@
import java.util.Arrays;
import java.util.List;
-import org.apache.helix.monitoring.mbeans.MonitorDomainNames;
+import org.apache.helix.monitoring.common.mbeans.MonitorDomainNames;
Review comment:
Why don't we just remove the conflicting classes from helix-core? That
is what my 2nd paragraph is saying. If you just change the package structure
for the conflicting classes in metrics-common, you'd be masking the problem
(the duality still exists) instead of solving the problem.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
