codeclimate-poseidon 0.0.9 → 0.0.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 24d819569b92bc26b634a8914fe26cb06f20ffb7
-  data.tar.gz: 32f521faf32658fe3bef867e68680c566039e97b
+  metadata.gz: 13660b540273d883d04fc215d955661f274e19d2
+  data.tar.gz: d42aac80cd5a5da6ca54eb68117c936c3b845c96
 SHA512:
-  metadata.gz: c2a37c2ac0b556382a0cdfa5db90c18a2e95b81f6d7a3e386b30fb8fa1bbd660d985daffced2072c0f913ac243df77824d2cdbbd5aeb90a1bde03e966d286c0e
-  data.tar.gz: d03477476c96df18826b9c9406ce7585242e5893673a7be35661a4ac82736c797d318e2579875f09fe3a17f692a04a346b3f5e9230ad22e81c736a1c865e02e4
+  metadata.gz: 711f1fa68fd8a26e47a42fb259b1296ccd01e02ed4137d26d9fc279158fe60b50fc94a6d4a73ff5118114674265dabf687e9f08fe224d655f73212096df2e108
+  data.tar.gz: 54c58a0f9fe5650dac5e498b246df497032336e347307f94b2c7aa2181af31276d6b537e14681df6f0dfb8df122bae7ccfc5f1411029f31cdb7b0607d8392bcc

data/Dockerfile

@@ -0,0 +1,21 @@
+FROM ruby:2.3.1-alpine
+MAINTAINER Code Climate <hello@codeclimate.com>
+
+RUN apk --update add git build-base autoconf automake libtool ruby-dev ruby-bundler snappy openjdk8 && \
+  rm -fr /usr/share/ri
+
+ENV KAFKA_SRC=http://apache.cs.utah.edu/kafka/0.8.2.1/kafka_2.10-0.8.2.1.tgz
+ENV KAFKA_PATH=/usr/src/kafka
+
+RUN curl "$KAFKA_SRC" | tar xvzf - && mv kafka_* "$KAFKA_PATH"
+
+WORKDIR /usr/src/poseidon/
+
+COPY .git/ /usr/src/poseidon/
+COPY Gemfile* /usr/src/poseidon/
+COPY poseidon.gemspec /usr/src/poseidon/
+COPY lib/poseidon/version.rb /usr/src/poseidon/lib/poseidon/
+
+RUN bundle install -j 4
+
+COPY . /usr/src/poseidon/

data/Makefile

@@ -0,0 +1,7 @@
+.PHONY: image test
+
+image:
+	docker build -t codeclimate/codeclimate-poseidon .
+
+test: image
+	docker run --rm -t codeclimate/codeclimate-poseidon bundle exec rake spec:all

data/circle.yml

@@ -0,0 +1,11 @@
+machine:
+  services:
+    - docker
+
+dependencies:
+  override:
+    - make image
+
+test:
+  override:
+    - make test

data/lib/poseidon/connection.rb

@@ -51,7 +51,7 @@ module Poseidon
       req = ProduceRequest.new( request_common(:produce),
                                 required_acks,
                                 timeout,
-                                messages_for_topics) 
+                                messages_for_topics)
       send_request(req)
       if required_acks != 0
         read_response(ProduceResponse)
@@ -71,7 +71,7 @@ module Poseidon
                                 REPLICA_ID,
                                 max_wait_time,
                                 min_bytes,
-                                topic_fetches) 
+                                topic_fetches)
       send_request(req)
       read_response(FetchResponse)
     end
@@ -103,8 +103,8 @@ module Poseidon
       if @socket.nil? || @socket.closed?
         begin
           @socket = TCPSocket.new(@host, @port)
-        rescue SystemCallError
-          raise_connection_failed_error
+        rescue SystemCallError => ex
+          raise_connection_failed_from_exception(ex)
         end
       end
     end
@@ -112,15 +112,15 @@ module Poseidon
     def read_response(response_class)
       r = ensure_read_or_timeout(4)
       if r.nil?
-        raise_connection_failed_error
+        raise_connection_failed_error("Could not read from socket")
       end
       n = r.unpack("N").first
       s = ensure_read_or_timeout(n)
       buffer = Protocol::ResponseBuffer.new(s)
       response_class.read(buffer)
-    rescue Errno::ECONNRESET, SocketError, TimeoutException
+    rescue Errno::ECONNRESET, SocketError, TimeoutException => ex
       @socket = nil
-      raise_connection_failed_error
+      raise_connection_failed_from_exception(ex)
     end
 
     def ensure_read_or_timeout(maxlen)
@@ -135,9 +135,9 @@ module Poseidon
       buffer = Protocol::RequestBuffer.new
       request.write(buffer)
       ensure_write_or_timeout([buffer.to_s.bytesize].pack("N") + buffer.to_s)
-    rescue Errno::EPIPE, Errno::ECONNRESET, TimeoutException
+    rescue Errno::EPIPE, Errno::ECONNRESET, TimeoutException => ex
       @socket = nil
-      raise_connection_failed_error
+      raise_connection_failed_from_exception(ex)
     end
 
     def ensure_write_or_timeout(data)
@@ -162,8 +162,12 @@ module Poseidon
       @correlation_id  += 1
     end
 
-    def raise_connection_failed_error
-      raise ConnectionFailedError, "Failed to connect to #{@host}:#{@port}"
+    def raise_connection_failed_from_exception(ex)
+      raise_connection_failed_error("Initial exception class=#{ex.class} message=#{ex.message}")
+    end
+
+    def raise_connection_failed_error(message)
+      raise ConnectionFailedError, "Failed to connect to #{@host}:#{@port}. #{message}"
     end
   end
 end

data/lib/poseidon/sync_producer.rb

@@ -155,8 +155,8 @@ module Poseidon
       else
         messages_for_broker.successfully_sent(response)
       end
-    rescue Connection::ConnectionFailedError
-      Poseidon.logger.warn { "Failed to send messages to #{messages_for_broker.broker_id} due to connection failure"  }
+    rescue Connection::ConnectionFailedError => ex
+      Poseidon.logger.warn { "Failed to send messages to #{messages_for_broker.broker_id} due to connection failure: message=#{ex.message}" }
       false
     end
   end

data/lib/poseidon/version.rb

@@ -1,4 +1,4 @@
 module Poseidon
   # Unstable! API May Change!
-  VERSION = "0.0.9"
+  VERSION = "0.0.10"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: codeclimate-poseidon
 version: !ruby/object:Gem::Version
-  version: 0.0.9
+  version: 0.0.10
 platform: ruby
 authors:
 - Bob Potter
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-25 00:00:00.000000000 Z
+date: 2016-07-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -92,11 +92,14 @@ files:
 - ".travis.yml"
 - ".yardopts"
 - CHANGES.md
+- Dockerfile
 - Gemfile
 - LICENSE.txt
+- Makefile
 - README.md
 - Rakefile
 - TODO.md
+- circle.yml
 - examples/consumer.rb
 - examples/producer.rb
 - lib/poseidon.rb