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

python connection with unix_socket #3735

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

python connection with unix_socket #3735

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

Comments

@monetdb-team
Copy link

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

Date: 2015-06-09 15:28:05 +0200
From: Guillaume Savary <<guillaume.savary>>
To: clients devs <>
Version: -- development

Last updated: 2015-08-28 13:41:53 +0200

Comment 20911

Date: 2015-06-09 15:28:05 +0200
From: Guillaume Savary <<guillaume.savary>>

It seems you cannot connect to the server with unix_socket. The hostname is forced to 'localhost'.

Here is the patch to apply on python2. (Do the same for python3) :

diff --git a/clients/python2/monetdb/mapi.py b/clients/python2/monetdb/mapi.py
index 1535836..1e6e515 100644
--- a/clients/python2/monetdb/mapi.py
+++ b/clients/python2/monetdb/mapi.py
@@ -73,7 +73,7 @@ class Connection(object):
hostname = None
if not unix_socket and os.path.exists("/tmp/.s.monetdb.%i" % port):
unix_socket = "/tmp/.s.monetdb.%i" % port

  •    elif not hostname:
    
  •    elif not unix_socket and not hostname:
           hostname = 'localhost'
    
       self.hostname = hostname
    

Comment 21069

Date: 2015-07-30 13:07:42 +0200
From: MonetDB Mercurial Repository <>

Changeset c1ba05ee9a5e made by Sjoerd Mullender sjoerd@acm.org in the MonetDB repo, refers to this bug.

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

Changeset description:

Applied patch from bug #3735.

Comment 21070

Date: 2015-07-30 13:08:18 +0200
From: @sjoerdmullender

Thanks. Your patch looks good.

Comment 21192

Date: 2015-08-28 13:41:53 +0200
From: @sjoerdmullender

Jul2015 has been released.

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