flydata 0.2.12 → 0.2.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 834f55f013a932c4bbd53408f020fe886ab2acf5
-  data.tar.gz: 702f346a6a072648956489fab6bf75776adb3351
+  metadata.gz: 7d20120a45eeb0e80acb2738e924ff664454d8bf
+  data.tar.gz: 96aadd0c65100102b2e659d063fbc1d9a391f56d
 SHA512:
-  metadata.gz: 3cafe705f68a2b86801e0af7bd93f5b450adaceb1f183876b44d01c3b921c437c78376b093b2e516e80446bc84fb18f18dfeeacc88ee1b4b6ff2922e0020fd06
-  data.tar.gz: 9aa3192f6524da454763d808941fdb847806166cfd369712835d37c36d33f2d5f307ac98fe31f44cea46a212c90b12b4c2c78594ebb691623369ab6705947cf5
+  metadata.gz: be0a985f146c83e734435c5a84d38d32b0daf8e0c32a59f8774c6b1570d360cfcab3b42fc327cd01b8a54ba4229718b6e7049cc9c635f11c1bf333d163d02e79
+  data.tar.gz: 09f3b5918fbe79bbdf86d0570f73524a646d7f507a89c3fe0ae4df9d846e3724577b52da4ee73dcd0f10172128664f2c9121b9d0f5d513ba38f5a5f84f4229d9

data/VERSION

@@ -1 +1 @@
-0.2.12
+0.2.13

data/flydata.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: flydata 0.2.12 ruby lib
+# stub: flydata 0.2.13 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "flydata"
-  s.version = "0.2.12"
+  s.version = "0.2.13"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]
   s.authors = ["Koichi Fujikawa", "Masashi Miyazaki", "Matthew Luu", "Mak Inada", "Sriram NS"]
-  s.date = "2014-11-03"
+  s.date = "2014-11-08"
   s.description = "FlyData Agent"
   s.email = "sysadmin@flydata.com"
   s.executables = ["fdmysqldump", "flydata", "serverinfo"]
@@ -33,6 +33,7 @@ Gem::Specification.new do |s|
     "flydata.gemspec",
     "lib/fly_data_model.rb",
     "lib/flydata.rb",
+    "lib/flydata/agent_errors.rb",
     "lib/flydata/api/base.rb",
     "lib/flydata/api/data_entry.rb",
     "lib/flydata/api/data_port.rb",
@@ -58,7 +59,6 @@ Gem::Specification.new do |s|
     "lib/flydata/compatibility_check.rb",
     "lib/flydata/credentials.rb",
     "lib/flydata/cron.rb",
-    "lib/flydata/errors.rb",
     "lib/flydata/fluent-plugins/in_mysql_binlog_flydata.rb",
     "lib/flydata/fluent-plugins/mysql/alter_table_query_handler.rb",
     "lib/flydata/fluent-plugins/mysql/binlog_position.rb",

data/lib/flydata/command/sender.rb

@@ -20,6 +20,11 @@ module Flydata
           return
         end
 
+        # Ends orphan_proceses if there is any
+        orphan_processes.each do |pid|
+          Process.kill(:TERM, pid)
+        end
+
         wait_until_server_ready(options)
 
         dp = flydata.data_port.get
@@ -29,8 +34,8 @@ module Flydata
         say('Starting sender process.') unless options[:quiet]
         Dir.chdir(FLYDATA_HOME){
           Kernel.system("bash #{File.dirname(__FILE__)}/../../../bin/serverinfo", :out => ["#{FLYDATA_HOME}/flydata.log",'a'], :err => ["#{FLYDATA_HOME}/flydata.log",'a'])
-          daemon_option = opts.no_daemon? ? "" : "-d #{FLYDATA_HOME}/flydata.pid"
-          Kernel.system("fluentd #{daemon_option} -l #{FLYDATA_HOME}/flydata.log -c #{FLYDATA_HOME}/flydata.conf -p #{File.dirname(__FILE__)}/../fluent-plugins")
+          daemon_opt = opts.no_daemon? ? "" : daemon_option
+          Kernel.system("ruby `which fluentd` #{daemon_opt} -l #{FLYDATA_HOME}/flydata.log -c #{FLYDATA_HOME}/flydata.conf -p #{File.dirname(__FILE__)}/../fluent-plugins")
         }
         Kernel.sleep 5
 
@@ -51,7 +56,7 @@ EOF
         end
 
         say('Stopping sender process.') unless options[:quiet]
-        if Kernel.system("kill `cat #{FLYDATA_HOME}/flydata.pid`") and wait_until_client_stop(options)
+        if Kernel.system("kill `cat #{pid_file}`") and wait_until_client_stop(options)
           say('Done.') unless options[:quiet]
           return true
         end
@@ -65,7 +70,7 @@ EOF
         end
 
         say('Stopping input plugins and flushing the client buffer.') unless options[:quiet]
-        Kernel.system("kill -USR1 `cat #{FLYDATA_HOME}/flydata.pid`")
+        Kernel.system("kill -USR1 `cat #{pid_file}`")
 
         retry_count = 12
         1.upto(retry_count) do |i|
@@ -79,7 +84,7 @@ EOF
       def restart(options = {})
         if process_exist?
           say('Restarting sender process.') unless options[:quiet]
-          if Kernel.system("kill -HUP `cat #{FLYDATA_HOME}/flydata.pid`")
+          if Kernel.system("kill -HUP `cat #{pid_file}`")
             say('Done.') unless options[:quiet]
             return true
           else
@@ -99,7 +104,7 @@ EOF
       end
       def process_exist?
         # Returns true if the process is running
-        `[ -f #{FLYDATA_HOME}/flydata.pid ] && pgrep -P \`cat #{FLYDATA_HOME}/flydata.pid\``.to_i > 0
+        `[ -f #{pid_file} ] && pgrep -P \`cat #{pid_file}\``.to_i > 0
       end
       def kill_all(optiosn = {})
         if Kernel.system("ps ax | grep 'flydata' | grep -v grep | awk '{print \"kill  \" $1}' | sh")
@@ -111,6 +116,19 @@ EOF
       end
 
       private
+      # Return a list of fluentd parent processes run by the same user for the
+      # same flydata.pid file but not the process managed by the file itself.
+      CMD = %Q{ps -u `whoami` -o ppid,pid,args | grep '^ *1 ' | grep '\\%s' | egrep -v "\\b`cat %s`\\b"}
+      def orphan_processes
+        cmd =  CMD % [ daemon_option, pid_file ]
+        result = `#{cmd}`
+        orphan_pids = []
+        result.each_line do |line|
+          orphan_pids << line.split[1].to_i
+        end
+        orphan_pids
+      end
+
       def wait_until_server_ready(options = {})
         retry_count = 10
         1.upto(retry_count) do |i|
@@ -181,6 +199,12 @@ EOF
         say("Checking the client buffer #{client_buffer}") unless options[:quiet]
         Dir.glob("#{client_buffer}/*").empty?
       end
+      def pid_file
+        "#{FLYDATA_HOME}/flydata.pid"
+      end
+      def daemon_option
+        "-d #{pid_file}"
+      end
     end
   end
 end

data/lib/flydata/command/sync.rb

@@ -3,7 +3,7 @@ require 'msgpack'
 require 'open3'
 require 'mysql2'
 require 'flydata/sync_file_manager'
-require 'flydata/errors'
+require 'flydata/agent_errors'
 require 'flydata/compatibility_check'
 require 'flydata/table_def'
 require 'flydata/output/forwarder'

data/lib/flydata/compatibility_check.rb

@@ -39,22 +39,26 @@ module Flydata
     SSL_PORT=45327
 
     def check_outgoing_ports
-      unless can_connect_to_port?(TCP_PORT) and
-             can_connect_to_port?(SSL_PORT)
-        raise AgentCompatibilityError,
-          "Cannot connect to outisde ports. Please check to make sure you have these outgoing ports open: #{TCP_PORT},#{SSL_PORT}"
-      end
-    end
+      ports = [TCP_PORT]
+      ports << SSL_PORT unless ENV['FLYDATA_ENV_KEY'] == 'development'
 
-    private
-    def can_connect_to_port?(port)
       url = @dp["servers"].first
-      begin
-        e = TCPSocket.new(url, port)
-        e.close
-        return true
-      rescue Errno::ETIMEDOUT => e
-        return false
+      errors = {}
+      ports.each do |port|
+        begin
+          e = TCPSocket.new(url, port)
+          e.close
+        rescue => e
+          errors[port] = e
+        end
+      end
+
+      unless errors.empty?
+        message = "Cannot connect to outisde ports. Please check to make sure you have these outgoing ports open."
+        errors.each do |port, e|
+          message += "  Port #{port}, Error #{e.class.name}: #{e.to_s}"
+        end
+        raise AgentCompatibilityError, message
       end
     end
   end

data/lib/flydata/parser/mysql/mysql_alter_table.treetop

@@ -100,7 +100,7 @@ grammar MysqlAlterTable
         action_hash
       end
     }
-    / add_key (sp col_key)? sp col_name_def ( sp pos_def )? {
+    / add_key opt_column sp col_name_def opt_place {
       def action
         ret = { action: :add_column }
         ret.merge!(col_name_def.column_def)
@@ -110,7 +110,7 @@ grammar MysqlAlterTable
         ret
       end
     }
-    / add_key (sp col_key)? nsp '(' nsp col_name_def ( comma col_name_def )* nsp ')' {
+    / add_key opt_column nsp '(' nsp col_name_def ( comma col_name_def )* nsp ')' {
       def action
         ret = [col_name_def.column_def]
         ret += 0.upto(elements[6].elements.count - 1).collect do |i|
@@ -119,6 +119,30 @@ grammar MysqlAlterTable
         ret.collect{|v| {action: :add_column}.merge(v) }
       end
     }
+    # Re-using col_name and col_name_def for now
+    # TODO Refactor later to match sql_yacc.yy more closely
+    / change_sym opt_column sp col_name sp col_name_def opt_place {
+      def action
+        ret = { action: :change_column }
+        ret[:old_column] = col_name.value
+        ret.merge!(col_name_def.column_def)
+        if elements[6].nonterminal?
+          ret.merge!(elements[6].elements[1].pos_def_option)
+        end
+        ret
+      end
+    }
+    / modify_sym opt_column sp col_name_def opt_place {
+      def action
+        ret = { action: :change_column }
+        ret.merge!(col_name_def.column_def)
+        ret[:old_column] = ret[:column]
+        if elements[4].nonterminal?
+          ret.merge!(elements[4].elements[1].pos_def_option)
+        end
+        ret
+      end
+    }
     / drop_key sp primary_sym sp key_sym {
       def action
         { action: :drop_primary_key }
@@ -138,7 +162,7 @@ grammar MysqlAlterTable
         }
       end
     }
-    / drop_key (sp col_key)? sp col_name {
+    / drop_key opt_column sp col_name {
       def action
         { action: :drop_column,
           column: col_name.value, }
@@ -578,6 +602,10 @@ grammar MysqlAlterTable
     }
   end
 
+  rule opt_place
+    ( sp pos_def )?
+  end
+
   rule pos_def
     'first'i {
       def pos_def_option
@@ -929,7 +957,6 @@ grammar MysqlAlterTable
   rule table_key 'table'i end
   rule add_key 'add'i end
   rule drop_key 'drop'i end
-  rule col_key 'column'i end
 
 
   ######## Common rules
@@ -967,6 +994,7 @@ grammar MysqlAlterTable
 
   rule value
     quoted_value { def raw_value; text_raw_value; end }
+    / double_quoted_value { def raw_value; text_raw_value; end }
     / ident_sym { def raw_value; text_value; end }
   end
 
@@ -974,10 +1002,22 @@ grammar MysqlAlterTable
     "'" text "'" { def text_raw_value; text.text_value; end }
   end
 
+  rule double_quoted_value
+    '"' text_within_double_quotes '"' {
+      def text_raw_value
+        text_within_double_quotes.text_value
+      end
+    }
+  end
+
   rule text
     ("\\\\" / "\\'" / !"'" . )*
   end
 
+  rule text_within_double_quotes
+    ('\\\\' / '\\"' / !'"' . )*
+  end
+
   rule ident
     ident_sys / keyword
   end
@@ -1298,4 +1338,23 @@ grammar MysqlAlterTable
     / extended_sym
     / use_frm_sym
   end
+
+  ## Change column related rules
+
+  rule change_sym
+    'change'i ![A-Za-z0-9_]
+  end
+
+  rule modify_sym
+    'modify'i ![A-Za-z0-9_]
+  end
+
+  rule column_sym
+    'column'i ![A-Za-z0-9_]
+  end
+
+  rule opt_column
+    ( sp column_sym )?
+  end
+
 end

data/spec/flydata/command/sender_spec.rb

@@ -26,7 +26,7 @@ module Flydata
             expect(Kernel).to receive(:system).with( Regexp.new(
 "bash .+/../../../bin/serverinfo"), :out => [Regexp.new(".+/flydata.log"),'a'], :err => [Regexp.new(".+/flydata.log"),'a'])
             expect(Kernel).to receive(:system).with( Regexp.new(
-"fluentd -d .+/flydata.pid -l .+/flydata.log -c .+/flydata.conf -p .+/\.\./fluent-plugins"))
+"ruby `which fluentd` -d .+/flydata.pid -l .+/flydata.log -c .+/flydata.conf -p .+/\.\./fluent-plugins"))
             subject.start(false)
           end
         end
@@ -36,7 +36,7 @@ module Flydata
             expect(Kernel).to receive(:system).with( Regexp.new(
 "bash .+/../../../bin/serverinfo"), :out => [Regexp.new(".+/flydata.log"),'a'], :err => [Regexp.new(".+/flydata.log"),'a'])
             expect(Kernel).to receive(:system).with(Regexp.new(
-"fluentd  -l .+/flydata.log -c .+/flydata.conf -p .+/\.\./fluent-plugins"))
+"ruby `which fluentd`  -l .+/flydata.log -c .+/flydata.conf -p .+/\.\./fluent-plugins"))
             subject.start(false)
           end
         end
@@ -46,7 +46,7 @@ module Flydata
             expect(Kernel).to receive(:system).with( Regexp.new(
 "bash .+/../../../bin/serverinfo"), :out => [Regexp.new(".+/flydata.log"),'a'], :err => [Regexp.new(".+/flydata.log"),'a'])
             expect(Kernel).to receive(:system).with(Regexp.new(
-"fluentd  -l .+/flydata.log -c .+/flydata.conf -p .+/\.\./fluent-plugins"))
+"ruby `which fluentd`  -l .+/flydata.log -c .+/flydata.conf -p .+/\.\./fluent-plugins"))
             subject.start(false)
           end
         end

data/spec/flydata/compatibility_check_spec.rb

@@ -7,17 +7,25 @@ module Flydata
      end
 
     describe "#check" do
-      subject { AgentCompatibilityCheck.new(:default_data_port) }
+      subject { AgentCompatibilityCheck.new("servers" => ['localhost']) }
       context "runs all check methods" do
-        it "successfully checks open ports" do
-          expect(subject).to receive(:can_connect_to_port?).with(Flydata::AgentCompatibilityCheck::TCP_PORT).and_return true
-          expect(subject).to receive(:can_connect_to_port?).with(Flydata::AgentCompatibilityCheck::SSL_PORT).and_return true
-          subject.check
+        context "when all ports are accessible" do
+          let(:sock) { double('sock') }
+          before do
+            allow(TCPSocket).to receive(:new).and_return(sock)
+            allow(sock).to receive(:close)
+          end
+          it "does nothing" do
+            subject.check
+          end
         end
-        it "successfully catches errors on open ports" do
-          expect(subject).to receive(:can_connect_to_port?).with(Flydata::AgentCompatibilityCheck::TCP_PORT).and_return true
-          expect(subject).to receive(:can_connect_to_port?).with(Flydata::AgentCompatibilityCheck::SSL_PORT).and_return false
-          expect{subject.check_outgoing_ports}.to raise_error(Flydata::AgentCompatibilityCheck::AgentCompatibilityError, /ports/)
+        context "when a port access fails" do
+          before do
+            allow(TCPSocket).to receive(:new).and_raise(Errno::ETIMEDOUT)
+          end
+          it do
+            expect{subject.check_outgoing_ports}.to raise_error(Flydata::AgentCompatibilityCheck::AgentCompatibilityError, /ports/)
+          end
         end
       end
     end

data/spec/flydata/parser/mysql/alter_table_parser_spec.rb

@@ -1333,5 +1333,261 @@ describe 'MysqlAlterTableParser' do
       let(:alter_table_action) { "REMOVE PARTITIONING" }
       it_behaves_like "a parser parsing a nonbreaking query"
     end
+
+    ## Change and modify column related tests
+    shared_examples "test optional column" do |*examples|
+      context "with column" do
+        it_behaves_like *examples do
+          let(:column) {" COLUMN"}
+        end
+      end
+      context "without column" do
+        it_behaves_like *examples do
+          let(:column) { "" }
+        end
+      end
+    end
+    shared_examples "test position" do |*examples|
+      context "with after" do
+        it_behaves_like *examples do
+          let(:position) {" after test_name"}
+          let(:position_hash){
+            { after: "test_name" }
+          }
+        end
+      end
+      context "with first" do
+        it_behaves_like *examples do
+          let(:position) {" first"}
+          let(:position_hash){
+            { position: :first }
+          }
+        end
+      end
+      context "with none" do
+        it_behaves_like *examples do
+          let(:position) {""}
+          let(:position_hash){
+            {}
+          }
+        end
+      end
+    end
+    shared_examples "test not null" do |*examples|
+      context "with not null option" do
+        it_behaves_like *examples do
+          let(:not_null) {" not null"}
+          let(:not_null_hash){
+            { not_null: true }
+          }
+        end
+      end
+      context "without not null option" do
+        it_behaves_like *examples do
+          let(:not_null) {""}
+          let(:not_null_hash){
+            {}
+          }
+        end
+      end
+    end
+    shared_examples "test default for timestamp" do |*examples|
+      context "timestamp" do
+        it_behaves_like *examples do
+          let(:default) {" default current_timestamp"}
+          let(:default_hash){
+            { default: "current_timestamp" }
+          }
+        end
+      end
+      context "timestamp" do
+        it_behaves_like *examples do
+          let(:default) {" default localtime"}
+          let(:default_hash){
+            { default: "current_timestamp" }
+          }
+        end
+      end
+      context "none" do
+        it_behaves_like *examples do
+          let(:default) {""}
+          let(:default_hash){
+            {}
+          }
+        end
+      end
+    end
+    shared_examples "test default for varchar" do |*examples|
+      context "null" do
+        it_behaves_like *examples do
+          let(:default) {" default NULL"}
+          let(:default_hash){
+            { default: nil }
+          }
+        end
+      end
+      context "empty string" do
+        it_behaves_like *examples do
+          let(:default) {" default ''"}
+          let(:default_hash){
+            { default: "" }
+          }
+        end
+      end
+      context "string with spaces" do
+        it_behaves_like *examples do
+          let(:default) {" default '0 0 0 0 0 0'"}
+          let(:default_hash){
+            { default: "0 0 0 0 0 0" }
+          }
+        end
+      end
+      context "none" do
+        it_behaves_like *examples do
+          let(:default) {""}
+          let(:default_hash){
+            {}
+          }
+        end
+      end
+    end
+    shared_examples "test auto increment" do |*examples|
+      context "with auto_increment" do
+        it_behaves_like *examples do
+          let(:auto_increment) {" auto_increment"}
+          let(:auto_increment_hash){
+            { auto_increment: true }
+          }
+        end
+      end
+      context "without auto_increment" do
+        it_behaves_like *examples do
+          let(:auto_increment) {""}
+          let(:auto_increment_hash){
+            {}
+          }
+        end
+      end
+    end
+    shared_examples "test comment" do |*examples|
+      context "with a comment" do
+        it_behaves_like *examples do
+          let(:comment) {" COMMENT 'Hello World!'"}
+        end
+      end
+      context "with a double quoted comment" do
+        it_behaves_like *examples do
+          let(:comment) {%q| comment "Hello World!"|}
+        end
+      end
+      context "with a backslash inside double quotes" do
+        it_behaves_like *examples do
+          let(:comment) {%q| comment "\\\\"|}
+        end
+      end
+      context "with special characters" do
+        it_behaves_like *examples do
+          let(:comment) {%q| comment "https://flydata.com $ \\\\'\"\n\t"|}
+        end
+      end
+      context "without a comment" do
+        it_behaves_like *examples do
+          let(:comment) {""}
+        end
+      end
+    end
+    shared_examples "test unique" do |*examples|
+      context "with unique" do
+        it_behaves_like *examples do
+          let(:unique) {" UNIQUE"}
+          let(:unique_hash){
+            { unique: true }
+          }
+        end
+      end
+      context "without unique" do
+        it_behaves_like *examples do
+          let(:unique) {""}
+          let(:unique_hash){
+            {}
+          }
+        end
+      end
+    end
+    shared_examples "test column definition" do |*examples|
+      context "varchar" do
+        it_behaves_like "test not null", "test default for varchar", "test comment", *examples do
+          let(:col_def) {"varchar(256)#{not_null}#{default}#{comment}"}
+          let(:col_def_hash) {
+            { type: "varchar(768)" }.merge!(not_null_hash).merge!(default_hash)
+          }
+        end
+      end
+      context "timestamp" do
+        it_behaves_like "test not null", "test default for timestamp", *examples do
+          let(:col_def) {"timestamp#{not_null}#{default}"}
+          let(:col_def_hash) {
+            { type: "datetime" }.merge!(not_null_hash).merge!(default_hash)
+          }
+        end
+      end
+      context "integer" do
+        it_behaves_like "test not null", "test auto increment", "test comment", *examples do
+          let(:col_def) {"int(11)#{not_null}#{auto_increment}#{comment}"}
+          let(:col_def_hash) {
+            { type: "int4(11)" }.merge!(not_null_hash).merge!(auto_increment_hash)
+          }
+        end
+      end
+      context "decimal" do
+        it_behaves_like "test not null", "test unique", *examples do
+          let(:col_def) {"decimal(10,2)#{not_null}#{unique}"}
+          let(:col_def_hash) {
+            { type: "numeric(10,2)" }.merge!(not_null_hash).merge!(unique_hash)
+          }
+        end
+      end
+    end
+    shared_examples "test parser parsing a change column" do
+      let(:action_hash) {
+        change_action_hash.merge!({
+          action: :change_column,
+          query: alter_table_action
+        })
+      }
+      it_behaves_like "a parser parsing a single action alter table query"
+    end
+    context 'change column' do
+      context 'change column name' do
+        let(:alter_table_action) { "change#{column} name name1 #{col_def}#{position}" }
+        let(:change_action_hash) {
+          {
+            old_column: "name",
+            column: "name1",
+          }.merge!(position_hash).merge!(col_def_hash)
+        }
+        it_behaves_like "test optional column", "test column definition", "test position", "test parser parsing a change column"
+      end
+      context 'no change to column name' do
+        let(:alter_table_action) { "change#{column} name1 name1 #{col_def}#{position}" }
+        let(:change_action_hash) {
+          {
+            old_column: "name1",
+            column: "name1",
+          }.merge!(position_hash).merge!(col_def_hash)
+        }
+        it_behaves_like "test optional column", "test column definition", "test position", "test parser parsing a change column"
+      end
+    end
+    context 'modify column' do
+      let(:alter_table_action) { "modify#{column} name1 #{col_def}#{position}" }
+      let(:change_action_hash) {
+        {
+          old_column: "name1",
+          column: "name1",
+        }.merge!(position_hash).merge!(col_def_hash)
+      }
+      it_behaves_like "test optional column", "test column definition", "test position", "test parser parsing a change column"
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: flydata
 version: !ruby/object:Gem::Version
-  version: 0.2.12
+  version: 0.2.13
 platform: ruby
 authors:
 - Koichi Fujikawa
@@ -12,7 +12,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-03 00:00:00.000000000 Z
+date: 2014-11-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rest-client
@@ -416,6 +416,7 @@ files:
 - flydata.gemspec
 - lib/fly_data_model.rb
 - lib/flydata.rb
+- lib/flydata/agent_errors.rb
 - lib/flydata/api/base.rb
 - lib/flydata/api/data_entry.rb
 - lib/flydata/api/data_port.rb
@@ -441,7 +442,6 @@ files:
 - lib/flydata/compatibility_check.rb
 - lib/flydata/credentials.rb
 - lib/flydata/cron.rb
-- lib/flydata/errors.rb
 - lib/flydata/fluent-plugins/in_mysql_binlog_flydata.rb
 - lib/flydata/fluent-plugins/mysql/alter_table_query_handler.rb
 - lib/flydata/fluent-plugins/mysql/binlog_position.rb