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

pyorient not working with orientdb 3.0.0 #270

Open
rrmerugu opened this issue May 13, 2018 · 34 comments
Open

pyorient not working with orientdb 3.0.0 #270

rrmerugu opened this issue May 13, 2018 · 34 comments

Comments

@rrmerugu
Copy link

When i try to establish a connection to the db from pyorient, the below error is showing up. im using pyorient==1.5.5, is it possible its not compatible with orientdb 3.0.0. Any plans to support pyorient for orientdb 3.0.0 version soon ?

pyorient.exceptions.PyOrientWrongProtocolVersionException: Protocol version 37 is not supported yet by this client.

PS: This is the version of the orientdb i'm using https://orientdb.com/download.php?file=orientdb-community-gremlin-3.0.0.tar.gz

@ghost
Copy link

ghost commented May 28, 2018

can you share your code please?

@XhrCkYsSgZokJL
Copy link

XhrCkYsSgZokJL commented Jun 5, 2018

client = pyorient.OrientDB('localhost', 2424)
session_id = client.connect('user', 'password')

pyorient.exceptions.PyOrientWrongProtocolVersionException: Protocol version 37 is not supported yet by this client.

pyorient 1.5.5
orientdb 3.0.0

@ghost
Copy link

ghost commented Jun 7, 2018

Same problem here.
pyorient 1.5.5
orientdb 3.0.2

@ghost
Copy link

ghost commented Jun 11, 2018

Same issue.

@vinitshetty
Copy link

+1

@vinitshetty
Copy link

i am using 3.0.2 and pyorient 1.5.5

@dersmon
Copy link

dersmon commented Jun 22, 2018

Same issue using the current Docker image.

I just wanted to try OrientDB and am not sure what a patch would encompass. No matter how complicated that may be, the Repository README states:

Pyorient works with orientdb version 1.7 and later.

As a first measure: Could someone adjust that line, pointing to the last working OrientDB (2.x) version?

For Docker it works with orientdb:2.2.35.

@Ostico
Copy link
Collaborator

Ostico commented Jun 22, 2018

@dersmon good point, since at moment i have not enough time to advance the version at least until august i think, this is the best option.

@softwareCobbler
Copy link

softwareCobbler commented Jul 28, 2018

I changed SUPPORTED_PROTOCOL in constants.py from 36 to 37 and it connects... now, is there going to be a bunch of buggy behavior?...Not the kind of thing for a production environment, but it got me back to my project.

@Ostico
Copy link
Collaborator

Ostico commented Aug 2, 2018

Hi @softwareCobbler ,

not tried, not tested and I don't know which issues there could be. Could be useful to know which problems you find as a good starting point to short cut the driver improvement.

@Froskekongen
Copy link

@mogui: Is there any plan to update pyorient to support version 3.x reliably?

@alanmeeson
Copy link

I have looked into this a bit. This client throws an error if the OrientDB server is of a more recent version than the client. The official Java client has slightly different behaviour, it just raises a warning that you will not be able to use the full features of the newer version.

It looks like both the python and Java clients attempt to connect to the server using the clients preferred protocol version, and expect the server to handle it.

I have tweaked the connection version checking in the python client to mirror that in the java client, and I'm going to give it a shot and see how it goes. (So far I have tried connecting to a V3.0.5 OrientDB and querying a graph successfully).

Note: links are to the relevant sections of code.

@wave-DmP
Copy link

@softwareCobbler I tried your approach and got the following

pyorient.exceptions.PyOrientDatabaseException: com.orientechnologies.orient.core.exception.OConfigurationException - You can use connect as first operation only for protocol < 37 please use handshake for protocol >= 37

@mogui
Copy link
Owner

mogui commented Sep 20, 2018 via email

@klkludde
Copy link

Hi Mogui!
Any updates on this yet? Our development team really looks forward to testing pyorient with our OrientDB-server, version 3.0.7

Yours sincerely
/Kjell

@montequie
Copy link

Hi Mogui,

Do you guys have any estimations about fixing it?
Or any proper python temporary solution to use mean while?

Thanks!!

@TariqAHassan
Copy link

TariqAHassan commented Sep 27, 2018

@mogui

Thanks for the reply. I ended up deleting my comment because I found that commenting out the error message (or invoking warning.warn() instead of raising in my case) will work.

@klkludde @montequie: What I suggested above is not particularly elegant, but it should work as a stopgap measure.

@montequie
Copy link

@TariqAHassan

Can you repost your suggestion? can't find it here

@TariqAHassan
Copy link

Hi @montequie

See here.

@montequie
Copy link

@TariqAHassan Thanks a lot!
It worked.

@dtanase
Copy link

dtanase commented Oct 25, 2018

+1
Any updates?

@dustyny
Copy link

dustyny commented Dec 8, 2018

+1

Let me add my encouragement to update to 3.x as well.. :)

I'm a new user to OrientDB. I have a project that I'm working on and I'm nervous to build and then have to migrate to 3.x later on.

@shashwatwork
Copy link

Is there any fix for pyorient with Orientdb 3.0
I got error like Protocol version 37 is not supported yet by this client

@mn3mos
Copy link

mn3mos commented Aug 19, 2019

+1

@upx86
Copy link

upx86 commented Sep 17, 2019

Is there any plan to fix this?

@laurendelong21
Copy link

+1

@mpomet
Copy link

mpomet commented Dec 10, 2019

If someone needs to use odb3 with python, just use master branch instead of the last release.
pip install git+https://github.com/orientechnologies/pyorient

If you just start a new project, I recommend using a more up-to-date dbsm (redisgraph or other).

@CafeLungo
Copy link

I'm sorry, but... what the hell is going on here?

Try out orientdb -> use pyorient -> latest, and right off the bat I run into a non-starter because of a constant that hasn't been updated in almost 2 years now? (Along with other changes that haven't been implemented along with that constant bump)

@jbonfardeci
Copy link

jbonfardeci commented Mar 29, 2020

The issue is back. You can fix this by overriding the connect method in OrientSocket:

from pyorient import OrientSocket

class PySocket(OrientSocket):
    def __init__(self, host, port, protocol=36, timeout=300):
        """
        Override OrientSocket protocol version number
        to fix PyOrientWrongProtocolVersionException where:
            'Protocol version 37 is not supported yet by this client.'
        """
        super(PySocket, self).__init__(host, port)
        self.protocol = protocol
        self.host = host
        self.port = port
        self.timeout = timeout

    def connect(self):
        """
        Override OrientSocket connect method
        to fix PyOrientWrongProtocolVersionException where:
            'Protocol version 37 is not supported yet by this client.'
        """

        self._socket.settimeout(self.timeout)
        self._socket.connect( (self.host, self.port) )
        _value = self._socket.recv(2)

        if len(_value) != 2:
            self._socket.close()

        self.connected = True

then

HOST = "localhost"
PORT = 2424
DATABASE_NAME = "<dbname>"
DB_USER = "admin"
DB_PWD = "admin"

socket = PySocket(HOST, PORT)
socket.connect()
client = OrientDB(socket)
client.db_open(DATABASE_NAME, DB_USER, DB_PWD)

@styk-tv
Copy link

styk-tv commented Apr 22, 2020

OrientTechnologies removed (in their fork) the if statement causing the error :)
orientechnologies@5bbe000 (long time ago it seems)

Orient 3+ documentation page links their own fork as the repo to be used (just passing on the message) Probably because this current bug has not been fixed. Check this link under Developers/Python (change of repo for 3+) http://orientdb.com/docs/3.0.x/

Tested on:
Orientdb 3.0.30
Pyorient 1.5.5 (not working as per ticket) then upgraded to version below.

If you want to try it below, it works, i checked. Just change your python package location from pypi package of this repo to git-based package of OrientTechnologies repo.

In your requirements.txt find line
pyorient=1.5.5

and change to

-e git+https://github.com/orientechnologies/pyorient.git@master#egg=pyorient

then

pip uninstall pyorient && pip install -r requirements.txt

Now the connection as plain as below is working without the error. And be sure to check the first link with the funny fix to have a laugh.

import pyorient
from pyorient.ogm import Graph, Config

graph = Graph(Config.from_url('plocal://127.0.0.1:2424/mydb', 'root', 'supersecret'))

@aniketgambhire
Copy link

Hi @rrmerugu. Install pyorient from source as given here PyOrient Installation after this you no longer will face this issue.

@sanvir10
Copy link

Or use pyorient=1.5.1 with this version this work fine.

@imanabu
Copy link

imanabu commented Jan 5, 2021

@styk-tv's solution fixes the connection problem (thank you for that), but still there are several more issues that's the driver itself.

You need to now get a token and then set it, I did the following and got around that,

    token = client.get_session_token()
    client.set_session_token(token)

But, after that you will still get another exception if you do a Query.

  pyorient.exceptions.PyOrientBadMethodCallException: Unable to find command 'DbQueryMessage'

Granted these are different bugs, I need to warn that you will run into more issues even if you get around this specific issue on this bug.

@DatoAkobiaQualcommAtheros

Getting same issue in 2024. No solutions, huh?

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

No branches or pull requests