在WEB应用中实现线程停止，线程终止登记表是否会内存溢出？

[chunjie008]

https://github.com/Viscent/javamtia/blob/06faaaa9610a8b9af2ee0eec551a0365154ef8fa/JavaMultiThreadInAction/src/io/github/viscent/mtia/ch5/ThreadTerminationRegistry.java
有add方法
public synchronized void register(Handler handler) {
handlers.add(handler);
}
但并没有
public synchronized void remove(Handler handler) {
handlers.remove(handler);
}
如果在生产环境中，handlers会越来越大，是否会内存溢出？

private final Set handlers = new HashSet();

能否改为

private final Set handlers = new ConcurrentHashSet();

register 和 remove 方法去掉 synchronized 关键字是否也能达到线程安全，并且效率更高。

比如以下写法：
private final Set handlers = Collections.newSetFromMap(new ConcurrentHashMap<Handler,Boolean>());

  public void register(Handler handler) {
		  handlers.add(handler);
  }
  
  public void remove(Handler handler) {
	      handlers.remove(handler);
  }