jbmusso / gremlin-javascript

JavaScript tools for graph processing in Node.js and the browser inspired by the Apache TinkerPop API
MIT License
215 stars 62 forks source link

Use with gremlin-server-3.2 #58

Open gregory-h opened 8 years ago

gregory-h commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

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:

Cheers.

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

dmill-bz commented 8 years ago

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 commented 8 years ago

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 :

gregory-h commented 8 years ago

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 :

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

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

gregory-h commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

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 commented 8 years ago

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