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

Use with gremlin-server-3.2 #58

Open
gregory-h opened this issue Sep 23, 2016 · 9 comments
Open

Use with gremlin-server-3.2 #58

gregory-h opened this issue Sep 23, 2016 · 9 comments

Comments

@gregory-h
Copy link

This is a question rather than an issue, but I'm not sure how to mark it as such.

I have been using gremlin-client with the titan-1.0.0.0-hadoop1 release, communicating through a standalone gremlin-server-3.2. I realize that this is not a supported configuration but it has been working well enough for a POC.

I'm wanting to migrate to a source build of titan with gremlin server 3.2, per https://groups.google.com/forum/#!topic/gremlin-users/ajs0C4-0vzY, but when I do so I lose the ability to use arrays as bound properties via gremlin client, in spite of the fact that they work in the gremlin console.
For example the following works in gremlin console but fails when issued via gremlin client using bound parameters.

// works in gremlin console
g.addV( "name","vn", "read",["p1"])

// works via gremlin client
g.addV( "name","vn", "read",["p1"])

// fails with bound parameter via gremlin client
g.addV( "name","vn", "read", read), {read:[ 'p1' ]}
=>
Error: Property value [[p1]] is of type class java.util.ArrayList is not supported (Error 500)

The query works via gremlin-client against my original titan-1.0.0-hadoop1 + gremlin-server-3.2 installation, so I'm wondering if the error with the custom build is some artifact of the build or if I am missing some gremlin server config or ?

Help appreciated.

@dmill-bz
Copy link
Collaborator

dmill-bz commented Sep 24, 2016

Hey,

I've got a few questions about this:

  1. Do you get this against gremlin-server 3.2.0 without titan?
  2. Is read indexed? If so can you show us how it's indexed?

Cheers.

FYI: I know the php driver tests these scenarios against gremlin server 3.2.1 (https://github.com/PommeVerte/gremlin-php/blob/master/tests/GremlinServerTest.php#L53-L56) and everything seems to work ok so we should probably be looking somewhere else. (1. should confirm this)

@gregory-h
Copy link
Author

Hi Dylan

There is no index on "read", and the test passes against gremlin-server 3.2.0 with no titan instance running.

Thx,

Greg


From: Dylan Millikin notifications@github.com
Sent: September 24, 2016 5:30 AM
To: jbmusso/gremlin-javascript
Cc: Gregory; Author
Subject: Re: [jbmusso/gremlin-javascript] Use with gremlin-server-3.2 (#58)

Hey,

I've got a few questions about this:

  • Do you get this against gremlin-server 3.2.0 without titan?
  • Is "read" indexed? If so can you show us how it's indexed?

Cheers.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com//issues/58#issuecomment-249360135, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAW5DUoe1a9zi8djq3yVXjq6eKYeGO1Gks5qtQnvgaJpZM4KE49e.

@dmill-bz
Copy link
Collaborator

Ok sounds like we should try to run these tests against the new titan then. I'll see if I can test this over the weekend.

@dmill-bz
Copy link
Collaborator

If you're in a bit more of a hurry here are a couple of things you can look at. These usually spawn different errors, but on the offchance :

  • Does the GraphSONMessageSerializer have useMapperFromGraphconfigured? :

    { className: org.apache.tinkerpop.gremlin.driver.ser.GraphSONMessageSerializerV1d0, config: { useMapperFromGraph: graph, ioRegistries: [com.thinkaurelius.titan.graphdb.tinkerpop.TitanIoRegistry] }}
  • Has read had a type either manually defined or automatically defined by setting it with a different value? (ie: if you reset your DB completely by removing the db folder and your test does the first write of the read field does it still fail?)

@gregory-h
Copy link
Author

Hi Dylan,

The usermapperFromGraph property was not specified in gremlin-server.yaml but adding it didn't produce any change in the test result. Similarly deleting my db folder and running the test again also didn't produce any change.

Thanks though!

Cheers,

Greg


From: Dylan Millikin notifications@github.com
Sent: September 24, 2016 12:00 PM
To: jbmusso/gremlin-javascript
Cc: Gregory; Author
Subject: Re: [jbmusso/gremlin-javascript] Use with gremlin-server-3.2 (#58)

If you're in a bit more of a hurry here are a couple of things you can look at. These usually spawn different errors, but on the offchance :

  • Does the GraphSONMessageSerializer have useMapperFromGraphconfigured? :

{ className: org.apache.tinkerpop.gremlin.driver.ser.GraphSONMessageSerializerV1d0, config: { useMapperFromGraph: graph, ioRegistries: [com.thinkaurelius.titan.graphdb.tinkerpop.TitanIoRegistry] }}

  • Has read had a type either manually defined or automatically defined by setting it with a different value? (ie: if you reset your DB completely by removing the db folder and your test does the first write of the read field does it still fail?)

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com//issues/58#issuecomment-249378538, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAW5DWZ366QNjLzQ-6UpLcgGgmaSsRcBks5qtWUrgaJpZM4KE49e.

@gregory-h
Copy link
Author

I'm stuck without Geo support for my project until I can get past this; wondering how to gauge the severity? Will the solution require an official Titan release with tinkerpop 3.2 support, or is this still possibly a titan/gremlin-server configuration issue?

@dmill-bz
Copy link
Collaborator

dmill-bz commented Oct 4, 2016

I would look into this but I'm stuck working out another issue which implies running a batch process that's chewing up all my ressources >.<

If this works ok with gremlin-server 3.2.x (whichever version is being used with your titan) then chances are you should look into the titan end of things.

When I find the time I'll try building that branch and I'll do tests with another driver to see if it's also an issue there.

@gregory-h
Copy link
Author

Thanks Dylan.

Do you happen to know if TinkerGraph supports geo queries? I could use that for awhile but can't find anything to support that it does so I assume not..

@dmill-bz
Copy link
Collaborator

dmill-bz commented Oct 5, 2016

Pretty sure it doesn't.
I need to upgrade my titan today so I might have more information in regards to titan+ gs 3.2.x

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

2 participants