opentelemetry-instrumentation-dalli 0.22.0 → 0.22.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c7db76142dadc45f67291aee6d863f69f852043d2fa2f8f41243f339f5ccef77
-  data.tar.gz: d592877e112281e939d18d3b89e819142a48d5fd1a389946bb65079499760db4
+  metadata.gz: 241bf85a863e4ddc2382d502a348fad2eec6eaf83a27c57e7da8daf9ff5012a2
+  data.tar.gz: c3b15aa63991b01313b12194dd0e2556cdb3b97211cff940177ff32622575d34
 SHA512:
-  metadata.gz: 7f7a666b3a4cda1882e36c9de578192729c4345639c7ac9e3cdcf14dd2da33f39d62a4ba84f39738600b62e7a061d0cd661f17fefdab25d93c4623315873db33
-  data.tar.gz: 742b4191641ef4acceee51ba9eda176b61f9980f05acd375416c7ddb4fca3b3083be30b7a951189315b58820c88d4e29f2054c702aabf75a5b0f02c5c05d9a5b
+  metadata.gz: 0d6b5e4c4772152e8f1684a6dad7e306572a65248f55f9b126c9c78521ae03904ba916a677ad66c5adb4a925191d7c7b96b18708c862ecb6b0a5c2dbb7457ac2
+  data.tar.gz: ac0646b81b28d7b426a4f157590a7b25d0db47d3d4862762f7bf809bd136deaa1fbf89f2458e50f8547cafef034263ad2bd917e79af8fce2f001e52ebe2a67c3

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Release History: opentelemetry-instrumentation-dalli
 
+### v0.22.1 / 2022-11-28
+
+* FIXED: `format_command`'s terrible performance 
+
 ### v0.22.0 / 2022-06-09
 
 * Upgrading Base dependency version

data/README.md

@@ -43,7 +43,7 @@ Apache 2.0 license. See [LICENSE][license-github] for more information.
 [dalli-home]: https://github.com/petergoldstein/dalli
 [bundler-home]: https://bundler.io
 [repo-github]: https://github.com/open-telemetry/opentelemetry-ruby
-[license-github]: https://github.com/open-telemetry/opentelemetry-ruby/blob/main/LICENSE
+[license-github]: https://github.com/open-telemetry/opentelemetry-ruby-contrib/blob/main/LICENSE
 [ruby-sig]: https://github.com/open-telemetry/community#ruby-sig
 [community-meetings]: https://github.com/open-telemetry/community#community-meetings
 [discussions-url]: https://github.com/open-telemetry/opentelemetry-ruby/discussions

data/lib/opentelemetry/instrumentation/dalli/patches/server.rb

@@ -10,7 +10,7 @@ module OpenTelemetry
       module Patches
         # Module to prepend to Dalli::Server (or Dalli::Protocol::Binary in 3.0+) for instrumentation
         module Server
-          def request(op, *args)
+          def request(op, *args) # rubocop:disable Naming/MethodParameterName
             operation = Utils.opname(op, multi?)
             attributes = {
               'db.system' => 'memcached',

data/lib/opentelemetry/instrumentation/dalli/utils.rb

@@ -53,7 +53,17 @@ module OpenTelemetry
 
       def format_command(operation, args)
         placeholder = "#{operation} BLOB (OMITTED)"
-        command = [operation, *args].join(' ').strip
+
+        command = +operation.to_s
+        args.flatten.each do |arg|
+          str = arg.to_s
+          if str.bytesize >= CMD_MAX_LEN
+            command << ' BLOB (OMITTED)'
+          elsif !str.empty?
+            command << ' ' << str
+          end
+        end
+
         command = OpenTelemetry::Common::Utilities.utf8_encode(command, binary: true, placeholder: placeholder)
         OpenTelemetry::Common::Utilities.truncate(command, CMD_MAX_LEN)
       rescue StandardError => e

data/lib/opentelemetry/instrumentation/dalli/version.rb

@@ -7,7 +7,7 @@
 module OpenTelemetry
   module Instrumentation
     module Dalli
-      VERSION = '0.22.0'
+      VERSION = '0.22.1'
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: opentelemetry-instrumentation-dalli
 version: !ruby/object:Gem::Version
-  version: 0.22.0
+  version: 0.22.1
 platform: ruby
 authors:
 - OpenTelemetry Authors
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-06-09 00:00:00.000000000 Z
+date: 2022-11-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: opentelemetry-api
@@ -142,14 +142,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.73.0
+        version: 1.3.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.73.0
+        version: 1.3.0
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -210,14 +210,14 @@ files:
 - lib/opentelemetry/instrumentation/dalli/patches/server.rb
 - lib/opentelemetry/instrumentation/dalli/utils.rb
 - lib/opentelemetry/instrumentation/dalli/version.rb
-homepage: https://github.com/open-telemetry/opentelemetry-ruby
+homepage: https://github.com/open-telemetry/opentelemetry-contrib
 licenses:
 - Apache-2.0
 metadata:
-  changelog_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-instrumentation-dalli/v0.22.0/file.CHANGELOG.html
-  source_code_uri: https://github.com/open-telemetry/opentelemetry-ruby/tree/main/instrumentation/dalli
-  bug_tracker_uri: https://github.com/open-telemetry/opentelemetry-ruby/issues
-  documentation_uri: https://open-telemetry.github.io/opentelemetry-ruby/opentelemetry-instrumentation-dalli/v0.22.0
+  changelog_uri: https://open-telemetry.github.io/opentelemetry-ruby-contrib/opentelemetry-instrumentation-dalli/v0.22.1/file.CHANGELOG.html
+  source_code_uri: https://github.com/open-telemetry/opentelemetry-ruby-contrib/tree/main/instrumentation/dalli
+  bug_tracker_uri: https://github.com/open-telemetry/opentelemetry-ruby-contrib/issues
+  documentation_uri: https://open-telemetry.github.io/opentelemetry-ruby-contrib/opentelemetry-instrumentation-dalli/v0.22.1
 post_install_message: 
 rdoc_options: []
 require_paths: