Thanks for doing that Dylan. I've been sick the past weekend and I haven't gotten anything done.
On Mon, Nov 9, 2015 at 10:03 AM, Dylan Millikin <dylan.milli...@gmail.com> wrote: > I create a JIRA ticket for this here : > https://issues.apache.org/jira/browse/TINKERPOP3-950 > > On Thu, Nov 5, 2015 at 12:17 PM, Stephen Mallette <spmalle...@gmail.com> > wrote: > > > My suggestion would be to create an issue in JIRA for this. If we can > get > > it into 3.1.0 then let's try, but it may have to wait for 3.1.1 given > that > > no one seems to have a clue why this is failing. > > > > Can any other driver providers give this a try? Maybe it works for the > JS > > driver? or the Go driver? > > > > On Thu, Nov 5, 2015 at 3:53 AM, Dylan Millikin <dylan.milli...@gmail.com > > > > wrote: > > > > > Yeah I had a look at the requests and they're identical between > > gremlin-php > > > and gremlin-driver. Everything seems to be done over the same channel. > > The > > > only thing that could differ might be the handshakes? Weird. > > > > > > On Wed, Nov 4, 2015 at 10:52 PM, Stephen Mallette < > spmalle...@gmail.com> > > > wrote: > > > > > > > thanks dylan, very strange that these two tests pass for the java > > driver > > > > but dont' pass for the drivers you both have developed. can't think > of > > > > what could be different........ > > > > > > > > On Wed, Nov 4, 2015 at 4:41 PM, Dylan Millikin < > > dylan.milli...@gmail.com > > > > > > > > wrote: > > > > > > > > > I added the two gremlin-driver tests here : > > > > > > > > > > > > > > > > > > > > https://github.com/apache/incubator-tinkerpop/commit/4cb087d039990f643494146e48ab95264cc99805 > > > > > > > > > > These pass so we'll have to break it down a bit to reproduce this. > > > > > > > > > > On Wed, Nov 4, 2015 at 10:07 PM, David Brown <davebs...@gmail.com> > > > > wrote: > > > > > > > > > > > I had the same results as Dylan here. Using def works, but state > > > isn't > > > > > > maintained across requests. This also brings up the question: > isn't > > > it > > > > a > > > > > > bit weird that you have to declare the var in secure, but not in > > > > > > non-secure...? > > > > > > > > > > > > On Wed, Nov 4, 2015 at 3:23 PM, Dylan Millikin < > > > > dylan.milli...@gmail.com > > > > > > > > > > > > wrote: > > > > > > > > > > > > > Declaring the var does work. The problem is that between two > > > requests > > > > > the > > > > > > > bindings seem to vanish (even though we're in the same > session). > > So > > > > the > > > > > > > second query will always complain about v > > > > > > > > > > > > > > On Wed, Nov 4, 2015 at 9:21 PM, Jason Plurad < > plur...@gmail.com> > > > > > wrote: > > > > > > > > > > > > > > > Did you try: def v=graph.addVertex('name', 'Dave'); v > > > > > > > > > > > > > > > > Vaguely recall having to declare the var with secure. > > > > > > > > On Wed, Nov 4, 2015 at 2:44 PM Stephen Mallette < > > > > > spmalle...@gmail.com> > > > > > > > > wrote: > > > > > > > > > > > > > > > > > I have no idea off hand especially if: > > > > > > > > > > > > > > > > > > > I went to add the tests to the integration suit for > > > > > gremlin-server > > > > > > > and > > > > > > > > > couldn't > > > > > > > > > recreate this behavior using gremlin-driver (both tests I > set > > > up > > > > > > pass). > > > > > > > > > > > > > > > > > > Interesting that you were able to create tests that passed > > for > > > > the > > > > > > > > driver. > > > > > > > > > I think those are worth committing and keeping if you still > > > have > > > > > > them. > > > > > > > > You > > > > > > > > > can CTR those to master - i'll review after the fact if you > > > send > > > > > me a > > > > > > > > link > > > > > > > > > to the commit. > > > > > > > > > > > > > > > > > > Does the PHP driver have this problem? can you write a test > > > from > > > > > that > > > > > > > > > perspective? > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Wed, Nov 4, 2015 at 2:28 PM, Dylan Millikin < > > > > > > > dylan.milli...@gmail.com > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > I just tried this with the php driver and I get the same > > > > result. > > > > > > Both > > > > > > > > on > > > > > > > > > > 3.0.2-incubating and 3.1.0-SNAPSHOT. (I even tried > > > > authenticating > > > > > > > > against > > > > > > > > > > the session processor "just in case") > > > > > > > > > > > > > > > > > > > > I went to add the tests to the integration suit for > > > > > gremlin-server > > > > > > > and > > > > > > > > > > couldn't recreate this behavior using gremlin-driver > (both > > > > tests > > > > > I > > > > > > > set > > > > > > > > up > > > > > > > > > > pass). > > > > > > > > > > > > > > > > > > > > I'm not really sure why this is happening at this point. > I > > > will > > > > > > > > probably > > > > > > > > > > have to write another set of tests using the simple > client > > to > > > > > > figure > > > > > > > > out > > > > > > > > > > what is going on. > > > > > > > > > > > > > > > > > > > > Any ideas Stephen? > > > > > > > > > > > > > > > > > > > > On Wed, Nov 4, 2015 at 6:08 PM, David Brown < > > > > davebs...@gmail.com > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > I am having a bit of trouble getting my SessionClient > > tests > > > > > > running > > > > > > > > > using > > > > > > > > > > > Gremlin Server 3.0.2-incubating when I use the > > > > > > > > > > > conf/gremlin-server-secure.yaml. Maybe I am missing > > > > something, > > > > > so > > > > > > > any > > > > > > > > > > help > > > > > > > > > > > would be appreciated. > > > > > > > > > > > > > > > > > > > > > > When using the regular conf/gremlin-server.yaml > config, I > > > > send > > > > > a > > > > > > > > series > > > > > > > > > > of > > > > > > > > > > > messages to make sure state is being maintained across > > > > messages > > > > > > as > > > > > > > > > > follows: > > > > > > > > > > > > > > > > > > > > > > Message: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > b'\x10application/json{"processor":"session","op":"eval","requestId":"eecf7155-46ee-4ad1-a792-d9babd85b72d","args":{"bindings":null,"gremlin":"v=graph.addVertex(\'name\', > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > \'Dave\')","rebindings":{},"language":"gremlin-groovy","session":"83440e37-15d1-4d94-9f2a-c38483b1e8f5"}}' > > > > > > > > > > > > > > > > > > > > > > Response: > > > > > > > > > > > {'result': {'data': [{'label': 'vertex', 'type': > > 'vertex', > > > > > 'id': > > > > > > 4, > > > > > > > > > > > 'properties': {'name': [{'id': 5, 'value': 'Dave'}]}}], > > > > 'meta': > > > > > > > {}}, > > > > > > > > > > > 'status': {'code': 200, 'message': '', 'attributes': > {}}, > > > > > > > > 'requestId': > > > > > > > > > > > 'eecf7155-46ee-4ad1-a792-d9babd85b72d'} > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Message: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > b'\x10application/json{"processor":"session","op":"eval","requestId":"e7bd76f4-2ce4-4129-b065-a5418627ac4c","args":{"bindings":null,"gremlin":"v.property(\'name\')","rebindings":{},"language":"gremlin-groovy","session":"83440e37-15d1-4d94-9f2a-c38483b1e8f5"}}' > > > > > > > > > > > > > > > > > > > > > > Response: > > > > > > > > > > > {'result': {'data': [{'label': 'name', 'id': 5, > 'value': > > > > > > 'Dave'}], > > > > > > > > > > 'meta': > > > > > > > > > > > {}}, 'status': {'code': 200, 'message': '', > 'attributes': > > > > {}}, > > > > > > > > > > 'requestId': > > > > > > > > > > > 'e7bd76f4-2ce4-4129-b065-a5418627ac4c'} > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > This is exactly the behaviour I expect. However, when I > > run > > > > the > > > > > > > > server > > > > > > > > > > with > > > > > > > > > > > the the secure conf, the user auth works fine, but > then I > > > > > receive > > > > > > > an > > > > > > > > > > error > > > > > > > > > > > stating that the variable `v` is undeclared as follows: > > > > > > > > > > > > > > > > > > > > > > Message: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > b'\x10application/json{"requestId":"1afa7873-df64-43a1-82a2-4aaa3cc35d82","processor":"session","args":{"bindings":null,"language":"gremlin-groovy","session":"e4deb7ef-c551-49bc-896d-089dba9cb5e6","gremlin":"v=graph.addVertex(\'name\', > > > > > > > > > > > \'Dave\')","rebindings":{}},"op":"eval"}' > > > > > > > > > > > > > > > > > > > > > > Response: > > > > > > > > > > > {'status': {'message': '', 'code': 407, 'attributes': > > {}}, > > > > > > > > 'requestId': > > > > > > > > > > > '1afa7873-df64-43a1-82a2-4aaa3cc35d82', 'result': > > {'meta': > > > > {}, > > > > > > > > 'data': > > > > > > > > > > > None}} > > > > > > > > > > > > > > > > > > > > > > Message: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > b'\x10application/json{"requestId":"78e3e4e1-08d9-4222-a57a-3fd07180d974","processor":"","args":{"sasl":"AHN0ZXBoZW4AcGFzc3dvcmQ="},"op":"authentication"}' > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Response: > > > > > > > > > > > {'status': {'message': "startup > failed:\nScript2.groovy: > > 1: > > > > > > [Static > > > > > > > > > type > > > > > > > > > > > checking] - The variable [v] is undeclared.\n @ line 1, > > > > column > > > > > > 1.\n > > > > > > > > > > > v=graph.addVertex('name', 'Dave')\n ^\n\n1 error\n", > > > > 'code': > > > > > > 597, > > > > > > > > > > > 'attributes': {}}, 'requestId': > > > > > > > > '1afa7873-df64-43a1-82a2-4aaa3cc35d82', > > > > > > > > > > > 'result': {'meta': {}, 'data': None}} > > > > > > > > > > > > > > > > > > > > > > Any insights into why this happens? Am I doing > something > > > > wrong? > > > > > > > > > > > > > > > > > > > > > > TIA > > > > > > > > > > > > > > > > > > > > > > Dave > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > > > David M. Brown > > > > > > > > > > > R.A. CulturePlex Lab, Western University > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > David M. Brown > > > > > > R.A. CulturePlex Lab, Western University > > > > > > > > > > > > > > > > > > > > > -- David M. Brown R.A. CulturePlex Lab, Western University
