addressing comments

Author: yue9944882

e2e/src/test/groovy/io/kubernetes/client/e2e/util/ModelMapperTest.groovy

@@ -24,14 +24,13 @@ import spock.lang.Specification
 
 class ModelMapperTest extends Specification {
 
-    def "api-disocvery should work"() {
+    def "api-discovery should work"() {
         given:
         def apiClient = ClientBuilder.defaultClient();
         def discovery = new Discovery(apiClient)
         ModelMapper.refresh(discovery);
 
         expect:
-
         new GroupVersionKind("", "v1", "Pod") == ModelMapper.getGroupVersionKindByClass(V1Pod.class)
         new GroupVersionResource("", "v1", "pods") == ModelMapper.getGroupVersionResourceByClass(V1Pod.class)
 

util/src/main/java/io/kubernetes/client/util/ModelMapper.java

@@ -53,8 +53,10 @@ public class ModelMapper {
   // Model's api-version midfix to kubernetes api-version
   private static List<String> preBuiltApiVersions = new ArrayList<>();
 
+  // TODO(yue9944882): make the map bi-directional
   private static Map<GroupVersionKind, Class<?>> classesByGVK = new ConcurrentHashMap<>();
 
+  // TODO(yue9944882): make the map bi-directional
   private static Map<GroupVersionResource, Class<?>> classesByGVR = new ConcurrentHashMap<>();
 
   private static Set<Discovery.APIResource> lastAPIDiscovery = new HashSet<>();