Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Concurrent DB Tests Failing #36

Open
allenwyma opened this issue Apr 14, 2018 · 1 comment
Open

Concurrent DB Tests Failing #36

allenwyma opened this issue Apr 14, 2018 · 1 comment
Labels

Comments

@allenwyma
Copy link

Cannot run tests after swapping postgres to MS SQL.

Expected Behavior

Tests should run without issue.

Current Behavior

Tests are failing, saying:

18:42:34.953 [error] GenServer #PID<0.322.0> terminating
** (stop) exited in: :gen_server.call(#PID<0.252.0>, {:checkout, #Reference<0.3768785716.1265631238.47152>, true, :infinity}, 5000)
    ** (EXIT) time out
    (db_connection) lib/db_connection/poolboy.ex:112: DBConnection.Poolboy.checkout/3
    (ecto) lib/ecto/adapters/sql/sandbox.ex:386: Ecto.Adapters.SQL.Sandbox.Pool.checkout/2
    (db_connection) lib/db_connection/ownership/proxy.ex:108: DBConnection.Ownership.Proxy.handle_call/3
    (stdlib) gen_server.erl:636: :gen_server.try_handle_call/4
    (stdlib) gen_server.erl:665: :gen_server.handle_msg/6
    (stdlib) proc_lib.erl:247: :proc_lib.init_p_do_apply/3
Last message (from #PID<0.321.0>): {:init, 15000}

Possible Solution

Do not check out DB connection.

Steps to Reproduce (for bugs)

  1. Clone https://github.com/HangingClowns/mssql_ecto_issue
  2. Run docker-compose up to turn on MSSql DB
  3. Run mix test from sample project:

Context

Just trying to run the tests.

Your Environment

  • MssqlEcto version: 1.1.0
  • Ecto version: 2.2.9
  • Elixir version: 1.6.4
  • Erlang version: 9.3 (OTP 20.3.2)
  • Microsoft SQL Server version: whatever is in docker image ab22b8353bbd, was tagged as latest 8 weeks ago
  • Operating System and version: Mac 10.13.4
  • Link to your project: https://github.com/HangingClowns/mssql_ecto_issue
@jbachhardie jbachhardie added the bug label May 7, 2018
@dimun
Copy link

dimun commented Nov 15, 2018

check out this issue elixir-ecto/db_connection#127, according to this you must update your OTP version to 21.0.2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

3 participants