-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
SqlServer for Ubuntu: DbContextPooling & SequenceEndToEnd tests hits frequent timeouts #10518
Comments
Stack trace from Can_use_sequence_end_to_end_from_multiple_contexts_concurrently_async
|
@saurabh500 @geleems These are all tests which hit the database pretty hard with multiple connections. Do you think these failures might be related to the Linux/SQL Server issues that are being investigated? |
@saurabh500 @geleems Any updates on this? |
@ajcvickers are these tests targeting the local sql server? |
@saurabh500 Not sure, but I don't think so. @smitpatel? |
It seems this issue was already reported, and the fix was merged to both master and release/2.0.0. SqlConnection failure issue when having multiple concurrent connectionsWe had an SqlConnection failure issue when having multiple concurrent connections. It was due to attempting to connect by asynchronously to server even when synchronous Performance issue when targeting local SQL ServerThere was a performance issue on .NET Core Linux when it specifically targets local SQL Server. It was due to socket packet size setup (Issue https://github.com/dotnet/corefx/issues/24480), and the fix was merged 1.5 month ago to master branch (dotnet/corefx#25621). |
@geleems Thanks for the update. Assigning to @smitpatel to re-enable the tests and monitor. |
This did not work. Test still failed. Reverted in b4a4a8d |
@saurabh500 @geleems These tests are still failing for us on SQL Server running on Linux. The tests pass on SQL Server for Windows, so it seems like this is an issue somewhere below the level of EF--either in SQLClient or SQL Server. We are closing this on our side since we have test coverage for EF code, but it would probably be a good idea for you guys to follow up some more on why this is failing on Linux. |
@ajcvickers this is the next item we are picking up for perf investigation. |
@saurabh500 We only test everything on Linux or everything on Windows for this case so we don't have any info if the tests also fail when running Windows pointing at Linux or vice versa. |
@smitpatel, @ajcvickers |
@geleems It's my understanding that you guys are able to run EF tests--at least, that was the case last time I talked with @saurabh500. Given that, you should be able to run the given tests on Linux and reproduce the issue. Or have you tried this and are not finding it to repro? |
@geleems You need to run the EF tests. You can build the EF Clone and run
You may want to run this particular failing test using following instructions After this you run the test, which may or may not reproduce the issue based on the configuration of the VM. I would start by running the tests on Linux against a local Linux server vs a Windows SQL Server to see if the server is acting up in this case. If there is a difference in the server side behavior, then this may be a server issue. Else we need to isolate the problem on the SqlClient. If you figure that this is a SqlClient perf issue, you could enable managed SNI on Windows and run the tests and collect tracing using PerfView.exe That should give you insights on CPU hotspots if any. Else you may want to capture the thread time in PerfView to figure out if we are waiting for any thread scheduling. As a first step, I recommend setting up the tests on Linux and run this specific test against the two server SKUs to find what component is a bottle neck |
I am testing out with code on Ubuntu and server on widows. |
Disabled tests on Ubuntu
Double_dispose_concurrency_test
Concurrency_test
Can_use_sequence_end_to_end
Can_use_sequence_end_to_end_from_multiple_contexts_concurrently_async
Using SqlServer 2017 on Ubuntu 16.04
The text was updated successfully, but these errors were encountered: