google-cloud-pubsub 0.37.0 → 0.37.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ff2481f99b8def0df9c9203a64569dc317907152518da6b522d63314524ebea5
-  data.tar.gz: 8773411fca063ab56c81d94951406866747f9137f4c9d9e9860b011412a0bae3
+  metadata.gz: 9ca95d3d409fb3fd3061e5e9bb0e34eed7d8efdd3677c4de6e34dbed26499eae
+  data.tar.gz: e7a16095e21f9a166461cbecbc4c95a4895421762da8c644a866cbce95a0e19b
 SHA512:
-  metadata.gz: 47976e8e93c6a68707a24f26b92110fca18eef669c41c86f93cf9d746c55596abbddde9f067a42b47c68c6472d5655616380f0e334623872455d5932b97ffd9c
-  data.tar.gz: ebf63bd135e128d19f56c9ece11aad5f1fc8cf2489fa7b8c726ccc68e98632ee4751c28079d9dfc17379dbf40cf9871a44b9d1c03d50dc0b1a6c9215de45f25d
+  metadata.gz: 2ad6bd6779827fcd87e613832f7a47cc73331615dcd3564d223776cea9d8469890829a7c2f8dd5bd989f1ffb5147c2fc8197086ce3e32994d40786c38141e1d7
+  data.tar.gz: f056480612e86fac7b2339067dae67c6d393cfc0864d30dcba8a492f3db25d6e6e269385d00dd5e85fd9de22d00cc577227b557e18aa4f3fe5f0244d1416c0d2

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Release History
 
+### 0.37.1 / 2019-07-09
+
+* Add IAM GetPolicyOptions in the lower-level interface.
+* Support overriding service host and port in the low-level interface.
+* Fixed race in TimedUnaryBuffer.
+
 ### 0.37.0 / 2019-06-17
 
 * Add Topic#persistence_regions

data/lib/google/cloud/pubsub/subscriber/timed_unary_buffer.rb

@@ -14,6 +14,7 @@
 
 
 require "concurrent"
+require "monitor"
 
 module Google
   module Cloud
@@ -22,9 +23,13 @@ module Google
         ##
         # @private
         class TimedUnaryBuffer
+          include MonitorMixin
+
           attr_reader :max_bytes, :interval
 
           def initialize subscriber, max_bytes: 10000000, interval: 1.0
+            super() # to init MonitorMixin
+
             @subscriber = subscriber
             @max_bytes = max_bytes
             @interval = interval
@@ -42,9 +47,11 @@ module Google
           def acknowledge ack_ids
             return if ack_ids.empty?
 
-            ack_ids.each do |ack_id|
-              # ack has no deadline set, use :ack indicate it is an ack
-              @register[ack_id] = :ack
+            synchronize do
+              ack_ids.each do |ack_id|
+                # ack has no deadline set, use :ack indicate it is an ack
+                @register[ack_id] = :ack
+              end
             end
 
             true
@@ -53,8 +60,10 @@ module Google
           def modify_ack_deadline deadline, ack_ids
             return if ack_ids.empty?
 
-            ack_ids.each do |ack_id|
-              @register[ack_id] = deadline
+            synchronize do
+              ack_ids.each do |ack_id|
+                @register[ack_id] = deadline
+              end
             end
 
             true
@@ -63,9 +72,11 @@ module Google
           def renew_lease deadline, ack_ids
             return if ack_ids.empty?
 
-            ack_ids.each do |ack_id|
-              # Do not overwrite pending actions when renewing leased messages.
-              @register[ack_id] ||= deadline
+            synchronize do
+              ack_ids.each do |ack_id|
+                # Don't overwrite pending actions when renewing leased messages.
+                @register[ack_id] ||= deadline
+              end
             end
 
             true
@@ -120,10 +131,13 @@ module Google
           private
 
           def flush_requests!
-            return {} if @register.empty?
-
-            prev_reg = @register
-            @register = Concurrent::Map.new
+            prev_reg =
+              synchronize do
+                return {} if @register.empty?
+                reg = @register
+                @register = Concurrent::Map.new
+                reg
+              end
 
             groups = prev_reg.each_pair.group_by { |_ack_id, delay| delay }
             req_hash = Hash[groups.map { |k, v| [k, v.map(&:first)] }]

data/lib/google/cloud/pubsub/v1/doc/google/iam/v1/options.rb

@@ -0,0 +1,21 @@
+# Copyright 2019 Google LLC
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     https://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+
+module Google
+  module Iam
+    module V1
+    end
+  end
+end

data/lib/google/cloud/pubsub/v1/publisher_client.rb

@@ -146,6 +146,10 @@ module Google
           #   The default timeout, in seconds, for calls made through this client.
           # @param metadata [Hash]
           #   Default metadata to be sent with each request. This can be overridden on a per call basis.
+          # @param service_address [String]
+          #   Override for the service hostname, or `nil` to leave as the default.
+          # @param service_port [Integer]
+          #   Override for the service port, or `nil` to leave as the default.
           # @param exception_transformer [Proc]
           #   An optional proc that intercepts any exceptions raised during an API call to inject
           #   custom error handling.
@@ -155,6 +159,8 @@ module Google
               client_config: {},
               timeout: DEFAULT_TIMEOUT,
               metadata: nil,
+              service_address: nil,
+              service_port: nil,
               exception_transformer: nil,
               lib_name: nil,
               lib_version: ""
@@ -211,8 +217,8 @@ module Google
             end
 
             # Allow overriding the service path/port in subclasses.
-            service_path = self.class::SERVICE_ADDRESS
-            port = self.class::DEFAULT_SERVICE_PORT
+            service_path = service_address || self.class::SERVICE_ADDRESS
+            port = service_port || self.class::DEFAULT_SERVICE_PORT
             interceptors = self.class::GRPC_INTERCEPTORS
             @iam_policy_stub = Google::Gax::Grpc.create_stub(
               service_path,
@@ -698,6 +704,11 @@ module Google
           # @param resource [String]
           #   REQUIRED: The resource for which the policy is being requested.
           #   See the operation documentation for the appropriate value for this field.
+          # @param options_ [Google::Iam::V1::GetPolicyOptions | Hash]
+          #   OPTIONAL: A `GetPolicyOptions` object for specifying options to
+          #   `GetIamPolicy`. This field is only used by Cloud IAM.
+          #   A hash of the same form as `Google::Iam::V1::GetPolicyOptions`
+          #   can also be provided.
           # @param options [Google::Gax::CallOptions]
           #   Overrides the default settings for this call, e.g, timeout,
           #   retries, etc.
@@ -715,10 +726,12 @@ module Google
 
           def get_iam_policy \
               resource,
+              options_: nil,
               options: nil,
               &block
             req = {
-              resource: resource
+              resource: resource,
+              options: options_
             }.delete_if { |_, v| v.nil? }
             req = Google::Gax::to_proto(req, Google::Iam::V1::GetIamPolicyRequest)
             @get_iam_policy.call(req, options, &block)

data/lib/google/cloud/pubsub/v1/subscriber_client.rb

@@ -170,6 +170,10 @@ module Google
           #   The default timeout, in seconds, for calls made through this client.
           # @param metadata [Hash]
           #   Default metadata to be sent with each request. This can be overridden on a per call basis.
+          # @param service_address [String]
+          #   Override for the service hostname, or `nil` to leave as the default.
+          # @param service_port [Integer]
+          #   Override for the service port, or `nil` to leave as the default.
           # @param exception_transformer [Proc]
           #   An optional proc that intercepts any exceptions raised during an API call to inject
           #   custom error handling.
@@ -179,6 +183,8 @@ module Google
               client_config: {},
               timeout: DEFAULT_TIMEOUT,
               metadata: nil,
+              service_address: nil,
+              service_port: nil,
               exception_transformer: nil,
               lib_name: nil,
               lib_version: ""
@@ -234,8 +240,8 @@ module Google
             end
 
             # Allow overriding the service path/port in subclasses.
-            service_path = self.class::SERVICE_ADDRESS
-            port = self.class::DEFAULT_SERVICE_PORT
+            service_path = service_address || self.class::SERVICE_ADDRESS
+            port = service_port || self.class::DEFAULT_SERVICE_PORT
             interceptors = self.class::GRPC_INTERCEPTORS
             @iam_policy_stub = Google::Gax::Grpc.create_stub(
               service_path,
@@ -1274,6 +1280,11 @@ module Google
           # @param resource [String]
           #   REQUIRED: The resource for which the policy is being requested.
           #   See the operation documentation for the appropriate value for this field.
+          # @param options_ [Google::Iam::V1::GetPolicyOptions | Hash]
+          #   OPTIONAL: A `GetPolicyOptions` object for specifying options to
+          #   `GetIamPolicy`. This field is only used by Cloud IAM.
+          #   A hash of the same form as `Google::Iam::V1::GetPolicyOptions`
+          #   can also be provided.
           # @param options [Google::Gax::CallOptions]
           #   Overrides the default settings for this call, e.g, timeout,
           #   retries, etc.
@@ -1291,10 +1302,12 @@ module Google
 
           def get_iam_policy \
               resource,
+              options_: nil,
               options: nil,
               &block
             req = {
-              resource: resource
+              resource: resource,
+              options: options_
             }.delete_if { |_, v| v.nil? }
             req = Google::Gax::to_proto(req, Google::Iam::V1::GetIamPolicyRequest)
             @get_iam_policy.call(req, options, &block)

data/lib/google/cloud/pubsub/version.rb

@@ -16,7 +16,7 @@
 module Google
   module Cloud
     module PubSub
-      VERSION = "0.37.0".freeze
+      VERSION = "0.37.1".freeze
     end
 
     Pubsub = PubSub unless const_defined? :Pubsub

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: google-cloud-pubsub
 version: !ruby/object:Gem::Version
-  version: 0.37.0
+  version: 0.37.1
 platform: ruby
 authors:
 - Mike Moore
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-06-18 00:00:00.000000000 Z
+date: 2019-07-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: google-cloud-core
@@ -31,14 +31,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '1.7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '1.7'
 - !ruby/object:Gem::Dependency
   name: googleapis-common-protos
   requirement: !ruby/object:Gem::Requirement
@@ -272,6 +272,7 @@ files:
 - lib/google/cloud/pubsub/v1.rb
 - lib/google/cloud/pubsub/v1/credentials.rb
 - lib/google/cloud/pubsub/v1/doc/google/iam/v1/iam_policy.rb
+- lib/google/cloud/pubsub/v1/doc/google/iam/v1/options.rb
 - lib/google/cloud/pubsub/v1/doc/google/iam/v1/policy.rb
 - lib/google/cloud/pubsub/v1/doc/google/protobuf/duration.rb
 - lib/google/cloud/pubsub/v1/doc/google/protobuf/empty.rb