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

minor bug into auto_commit() (php_monetdb.php) #2938

Closed
monetdb-team opened this issue Nov 30, 2020 · 0 comments
Closed

minor bug into auto_commit() (php_monetdb.php) #2938

monetdb-team opened this issue Nov 30, 2020 · 0 comments

Comments

@monetdb-team
Copy link

@monetdb-team monetdb-team commented Nov 30, 2020

Date: 2011-12-06 12:20:34 +0100
From: manuela <<m.sagona>>
To: clients devs <>
Version: 11.3.3 (Apr2011-SP1) [obsolete]

Last updated: 2012-01-26 15:32:07 +0100

Comment 16587

Date: 2011-12-06 12:20:34 +0100
From: manuela <<m.sagona>>

User-Agent: Mozilla/5.0 (Windows NT 5.1) AppleWebKit/535.2 (KHTML, like Gecko) Chrome/15.0.874.121 Safari/535.2
Build Identifier:

there was an error retrieving the socket for the mapi_write() call

Reproducible: Always

Comment 16588

Date: 2011-12-06 12:21:01 +0100
From: manuela <<m.sagona>>

Created attachment 85
proposed patch

Attached file: php_monetdb.php_patch_auto_commit.txt (text/plain, 390 bytes)
Description: proposed patch

Comment 16595

Date: 2011-12-06 13:22:10 +0100
From: manuela <<m.sagona>>

Changeset 554fd39f52ab made by Manuela Sagona m.sagona@acsys.it in the MonetDB repo, refers to this bug.

For complete details, see http//devmonetdborg/hg/MonetDB?cmd=changeset;node=554fd39f52ab

Changeset description:

auto_commit: reference existing variable

socket is a member for the $conn array.  Fixes bug #2938.

Comment 16596

Date: 2011-12-06 13:22:26 +0100
From: @grobian

Thanks, pushed

Comment 16832

Date: 2012-01-26 15:32:07 +0100
From: @sjoerdmullender

The Dec2011 version has been release, so declaring this bug as FIXED.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant