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

Two Azure SQL Server "connection lost" messages #234

Closed
wants to merge 4,286 commits into from

Conversation

judgej
Copy link

@judgej judgej commented Apr 19, 2018

See laravel/framework#23925

Seeing these on MS Azure, with Azure SQL Server, and CentOS 7 in a VM with PHP 7.2 + pdo_sqlsrv. Cloud database connections seem to be made of water, and time out in many ways. I suspect there will be more to come.

VinceG and others added 30 commits November 17, 2017 07:24
- Currently if your JSON keys have a dash, the update fails due to this issue https://bugs.mysql.com/bug.php?id=81896
- this change ensures the keys are within double quotes
- before: “$.foo-bar”
- after: '$."foo-bar"'
- I changed the test to reflect the new format
* Fixes MySqlGrammar json accessor building when having a json field with nested columns. Related to #22118

* adds testMySqlUpdateWrappingNestedJson

* fix style
staudenmeir and others added 22 commits March 21, 2018 02:02
(cherry picked from commit 273b7fc)
* [5.6] Allow to disable `CREATED_AT`.

* [5.6] add unit tests for CREATED_AT and UPDATED_AT disabling.

* [5.6] code reformat.
Allow nested relations to be eager loaded for
morphTo() relationships of mixed classes.

e.g.,

class ActivityFeed
{
    function parentable()
    {
        return $this->morphTo();
    }
}

$collection = ActivityFeed::with('parentable')
    ->get()
    ->loadMorph('parentable', [
        Event::class => 'calendar',
        Photo::class => 'tags',
        Post::class => ['author', 'commentsCount'],
    ]);

or

$paginator = ActivityFeed::with('parentable')
    ->paginate()
    ->loadMorph('parentable', [
        // etc.
    ]);
[5.6] Fix typos and remove unused imports
There is a double || in the @param declaration of this method.
…ent (#23718)

* moved the soft delete column name to the arg to make it consistent

* added missing docblock

* added missing docblock 2

* added missing docblock
* Fix phpDocs

* Update Blueprint.php
See laravel/framework#23925

Seeing these on MS Azure, with Azure SQL Server, and CentOS 7 in a VM with PHP 7.2 + pdo_sqlsrv. Cloud database connections seem to be made of water, and time out in many ways. I suspect there will be more to come.
@judgej judgej changed the title Two frequent SQL Server connection lost messages Two frequent Azure SQL Server connection lost messages Apr 19, 2018
@judgej judgej changed the title Two frequent Azure SQL Server connection lost messages Two frequent Azure SQL Server "connection lost" messages Apr 19, 2018
@judgej judgej changed the title Two frequent Azure SQL Server "connection lost" messages Two Azure SQL Server "connection lost" messages Apr 19, 2018
@judgej
Copy link
Author

judgej commented Apr 20, 2018

Got error "SQLSTATE[08S01]: [Microsoft][ODBC Driver 13 for SQL Server]TCP Provider: Error code 0x68" twice in a pre-production queue:work process over the last 24 hours, and an automatic reconnect worked as expected with this PR change implemented there.

@madbarron
Copy link
Contributor

Seeing one of those messages from our on-premises SQL Server 2012 instance. Thanks for the PR!

PDOException: SQLSTATE[08S02]: [Microsoft][ODBC Driver 13 for SQL Server]SMux Provider: Physical connection is not usable [xFFFFFFFF]. in /var/www/laravel/vendor/laravel/framework/src/Illuminate/Database/Connection.php:326

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.