Remove action execute method

src/main/java/com/amazon/opendistro/elasticsearch/performanceanalyzer/decisionmaker/actions/Action.java

@@ -33,13 +33,6 @@ public interface Action {
   /** Time to wait since last recommendation, before suggesting this action again */
   int coolOffPeriodInSeconds();
 
-  /**
-   * Called when the action is invoked.
-   *
-   * <p>Specific implementation may include executing the action, or invoking downstream APIs
-   */
-  void execute();
-
   /** Returns a list of Elasticsearch nodes impacted by this action. */
   List<NodeKey> impactedNodes();
 

src/main/java/com/amazon/opendistro/elasticsearch/performanceanalyzer/decisionmaker/actions/ModifyQueueCapacityAction.java

@@ -81,13 +81,6 @@ public Map<NodeKey, ImpactVector> impact() {
     return Collections.singletonMap(esNode, impactVector);
   }
 
-  @Override
-  public void execute() {
-    // Making this a no-op for now
-    // TODO: Modify based on downstream agent API calls
-    assert true;
-  }
-
   @Override
   public String summary() {
     if (!isActionable()) {

src/main/java/com/amazon/opendistro/elasticsearch/performanceanalyzer/decisionmaker/deciders/Publisher.java

@@ -48,8 +48,6 @@ public EmptyFlowUnit operate() {
 
     Decision decision = collator.getFlowUnits().get(0);
     for (Action action : decision.getActions()) {
-//      LOG.info("Executing action: [{}]", action.name());
-//      action.execute();
       for (ActionListener listener: actionListeners) {
         listener.actionPublished(action);
       }