skyfall 0.4.0 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: db537537fcd4e38f184c6cdea3f51ba1b8554848d4c3cd1de20f63ca2d956fc2
-  data.tar.gz: 34c51f6c8c0152589562bf0a3b3c5246cd8a1e95664f1128c86bd17c6a313d59
+  metadata.gz: 1fa2648926fdc913472ec8fca4d6bdf3bbac188dbb0d9c82816538ca0c446d23
+  data.tar.gz: f35348c4834fb9dc26544105074bdc4f9ba07c36d595e5c248651d47e5dc188e
 SHA512:
-  metadata.gz: 11477293a1bc0377ef5d9eaa68cc2d374f76ac4d942006572e8b9e1668cb8f54a582abdeda81b65c1d72f6452ba2cad1153c403027ba5d7efbfa093624713105
-  data.tar.gz: 8d86b1f71a4fd5fa7f01077fefe8d8e292dc6b5007b79ee4d1ed1343f887c4104c0480b297e7ce4be61db0cce65211a568b642411fa5c3e7f62c4c91a4fe0f7b
+  metadata.gz: df775bdf85f5fe73f5fc5f27868b4ad8755219e98dfae72a32e7a9bb66295b463f65a2ebd9a8c956d74f200f667a9f02c00310e665a506f69a950db03eec4ad6
+  data.tar.gz: 6dfe5acf800a9765d0ad6f3ce78e7a6aae445a2dde4028ef3af39ff81a5fc3947080ecfb0ceb0dcf439a2db3e6d65e17376b187455fa1eaf56889c87d0137be1

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## [0.4.1] - 2024-10-04
+
+- performance fix - don't decode CAR sections which aren't needed, which is most of them; this cuts the amount of memory that GC has to free up by about one third, and should speed up processing by around ~10%
+
 ## [0.4.0] - 2024-09-23
 
 - (re)added a "hearbeat" feature (removed earlier in 0.2.0) to fix the occasional issue when the websocket stops receiving data, but doesn't disconnect (not enabled by default, turn it on by setting `check_heartbeat` to true)

data/lib/skyfall/car_archive.rb

@@ -11,11 +11,15 @@ require 'stringio'
 
 module Skyfall
   class CarSection
-    attr_reader :cid, :body
+    attr_reader :cid
 
-    def initialize(cid, body)
+    def initialize(cid, body_data)
       @cid = cid
-      @body = body
+      @body_data = body_data
+    end
+
+    def body
+      @body ||= CarArchive.convert_data(CBOR.decode(@body_data))
     end
   end
 
@@ -37,9 +41,7 @@ module Skyfall
       section && section.body
     end
 
-    private
-
-    def convert_data(object)
+    def self.convert_data(object)
       if object.is_a?(Hash)
         object.each do |k, v|
           if v.is_a?(Hash) || v.is_a?(Array)
@@ -65,14 +67,16 @@ module Skyfall
       end
     end
 
-    def make_cid_link(cid)
+    def self.make_cid_link(cid)
       { '$link' => CID.from_cbor_tag(cid) }
     end
 
-    def make_bytes(data)
+    def self.make_bytes(data)
       { '$bytes' => Base64.encode64(data).chomp.gsub(/=+$/, '') }
     end
 
+    private
+
     def read_header(buffer)
       len = buffer.read_varint
 
@@ -112,10 +116,7 @@ module Skyfall
       cid = CID.new(prefix + cid_data)
 
       body_data = sbuffer.read
-      body = CBOR.decode(body_data)
-      convert_data(body)
-
-      @sections << CarSection.new(cid, body)
+      @sections << CarSection.new(cid, body_data)
     end
   end
 end

data/lib/skyfall/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Skyfall
-  VERSION = "0.4.0"
+  VERSION = "0.4.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: skyfall
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.1
 platform: ruby
 authors:
 - Kuba Suder
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-09-23 00:00:00.000000000 Z
+date: 2024-10-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: base32