influxer 1.1.5 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7088f24ca98f504a152e38aff5f5ccc895eef65f1916aa5986bc425d9ef2d532
-  data.tar.gz: 733e79c1bc8f7023fcb23032f519b7cfe5b7a61e373b1e7b213a821adcb29c43
+  metadata.gz: af367b48817287138ec7eddf616361dd101b69e90db4610f2640899a0c4bfcd1
+  data.tar.gz: 4a74d454aa9a9fb044530295878c0c75ab8a2a4705115dfeddf5c358c4373c01
 SHA512:
-  metadata.gz: 9f4217519c7c72281f1ed91fe9a22134a89be30f1e473f1ac06fc227599403d0b0d0938da7ebbbcc5f3e1c98f13916c560fb6692edbf59360d883be92766e65e
-  data.tar.gz: 9e88838a7c84d1ac73be7a583e4bebcc73c48ef22f3bdab12eab0adce32350bb694094c6833a2c38089524d7c88dc58fb8917ba35a74c9e353a7c2a809ab2bfc
+  metadata.gz: dd988870b77abda0c5db4d0f0f9d2d9c88796ed7d0833c2f32e54c27bbd9bbdf7de8fe0344bc67e62f4c85a45af000afe9f9aa02a17eceedb47b00ddb89422b7
+  data.tar.gz: c290df58fc6f0946ae88097fdbfc698a72d760fffb71451e6ab4fb7ff1cf1c256d500659c8bd59b8844bf837b4bb240e2e88826869c1d916214f4c76831e8406

data/CHANGELOG.md

@@ -0,0 +1,102 @@
+# Change log
+
+## master (unreleased)
+
+## 1.3.0 (2020-10-27)
+
+- Fixes [#53](https://github.com/palkan/influxer/issues/53) Influxer client configuration issue with anyway config v2 and higher.([@AlexanderShvaykin][])
+
+## 1.2.2 (2020-10-27)
+
+- Fixes [#49](https://github.com/palkan/influxer/issues/49)  Cache hash configuration cannot be applied.([@AlexanderShvaykin][])
+- Fixes [#47](https://github.com/palkan/influxer/issues/47) Can't delete data when retention policy is set for a metric. ([@MPursche][])
+
+## 1.2.1 (2020-07-09)
+
+- Support for setting timezone in queries to configure influx time calculations, e.g., time epoch aggregation ([@jklimke][])
+
+  [PR](https://github.com/palkan/influxer/pull/46)
+
+## 1.2.0 (2019-05-20)
+
+- **Require Ruby 2.4+**
+
+## 1.1.6
+
+- [Fixes [#41](https://github.com/palkan/influxer/issues/41)] Fix query building with empty arrays in `where` clause ([@dimiii][])
+
+  [PR](https://github.com/palkan/influxer/pull/44)
+
+  **BREAKING:** `where.not` now returns non-empty result for an empty array.
+
+- Support of year alias in queries ([@dimiii][])
+
+- [Fixes [#40](https://github.com/palkan/influxer/issues/40)] Avoid adding precision suffix to write queries. ([@palkan][])
+
+## 1.1.5
+
+- [Fixes [#37](https://github.com/palkan/influxer/issues/37)] Timestamp ranges are quoted again. ([@jklimke][])
+
+## 1.1.4
+
+- [Fixes [#35](https://github.com/palkan/influxer/issues/35)] Support time duration suffix and handle `'s'` and `'ms'` precision. ([@palkan][])
+
+  [PR](https://github.com/palkan/influxer/pull/36)
+
+  **BREAKING:** `Time`-like value are only type-casted for `time` key.
+
+## 1.1.2
+
+- Support exclusive ranges as `where` arguments. ([@MPursche][])
+
+```ruby
+# range including the end
+where(a: 1..4)
+#=> ... WHERE a >= 1 AND a <= 4
+
+#range excluding the end
+where(a: 1...4)
+#=> ... WHERE a >= 1 AND a < 4
+```
+
+## 1.1.1
+
+- [Fixes [#31](https://github.com/palkan/influxer/issues/31)] Fix bug with empty arrays in `where` clause
+
+- Introduce `Relation#none` method
+
+## 1.1.0
+
+### Features
+
+- Add ability to specify per-metrics retention-policy, precision and database
+
+Now you can override default configuration for a specific metrics class:
+
+```ruby
+class CustomMetrics < Influxer::Metrics
+  set_database "custom_db"
+  set_retention_policy :yearly
+  set_precision "ms"
+end
+```
+
+### Fixes
+
+- [Fixes [#30](https://github.com/palkan/influxer/issues/30)] Fix writing points with custom retention policy
+
+### Misc
+
+- Update Rubocop configuration and add Rubocop Rake task to defaults
+
+## 1.0.1
+
+- Fix missing `#delegate` in ActiveRecord 3.2
+
+See [changelog](https://github.com/palkan/influxer/blob/1.0.0/Changelog.md) for earlier versions.
+
+[@palkan]: https://github.com/palkan
+[@MPursche]: https://github.com/MPursche
+[@jklimke]: https://github.com/jklimke
+[@dimiii]: https://github.com/dimiii
+[@AlexanderShvaykin]: https://github.com/AlexanderShvaykin

data/{MIT-LICENSE → LICENSE.txt}

@@ -1,4 +1,4 @@
-Copyright 2014 palkan
+Copyright 2014-2020 Vladimir Dementyev
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/README.md

@@ -1,50 +1,109 @@
-[![Gem Version](https://badge.fury.io/rb/influxer.svg)](https://rubygems.org/gems/influxer) [![Build Status](https://travis-ci.org/palkan/influxer.svg?branch=master)](https://travis-ci.org/palkan/influxer) [![Dependency Status](https://dependencyci.com/github/palkan/influxer/badge)](https://dependencyci.com/github/palkan/influxer)
-## Influxer
+![Build](https://github.com/palkan/influxer/workflows/Build/badge.svg)
 
-**NOTE**: Version 0.3.x supports InfluxDB >= 0.9.0. For InfluxDB 0.8.x use [version 0.2.5](https://github.com/palkan/influxer/tree/0.2.5).
-
-**NOTE**: Influxer is Rails 4+ compatible!
+# Influxer
 
 Influxer provides an ActiveRecord-style way to work with [InfluxDB](https://influxdb.com/) with many useful features, such as:
-- Familar query language (use `select`, `where`, `not`, `group` etc).
-- Support for Regex conditions: `where(page_id: /^home\/.*/) #=> select * ... where page_id=~/^home\/.*/`.
-- Special query methods for InfluxDB: 
-  - `time` - group by time (e.g. `Metrics.time(:hour) => # select * ... group by time(1h)`);
-  - `past` - get only points for last hour/minute/whatever (e.g. `Metrics.past(:day) => # select * ... where time > now() - 1d`);
-  - `since` - get only points since date (e.g. `Metrics.since(Time.utc(2014,12,31)) => # select * ... where time > 1419984000s`);
-  - `merge` - merge series.
-- Scopes support
-  ```ruby
-  class Metrics < Influxer::Metrics
-    default_scope -> { time(:hour).limit(1000) }
-    tags :account_id
-    attributes :value
-    scope :unlimited, -> { limit(nil) }
-    scope :by_account, -> (id) { where(account_id: id) if id.present? }
-  end
-
-  Metrics.by_account(1)
-  # => select * from "metrics" group by time(1h) where account_id=1 limit 1000
-
-  Metrics.unlimited.by_account(1).time(:week)
-  # => select * from "metrics" group by time(1w) where account_id=1
-
-  ```
-- Integrate with your model:
-  ```ruby
-  class UserVisits < Influxer::Metrics
-  end
-
-  class User < ActiveRecord::Base
-    has_metrics :visits
-  end
-
-  user = User.find(1)
-  user.visits.write(page_id: 'home')
-  #=> < creates point {user_id: 1, page_id: 'home'} in 'user_visits' series >
-
-  user.visits.where(page_id: 'home')
-  #=> select * from user_visits where page_id='home'
-  ```
-    
-Find more on [Wiki](https://github.com/palkan/influxer/wiki).
+
+## Installation
+
+Adding to a gem:
+
+```ruby
+# my-cool-gem.gemspec
+Gem::Specification.new do |spec|
+  # ...
+  spec.add_dependency "influxer", ">= 1.2.0"
+  # ...
+end
+```
+
+Or adding to your project:
+
+```ruby
+# Gemfile
+gem "influxer", "~> 1.2"
+```
+
+## Usage
+
+### Metrics classes
+
+To query InfluxDB or write to it, you should define a metrics class first. Each metrics class represents a measurement/series (or multiple related measurements):
+
+```ruby
+class VisitsMetrics < Influxer::Metrics
+  # Define tags...
+  tags :account_id, :page_id
+  # ...and attributes
+  attributes :user_id, :browser
+end
+```
+
+### Querying
+
+Now you can use your metrics classes in a similar way to Active Record models to build queries. For example:
+
+```ruby
+VisitsMetrics.select(:account_id, :user_id).where(page_id: /^home\/.*/)
+```
+
+Influxer provides special query methods for dealing with time series:
+
+- Group by time: `Metrics.time(:hour) => # select * ... group by time(1h)`.
+- Select only points for the last hour/minute/whatever: `Metrics.past(:day) => # select * ... where time > now() - 1d`.
+- Select only points since the specified time: `Metrics.since(Time.utc(2014,12,31)) => # select * ... where time > 1419984000s`.
+- and more.
+
+See [our Wiki](https://github.com/palkan/influxer/wiki/Query-methods) for more.
+
+### Scopes support
+
+You can define scopes to re-use query conditions:
+
+```ruby
+class Metrics < Influxer::Metrics
+  tags :account_id
+  attributes :value
+
+  default_scope -> { time(:hour).limit(1000) }
+
+  scope :unlimited, -> { limit(nil) }
+  scope :by_account, ->(id) { where(account_id: id) if id.present? }
+end
+
+Metrics.by_account(1)
+# => select * from "metrics" group by time(1h) where account_id=1 limit 1000
+
+Metrics.unlimited.by_account(1).time(:week)
+# => select * from "metrics" group by time(1w) where account_id=1
+```
+
+### Active Record integration
+
+You can association metrics with Active Record models:
+
+```ruby
+class UserVisits < Influxer::Metrics
+end
+
+class User < ActiveRecord::Base
+  has_metrics :visits
+end
+
+user = User.find(1)
+user.visits.write(page_id: "home")
+#=> < creates point {user_id: 1, page_id: 'home'} in 'user_visits' series >
+
+user.visits.where(page_id: "home")
+#=> select * from user_visits where page_id='home'
+```
+
+Find more on [Wiki](https://github.com/palkan/influxer/wiki/ActiveRecord-integration).
+
+## Contributing
+
+Bug reports and pull requests are welcome on GitHub at [https://github.com/palkan/influxer](https://github.com/palkan/influxer).
+
+## License
+
+The gem is available as open source under the terms of the [MIT License](https://opensource.org/licenses/MIT).

data/lib/influxer.rb

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
 
-require 'influxer/version'
-require 'active_model'
+require "influxer/version"
+require "active_model"
+require "active_support/core_ext"
 
 # Rails client for InfluxDB
 module Influxer
@@ -9,16 +10,16 @@ module Influxer
     ActiveModel::VERSION::MAJOR == 3
   end
 
-  require 'influxer/config'
-  require 'influxer/client'
-  require 'influxer/metrics/metrics'
+  require "influxer/config"
+  require "influxer/client"
+  require "influxer/metrics/metrics"
 
   module Model # :nodoc:
-    require 'influxer/model'
+    require "influxer/model"
   end
 
-  require 'influxer/rails/client' if defined?(Rails)
-  require 'influxer/engine' if defined?(Rails)
+  require "influxer/rails/client" if defined?(Rails)
+  require "influxer/engine" if defined?(Rails)
 
   def self.config
     @config ||= Config.new
@@ -33,7 +34,7 @@ module Influxer
   end
 
   def self.reset!
-    @client.stop! unless @client.nil?
+    @client&.stop!
     @config = nil
     @client = nil
   end

data/lib/influxer/client.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
-require 'influxdb'
+require "influxdb"
 
 module Influxer
   # InfluxDB API client
   class Client < ::InfluxDB::Client
     def initialize
-      super Influxer.config.as_json.symbolize_keys!
+      super Influxer.config.to_h.symbolize_keys
     end
 
     def time_precision

data/lib/influxer/config.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'anyway_config'
+require "anyway_config"
 
 module Influxer
   # Influxer configuration
@@ -28,15 +28,28 @@ module Influxer
                 :denormalize,
                 :udp,
                 :async,
-                database: 'db',
-                time_precision: 'ns',
-                cache: false,
+                :cache_enabled,
+                :cache,
+                database: "db",
+                time_precision: "ns",
                 time_duration_suffix_enabled: false
 
     def load(*)
       super
-      # we want pass @cache value as options to cache store, so we want it to be a Hash
-      @cache = {}.with_indifferent_access if @cache == true
+      if cache_enabled.nil?
+        self.cache_enabled = cache_enabled_value
+      end
+    end
+
+    def cache=(value)
+      super
+      self.cache_enabled = cache_enabled_value
+    end
+
+    private
+
+    def cache_enabled_value
+      !!cache
     end
   end
 end

data/lib/influxer/engine.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'influxer'
+require "influxer"
 
 module Influxer
   class Engine < Rails::Engine # :nodoc:

data/lib/influxer/metrics/active_model3/model.rb

@@ -5,10 +5,8 @@ module Influxer
     # Replacement of ActiveModel::Model for ActiveModel 3
     module Model
       def initialize(attributes = {})
-        if attributes
-          attributes.each do |attr, value|
-            public_send("#{attr}=", value)
-          end
+        attributes&.each do |attr, value|
+          public_send("#{attr}=", value)
         end
 
         super()

data/lib/influxer/metrics/metrics.rb

@@ -1,16 +1,15 @@
 # frozen_string_literal: true
 
-require 'influxer/metrics/relation'
-require 'influxer/metrics/scoping'
-require 'influxer/metrics/quoting/timestamp'
-require 'influxer/metrics/active_model3/model'
+require "influxer/metrics/relation"
+require "influxer/metrics/scoping"
+require "influxer/metrics/quoting/timestamp"
+require "influxer/metrics/active_model3/model"
 
 module Influxer
   class MetricsError < StandardError; end
   class MetricsInvalid < MetricsError; end
 
   # Base class for InfluxDB querying and writing
-  # rubocop:disable Metrics/ClassLength
   class Metrics
     TIME_FACTOR = 1_000_000_000
     if Influxer.active_model3?
@@ -18,7 +17,6 @@ module Influxer
     else
       include ActiveModel::Model
     end
-    # rubocop:enable Style/MixinUsage
 
     include ActiveModel::Validations
     extend ActiveModel::Callbacks
@@ -35,7 +33,7 @@ module Influxer
           [
             :write, :write!, :select, :where,
             :group, :time, :past, :since,
-            :limit, :offset, :fill, :delete_all, :epoch
+            :limit, :offset, :fill, :delete_all, :epoch, :timezone
           ] + Influxer::Calculations::CALCULATION_METHODS
         ),
         to: :all
@@ -125,7 +123,7 @@ module Influxer
           quoted_series(val.call(instance), write: write)
         when Array
           if val.length > 1
-            "merge(#{val.map { |s| quoted_series(s, write: write) }.join(',')})"
+            "merge(#{val.map { |s| quoted_series(s, write: write) }.join(",")})"
           else
             quoted_series(val.first, write: write)
           end
@@ -133,7 +131,7 @@ module Influxer
           # Only add retention policy when selecting points
           # and not writing
           if !write && retention_policy.present?
-            [quote(@retention_policy), quote(val)].join('.')
+            [quote(@retention_policy), quote(val)].join(".")
           else
             quote(val)
           end
@@ -169,6 +167,7 @@ module Influxer
 
     def write!
       raise MetricsInvalid if invalid?
+
       write
     end
 
@@ -225,13 +224,11 @@ module Influxer
     end
 
     def parsed_timestamp
-      quote_timestamp @timestamp, client
+      quote_timestamp_for_write @timestamp, client
     end
-    # rubocop:enable Metrics/MethodLength
-    # rubocop:enable Metrics/AbcSize
 
     def unquote(name)
-      name.gsub(/(\A['"]|['"]\z)/, '')
+      name.gsub(/(\A['"]|['"]\z)/, "")
     end
   end
 end