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

Return more info in connection init errores #10

Merged

Conversation

zmstone
Copy link
Member

@zmstone zmstone commented Oct 7, 2024

When there are more than one set of MySQL configs, it's quite hard to tell which one is for which.
before: timeout
after: #{cause => timeout,port => 3306,user => <<"root">>,host => "1.1.1.1",database => <<"test">>}

e.g.

2024-10-07T07:12:52.857393+00:00 [error] crasher: initial call: mysql_conn:init/1, pid: <0.6494.0>, registered_name: [], exit: {#{cause => timeout,port => 3306,user => <<"root">>,host => "1.1.1.1",database => <<"test">>},[{gen_server,init_it,6,[{file,"gen_server.erl"},{line,961}]},{proc_lib,init_p_do_apply,3,[{file,"proc_lib.erl"},{line,241}]}]}, ancestors: [<0.6493.0>,<0.6492.0>,<0.6490.0>,ecpool_sup,<0.4205.0>], message_queue_len: 0, messages: [], links: [<0.6493.0>], dictionary: [], trap_exit: false, status: running, heap_size: 376, stack_size: 28, reductions: 286; neighbours:

@zmstone zmstone force-pushed the return-more-info-in-connection-init-errores branch from 99f829a to 3b22690 Compare October 7, 2024 07:29
@zmstone zmstone force-pushed the return-more-info-in-connection-init-errores branch 8 times, most recently from 10af525 to ecc2908 Compare October 7, 2024 08:16
when in a system there are more than one set of MySQL database
connections, looking at a e.g. 'timeout' log is not quite helpful
to tell which exact config set went wrong
@zmstone zmstone force-pushed the return-more-info-in-connection-init-errores branch from ecc2908 to 0953d99 Compare October 7, 2024 08:18
@zmstone zmstone force-pushed the return-more-info-in-connection-init-errores branch from 6bf3b41 to e204ec5 Compare October 7, 2024 08:49
@zmstone zmstone force-pushed the return-more-info-in-connection-init-errores branch from e204ec5 to 1ee4047 Compare October 7, 2024 08:51
@zmstone zmstone merged commit 0f6639b into emqx:master Oct 7, 2024
2 checks passed
@zmstone zmstone deleted the return-more-info-in-connection-init-errores branch October 7, 2024 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants