google-cloud-spanner-admin-database-v1 1.2.1 → 1.3.0

data/lib/google/cloud/spanner/admin/database/v1/database_admin/rest/client.rb

@@ -272,8 +272,19 @@ module Google
                       endpoint: @config.endpoint,
                       endpoint_template: DEFAULT_ENDPOINT_TEMPLATE,
                       universe_domain: @config.universe_domain,
-                      credentials: credentials
+                      credentials: credentials,
+                      logger: @config.logger
                     )
+
+                    @database_admin_stub.logger(stub: true)&.info do |entry|
+                      entry.set_system_name
+                      entry.set_service
+                      entry.message = "Created client for #{entry.service}"
+                      entry.set_credentials_fields credentials
+                      entry.set "customEndpoint", @config.endpoint if @config.endpoint
+                      entry.set "defaultTimeout", @config.timeout if @config.timeout
+                      entry.set "quotaProject", @quota_project_id if @quota_project_id
+                    end
                   end
 
                   ##
@@ -283,6 +294,15 @@ module Google
                   #
                   attr_reader :operations_client
 
+                  ##
+                  # The logger used for request/response debug logging.
+                  #
+                  # @return [Logger]
+                  #
+                  def logger
+                    @database_admin_stub.logger
+                  end
+
                   # Service calls
 
                   ##
@@ -372,7 +392,7 @@ module Google
                     @database_admin_stub.list_databases request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @database_admin_stub, :list_databases, "databases", request, result, options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -500,7 +520,7 @@ module Google
                     @database_admin_stub.create_database request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -580,7 +600,6 @@ module Google
 
                     @database_admin_stub.get_database request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -707,7 +726,7 @@ module Google
                     @database_admin_stub.update_database request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -840,7 +859,7 @@ module Google
                     @database_admin_stub.update_database_ddl request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -923,7 +942,6 @@ module Google
 
                     @database_admin_stub.drop_database request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1006,7 +1024,6 @@ module Google
 
                     @database_admin_stub.get_database_ddl request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1103,7 +1120,6 @@ module Google
 
                     @database_admin_stub.set_iam_policy request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1193,7 +1209,6 @@ module Google
 
                     @database_admin_stub.get_iam_policy request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1287,7 +1302,6 @@ module Google
 
                     @database_admin_stub.test_iam_permissions request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1402,7 +1416,7 @@ module Google
                     @database_admin_stub.create_backup request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1525,7 +1539,7 @@ module Google
                     @database_admin_stub.copy_backup request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1607,7 +1621,6 @@ module Google
 
                     @database_admin_stub.get_backup request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1696,7 +1709,6 @@ module Google
 
                     @database_admin_stub.update_backup request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1778,7 +1790,6 @@ module Google
 
                     @database_admin_stub.delete_backup request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -1915,7 +1926,7 @@ module Google
                     @database_admin_stub.list_backups request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @database_admin_stub, :list_backups, "backups", request, result, options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2037,7 +2048,7 @@ module Google
                     @database_admin_stub.restore_database request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2184,7 +2195,7 @@ module Google
                     @database_admin_stub.list_database_operations request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @database_admin_stub, :list_database_operations, "operations", request, result, options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2358,7 +2369,7 @@ module Google
                     @database_admin_stub.list_backup_operations request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @database_admin_stub, :list_backup_operations, "operations", request, result, options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2452,7 +2463,7 @@ module Google
                     @database_admin_stub.list_database_roles request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @database_admin_stub, :list_database_roles, "database_roles", request, result, options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2537,7 +2548,6 @@ module Google
 
                     @database_admin_stub.create_backup_schedule request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2618,7 +2628,6 @@ module Google
 
                     @database_admin_stub.get_backup_schedule request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2705,7 +2714,6 @@ module Google
 
                     @database_admin_stub.update_backup_schedule request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2786,7 +2794,6 @@ module Google
 
                     @database_admin_stub.delete_backup_schedule request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2881,7 +2888,7 @@ module Google
                     @database_admin_stub.list_backup_schedules request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @database_admin_stub, :list_backup_schedules, "backup_schedules", request, result, options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -2961,6 +2968,11 @@ module Google
                   #   default endpoint URL. The default value of nil uses the environment
                   #   universe (usually the default "googleapis.com" universe).
                   #   @return [::String,nil]
+                  # @!attribute [rw] logger
+                  #   A custom logger to use for request/response debug logging, or the value
+                  #   `:default` (the default) to construct a default logger, or `nil` to
+                  #   explicitly disable logging.
+                  #   @return [::Logger,:default,nil]
                   #
                   class Configuration
                     extend ::Gapic::Config
@@ -2982,6 +2994,7 @@ module Google
                     config_attr :retry_policy,  nil, ::Hash, ::Proc, nil
                     config_attr :quota_project, nil, ::String, nil
                     config_attr :universe_domain, nil, ::String, nil
+                    config_attr :logger, :default, ::Logger, nil, :default
 
                     # @private
                     def initialize parent_config = nil

data/lib/google/cloud/spanner/admin/database/v1/database_admin/rest/operations.rb

@@ -117,14 +117,6 @@ module Google
                   # Lists operations that match the specified filter in the request. If the
                   # server doesn't support this method, it returns `UNIMPLEMENTED`.
                   #
-                  # NOTE: the `name` binding allows API services to override the binding
-                  # to use different resource name schemes, such as `users/*/operations`. To
-                  # override the binding, API services can add a binding such as
-                  # `"/v1/{name=users/*}/operations"` to their service configuration.
-                  # For backwards compatibility, the default name includes the operations
-                  # collection id, however overriding users must ensure the name binding
-                  # is the parent resource, without the operations collection id.
-                  #
                   # @overload list_operations(request, options = nil)
                   #   Pass arguments to `list_operations` via a request object, either of type
                   #   {::Google::Longrunning::ListOperationsRequest} or an equivalent Hash.
@@ -206,7 +198,7 @@ module Google
                     @operations_stub.list_operations request, options do |result, operation|
                       result = ::Gapic::Rest::PagedEnumerable.new @operations_stub, :list_operations, "operations", request, result, options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -295,7 +287,7 @@ module Google
                     @operations_stub.get_operation request, options do |result, operation|
                       result = ::Gapic::Operation.new result, @operations_client, options: options
                       yield result, operation if block_given?
-                      return result
+                      throw :response, result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -377,7 +369,6 @@ module Google
 
                     @operations_stub.delete_operation request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -392,8 +383,9 @@ module Google
                   # other methods to check whether the cancellation succeeded or whether the
                   # operation completed despite cancellation. On successful cancellation,
                   # the operation is not deleted; instead, it becomes an operation with
-                  # an {::Google::Longrunning::Operation#error Operation.error} value with a {::Google::Rpc::Status#code google.rpc.Status.code} of 1,
-                  # corresponding to `Code.CANCELLED`.
+                  # an {::Google::Longrunning::Operation#error Operation.error} value with a
+                  # {::Google::Rpc::Status#code google.rpc.Status.code} of `1`, corresponding to
+                  # `Code.CANCELLED`.
                   #
                   # @overload cancel_operation(request, options = nil)
                   #   Pass arguments to `cancel_operation` via a request object, either of type
@@ -465,7 +457,6 @@ module Google
 
                     @operations_stub.cancel_operation request, options do |result, operation|
                       yield result, operation if block_given?
-                      return result
                     end
                   rescue ::Gapic::Rest::Error => e
                     raise ::Google::Cloud::Error.from_error(e)
@@ -545,6 +536,11 @@ module Google
                   #   default endpoint URL. The default value of nil uses the environment
                   #   universe (usually the default "googleapis.com" universe).
                   #   @return [::String,nil]
+                  # @!attribute [rw] logger
+                  #   A custom logger to use for request/response debug logging, or the value
+                  #   `:default` (the default) to construct a default logger, or `nil` to
+                  #   explicitly disable logging.
+                  #   @return [::Logger,:default,nil]
                   #
                   class Configuration
                     extend ::Gapic::Config
@@ -566,6 +562,7 @@ module Google
                     config_attr :retry_policy,  nil, ::Hash, ::Proc, nil
                     config_attr :quota_project, nil, ::String, nil
                     config_attr :universe_domain, nil, ::String, nil
+                    config_attr :logger, :default, ::Logger, nil, :default
 
                     # @private
                     def initialize parent_config = nil
@@ -685,16 +682,18 @@ module Google
 
                     response = @client_stub.make_http_request(
                       verb,
-                      uri:     uri,
-                      body:    body || "",
-                      params:  query_string_params,
+                      uri: uri,
+                      body: body || "",
+                      params: query_string_params,
+                      method_name: "list_operations",
                       options: options
                     )
                     operation = ::Gapic::Rest::TransportOperation.new response
                     result = ::Google::Longrunning::ListOperationsResponse.decode_json response.body, ignore_unknown_fields: true
-
-                    yield result, operation if block_given?
-                    result
+                    catch :response do
+                      yield result, operation if block_given?
+                      result
+                    end
                   end
 
                   ##
@@ -723,16 +722,18 @@ module Google
 
                     response = @client_stub.make_http_request(
                       verb,
-                      uri:     uri,
-                      body:    body || "",
-                      params:  query_string_params,
+                      uri: uri,
+                      body: body || "",
+                      params: query_string_params,
+                      method_name: "get_operation",
                       options: options
                     )
                     operation = ::Gapic::Rest::TransportOperation.new response
                     result = ::Google::Longrunning::Operation.decode_json response.body, ignore_unknown_fields: true
-
-                    yield result, operation if block_given?
-                    result
+                    catch :response do
+                      yield result, operation if block_given?
+                      result
+                    end
                   end
 
                   ##
@@ -761,16 +762,18 @@ module Google
 
                     response = @client_stub.make_http_request(
                       verb,
-                      uri:     uri,
-                      body:    body || "",
-                      params:  query_string_params,
+                      uri: uri,
+                      body: body || "",
+                      params: query_string_params,
+                      method_name: "delete_operation",
                       options: options
                     )
                     operation = ::Gapic::Rest::TransportOperation.new response
                     result = ::Google::Protobuf::Empty.decode_json response.body, ignore_unknown_fields: true
-
-                    yield result, operation if block_given?
-                    result
+                    catch :response do
+                      yield result, operation if block_given?
+                      result
+                    end
                   end
 
                   ##
@@ -799,16 +802,18 @@ module Google
 
                     response = @client_stub.make_http_request(
                       verb,
-                      uri:     uri,
-                      body:    body || "",
-                      params:  query_string_params,
+                      uri: uri,
+                      body: body || "",
+                      params: query_string_params,
+                      method_name: "cancel_operation",
                       options: options
                     )
                     operation = ::Gapic::Rest::TransportOperation.new response
                     result = ::Google::Protobuf::Empty.decode_json response.body, ignore_unknown_fields: true
-
-                    yield result, operation if block_given?
-                    result
+                    catch :response do
+                      yield result, operation if block_given?
+                      result
+                    end
                   end
 
                   ##