grssam commented on code in PR #23120: URL: https://github.com/apache/pulsar/pull/23120#discussion_r1736164657
########## pulsar-broker/src/test/java/org/apache/pulsar/broker/admin/AdminApi2Test.java: ########## @@ -22,6 +22,7 @@ import static org.apache.commons.lang3.StringUtils.isBlank; import static org.apache.pulsar.broker.BrokerTestUtil.newUniqueName; import static org.apache.pulsar.broker.resources.LoadBalanceResources.BUNDLE_DATA_BASE_PATH; +import static org.apache.pulsar.common.policies.data.NamespaceIsolationPolicyUnloadScope.*; Review Comment: Any reason why a star import is bad here? there can't be any conflict as the enum would only be usable in a place where it is expecting an enum of class `NamespaceIsolationPolicyUnloadScope`. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
