receptor_controller-client 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aeb9b0b9d6b9ababec5a45682b5674596687374fd6ff6285b3ba0d053f01d0ca
-  data.tar.gz: ec19064ed6c2fa29a3eac4c3e7aad616767946e1538b234fd15d8aec923f5012
+  metadata.gz: 7e0fca40eecf936d8ef48f10f5ff0a9c0a8f285496d504a4942b85f5ad43b679
+  data.tar.gz: c48de26017a7f7b2193d13603abe1ebfafaf55974efd1e1cc5700e89b3097394
 SHA512:
-  metadata.gz: 472a2f6e751379bc75b17cc318ae0bf6880705099fa238891af27bc98e27abd65a675d5ad3058c6ef55f6241b6b424cd27f220d0834dd63b53e09a43aba7070f
-  data.tar.gz: 657b3fb273f5ce72b50460e8483a20cdbdce1d5b22b1f552e52655f6551a10e7e7c9f3038f84e59af12e0a7e2e490f529e69c504d2bb5e23a528f4c014b4b132
+  metadata.gz: 9cabdb3d3176a0d6591a4b4fb06422481cf51f248edf669db455fc9f1ab9a98c19887d2b3b4780568be92058477d424164c158f49bc5ec9afbe1630becd273d6
+  data.tar.gz: fbe4d361bb3dfff586d6e6da8391623a5a92fcdc224a15c6c3ace6379105748562b6bc87f360d84556170b271298b3661985b88bddd3b8d8fe9ab60e0e8f4e6c

data/CHANGELOG.md

@@ -6,11 +6,21 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.0.4] - 2020-08-04
+### Added
+- Default Kafka ACK to true
+
+## [0.0.3] - 2020-08-03
+### Added
+- Do not send EOF message until all messages have been received
+
 ## [0.0.2] - 2020-07-31
 ### Initial release to rubygems.org
 
 ### Added
 - Added extra logging for messages
 
-[Unreleased]: https://github.com/RedHatInsights/receptor_controller-client-ruby/compare/v0.0.2...HEAD
+[Unreleased]: https://github.com/RedHatInsights/receptor_controller-client-ruby/compare/v0.0.4...HEAD
+[0.0.4]: https://github.com/RedHatInsights/receptor_controller-client-ruby/releases/tag/v0.0.4
+[0.0.3]: https://github.com/RedHatInsights/receptor_controller-client-ruby/releases/tag/v0.0.3
 [0.0.2]: https://github.com/RedHatInsights/receptor_controller-client-ruby/releases/tag/v0.0.2

data/lib/receptor_controller/client/configuration.rb

@@ -39,7 +39,7 @@ module ReceptorController
       @job_path               = '/job'
       @pre_shared_key         = nil
 
-      @queue_auto_ack    = false
+      @queue_auto_ack    = true
       @queue_host        = nil
       @queue_max_bytes   = nil
       @queue_persist_ref = nil

data/lib/receptor_controller/client/response_worker.rb

@@ -119,6 +119,9 @@ module ReceptorController
               payload = unpack_payload(payload) if payload.kind_of?(String)
               callbacks[:msg_size] ? callbacks[:msg_size] += payload.size : callbacks[:msg_size] = payload.size
               callbacks[:receiver].send(callbacks[:response_callback], message_id, message_type, payload)
+            else
+              # Send the callback to release the thread.
+              callbacks[:receiver].send(callbacks[:response_callback], message_id, message_type, payload)
             end
 
             # We received all the messages, complete the message.

data/lib/receptor_controller/client/version.rb

@@ -1,5 +1,5 @@
 module ReceptorController
   class Client
-    VERSION = "0.0.3".freeze
+    VERSION = "0.0.4".freeze
   end
 end

data/spec/receptor_controller/directive_blocking_spec.rb

@@ -1,6 +1,6 @@
 require "receptor_controller/client/directive_blocking"
 
-RSpec.xdescribe ReceptorController::Client::DirectiveBlocking do
+RSpec.describe ReceptorController::Client::DirectiveBlocking do
   # TODO: definitions below contain the same like non-blocking spec
   let(:external_tenant) { '0000001' }
   let(:organization_id) { '000001' }
@@ -106,7 +106,8 @@ RSpec.xdescribe ReceptorController::Client::DirectiveBlocking do
         response_message = {'code'           => 0,
                             'in_response_to' => http_response['id'],
                             'message_type'   => subject.class::MESSAGE_TYPE_EOF,
-                            'payload'        => 'Unimportant'}
+                            'payload'        => 'Unimportant',
+                            'serial'         => 1}
 
         allow(kafka_response).to receive(:payload).and_return(response_message.to_json)
 

data/spec/receptor_controller/directive_non_blocking_spec.rb

@@ -1,6 +1,6 @@
 require "receptor_controller/client/directive_non_blocking"
 
-RSpec.xdescribe ReceptorController::Client::DirectiveNonBlocking do
+RSpec.describe ReceptorController::Client::DirectiveNonBlocking do
   let(:external_tenant) { '0000001' }
   let(:organization_id) { '000001' }
   let(:identity) do
@@ -101,13 +101,13 @@ RSpec.xdescribe ReceptorController::Client::DirectiveNonBlocking do
       # Kafka response - 'response' type
       expect(subject).to receive(:response_success).twice.and_call_original
       message_type, payload = 'response', 'Testing payload'
-      response_payload = {'code' => 0, 'in_response_to' => response_id, 'message_type' => message_type, 'payload' => payload}
+      response_payload = {'code' => 0, 'in_response_to' => response_id, 'message_type' => message_type, 'payload' => payload, :serial => 1}
       allow(response).to receive(:payload).and_return(response_payload.to_json)
       subject.response_worker.send(:process_message, response)
 
       # Kafka response - 'eof' type
       message_type = 'eof'
-      response_payload = {'code' => 0, 'in_response_to' => response_id, 'message_type' => message_type, 'payload' => nil}
+      response_payload = {'code' => 0, 'in_response_to' => response_id, 'message_type' => message_type, 'payload' => nil, :serial => 2}
       allow(response).to receive(:payload).and_return(response_payload.to_json)
       subject.response_worker.send(:process_message, response)
 
@@ -142,7 +142,7 @@ RSpec.xdescribe ReceptorController::Client::DirectiveNonBlocking do
 
       # Kafka response - 'response' type - success
       message_type, payload = 'response', 'Testing payload'
-      response_payload = {'code' => 0, 'in_response_to' => response_id, 'message_type' => message_type, 'payload' => payload}
+      response_payload = {'code' => 0, 'in_response_to' => response_id, 'message_type' => message_type, 'payload' => payload, :serial => 1}
       allow(response).to receive(:payload).and_return(response_payload.to_json)
       success_calls.times do
         subject.response_worker.send(:process_message, response)
@@ -150,7 +150,7 @@ RSpec.xdescribe ReceptorController::Client::DirectiveNonBlocking do
 
       # Kafka response - 'eof' type
       message_type = 'eof'
-      response_payload = {'code' => 0, 'in_response_to' => response_id, 'message_type' => message_type, 'payload' => nil}
+      response_payload = {'code' => 0, 'in_response_to' => response_id, 'message_type' => message_type, 'payload' => nil, :serial => 4}
       allow(response).to receive(:payload).and_return(response_payload.to_json)
       subject.response_worker.send(:process_message, response)
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: receptor_controller-client
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Martin Slemr