mysql - 我应该使用 Ruby 进行计算，还是应该使用 MySQL 进行计算？

Question

我正在为 Ruby on Rails 中的聊天构建一个报告系统，但收到一些评论，告诉我我的方法效率低下。

以下是我的报告如何工作的一个小示例:

我有一个每月都会调用的处理程序，它会调用报告邮件程序，如下所示:

ReportMailer.monthly_report(user).deliver_later

这是邮件程序的外观:

class ReportMailer < ApplicationMailer
  default from: ENV["DEFAULT_MAILER_FROM"],
          template_path: 'mailers/report_mailer'

  def monthly_report(agent)
    @agent = agent
    @organization = agent.organization
    @report = Report.new @organization
    mail(to: agent.email, subject: @report.email_subject)
  end
end

我正在尝试使用“普通旧”Ruby 类来计算数据:

module Reports
  class Component < Report

    def initialize(subject)
      @component = subject
      @cache = {}
    end

    attr_reader :component

    # DELEGATIONS
    # -----------------------

    delegate :chat_messages, to: :component

    def response_count
      count = 0
      explore_msgs { |msg, next_msg| count += 1 if response? msg, next_msg }
      return count
    end

    def response_time
      time = 0
      explore_msgs { |msg, next_msg| time += time_difference msg, next_msg if response? msg, next_msg }
      return time.to_i.seconds
    end

    def avg_response_time
      @cache[__method__] ||= (response_time / response_count if response_count > 0)
    end

    private

    def response?(msg, next_msg)
      next_msg&.user_type == 'Agent' && msg.user_type == 'User' && msg.conversation_id == next_msg.conversation_id && time_difference(msg, next_msg).seconds < 8.hours
    end

    def time_difference(msg, next_msg)
      (next_msg.created_at - msg.created_at).abs
    end

    def explore_msgs
      chat_messages.each_with_index do |msg, i| 
        next_msg = chat_messages[i+1]
        yield msg, next_msg
      end
    end

  end
end

我关心的是提高性能。我在负责进行计算的类中实现了一个简单的缓存系统，这极大地提高了系统效率，但是，我担心在 Ruby 中进行这些计算可能会产生瓶颈，或者它可能不是一个可扩展的解决方案。

Answer 1

它可能会更快。我看到的问题是您正在查找一条记录和下一条记录。那么如何让数据库来比较两条记录呢？

在直接 SQL 中，我会将表连接到自身，按表的第一个实例进行分组，并对表的第二个实例执行 min(created_at)。

使用我们的 companies 表，SQL 如下所示:

select rc1.id, rc1.created_at, min(rc2.created_at)
from companies rc1 inner join companies rc2 on rc1.created_at < rc2.created_at
group by rc1.id

您可以将差异添加到选择中。

如果created_at字段没有索引并且表中的记录数量很大，这肯定会很慢。

您可以将 Agent 和 User 的测试添加到having 子句中。

该查询很棘手，数据库可能无法快速完成此操作。如果您尝试让 ActiveRecord 为您构建查询，这也会很棘手。

但是，我认为您在代码中尝试执行的所有操作都可以通过数据库以这种方式完成。

您的查询可能如下所示:

select chat_messages.*,
       min(next_msg.created_at) as next_created_at,
       next_msg.created_at - chat_messages.created_at as created_at_diff
from chat_messages inner join chat_messages next_msg
  on chat_messages.created_at < chat_messages.created_at
     and chat_messages.user_type = 'User'
group by chat_messages.id
having next_msg.user_type = 'Agent'
   and TIMESTAMPDIFF(HOUR, min(next_msg.created_at), chat_messages.created_at) < 8