nit: remove unused code

[turboFei]

Why are the changes needed?

Close #xxx.

Remove unused/duplicate code in OptimizerController and OptimizerGroupController

1. getContainers

• amoro/amoro-ams/src/main/java/org/apache/amoro/server/dashboard/controller/OptimizerController.java

  Lines 102 to 109 in 2a68423

  	/** check if optimizerGroup can be deleted url = /optimize/containers/get */
  	public void getContainers(Context ctx) {
  	ctx.json(
  	OkResponse.of(
  	Containers.getMetadataList().stream()
  	.map(ContainerMetadata::getName)
  	.collect(Collectors.toList())));
  	}

• amoro/amoro-ams/src/main/java/org/apache/amoro/server/dashboard/controller/OptimizerGroupController.java

  Lines 309 to 316 in 2a68423

  	/** check if optimizerGroup can be deleted url = /optimize/containers/get */
  	public void getContainers(Context ctx) {
  	ctx.json(
  	OkResponse.of(
  	Containers.getMetadataList().stream()
  	.map(ContainerMetadata::getName)
  	.collect(Collectors.toList())));
  	}

2. releaseOptimizer

• amoro/amoro-ams/src/main/java/org/apache/amoro/server/dashboard/controller/OptimizerGroupController.java

  Lines 187 to 217 in 2a68423

  	/**
  	* release optimizer.
  	*
  	* @pathParam jobId
  	*/
  	public void releaseOptimizer(Context ctx) {
  	String resourceId = ctx.pathParam("jobId");
  	Preconditions.checkArgument(
  	!resourceId.isEmpty(), "resource id can not be empty, maybe it's a external optimizer");
  	List<OptimizerInstance> optimizerInstances =
  	optimizerManager.listOptimizers().stream()
  	.filter(e -> resourceId.equals(e.getResourceId()))
  	.collect(Collectors.toList());
  	Preconditions.checkState(
  	!optimizerInstances.isEmpty(),
  	String.format(
  	"The resource ID %s has not been indexed" + " to any optimizer.", resourceId));
  	Resource resource = optimizerManager.getResource(resourceId);
  	resource.getProperties().putAll(optimizerInstances.get(0).getProperties());
  	ResourceContainer rc = Containers.get(resource.getContainerName());
  	Preconditions.checkState(
  	rc instanceof AbstractOptimizerContainer,
  	"Cannot release optimizer on non-optimizer resource container %s.",
  	resource.getContainerName());
  	((AbstractOptimizerContainer) rc).releaseResource(resource);
  	optimizerManager.deleteResource(resourceId);
  	optimizerManager.deleteOptimizer(resource.getGroupName(), resourceId);
  	ctx.json(OkResponse.of("Success to release optimizer"));
  	}

• amoro/amoro-ams/src/main/java/org/apache/amoro/server/dashboard/controller/OptimizerController.java

  Lines 47 to 77 in 2a68423

  	/**
  	* release optimizer.
  	*
  	* @pathParam jobId
  	*/
  	public void releaseOptimizer(Context ctx) {
  	String resourceId = ctx.pathParam("jobId");
  	Preconditions.checkArgument(
  	!resourceId.isEmpty(), "resource id can not be empty, maybe it's a external optimizer");
  	List<OptimizerInstance> optimizerInstances =
  	optimizerManager.listOptimizers().stream()
  	.filter(e -> resourceId.equals(e.getResourceId()))
  	.collect(Collectors.toList());
  	Preconditions.checkState(
  	!optimizerInstances.isEmpty(),
  	String.format(
  	"The resource ID %s has not been indexed" + " to any optimizer.", resourceId));
  	Resource resource = optimizerManager.getResource(resourceId);
  	resource.getProperties().putAll(optimizerInstances.get(0).getProperties());
  	ResourceContainer rc = Containers.get(resource.getContainerName());
  	Preconditions.checkState(
  	rc instanceof AbstractOptimizerContainer,
  	"Cannot release optimizer on non-optimizer resource container %s.",
  	resource.getContainerName());
  	((AbstractOptimizerContainer) rc).releaseResource(resource);
  	optimizerManager.deleteResource(resourceId);
  	optimizerManager.deleteOptimizer(resource.getGroupName(), resourceId);
  	ctx.json(OkResponse.of("Success to release optimizer"));
  	}

Brief change log

• remove unused/duplicate code

How was this patch tested?

• Add some test cases that check the changes thoroughly including negative and positive cases if possible

• Add screenshots for manual tests if appropriate

• Run test locally before making a pull request

Documentation

• Does this pull request introduce a new feature? (yes / no)
• If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)