[grid] Removing grid-model flag. Is not being used.

diemol · diemol · commit ffa9f47424ad · 2022-08-15T13:25:01.000+02:00
It is also misleading because someone might think that
it is implemented, and it is not. In addition, no user
has requested to provide their own GridModel implementation.
diff --git a/java/src/org/openqa/selenium/grid/distributor/config/DistributorFlags.java b/java/src/org/openqa/selenium/grid/distributor/config/DistributorFlags.java
@@ -18,8 +18,10 @@
 package org.openqa.selenium.grid.distributor.config;
 
 
-import com.beust.jcommander.Parameter;
 import com.google.auto.service.AutoService;
+
+import com.beust.jcommander.Parameter;
+
 import org.openqa.selenium.grid.config.ConfigValue;
 import org.openqa.selenium.grid.config.HasRoles;
 import org.openqa.selenium.grid.config.Role;
@@ -30,7 +32,6 @@
 
 import static org.openqa.selenium.grid.config.StandardGridRoles.DISTRIBUTOR_ROLE;
 import static org.openqa.selenium.grid.distributor.config.DistributorOptions.DEFAULT_DISTRIBUTOR_IMPLEMENTATION;
-import static org.openqa.selenium.grid.distributor.config.DistributorOptions.DEFAULT_GRID_MODEL_IMPLEMENTATION;
 import static org.openqa.selenium.grid.distributor.config.DistributorOptions.DEFAULT_HEALTHCHECK_INTERVAL;
 import static org.openqa.selenium.grid.distributor.config.DistributorOptions.DEFAULT_REJECT_UNSUPPORTED_CAPS;
 import static org.openqa.selenium.grid.distributor.config.DistributorOptions.DEFAULT_SLOT_MATCHER;
@@ -66,12 +67,6 @@ public class DistributorFlags implements HasRoles {
     example = DEFAULT_DISTRIBUTOR_IMPLEMENTATION)
   private String implementation = DEFAULT_DISTRIBUTOR_IMPLEMENTATION;
 
-  @Parameter(
-    names = {"--grid-model"},
-    description = "Full classname of non-default grid model. This is used to store states of the all the registered Nodes.")
-  @ConfigValue(section = DISTRIBUTOR_SECTION, name = "grid-model", example = DEFAULT_GRID_MODEL_IMPLEMENTATION)
-  private String gridModel = DEFAULT_GRID_MODEL_IMPLEMENTATION;
-
   @Parameter(
     names = {"--slot-matcher"},
     description = "Full classname of non-default slot matcher to use. This is used to determine whether a Node can support a particular session.")
diff --git a/java/src/org/openqa/selenium/grid/distributor/config/DistributorOptions.java b/java/src/org/openqa/selenium/grid/distributor/config/DistributorOptions.java
@@ -37,8 +37,6 @@ public class DistributorOptions {
   static final String DEFAULT_SLOT_MATCHER = "org.openqa.selenium.grid.data.DefaultSlotMatcher";
   static final String DEFAULT_SLOT_SELECTOR_IMPLEMENTATION =
     "org.openqa.selenium.grid.distributor.selector.DefaultSlotSelector";
-  static final String DEFAULT_GRID_MODEL_IMPLEMENTATION =
-    "org.openqa.selenium.grid.distributor.GridModel";
   static final boolean DEFAULT_REJECT_UNSUPPORTED_CAPS = false;
   private final Config config;
 
