Hi,

Good job,
Utils should be used as a tool class and it is better not to involve business process.
Is it better to implement this part in the business process class?

• Hello, this is exclusive to net, other business is not involved, so I think it is better to achieve here,how do you think？

• Hello, this is exclusive to net, other business is not involved, so I think it is better to achieve here,how do you think？

Hi,

I think it is better not to add the isSpecifyNetworkInterface in findNetworkInterface of utils, but I find there is only one call of findNetworkInterface, so I don't think of a better place to add this at present.
I think we can think about the implement scheme again later.

I think this modify like this, the name is more better.

 private static boolean isSpecifyNetworkInterface(NetworkInterface networkInterface) {
        String preferredNetworkInterface = PropertyUtils.getString(DOLPHIN_SCHEDULER_PREFERRED_NETWORK_INTERFACE);
        logger.info("config name:{}, net work interface: {}, addrs: {}",
                preferredNetworkInterface,  networkInterface, networkInterface.getInterfaceAddresses());
        return Objects.equals(networkInterface.getName(), preferredNetworkInterface) || 
                Objects.equals(networkInterface.getDisplayName(), preferredNetworkInterface);
    }

common.properties add interface config

# Network Interface name
#dolphin.scheduler.network.interface.preferred=eth0

Hi,

Is it better to move DOLPHIN_SCHEDULER_PREFERRED_NETWORK_INTERFACE to Constants.java?

Thanks for your suggestions, I have completed the changes.