zdennis/activerecord-import

Support `trilogy` adapter

tmimura39 opened this issue ยท 7 comments

A new MySQL adapter (trilogy) has been introduced to rails. rails/rails#47880

activerecord-import should also support trilogy.

Trilogy is a client library for MySQL-compatible database servers, designed for performance, flexibility, and ease of embedding.

I have verified that the application works correctly with the following code.

require "active_record/connection_adapters/trilogy_adapter"
require "activerecord-import/adapters/mysql2_adapter"

class ActiveRecord::ConnectionAdapters::TrilogyAdapter
  include ActiveRecord::Import::Mysql2Adapter
end

Any input on PR #811?

Oops, I missed that pull request.

thanks ๐Ÿ‘

I'm currently trying to figure out how to properly test this change. The changes I have currently on my PR works as is with a rather large Rails application.

When running he mysql test in a Docker container I see the following:

201 runs, 289 assertions, 1 failures, 56 errors, 0 skips
rake aborted!

I still need to investigate why the CI is โŒ, but I'll try to identify the issue and report back.

I almost have a ๐ŸŸข test running. I discovered a small issue with the Trilogy adapter or composite_primary_keys. Opening a PR against the project to see if I can get that merged. ๐Ÿคž๐Ÿผ I can, after that I can hopefully merge this PR.

I have opened a PR around add trilogy support here

@jkowens with my PR merged now can we close this issue?

Yep ๐Ÿ‘