embulk-input-mixpanel 0.2.1 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9a688fb2eb175f13ee5c82b8ea2da6c74ed69839
-  data.tar.gz: af0fdae8cc6fba0911ab269d7d8f354d51803f25
+  metadata.gz: 878c5ed629d13780bfccd6b0c03e41e6c07345cf
+  data.tar.gz: bd30844c67525e74f702b39563a56c166b48a5e9
 SHA512:
-  metadata.gz: 25381a19a9743a0b945d5d453931b1f93ab87009a79185b72eba32577a496550f1050b38ff8baa81c164d628a1c67714e785c27bf6833a04e69431bcad2e6ace
-  data.tar.gz: d69d65730c8c3a3d1ea1c85182e274ea8bde716662c132ee628e1c0be7d25e2b97c0a2c15dc0a7269b714677cf1635f04fcdd8a8237ec7353714d4f1ede05094
+  metadata.gz: 40fe5641a663dbb10b1489e06f1dcf36ffa053bdd8495b6a0a39c1edce1e39f76264a486b3d1f8a4cc203613ed6403bae0112deb72a3c4f3d30d1ecc1d16ae92
+  data.tar.gz: fe18510a1491d008bd58c5abd54b6cab2dd15f9d9ad4dbbb5767306134f4e3744615e0078f5abd9df4af90d69e838e6035fe5ab9060c09f60e1731687393b90c

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 0.3.0 - 2015-08-31
+
+This version breaks backword compatibility of mixpanel. `days` key in config was changed to `fetch_days`. For detail, please check README.md to modify your config.
+
+* [fixed] Fix the bug 1 day data can't be fetched [#21](https://github.com/treasure-data/embulk-input-mixpanel/pull/21)
+
 ## 0.2.1 - 2015-08-26
 
 * [fixed] Fix guess with recently from date [#18](https://github.com/treasure-data/embulk-input-mixpanel/pull/18)

data/README.md

@@ -36,7 +36,7 @@ To get it, you should log in mixpanel website, and click gear icon at the lower
 - **timezone**: project timezone(string, required)
 - **from_date**: From date to export (string, optional, default: today - 2)
   - NOTE: Mixpanel API supports to export data from at least 2 days before to at most the previous day.
-- **days**: Count of days range for exporting (integer, optional, default: from_date - (today - 1))
+- **fetch_days**: Count of days range for exporting (integer, optional, default: from_date - (today - 1))
   - NOTE: Mixpanel doesn't support to from_date > today - 2
 - **event**: The event or events to filter data (array, optional, default: nil)
 - **where**: Expression to filter data (c.f. https://mixpanel.com/docs/api-documentation/data-export-api#segmentation-expressions) (string, optional, default: nil)
@@ -51,7 +51,7 @@ in:
   api_secret: "API_SECRET"
   timezone: "US/Pacific"
   from_date: "2015-07-19"
-  days: 5
+  fetch_days: 5
 ```
 
 ## Run test

data/embulk-input-mixpanel.gemspec

@@ -1,7 +1,7 @@
 
 Gem::Specification.new do |spec|
   spec.name          = "embulk-input-mixpanel"
-  spec.version       = "0.2.1"
+  spec.version       = "0.3.0"
   spec.authors       = ["yoshihara", "uu59"]
   spec.summary       = "Mixpanel input plugin for Embulk"
   spec.description   = "Loads records from Mixpanel."

data/lib/embulk/input/mixpanel.rb

@@ -29,7 +29,7 @@ module Embulk
           Embulk.logger.warn "Mixpanel allow 2 days before to from_date, so no data is input."
           target_dates = []
         else
-          days = config.param(:days, :integer, default: nil)
+          days = config.param(:fetch_days, :integer, default: nil)
 
           if days.nil?
             # When no 'days' is specified in config file, so dates is
@@ -40,7 +40,7 @@ module Embulk
           else
             # When 'days' is specified in config file and it is satisfied,
             # so it is used for dates.
-            dates = from_date..(from_date + days)
+            dates = from_date..(from_date + days - 1)
           end
 
           target_dates = dates.find_all {|date| date < Date.today}

data/test/embulk/input/test_mixpanel.rb

@@ -10,7 +10,7 @@ module Embulk
       FROM_DATE = "2015-02-22".freeze
       TO_DATE = "2015-03-02".freeze
       DAYS = 8
-      DATES = Date.parse(FROM_DATE)..(Date.parse(FROM_DATE) + DAYS)
+      DATES = Date.parse(FROM_DATE)..(Date.parse(FROM_DATE) + DAYS - 1)
       TIMEZONE = "Asia/Tokyo".freeze
 
       DURATIONS = [
@@ -165,7 +165,7 @@ module Embulk
 
         def test_negative_days
           assert_raise(Embulk::ConfigError) do
-            Mixpanel.transaction(transaction_config((Date.today - 1).to_s).merge(days: -1))
+            Mixpanel.transaction(transaction_config((Date.today - 1).to_s).merge(fetch_days: -1))
           end
         end
 
@@ -213,7 +213,7 @@ module Embulk
           def transaction_config
             _config = config.merge(
               from_date: dates.first.to_s,
-              days: dates.to_a.size,
+              fetch_days: dates.to_a.size,
               timezone: TIMEZONE,
               columns: schema
             )
@@ -279,7 +279,7 @@ module Embulk
           def transaction_task(days)
             from_date = Date.parse(FROM_DATE)
             task.merge(
-              dates: (from_date..(from_date + days)).map {|date| date.to_s},
+              dates: (from_date..(from_date + days - 1)).map {|date| date.to_s},
               api_key: API_KEY,
               api_secret: API_SECRET,
               timezone: TIMEZONE,
@@ -289,7 +289,7 @@ module Embulk
 
           def transaction_config(days)
             _config = config.merge(
-              days: days,
+              fetch_days: days,
               columns: schema,
               timezone: TIMEZONE,
             )
@@ -449,7 +449,7 @@ module Embulk
           api_key: API_KEY,
           api_secret: API_SECRET,
           from_date: FROM_DATE,
-          days: DAYS,
+          fetch_days: DAYS,
         }
       end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: embulk-input-mixpanel
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - yoshihara
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2015-08-26 00:00:00.000000000 Z
+date: 2015-08-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   requirement: !ruby/object:Gem::Requirement