prometheus-client-mmap 0.28.1 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 397a7f8162a7c2ef33bb35dbc4a1985891e9b96fc492319dfc1b6a8e9ec0cbb7
-  data.tar.gz: 2848f51578cdd4aabf3e13102f42b7b1f07ffc932a15f84cf2c85b2ef92b6c78
+  metadata.gz: 97fc31e7c9aad5a4b25d1d02cc7aa5b44f2a74d05a80776aa6d9f961b51a3631
+  data.tar.gz: '010394948353bb52e8aa4cd12dd45f22d3026099ae05b6d600905872ac094fc2'
 SHA512:
-  metadata.gz: c61ba2713843070ff41449a82a62ee0cb23502d6194e0c2ad79644e3254178f945aa34f2d6af123115389453c13441bf54c22aab36bf05ef33464e3e06f20b58
-  data.tar.gz: 9e504a92a22eaa933099786e0858bd6d59539a510a29362756cd1636c423e5b5f742a50ce1dac1f19bcef633d9354b11d3ba19d2d1ad4a99d495be9f7d7c69f4
+  metadata.gz: fcd7c42ca842979e64b2e510f18cd9113ace7ab6d68a1f017555defd16940a52aac7a081d86a50654045229f098db2bc53de38e7a47e63edab8ed049b1e32629
+  data.tar.gz: b2f6438a636775e0bf902f244fcd58b0c97fbe3f1f46edf593a5838f0db9c35219de452322cf03b627e2053bbc72b81e2ff98ac7f1f2bbdb20e26c53bdce79df

data/.tool-versions

@@ -1 +1 @@
-rust 1.65.0
+rust 1.73.0

data/README.md

@@ -53,19 +53,9 @@ http_requests.increment
 
 ## Rust extension
 
-In an effort to improve maintainability, there is now a Rust
+This gem now uses a rewritten Rust extension instead of C.
 implementation that reads the metric files and outputs the multiprocess
-metrics to text. If `rustc` is available, then the Rust extension will
-be built automatically. The `use_rust` keyword argument can be used:
-
-```ruby
-puts Prometheus::Client::Formats::Text.marshal_multiprocess(use_rust: true)
-```
-
-Starting with v0.26.0, `use_rust` defaults to `true`.
-
-Note that this parameter will likely be deprecated and removed once the Rust
-extension becomes the only supported mode.
+metrics to text. This implementation is significantly faster than the C extension.
 
 ### Rack middleware
 
@@ -219,24 +209,6 @@ Set `prometheus_multiproc_dir` environment variable to the path where you want m
 prometheus_multiproc_dir=/tmp
 ```
 
-### Multiprocess metrics via Rust extension
-
-By default, the Rust extension will be used to read and write multiprocess
-metrics if the `fast_mmaped_file_rs` extension is available. This should
-be significantly faster than the C extension.
-
-If the environment variable `prometheus_rust_multiprocess_metrics=false`
-is set or if the `rust_multiprocess_metrics` configuration setting is
-`false` , the C extension will be used.
-
-### Read and write metrics via Rust extension
-
-By default, the Rust extension will be used to read and write metrics
-from the mmaped file if the `fast_mmaped_file_rs` extension is available.
-
-To use the C extension, set the environment variable
-`prometheus_rust_mmaped_file=false`.
-
 ## Pitfalls
 
 ### PID cardinality

data/ext/fast_mmaped_file_rs/extconf.rb

@@ -24,7 +24,5 @@ if find_executable('rustc')
     ARGV = trimmed_argv
   end
 else
-  puts 'rustc not found, skipping Rust extension.'
-
-  File.write('Makefile', dummy_makefile($srcdir).join(''))
+  raise 'rustc not found. prometheus-client-mmap now requires Rust.'
 end

data/ext/fast_mmaped_file_rs/src/error.rs

@@ -100,6 +100,13 @@ pub enum MmapError {
     /// No mmap open.
     #[error("unmapped file")]
     UnmappedFile,
+    /// File length changed by another process.
+    #[error("size of {path} {len} does not match previous size {expected_len}. Another process may have modified it.")]
+    UnexpectedFileLength {
+        path: String,
+        len: usize,
+        expected_len: usize,
+    },
     /// A custom error message with `strerror(3)` appended.
     #[error("{0}")]
     WithErrno(String),
@@ -162,6 +169,7 @@ impl MmapError {
             MmapError::Other(_) => RubyError::Arg,
             MmapError::PromParsing(_) => RubyError::PromParsing,
             MmapError::UnmappedFile => RubyError::Io,
+            MmapError::UnexpectedFileLength { .. } => RubyError::Runtime,
             MmapError::WithErrno(_) => RubyError::Io,
         }
     }

data/ext/fast_mmaped_file_rs/src/mmap/inner.rs

@@ -5,7 +5,7 @@ use std::fs::File;
 use std::mem::size_of;
 use std::ops::Range;
 use std::os::unix::prelude::{AsRawFd, RawFd};
-use std::path::PathBuf;
+use std::path::{Path, PathBuf};
 
 use crate::error::{MmapError, RubyError};
 use crate::raw_entry::RawEntry;
@@ -34,6 +34,8 @@ pub(super) struct InnerMmap {
     ///
     /// Equivalent to `i_mm->t->real` in the C implementation.
     len: usize,
+    /// The length of the file being mmapped at the time of mapping.
+    file_len: usize,
 }
 
 impl InnerMmap {
@@ -41,19 +43,12 @@ impl InnerMmap {
     /// Use when mmapping a file for the first time. When re-mapping a file
     /// after expanding it the `reestablish` function should be used.
     pub fn new(path: PathBuf, file: File) -> Result<Self> {
-        let stat = file.metadata().map_err(|e| {
-            MmapError::legacy(
-                format!("Can't stat {}: {e}", path.display()),
-                RubyError::Arg,
-            )
-        })?;
-
-        let file_size = util::cast_chk::<_, usize>(stat.len(), "file length")?;
+        let initial_file_len = get_file_len(&path, &file)?;
 
         // We need to ensure the underlying file descriptor is at least a page size.
         // Otherwise, we could get a SIGBUS error if mmap() attempts to read or write
         // past the file.
-        let reserve_size = Self::next_page_boundary(file_size)?;
+        let reserve_size = Self::next_page_boundary(initial_file_len)?;
 
         // Cast: no-op.
         Self::reserve_mmap_file_bytes(file.as_raw_fd(), reserve_size as off_t).map_err(|e| {
@@ -67,21 +62,23 @@ impl InnerMmap {
         })?;
 
         // Ensure we always have space for the header.
-        let map_len = file_size.max(HEADER_SIZE);
+        let map_len = initial_file_len.max(HEADER_SIZE);
 
         // SAFETY: There is the possibility of UB if the file is modified outside of
         // this program.
         let map = unsafe { MmapOptions::new().len(map_len).map_mut(&file) }.map_err(|e| {
             MmapError::legacy(format!("mmap failed ({}): {e}", errno()), RubyError::Arg)
         })?;
+        let file_len = util::cast_chk::<_, usize>(reserve_size, "file length")?;
 
-        let len = file_size;
+        let len = initial_file_len;
 
         Ok(Self {
             file,
             path,
             map,
             len,
+            file_len,
         })
     }
 
@@ -97,12 +94,14 @@ impl InnerMmap {
         // to the old length at this point if closing the file. Matching C implementation
         // for now.
         let len = map_len;
+        let file_len = get_file_len(&path, &file)?;
 
         Ok(Self {
             file,
             path,
             map,
             len,
+            file_len,
         })
     }
 
@@ -186,7 +185,7 @@ impl InnerMmap {
         self.len
     }
 
-    /// The total length in bytes of the mmapped file.
+    /// The total length in bytes of the mmapped region of the file.
     ///
     /// Equivalent to `i_mm->t->len` in the C implementation.
     #[inline]
@@ -194,6 +193,21 @@ impl InnerMmap {
         self.map.len()
     }
 
+    /// `stat` the mmapped file and return its length.
+    pub fn check_file_len(&self) -> Result<usize> {
+        get_file_len(&self.path, &self.file)
+    }
+
+    /// The length of the file at the time it was mmapped.
+    pub fn file_len(&self) -> usize {
+        self.file_len
+    }
+
+    /// The path of the mmapped file.
+    pub fn path(&self) -> String {
+        self.path.to_string_lossy().to_string()
+    }
+
     /// Update the length of the mmap considered to be written.
     pub fn set_len(&mut self, len: usize) {
         self.len = len;
@@ -312,6 +326,17 @@ impl InnerMmap {
     }
 }
 
+fn get_file_len(path: &Path, file: &File) -> Result<usize> {
+    let stat = file.metadata().map_err(|e| {
+        MmapError::legacy(
+            format!("Can't stat {}: {e}", path.display()),
+            RubyError::Arg,
+        )
+    })?;
+
+    util::cast_chk::<_, usize>(stat.len(), "file length")
+}
+
 #[cfg(test)]
 mod test {
     use nix::unistd::{self, SysconfVar};

data/ext/fast_mmaped_file_rs/src/mmap.rs

@@ -441,17 +441,49 @@ impl MmapedFile {
     }
 
     /// Check that the mmap is large enough to contain the value to be added,
-    /// and expand it to fit if necessary.
+    /// and expand it to fit if necessary. Will return an error if the file
+    /// has been resized by another process.
     fn check_expand(&self, rb_self: Obj<Self>, key_len: usize) -> magnus::error::Result<()> {
         // CAST: no-op on 32-bit, widening on 64-bit.
         let used = self.inner(|inner| inner.load_used())? as usize;
         let entry_len = RawEntry::calc_total_len(key_len)?;
+        let required_len = used.add_chk(entry_len)?;
+        let mut target_cap = self.capacity();
 
+        let mut needs_resize = false;
         // We need the mmapped region to contain at least one byte beyond the
-        // written data to create a NUL- terminated C string. Validate that
-        // new length does not exactly match or exceed the length of the mmap.
-        while self.capacity() <= used.add_chk(entry_len)? {
-            self.expand_to_fit(rb_self, self.capacity().mul_chk(2)?)?;
+        // written data to create a NUL-terminated C string. Validate that
+        // new length does not exactly match the length of the mmap.
+        if target_cap <= required_len {
+            needs_resize = true;
+            target_cap = required_len + 1;
+        }
+
+        let (current_file_len, original_file_len) =
+            self.inner(|inner| Ok((inner.check_file_len()?, inner.file_len())))?;
+
+        let file_modified = current_file_len != original_file_len;
+        if file_modified {
+            // Ensure our new capacity is at least as large as the actual file.
+            // Erroring out without first resizing the mmap would leave us stuck
+            // in a bad state.
+            target_cap = current_file_len
+        }
+
+        if needs_resize {
+            self.expand_to_fit(rb_self, target_cap)?;
+        }
+
+        if file_modified {
+            // Return an error so the caller is alerted that there were concurrent writes to the
+            // file.
+            let path = self.inner(|inner| Ok(inner.path()))?;
+            return Err(MmapError::UnexpectedFileLength {
+                path,
+                len: current_file_len,
+                expected_len: original_file_len,
+            }
+            .into());
         }
 
         Ok(())
@@ -652,6 +684,7 @@ mod test {
     use super::*;
     use crate::raw_entry::RawEntry;
     use crate::testhelper::TestFile;
+    use std::os::unix::prelude::AsRawFd;
 
     /// Create a wrapped MmapedFile object.
     fn create_obj() -> Obj<MmapedFile> {
@@ -881,4 +914,37 @@ mod test {
         // Validate that we have expanded the mmap, ensuring a trailing NUL.
         assert!(rs_self.capacity() > current_cap);
     }
+
+    #[test]
+    fn test_detect_external_resize() {
+        let _cleanup = unsafe { magnus::embed::init() };
+        let ruby = magnus::Ruby::get().unwrap();
+        crate::init(&ruby).unwrap();
+
+        let TestFile {
+            file,
+            path,
+            dir: _dir,
+        } = TestFile::new(&[0u8; 8]);
+
+        let path_str = path.display().to_string();
+        let rpath = RString::new(&path_str);
+
+        // Object created successfully
+        let result: std::result::Result<Obj<MmapedFile>, Error> =
+            eval!("FastMmapedFileRs.new(path)", path = rpath);
+        assert!(result.is_ok());
+        let obj = result.unwrap();
+
+        let rs_self = &*obj;
+
+        let file_len = 1 << 32;
+        nix::unistd::ftruncate(file.as_raw_fd(), file_len).unwrap();
+
+        // We error when the file is externally modified.
+        assert!(rs_self.check_expand(obj, 512).is_err());
+
+        // We updated the object to use the new size.
+        assert_eq!(rs_self.capacity(), file_len as usize);
+    }
 }

data/lib/prometheus/client/configuration.rb

@@ -7,14 +7,13 @@ require 'tmpdir'
 module Prometheus
   module Client
     class Configuration
-      attr_accessor :value_class, :multiprocess_files_dir, :initial_mmap_file_size, :logger, :pid_provider, :rust_multiprocess_metrics
+      attr_accessor :value_class, :multiprocess_files_dir, :initial_mmap_file_size, :logger, :pid_provider
 
       def initialize
         @value_class = ::Prometheus::Client::MmapedValue
         @initial_mmap_file_size = ::Prometheus::Client::PageSize.page_size(fallback_page_size: 4096)
         @logger = Logger.new($stdout)
         @pid_provider = Process.method(:pid)
-        @rust_multiprocess_metrics = ENV.fetch('prometheus_rust_multiprocess_metrics', 'true') == 'true'
         @multiprocess_files_dir = ENV.fetch('prometheus_multiproc_dir') do
           Dir.mktmpdir("prometheus-mmap")
         end

data/lib/prometheus/client/formats/text.rb

@@ -1,6 +1,5 @@
 require 'prometheus/client/uses_value_type'
 require 'prometheus/client/helper/json_parser'
-require 'prometheus/client/helper/loader'
 require 'prometheus/client/helper/plain_file'
 require 'prometheus/client/helper/metrics_processing'
 require 'prometheus/client/helper/metrics_representation'
@@ -27,22 +26,12 @@ module Prometheus
             Helper::MetricsRepresentation.to_text(metrics)
           end
 
-          def marshal_multiprocess(path = Prometheus::Client.configuration.multiprocess_files_dir, use_rust: true)
+          def marshal_multiprocess(path = Prometheus::Client.configuration.multiprocess_files_dir)
             file_list = Dir.glob(File.join(path, '*.db')).sort
               .map {|f| Helper::PlainFile.new(f) }
               .map {|f| [f.filepath, f.multiprocess_mode.to_sym, f.type.to_sym, f.pid] }
 
-            if use_rust && Prometheus::Client::Helper::Loader.rust_impl_available?
               FastMmapedFileRs.to_metrics(file_list.to_a)
-            else
-              FastMmapedFile.to_metrics(file_list.to_a)
-            end
-          end
-
-          def rust_impl_available?
-            return @rust_available unless @rust_available.nil?
-
-            check_for_rust
           end
 
           private

data/lib/prometheus/client/helper/mmaped_file.rb

@@ -1,29 +1,18 @@
 require 'prometheus/client/helper/entry_parser'
 require 'prometheus/client/helper/file_locker'
-require 'prometheus/client/helper/loader'
 
 # load precompiled extension if available
 begin
   ruby_version = /(\d+\.\d+)/.match(RUBY_VERSION)
-  require_relative "../../../#{ruby_version}/fast_mmaped_file"
+  require_relative "../../../#{ruby_version}/fast_mmaped_file_rs"
 rescue LoadError
-  require 'fast_mmaped_file'
+  require 'fast_mmaped_file_rs'
 end
 
 module Prometheus
   module Client
     module Helper
-      # We can't check `Prometheus::Client.configuration` as this creates a circular dependency
-      if (ENV.fetch('prometheus_rust_mmaped_file', 'true') == "true" &&
-          Prometheus::Client::Helper::Loader.rust_impl_available?)
-        class MmapedFile < FastMmapedFileRs
-        end
-      else
-        class MmapedFile < FastMmapedFile
-        end
-      end
-
-      class MmapedFile
+      class MmapedFile < FastMmapedFileRs
         include EntryParser
 
         attr_reader :filepath, :size

data/lib/prometheus/client/rack/exporter.rb

@@ -62,10 +62,8 @@ module Prometheus
         end
 
         def respond_with(format)
-          rust_enabled = Prometheus::Client.configuration.rust_multiprocess_metrics
-
           response = if Prometheus::Client.configuration.value_class.multiprocess
-                       format.marshal_multiprocess(use_rust: rust_enabled)
+                       format.marshal_multiprocess
                      else
                        format.marshal
                      end

data/lib/prometheus/client/version.rb

@@ -1,5 +1,5 @@
 module Prometheus
   module Client
-    VERSION = '0.28.1'.freeze
+    VERSION = '1.0.0'.freeze
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: prometheus-client-mmap
 version: !ruby/object:Gem::Version
-  version: 0.28.1
+  version: 1.0.0
 platform: ruby
 authors:
 - Tobias Schmidt
@@ -11,7 +11,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-10-05 00:00:00.000000000 Z
+date: 2023-11-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rb_sys
@@ -125,29 +125,11 @@ email:
 - wchandler@gitlab.com
 executables: []
 extensions:
-- ext/fast_mmaped_file/extconf.rb
 - ext/fast_mmaped_file_rs/extconf.rb
 extra_rdoc_files: []
 files:
 - ".tool-versions"
 - README.md
-- ext/fast_mmaped_file/extconf.rb
-- ext/fast_mmaped_file/fast_mmaped_file.c
-- ext/fast_mmaped_file/file_format.c
-- ext/fast_mmaped_file/file_format.h
-- ext/fast_mmaped_file/file_parsing.c
-- ext/fast_mmaped_file/file_parsing.h
-- ext/fast_mmaped_file/file_reading.c
-- ext/fast_mmaped_file/file_reading.h
-- ext/fast_mmaped_file/globals.h
-- ext/fast_mmaped_file/mmap.c
-- ext/fast_mmaped_file/mmap.h
-- ext/fast_mmaped_file/rendering.c
-- ext/fast_mmaped_file/rendering.h
-- ext/fast_mmaped_file/utils.c
-- ext/fast_mmaped_file/utils.h
-- ext/fast_mmaped_file/value_access.c
-- ext/fast_mmaped_file/value_access.h
 - ext/fast_mmaped_file_rs/.cargo/config.toml
 - ext/fast_mmaped_file_rs/Cargo.lock
 - ext/fast_mmaped_file_rs/Cargo.toml
@@ -174,7 +156,6 @@ files:
 - lib/prometheus/client/helper/entry_parser.rb
 - lib/prometheus/client/helper/file_locker.rb
 - lib/prometheus/client/helper/json_parser.rb
-- lib/prometheus/client/helper/loader.rb
 - lib/prometheus/client/helper/metrics_processing.rb
 - lib/prometheus/client/helper/metrics_representation.rb
 - lib/prometheus/client/helper/mmaped_file.rb
@@ -195,26 +176,6 @@ files:
 - lib/prometheus/client/support/unicorn.rb
 - lib/prometheus/client/uses_value_type.rb
 - lib/prometheus/client/version.rb
-- vendor/c/hashmap/.gitignore
-- vendor/c/hashmap/LICENSE
-- vendor/c/hashmap/README.md
-- vendor/c/hashmap/_config.yml
-- vendor/c/hashmap/src/hashmap.c
-- vendor/c/hashmap/src/hashmap.h
-- vendor/c/hashmap/test/Makefile
-- vendor/c/hashmap/test/hashmap_test.c
-- vendor/c/jsmn/.travis.yml
-- vendor/c/jsmn/LICENSE
-- vendor/c/jsmn/Makefile
-- vendor/c/jsmn/README.md
-- vendor/c/jsmn/example/jsondump.c
-- vendor/c/jsmn/example/simple.c
-- vendor/c/jsmn/jsmn.c
-- vendor/c/jsmn/jsmn.h
-- vendor/c/jsmn/library.json
-- vendor/c/jsmn/test/test.h
-- vendor/c/jsmn/test/tests.c
-- vendor/c/jsmn/test/testutil.h
 homepage: https://gitlab.com/gitlab-org/prometheus-client-mmap
 licenses:
 - Apache-2.0
@@ -237,6 +198,6 @@ requirements: []
 rubygems_version: 3.3.26
 signing_key:
 specification_version: 4
-summary: A suite of instrumentation metric primitivesthat can be exposed through a
-  web services interface.
+summary: A suite of instrumentation metric primitives that can be exposed through
+  a web services interface.
 test_files: []

data/ext/fast_mmaped_file/extconf.rb

@@ -1,30 +0,0 @@
-require 'mkmf'
-require 'fileutils'
-
-$CFLAGS << ' -std=c99 -D_POSIX_C_SOURCE=200809L -Wall -Wextra'
-
-if enable_config('fail-on-warning')
-  $CFLAGS << ' -Werror'
-end
-
-if enable_config('debug')
-  $CFLAGS << ' -O0 -g'
-end
-
-if enable_config('address-sanitizer')
-  $CFLAGS << ' -O -fsanitize=address -fno-omit-frame-pointer -g'
-end
-
-CONFIG['warnflags'].slice!(/ -Wdeclaration-after-statement/)
-
-cwd = File.expand_path(File.dirname(__FILE__))
-vendor_dir = File.join(cwd, '../../vendor/c')
-src_dir = File.join(cwd, '../../ext/fast_mmaped_file')
-
-src_files = %W[#{vendor_dir}/jsmn/jsmn.c #{vendor_dir}/hashmap/src/hashmap.c]
-FileUtils.cp(src_files, src_dir)
-
-$INCFLAGS << " -I#{vendor_dir}/jsmn -I#{vendor_dir}/hashmap/src"
-
-dir_config('fast_mmaped_file')
-create_makefile('fast_mmaped_file')

data/ext/fast_mmaped_file/fast_mmaped_file.c

@@ -1,122 +0,0 @@
-#include <errno.h>
-#include <hashmap.h>
-#include <jsmn.h>
-#include <ruby.h>
-#include <ruby/intern.h>
-#include <sys/mman.h>
-
-#include "file_parsing.h"
-#include "file_reading.h"
-#include "globals.h"
-#include "mmap.h"
-#include "rendering.h"
-#include "utils.h"
-#include "value_access.h"
-
-VALUE MMAPED_FILE = Qnil;
-
-ID sym_min;
-ID sym_max;
-ID sym_livesum;
-ID sym_gauge;
-ID sym_pid;
-ID sym_samples;
-
-VALUE prom_eParsingError;
-
-int aggregate_files(struct hashmap *map, VALUE list_of_files) {
-    buffer_t reading_buffer;
-    memset(&reading_buffer, 0, sizeof(buffer_t));
-
-    for (int i = 0; i < RARRAY_LEN(list_of_files); i++) {
-        VALUE params = RARRAY_PTR(list_of_files)[i];
-        file_t file;
-
-        if (!file_open_from_params(&file, params)) {
-            buffer_dispose(&reading_buffer);
-            return 0;
-        }
-
-        if (!read_from_file(&file, &reading_buffer)) {
-            buffer_dispose(&reading_buffer);
-            file_close(&file);
-            return 0;
-        }
-
-        if (!process_buffer(&file, &reading_buffer, map)) {
-            buffer_dispose(&reading_buffer);
-            file_close(&file);
-            return 0;
-        }
-
-        if (!file_close(&file)) {
-            buffer_dispose(&reading_buffer);
-            return 0;
-        }
-    }
-
-    buffer_dispose(&reading_buffer);
-    return 1;
-}
-
-VALUE method_to_metrics(VALUE UNUSED(self), VALUE file_list) {
-    struct hashmap map;
-    hashmap_setup(&map);
-
-    if (!aggregate_files(&map, file_list)) {  // all entries in map are now copies that need to be disposed
-        hashmap_destroy(&map);
-        raise_last_exception();
-        return Qnil;
-    }
-
-    entry_t **sorted_entries;
-
-    if (!sort_map_entries(&map, &sorted_entries)) {
-        hashmap_destroy(&map);
-
-        raise_last_exception();
-        return Qnil;
-    }
-
-    VALUE rv = rb_str_new("", 0);
-    if (!entries_to_string(rv, sorted_entries, hashmap_size(&map))) {
-        free(sorted_entries);
-        hashmap_destroy(&map);
-
-        raise_last_exception();
-        return Qnil;
-    }
-
-    RB_GC_GUARD(file_list);  // ensure file list is not GCed before this point
-    free(sorted_entries);
-    hashmap_destroy(&map);
-    return rv;
-}
-
-void Init_fast_mmaped_file() {
-    sym_gauge = rb_intern("gauge");
-    sym_min = rb_intern("min");
-    sym_max = rb_intern("max");
-    sym_livesum = rb_intern("livesum");
-    sym_pid = rb_intern("pid");
-    sym_samples = rb_intern("samples");
-
-    prom_eParsingError = rb_define_class("PrometheusParsingError", rb_eRuntimeError);
-
-    MMAPED_FILE = rb_define_class("FastMmapedFile", rb_cObject);
-    rb_define_const(MMAPED_FILE, "MAP_SHARED", INT2FIX(MAP_SHARED));
-
-    rb_define_singleton_method(MMAPED_FILE, "to_metrics", method_to_metrics, 1);
-
-    rb_define_alloc_func(MMAPED_FILE, mm_s_alloc);
-    rb_define_singleton_method(MMAPED_FILE, "new", mm_s_new, -1);
-    rb_define_method(MMAPED_FILE, "initialize", mm_init, 1);
-    rb_define_method(MMAPED_FILE, "slice", mm_aref_m, -1);
-    rb_define_method(MMAPED_FILE, "sync", mm_msync, -1);
-    rb_define_method(MMAPED_FILE, "munmap", mm_unmap, 0);
-
-    rb_define_method(MMAPED_FILE, "used", method_load_used, 0);
-    rb_define_method(MMAPED_FILE, "used=", method_save_used, 1);
-    rb_define_method(MMAPED_FILE, "fetch_entry", method_fetch_entry, 3);
-    rb_define_method(MMAPED_FILE, "upsert_entry", method_upsert_entry, 3);
-}

data/ext/fast_mmaped_file/file_format.c

@@ -1,5 +0,0 @@
-#include "file_format.h"
-
-inline uint32_t padding_length(uint32_t key_length) {
-    return 8 - (sizeof(uint32_t) + key_length) % 8;  // padding | 8 byte aligned
-}