avro_turf 1.7.0 → 1.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3eea429db02f7f6eba5f13f46b2bf3e82602371ef9306e374c1102561f508675
-  data.tar.gz: d798ef10b0e674f48ffdcc6bd3b87e59d9447c985d539f2ba01164e689fe4f8a
+  metadata.gz: 67ff8ee9b578227cd54a33d76a95fed8bd2c9a396aa360ad9df9f52d01e5c25a
+  data.tar.gz: 8bca24236ce42f19f6db457ad13d76d33e89b18703dc0eb74a8adca53b2c7b6e
 SHA512:
-  metadata.gz: e7cacb60d4e7890a700ebaba9a0cf1d46606fb01157e0746a0d6a6c900895e9ca1268f0a04dd4d03f5d1a9cec9bf6795d9c5d70acb6884965353c72bb1de4095
-  data.tar.gz: c4dbd604b66916355a6cba44ec4e7225eb439c2d9bb0682e62414b6ffa3018bb1d2a84b78d943983c99d7af3897a67658db9cb2205f7d92d6c13661ae44ba602
+  metadata.gz: 859e8fa6938679007704d35b1b4a54616da7637c83f41c58cbd10828216e0af198846e755334f465613ceb9086ec0c63a823dc2cbf3ddbdc191a97423d2e3147
+  data.tar.gz: 7900a85abaf9dd55ac3b89bba2d04935ddbafc850a92fb1941ff8984df864cfaa4bb5527ca4d48a74ef3715915d598044b7c486363909f2705ece83b0ffca50f

data/CHANGELOG.md

@@ -2,9 +2,14 @@
 
 ## Unreleased
 
+## v1.8.0
+
+- add support for `Date` via appropriate logicalType defintion.  This is a backwards incompatible change  (#177)
+- Fixed schema file cache truncation on multiple running instances and parallel access to the cache files.
+
 ## v1.7.0
 
-- Added extra params for the validation message schem before encode (#169)
+- Added extra params for the validation message schema before encode (#169)
 - Fix infinite retry when loading schema with nested primary type in separate file (#165)
 
 ## v1.6.0
@@ -60,7 +65,7 @@
 ## v0.9.0
 
 - Compatibility with Avro v1.9.0 (#94)
-- Disable the auto registeration of schema (#95)
+- Disable the auto registration of schema (#95)
 - abstracted caching from CachedConfluentSchemaRegistry (#74)
 - Load avro-patches if installed to silence deprecation errors (#85)
 - Make schema store to be thread safe (#92)

data/lib/avro_turf/cached_confluent_schema_registry.rb

@@ -13,7 +13,7 @@ class AvroTurf::CachedConfluentSchemaRegistry
   # cache     - Optional user provided Cache object that responds to all methods in the AvroTurf::InMemoryCache interface.
   def initialize(upstream, cache: nil)
     @upstream = upstream
-    @cache = cache || AvroTurf::InMemoryCache.new()
+    @cache = cache || AvroTurf::InMemoryCache.new
   end
 
   # Delegate the following methods to the upstream
@@ -34,7 +34,7 @@ class AvroTurf::CachedConfluentSchemaRegistry
 
   def subject_version(subject, version = 'latest')
     return @upstream.subject_version(subject, version) if version == 'latest'
-    
+
     @cache.lookup_by_version(subject, version) ||
       @cache.store_by_version(subject, version, @upstream.subject_version(subject, version))
   end

data/lib/avro_turf/core_ext/date.rb

@@ -1,5 +1,5 @@
 class Date
   def as_avro
-    iso8601
+    self
   end
 end

data/lib/avro_turf/disk_cache.rb

@@ -1,20 +1,18 @@
 # A cache for the CachedConfluentSchemaRegistry.
 # Extends the InMemoryCache to provide a write-thru to disk for persistent cache.
-class AvroTurf::DiskCache < AvroTurf::InMemoryCache
+class AvroTurf::DiskCache
 
   def initialize(disk_path, logger: Logger.new($stdout))
-    super()
-
     @logger = logger
 
     # load the write-thru cache on startup, if it exists
     @schemas_by_id_path = File.join(disk_path, 'schemas_by_id.json')
     hash = read_from_disk_cache(@schemas_by_id_path)
-    @schemas_by_id = hash if hash
+    @schemas_by_id = hash || {}
 
     @ids_by_schema_path = File.join(disk_path, 'ids_by_schema.json')
     hash = read_from_disk_cache(@ids_by_schema_path)
-    @ids_by_schema = hash if hash
+    @ids_by_schema = hash || {}
 
     @schemas_by_subject_version_path = File.join(disk_path, 'schemas_by_subject_version.json')
     @schemas_by_subject_version = {}
@@ -24,15 +22,16 @@ class AvroTurf::DiskCache < AvroTurf::InMemoryCache
   # the write-thru cache (json) does not store keys in numeric format
   # so, convert id to a string for caching purposes
   def lookup_by_id(id)
-    super(id.to_s)
+    @schemas_by_id[id.to_s]
   end
 
   # override to include write-thru cache after storing result from upstream
   def store_by_id(id, schema)
     # must return the value from storing the result (i.e. do not return result from file write)
-    value = super(id.to_s, schema)
-    File.write(@schemas_by_id_path, JSON.pretty_generate(@schemas_by_id))
-    return value
+    @schemas_by_id[id.to_s] = schema
+    write_to_disk_cache(@schemas_by_id_path, @schemas_by_id)
+
+    schema
   end
 
   # override to use a json serializable cache key
@@ -45,7 +44,8 @@ class AvroTurf::DiskCache < AvroTurf::InMemoryCache
   def store_by_schema(subject, schema, id)
     key = "#{subject}#{schema}"
     @ids_by_schema[key] = id
-    File.write(@ids_by_schema_path, JSON.pretty_generate(@ids_by_schema))
+
+    write_to_disk_cache(@ids_by_schema_path, @ids_by_schema)
     id
   end
 
@@ -78,7 +78,7 @@ class AvroTurf::DiskCache < AvroTurf::InMemoryCache
              hash[key] = schema
              hash
            else
-             {key => schema}
+             { key => schema }
            end
 
     write_to_disk_cache(@schemas_by_subject_version_path, hash)
@@ -90,17 +90,27 @@ class AvroTurf::DiskCache < AvroTurf::InMemoryCache
   # Parse the file from disk, if it exists and is not zero length
   private def read_from_disk_cache(path)
     if File.exist?(path)
-      if File.size(path)!=0
-        return JSON.parse(File.read(path))
+      if File.size(path) != 0
+        json_data = File.open(path, 'r') do |file|
+          file.flock(File::LOCK_SH)
+          file.read
+        end
+
+        return JSON.parse(json_data)
       else
         # just log a message if skipping zero length file
         @logger.warn "skipping JSON.parse of zero length file at #{path}"
       end
     end
-    return nil
+
+    nil
   end
 
   private def write_to_disk_cache(path, hash)
-    File.write(path, JSON.pretty_generate(hash))
+    # don't use "w" because it truncates the file before lock
+    File.open(path, File::RDWR | File::CREAT, 0644) do |file|
+      file.flock(File::LOCK_EX)
+      file.write(JSON.pretty_generate(hash))
+    end
   end
 end

data/lib/avro_turf/messaging.rb

@@ -21,8 +21,20 @@ class AvroTurf
   # 1: https://github.com/confluentinc/schema-registry
   class Messaging
     MAGIC_BYTE = [0].pack("C").freeze
-    DecodedMessage = Struct.new(:schema_id, :writer_schema, :reader_schema, :message)
-    private_constant(:DecodedMessage)
+
+    class DecodedMessage
+      attr_reader :schema_id
+      attr_reader :writer_schema
+      attr_reader :reader_schema
+      attr_reader :message
+
+      def initialize(schema_id, writer_schema, reader_schema, message)
+        @schema_id = schema_id
+        @writer_schema = writer_schema
+        @reader_schema = reader_schema
+        @message = message
+      end
+    end
 
     # Instantiate a new Messaging instance with the given configuration.
     #
@@ -34,7 +46,7 @@ class AvroTurf
     # namespace            - The String default schema namespace.
     # registry_path_prefix - The String URL path prefix used to namespace schema registry requests (optional).
     # logger               - The Logger that should be used to log information (optional).
-    # proxy                - Forward the request via  proxy (optional).
+    # proxy                - Forward the request via  proxy (optional).
     # user                 - User for basic auth (optional).
     # password             - Password for basic auth (optional).
     # ssl_ca_file          - Name of file containing CA certificate (optional).
@@ -130,7 +142,7 @@ class AvroTurf
       writer.write(message, encoder)
 
       stream.string
-    rescue Excon::Errors::NotFound
+    rescue Excon::Error::NotFound
       if schema_id
         raise SchemaNotFoundError.new("Schema with id: #{schema_id} is not found on registry")
       else

data/lib/avro_turf/version.rb

@@ -1,3 +1,3 @@
 class AvroTurf
-  VERSION = "1.7.0"
+  VERSION = "1.8.0"
 end

data/spec/avro_turf_spec.rb

@@ -16,6 +16,13 @@ describe AvroTurf do
               {
                 "type": "string",
                 "name": "full_name"
+              },
+              {
+                "name": "birth_date",
+                "type": {
+                  "type": "int",
+                  "logicalType": "date"
+                }
               }
             ]
           }
@@ -24,7 +31,8 @@ describe AvroTurf do
 
       it "encodes data with Avro" do
         data = {
-          "full_name" => "John Doe"
+          "full_name" => "John Doe",
+          "birth_date" => Date.new(1934, 1, 2)
         }
 
         encoded_data = avro.encode(data, schema_name: "person")
@@ -36,7 +44,8 @@ describe AvroTurf do
         compressed_avro = AvroTurf.new(schemas_path: "spec/schemas/", codec: "deflate")
 
         data = {
-          "full_name" => "John Doe" * 100
+          "full_name" => "John Doe" * 100,
+          "birth_date" => Date.new(1934, 1, 2)
         }
 
         uncompressed_data = avro.encode(data, schema_name: "person")
@@ -329,5 +338,33 @@ describe AvroTurf do
       datum = { message: "hello" }
       expect(avro.valid?(datum, schema_name: "postcard")).to eq true
     end
+
+    it "handles logicalType of date in schema" do
+      define_schema "postcard.avsc", <<-AVSC
+        {
+          "name": "postcard",
+          "type": "record",
+          "fields": [
+            {
+              "name": "message",
+              "type": "string"
+            },
+            {
+              "name": "sent_date",
+              "type": {
+                "type": "int",
+                "logicalType": "date"
+              }
+            }
+          ]
+        }
+      AVSC
+
+      datum = { 
+        message: "hello",
+        sent_date: Date.new(2022, 9, 11)
+      }
+      expect(avro.valid?(datum, schema_name: "postcard")).to eq true
+    end
   end
 end

data/spec/core_ext/date_spec.rb

@@ -1,6 +1,6 @@
 describe Date, "#as_avro" do
-  it "returns an ISO8601 string describing the time" do
+  it "returns Date object describing the time" do
     date = Date.today
-    expect(date.as_avro).to eq(date.iso8601)
+    expect(date.as_avro).to eq(date)
   end
 end

data/spec/spec_helper.rb

@@ -22,6 +22,16 @@ module Helpers
   end
 end
 
+# gem `fakefs` does not support flock for the file, and require patch
+# https://github.com/fakefs/fakefs/issues/433
+module FakeFS
+  class File < StringIO
+    def flock(*)
+      true
+    end
+  end
+end
+
 RSpec.configure do |config|
   config.include FakeFS::SpecHelpers
   config.include Helpers

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: avro_turf
 version: !ruby/object:Gem::Version
-  version: 1.7.0
+  version: 1.8.0
 platform: ruby
 authors:
 - Daniel Schierbeck
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-07-25 00:00:00.000000000 Z
+date: 2022-11-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: avro