[INLONG-90] Remove unused codes in TubeBroker

• Type: Improvement
• Status: Closed
• Priority: Normal
• Resolution: Fixed
• Fix Version/s: 0.5.0
• Labels: pull-request-available
• Assignee: Guo Jiwei
• Reporter: Guo Jiwei
• Created: 07/May/20 15:26
• Updated: 28/Jul/21 11:19
• Resolved: 02/Jun/20 02:03

Description

1. TubeBroker implements Runnable interface, I think this is a little strange. If we have some extra work to do , we can create inner thread for it.
2. JVMClusterUtil is useless . If we support start more than one broker in one JVM, we have to change some params, we can't set different params for different brokers at the same time. And as we will support docker in the future, we decide to remove this class.
3. remove synchronized in start method.
4. remove below codes in stop method

if (this.shutdown.get()) {
    return;
}

5. remove unused run method.
6. remove unused waitForServerOnline method.

Issue Links

• links to: GitHub Pull Request #115
• links to: GitHub Pull Request #689