graphiti 1.6.2 → 1.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c7e9a22a29df9a53ef4c2da7a6bb102c4407f21201440cd0888015c2442895b5
-  data.tar.gz: 405ed0c5ee38118ef4b535bc5751dcb960630117ca9ca1706b69bb91eda6df4a
+  metadata.gz: d9fe56af10e2f476d55ff2d367e4efd0841f73afbe4d4f95cae0f9ab81f6a39e
+  data.tar.gz: 4d3037e59ed89ed1bfb5bd4af49d6cd80038661907d8983ecfd5a17afa96c061
 SHA512:
-  metadata.gz: fa1bbea7813069ac8c97bacdb7d298162bc55121771974693264c81b5b85da9420aef3d3c0509e96013f9266861e6a0e8dc61853e88dcb88b7cc7c9ddf83fa99
-  data.tar.gz: 9929dda99cd0fecbbac7939c1a161f43f20944921950cf8f4e8bb30c30251fcbf1f253364ee7fe4c7c442e4eafb7d74c6c2cb6347835f7cdbc71713c31938639
+  metadata.gz: f09aeaedda7fc10d97270fe89789fb1b038a2efe4615136fe1ccc576059c5af4b57c9bf97a0f79e38d240ba44f99b4d31f662b27a7d7f3128622bb8ae60429ef
+  data.tar.gz: 5c965ae2e16ceb97a7e2aa5341cff1e039a8f43d17c2766c884adfb9b0f1ae16fd5df16f1ca7973b3b27face19f80a6e541c43558f9dfbaedf72909b6ae8c361

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 graphiti changelog
 
+## [1.6.3](https://github.com/graphiti-api/graphiti/compare/v1.6.2...v1.6.3) (2024-03-26)
+
+
+### Bug Fixes
+
+* Remove thread pool executor logic until we get a better handle on what's causing thread pool hangs. refs [#469](https://github.com/graphiti-api/graphiti/issues/469) ([7941b6f](https://github.com/graphiti-api/graphiti/commit/7941b6f75ce1001b034ed6e83c148b893e9f3d99)), closes [#471](https://github.com/graphiti-api/graphiti/issues/471) [#470](https://github.com/graphiti-api/graphiti/issues/470)
+
 ## [1.6.2](https://github.com/graphiti-api/graphiti/compare/v1.6.1...v1.6.2) (2024-03-22)
 
 

data/lib/graphiti/configuration.rb

@@ -8,20 +8,6 @@ module Graphiti
     #   Defaults to false OR if classes are cached (Rails-only)
     attr_accessor :concurrency
 
-    # This number must be considered in accordance with the database
-    # connection pool size configured in `database.yml`. The connection
-    # pool should be large enough to accommodate both the foreground
-    # threads (ie. web server or job worker threads) and background
-    # threads. For each process, Graphiti will create one global
-    # executor that uses this many threads to sideload resources
-    # asynchronously. Thus, the pool size should be at least
-    # `thread_count + concurrency_max_threads + 1`. For example, if your
-    # web server has a maximum of 3 threads, and
-    # `concurrency_max_threads` is set to 4, then your pool size should
-    # be at least 8.
-    # @return [Integer] Maximum number of threads to use when fetching sideloads concurrently
-    attr_accessor :concurrency_max_threads
-
     attr_accessor :respond_to
     attr_accessor :context_for_endpoint
     attr_accessor :links_on_demand
@@ -40,7 +26,6 @@ module Graphiti
     def initialize
       @raise_on_missing_sideload = true
       @concurrency = false
-      @concurrency_max_threads = 4
       @respond_to = [:json, :jsonapi, :xml]
       @links_on_demand = false
       @pagination_links_on_demand = false

data/lib/graphiti/scope.rb

@@ -2,23 +2,6 @@ module Graphiti
   class Scope
     attr_accessor :object, :unpaginated_object
     attr_reader :pagination
-
-    @@thread_pool_executor_mutex = Mutex.new
-
-    def self.thread_pool_executor
-      return @thread_pool_executor if @thread_pool_executor
-
-      concurrency = Graphiti.config.concurrency_max_threads || 4
-      @@thread_pool_executor_mutex.synchronize do
-        @@thread_pool_executor ||= Concurrent::ThreadPoolExecutor.new(
-          min_threads: 0,
-          max_threads: concurrency,
-          max_queue: concurrency * 4,
-          fallback_policy: :caller_runs
-        )
-      end
-    end
-
     def initialize(object, resource, query, opts = {})
       @object = object
       @resource = resource
@@ -66,7 +49,7 @@ module Graphiti
           @resource.adapter.close if concurrent
         }
         if concurrent
-          promises << Concurrent::Promise.execute(executor: self.class.thread_pool_executor, &resolve_sideload)
+          promises << Concurrent::Promise.execute(&resolve_sideload)
         else
           resolve_sideload.call
         end

data/lib/graphiti/version.rb

@@ -1,3 +1,3 @@
 module Graphiti
-  VERSION = "1.6.2"
+  VERSION = "1.6.3"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: graphiti
 version: !ruby/object:Gem::Version
-  version: 1.6.2
+  version: 1.6.3
 platform: ruby
 authors:
 - Lee Richmond
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-03-22 00:00:00.000000000 Z
+date: 2024-03-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jsonapi-serializable