Avoid unnecessary hash values

This about halves the memory usage of the gem

Author: jhawthorn

lib/elastomer_client/client/rest_api_spec/api_spec.rb

@@ -12,9 +12,7 @@ class ApiSpec
     attr_reader :common_params
 
     def initialize
-      @rest_apis ||= {}
-      @common_params ||= {}
-      @common_params_set = Set.new(@common_params.keys)
+      @common_params_set = Set.new(@common_params)
     end
 
     # Given an API descriptor name and a set of request parameters, select those

lib/elastomer_client/client/rest_api_spec/api_spec_v5_6.rb

@@ -31,11 +31,11 @@ class Url
       attr_reader :parts_keys
       attr_reader :params_keys
 
-      def initialize(path:, paths: [], parts: {}, params: {})
+      def initialize(path:, paths: [], parts: [], params: [])
         @path = path
         @paths = Array(paths)
-        @parts_keys = parts.keys
-        @params_keys = params.keys
+        @parts_keys = parts
+        @params_keys = params
 
         @parts_set  = Set.new(@parts_keys)
         @params_set = Set.new(@params_keys)

lib/elastomer_client/client/rest_api_spec/api_spec_v7_17.rb

@@ -178,29 +178,29 @@ module ElastomerClient::Client::RestApiSpec
             path: "<%= generate_path(url) %>",
             paths: <%= generate_paths(url) %>,
 <% if (parts = generate_parts(url)) && !parts.empty? -%>
-            parts: {
+            parts: [
 <% parts.each do |k,v| -%>
-              "<%= k %>" => <%= v.to_s %>,
+              "<%= k %>",
 <% end -%>
-            },
+            ],
 <% end -%>
 <% params = generate_params(data) -%>
 <% if !params.nil? && !params.empty? -%>
-            params: {
+            params: [
 <% params.each do |k,v| -%>
-              "<%= k %>" => <%= v.to_s %>,
+              "<%= k %>",
 <% end -%>
-            }
+            ]
 <% end -%>
           }
         ),
 <% end -%>
       }
-      @common_params = {
+      @common_params = [
 <% each_common do |k,v| -%>
-        "<%= k %>" => <%= v.to_s %>,
+        "<%= k %>",
 <% end -%>
-      }
+      ]
       super
     end
   end

lib/elastomer_client/client/rest_api_spec/api_spec_v8_6.rb

@@ -11,18 +11,15 @@
         url: {
           path: "/_cluster/state",
           paths: ["/_cluster/state", "/_cluster/state/{metric}", "/_cluster/state/{metric}/{index}"],
-          parts: {
-            "index" => {"type"=>"list", "description"=>"A comma-separated list of index names; use `_all` or empty string to perform the operation on all indices"},
-            "metric" => {"type"=>"list", "options"=>["_all", "blocks", "metadata", "nodes", "routing_table", "routing_nodes", "master_node", "version"], "description"=>"Limit the information returned to the specified metrics"},
-          },
-          params: {
-            "local" => {"type"=>"boolean", "description"=>"Return local information, do not retrieve the state from master node (default: false)"},
-            "master_timeout" => {"type"=>"time", "description"=>"Specify timeout for connection to master"},
-            "flat_settings" => {"type"=>"boolean", "description"=>"Return settings in flat format (default: false)"},
-            "ignore_unavailable" => {"type"=>"boolean", "description"=>"Whether specified concrete indices should be ignored when unavailable (missing or closed)"},
-            "allow_no_indices" => {"type"=>"boolean", "description"=>"Whether to ignore if a wildcard indices expression resolves into no concrete indices. (This includes `_all` string or when no indices have been specified)"},
-            "expand_wildcards" => {"type"=>"enum", "options"=>["open", "closed", "none", "all"], "default"=>"open", "description"=>"Whether to expand wildcard expression to concrete indices that are open, closed or both."},
-          }
+          parts: ["index", "metric"],
+          params: [
+            "local",
+            "master_timeout",
+            "flat_settings",
+            "ignore_unavailable",
+            "allow_no_indices",
+            "expand_wildcards"
+          ]
         }
   end