Re: Code Freeze 3.2.3/3.1.5

2016-10-17 Thread Daniel Kuppitz 
Here's the fix:


$ git diff bin/validate-distribution.sh
diff --git a/bin/validate-distribution.sh b/bin/validate-distribution.sh
index 4daaf47..2148610 100755
--- a/bin/validate-distribution.sh
+++ b/bin/validate-distribution.sh
@@ -117,11 +117,11 @@ if [ "${TYPE}" = "SOURCE" ]; then
 cd ${DIR_NAME}
 echo -n "* building project ... "
 LOG_FILE="${TMP_DIR}/mvn-clean-install-${VERSION}.log"
-mvn clean install -q 2>&1 > "${LOG_FILE}" || {
*+mvn clean install -q > "${LOG_FILE}" 2>&1 || {*
   echo "failed"
   echo
   tail -n50 "${LOG_FILE}"
-  echo -e "\n\e[1mThe full log file can be inspected under
`pwd`/${LOG_FILE}.\e[0m\n"
*+  echo -e "\n\e[1mThe full log file can be inspected under
${LOG_FILE}.\e[0m\n"*
   exit 1
 }
 echo "OK"


Cheers,
Daniel


On Mon, Oct 17, 2016 at 11:24 PM, Stephen Mallette 
wrote:

> You've probably noted the VOTE thread is out for 3.2.3 - note that
> bin/validate-distribution.sh works, but I noticed it generates a bit of
> junk logging around python output. Not sure what that's all about. I've
> sent a note to kuppitz about it. At least it still works - you just have to
> ignore that bit of output as it ultimately ends with "OK".
>
> Kuppitz, if you have the fix, maybe post it as a gist here on this thread.
> Once we re-open master for development you can push the change to the repo.
>
> On Mon, Oct 17, 2016 at 2:06 PM, Stephen Mallette 
> wrote:
>
> > I got delayed on TINKERPOP-1511 - trying to get the release put together
> > now.
> >
> > On Sun, Oct 16, 2016 at 8:38 PM, Ted Wilmes  wrote:
> >
> >> No problem, thanks for the heads up.  I just had been doing some prep
> and
> >> dry runs.  I added it to the changelog too and I'll get things finished
> up
> >> tomorrow.
> >>
> >> --Ted
> >>
> >> On Sun, Oct 16, 2016 at 3:36 PM, Stephen Mallette  >
> >> wrote:
> >>
> >> > da
> >> >
> >> > ted - i just fixed this on tp31:
> >> >
> >> > https://issues.apache.org/jira/browse/TINKERPOP-1512
> >> >
> >> > sorry - i realized too late that you had done some work on firming up
> >> that
> >> > branch for release. hope you don't need to do too much re-work as a
> >> result
> >> > of my push.
> >> >
> >> > On Sun, Oct 16, 2016 at 11:14 AM, Robert Dale 
> >> wrote:
> >> >
> >> > > I think TINKERPOP-1511
> >> > >  should be
> >> fixed
> >> > in
> >> > > 3.2.3.
> >> > >
> >> > > On Fri, Oct 14, 2016 at 10:09 PM, Ted Wilmes 
> >> wrote:
> >> > >
> >> > > > A fresh 3.1.5-SNAPSHOT has been deployed along with the docs.
> >> > > >
> >> > > > --Ted
> >> > > >
> >> > > >
> >> > > >
> >> > > > On Fri, Oct 14, 2016 at 2:10 PM, pieter-gmail <
> >> pieter.mar...@gmail.com
> >> > >
> >> > > > wrote:
> >> > > >
> >> > > > > I just upgraded and things are looking good.
> >> > > > >
> >> > > > > Cheers
> >> > > > >
> >> > > > > Pieter
> >> > > > >
> >> > > > >
> >> > > > > On 14/10/2016 20:50, Stephen Mallette wrote:
> >> > > > > > It's been fairly quiet on this thread for this release for
> some
> >> > > > reason. I
> >> > > > > > assume that can only mean that this is to be the best release
> >> ever!
> >> > > > > >
> >> > > > > > I just published the latest 3.2.3-SNAPSHOT again to Apache
> >> > Snapshots
> >> > > > > > Repository and also republished the docs:
> >> > > > > >
> >> > > > > > http://tinkerpop.apache.org/docs/3.2.3-SNAPSHOT/
> >> > > > > >
> >> > > > > > and they look pretty good (there was a problem with upgrade
> doc
> >> > > > > > formatting). Upgrade docs look really solid for this release.
> >> > > > Hopefully,
> >> > > > > > master is fully stable now and we won't need any more changes
> >> > before
> >> > > I
> >> > > > > > build up the release for vote on monday.
> >> > > > > >
> >> > > > > >
> >> > > > > > On Tue, Oct 11, 2016 at 7:20 PM, Ted Wilmes <
> twil...@gmail.com>
> >> > > wrote:
> >> > > > > >
> >> > > > > >> I was getting failures earlier today off of master.  Just
> did a
> >> > pull
> >> > > > and
> >> > > > > >> things are looking good.
> >> > > > > >>
> >> > > > > >> --Ted
> >> > > > > >>
> >> > > > > >> On Tue, Oct 11, 2016 at 5:48 PM, Stephen Mallette <
> >> > > > spmalle...@gmail.com
> >> > > > > >
> >> > > > > >> wrote:
> >> > > > > >>
> >> > > > > >>> Push a commit to master earlier today to fix that issue we
> >> talked
> >> > > > about
> >> > > > > >>> last week regarding the failing TraversalInterruptionTest.
> >> Travis
> >> > > has
> >> > > > > >> been
> >> > > > > >>> happy and I can't seem to get it to fail locally. I think
> >> it's in
> >> > > > good
> >> > > > > >>> shape. If you were having problems with that before, please
> >> give
> >> > > it a
> >> > > > > try
> >> > > > > >>> now. Marko is still planning to some work to fix up
> >> PeerPressure
> >> > > test
> >> > > > > >>> (can't say I've have trouble with that one myself).
> >> > > > > >>>
> >> > > > > >>> Also, I

Re: Code Freeze 3.2.3/3.1.5

2016-10-17 Thread Stephen Mallette 
You've probably noted the VOTE thread is out for 3.2.3 - note that
bin/validate-distribution.sh works, but I noticed it generates a bit of
junk logging around python output. Not sure what that's all about. I've
sent a note to kuppitz about it. At least it still works - you just have to
ignore that bit of output as it ultimately ends with "OK".

Kuppitz, if you have the fix, maybe post it as a gist here on this thread.
Once we re-open master for development you can push the change to the repo.

On Mon, Oct 17, 2016 at 2:06 PM, Stephen Mallette 
wrote:

> I got delayed on TINKERPOP-1511 - trying to get the release put together
> now.
>
> On Sun, Oct 16, 2016 at 8:38 PM, Ted Wilmes  wrote:
>
>> No problem, thanks for the heads up.  I just had been doing some prep and
>> dry runs.  I added it to the changelog too and I'll get things finished up
>> tomorrow.
>>
>> --Ted
>>
>> On Sun, Oct 16, 2016 at 3:36 PM, Stephen Mallette 
>> wrote:
>>
>> > da
>> >
>> > ted - i just fixed this on tp31:
>> >
>> > https://issues.apache.org/jira/browse/TINKERPOP-1512
>> >
>> > sorry - i realized too late that you had done some work on firming up
>> that
>> > branch for release. hope you don't need to do too much re-work as a
>> result
>> > of my push.
>> >
>> > On Sun, Oct 16, 2016 at 11:14 AM, Robert Dale 
>> wrote:
>> >
>> > > I think TINKERPOP-1511
>> > >  should be
>> fixed
>> > in
>> > > 3.2.3.
>> > >
>> > > On Fri, Oct 14, 2016 at 10:09 PM, Ted Wilmes 
>> wrote:
>> > >
>> > > > A fresh 3.1.5-SNAPSHOT has been deployed along with the docs.
>> > > >
>> > > > --Ted
>> > > >
>> > > >
>> > > >
>> > > > On Fri, Oct 14, 2016 at 2:10 PM, pieter-gmail <
>> pieter.mar...@gmail.com
>> > >
>> > > > wrote:
>> > > >
>> > > > > I just upgraded and things are looking good.
>> > > > >
>> > > > > Cheers
>> > > > >
>> > > > > Pieter
>> > > > >
>> > > > >
>> > > > > On 14/10/2016 20:50, Stephen Mallette wrote:
>> > > > > > It's been fairly quiet on this thread for this release for some
>> > > > reason. I
>> > > > > > assume that can only mean that this is to be the best release
>> ever!
>> > > > > >
>> > > > > > I just published the latest 3.2.3-SNAPSHOT again to Apache
>> > Snapshots
>> > > > > > Repository and also republished the docs:
>> > > > > >
>> > > > > > http://tinkerpop.apache.org/docs/3.2.3-SNAPSHOT/
>> > > > > >
>> > > > > > and they look pretty good (there was a problem with upgrade doc
>> > > > > > formatting). Upgrade docs look really solid for this release.
>> > > > Hopefully,
>> > > > > > master is fully stable now and we won't need any more changes
>> > before
>> > > I
>> > > > > > build up the release for vote on monday.
>> > > > > >
>> > > > > >
>> > > > > > On Tue, Oct 11, 2016 at 7:20 PM, Ted Wilmes 
>> > > wrote:
>> > > > > >
>> > > > > >> I was getting failures earlier today off of master.  Just did a
>> > pull
>> > > > and
>> > > > > >> things are looking good.
>> > > > > >>
>> > > > > >> --Ted
>> > > > > >>
>> > > > > >> On Tue, Oct 11, 2016 at 5:48 PM, Stephen Mallette <
>> > > > spmalle...@gmail.com
>> > > > > >
>> > > > > >> wrote:
>> > > > > >>
>> > > > > >>> Push a commit to master earlier today to fix that issue we
>> talked
>> > > > about
>> > > > > >>> last week regarding the failing TraversalInterruptionTest.
>> Travis
>> > > has
>> > > > > >> been
>> > > > > >>> happy and I can't seem to get it to fail locally. I think
>> it's in
>> > > > good
>> > > > > >>> shape. If you were having problems with that before, please
>> give
>> > > it a
>> > > > > try
>> > > > > >>> now. Marko is still planning to some work to fix up
>> PeerPressure
>> > > test
>> > > > > >>> (can't say I've have trouble with that one myself).
>> > > > > >>>
>> > > > > >>> Also, I published a 3.2.3 -SNAPSHOT earlier today btw to the
>> > Apache
>> > > > > >>> snapshot repository for testing.
>> > > > > >>>
>> > > > > >>> On Fri, Oct 7, 2016 at 6:44 PM, Stephen Mallette <
>> > > > spmalle...@gmail.com
>> > > > > >
>> > > > > >>> wrote:
>> > > > > >>>
>> > > > >  We're supposed to start code freeze tomorrow, but we are a
>> > little
>> > > > > >> behind.
>> > > > >  Still have one PR left to merge and it needs a rebase:
>> > > > > 
>> > > > >  https://github.com/apache/tinkerpop/pull/448
>> > > > > 
>> > > > >  So expect that to get merged for 3.2.3 during code freeze
>> week,
>> > > but
>> > > > >  nothing in that PR should preclude providers from testing
>> their
>> > > > >  implementations.  Other than that, I think everything else of
>> > > > > substance
>> > > > > >>> is
>> > > > >  in.
>> > > > > 
>> > > > >  I do have one worry about that TraversalInterruption test
>> that
>> > has
>> > > > > been
>> > > > >  failing randomly since the LazyBarrierStrategy stuff went in
>> (i
>> > > > > think).
>> > > > >

[jira] [Created] (TINKERPOP-1518) Cache Graph.Features

2016-10-17 Thread stephen mallette (JIRA) 
stephen mallette created TINKERPOP-1518:
---

 Summary: Cache Graph.Features
 Key: TINKERPOP-1518
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1518
 Project: TinkerPop
  Issue Type: Improvement
  Components: test-suite
Affects Versions: 3.2.2
Reporter: stephen mallette
Assignee: stephen mallette


Determine if there's a way to cache graph features to avoid having to load a 
{{Graph}} instance for a test. That could save some test cycles for graphs that 
dont' support a test.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Created] (TINKERPOP-1517) Introduce a GraphProviderSource

2016-10-17 Thread stephen mallette (JIRA) 
stephen mallette created TINKERPOP-1517:
---

 Summary: Introduce a GraphProviderSource
 Key: TINKERPOP-1517
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1517
 Project: TinkerPop
  Issue Type: Improvement
  Components: test-suite
Affects Versions: 3.2.2
Reporter: stephen mallette
Assignee: stephen mallette


Taken right from the mailing list: 

{quote}
We currently tied the GraphProvider to a specific configuration of a Graph 
instance. So every time you want a slight permutation on that configuration, 
you need to create a new GraphProvider instance. I think that we can simplify 
that and cut down on the proliferation of those instances and in the same 
moment offer some added flexibility. I was digging through JUnit docs/code and 
I think there is a way for us to create a "GraphProviderSource" which would 
annotate a test (rather than @GraphProviderClass). The GraphProviderSource 
would produce a list of GraphProvider instances to run each test in a suite 
with. So, if the GraphProviderSource produced 4 different GraphProvider 
instances, it would execute each test in the suite 4 times (one for each 
GraphProvider).
{quote}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Created] (TINKERPOP-1516) Deprecate OptOut and OptIn

2016-10-17 Thread stephen mallette (JIRA) 
stephen mallette created TINKERPOP-1516:
---

 Summary: Deprecate OptOut and OptIn
 Key: TINKERPOP-1516
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1516
 Project: TinkerPop
  Issue Type: Improvement
  Components: structure, test-suite
Affects Versions: 3.2.2
Reporter: stephen mallette
Assignee: stephen mallette


Not clear on how this will work at the moment other than it should be something 
that we do given consensus on the mailing list. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (TINKERPOP-1516) Deprecate OptOut and OptIn

2016-10-17 Thread stephen mallette (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1516?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15583082#comment-15583082
 ] 

stephen mallette commented on TINKERPOP-1516:
-

This could be a breaking change when implemented, but I won't tag it as such 
now as I'm not clear how just deprecation would work along the 3.2.x line. It 
might just be a 3.3.x change and then it's just straight breaking - we'll see 
what's possible.

> Deprecate OptOut and OptIn
> --
>
> Key: TINKERPOP-1516
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1516
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: structure, test-suite
>Affects Versions: 3.2.2
>Reporter: stephen mallette
>Assignee: stephen mallette
>
> Not clear on how this will work at the moment other than it should be 
> something that we do given consensus on the mailing list. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Created] (TINKERPOP-1515) One test suite to rule them all

2016-10-17 Thread stephen mallette (JIRA) 
stephen mallette created TINKERPOP-1515:
---

 Summary: One test suite to rule them all
 Key: TINKERPOP-1515
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1515
 Project: TinkerPop
  Issue Type: Improvement
  Components: test-suite
Affects Versions: 3.2.2
Reporter: stephen mallette
Assignee: stephen mallette
 Fix For: 3.3.0


Make an attempt at an uber test module that would collapse all provider related 
tests into a single module. I say "attempt" as I'm not completely aware of what 
kinds of dependency conflicts I'll see try to do this.  Consider TINKERPOP-1410 
in doing this.

Obviously this change would be breaking if modules are combined and package 
renaming occurs.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Created] (TINKERPOP-1514) Restructure for gremlin-tools module

2016-10-17 Thread stephen mallette (JIRA) 
stephen mallette created TINKERPOP-1514:
---

 Summary: Restructure for gremlin-tools module
 Key: TINKERPOP-1514
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1514
 Project: TinkerPop
  Issue Type: Improvement
  Components: build-release
Affects Versions: 3.2.2
Reporter: stephen mallette
Assignee: stephen mallette
 Fix For: 3.3.0


Need to restructure the project a bit to have a gremlin-tools module which will 
house both gremlin-benchmarks and a gremlin utility modules for internal 
consumption like gremlin-coverage (for test coverage reporting with jacoco). 
Not sure what else might go under here at the moment.

This is a breaking change in the sense that we're moving around 
gremlin-benchmark.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (TINKERPOP-1410) mvn install -Dmaven.test.skip=true doesn't work on a clean machine

2016-10-17 Thread stephen mallette (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15583041#comment-15583041
 ] 

stephen mallette commented on TINKERPOP-1410:
-

I really want this to change - causes extra hassles during release because you 
have to re-run tests you wouldn't otherwise run after you bump versions.

> mvn install -Dmaven.test.skip=true doesn't work on a clean machine
> --
>
> Key: TINKERPOP-1410
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1410
> Project: TinkerPop
>  Issue Type: Bug
>  Components: build-release
>Affects Versions: 3.2.1
>Reporter: Bryn Cooke
>
> {noformat}mvn install -Dmaven.test.skip=true{noformat}
> gives 
> {noformat}
> Failed to execute goal on project spark-gremlin: Could not resolve 
> dependencies for project 
> org.apache.tinkerpop:spark-gremlin:jar:3.2.2-SNAPSHOT: Could not find 
> artifact org.apache.tinkerpop:hadoop-gremlin:jar:tests:3.2.2-SNAPSHOT in 
> apache.snapshots (http://repository.apache.org/snapshots) -> [Help 1]
> [ERROR] {noformat}
> The reason for this is that skipping tests also skips creating the test jar 
> artefact for hadoop-gremlin.
> https://issues.apache.org/jira/browse/MJAR-138
> Really the prefferred way for creating test jars is to have them as a 
> separate project:
> https://maven.apache.org/plugins/maven-jar-plugin/examples/create-test-jar.html



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Reopened] (TINKERPOP-1511) Remote client addV, V()

2016-10-17 Thread stephen mallette (JIRA) 

 [ 
https://issues.apache.org/jira/browse/TINKERPOP-1511?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stephen mallette reopened TINKERPOP-1511:
-

> Remote client addV, V()
> ---
>
> Key: TINKERPOP-1511
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1511
> Project: TinkerPop
>  Issue Type: Bug
>  Components: server
>Affects Versions: 3.2.2
>Reporter: Robert Dale
>Assignee: stephen mallette
>Priority: Critical
> Fix For: 3.2.3
>
> Attachments: fail-time.groovy
>
>
> I could reproduce on remote neo4j, but not embedded, nor remote TinkerGraph.  
> server:
> ./bin/gremlin-server.sh conf/gremlin-server-neo4j.yaml
> console:
> ./bin/gremlin.sh
> gremlin> graph = EmptyGraph.instance()
> ==>emptygraph[empty]
> gremlin> g = graph.traversal().withRemote('conf/remote-graph.properties')
> ==>graphtraversalsource[emptygraph[empty], standard]
> gremlin> for (n in 0..1)
> { x = g.addV('block').property('hash', 
> '1234').property('txt','text').id().next(); g.V(x).id().next(); }
> java.util.NoSuchElementException
> Type ':help' or ':h' for help.
> Display stack trace? [yN]y
> java.util.NoSuchElementException
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:158)
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:140)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:118)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:103)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal.nextTraverser(DriverRemoteTraversal.java:95)
> at 
> org.apache.tinkerpop.gremlin.process.remote.traversal.step.map.RemoteStep.processNextStart(RemoteStep.java:65)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:128)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:38)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.util.DefaultTraversal.next(DefaultTraversal.java:186)
> at java_util_Iterator$next.call(Unknown Source)
> at groovysh_evaluate.run(groovysh_evaluate:3)
> at 
> org.codehaus.groovy.vmplugin.v7.IndyInterface.selectMethod(IndyInterface.java:232)
> at org.codehaus.groovy.tools.shell.Interpreter.evaluate(Interpreter.groovy:70)
> at org.codehaus.groovy.tools.shell.Groovysh.execute(Groovysh.groovy:190)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.super$3$execute(GremlinGroovysh.groovy)
> at sun.reflect.GeneratedMethodAccessor16.invoke(Unknown Source)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.execute(GremlinGroovysh.groovy:72)
> at org.codehaus.groovy.tools.shell.Shell.leftShift(Shell.groovy:122)
> at org.codehaus.groovy.tools.shell.ShellRunner.work(ShellRunner.groovy:95)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.super$2$work(InteractiveShellRunner.groovy)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuper0(ScriptBytecodeAdapter.java:152)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.work(InteractiveShellRunner.groovy:124)
> at org.codehaus.groovy.tools.shell.ShellRunner.run(ShellRunner.groovy:59)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.super$2$run(InteractiveShellRunner.groovy)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at

[jira] [Closed] (TINKERPOP-1511) Remote client addV, V()

2016-10-17 Thread stephen mallette (JIRA) 

 [ 
https://issues.apache.org/jira/browse/TINKERPOP-1511?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stephen mallette closed TINKERPOP-1511.
---
Resolution: Fixed

> Remote client addV, V()
> ---
>
> Key: TINKERPOP-1511
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1511
> Project: TinkerPop
>  Issue Type: Bug
>  Components: server
>Affects Versions: 3.2.2
>Reporter: Robert Dale
>Assignee: stephen mallette
>Priority: Critical
> Fix For: 3.2.3
>
> Attachments: fail-time.groovy
>
>
> I could reproduce on remote neo4j, but not embedded, nor remote TinkerGraph.  
> server:
> ./bin/gremlin-server.sh conf/gremlin-server-neo4j.yaml
> console:
> ./bin/gremlin.sh
> gremlin> graph = EmptyGraph.instance()
> ==>emptygraph[empty]
> gremlin> g = graph.traversal().withRemote('conf/remote-graph.properties')
> ==>graphtraversalsource[emptygraph[empty], standard]
> gremlin> for (n in 0..1)
> { x = g.addV('block').property('hash', 
> '1234').property('txt','text').id().next(); g.V(x).id().next(); }
> java.util.NoSuchElementException
> Type ':help' or ':h' for help.
> Display stack trace? [yN]y
> java.util.NoSuchElementException
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:158)
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:140)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:118)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:103)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal.nextTraverser(DriverRemoteTraversal.java:95)
> at 
> org.apache.tinkerpop.gremlin.process.remote.traversal.step.map.RemoteStep.processNextStart(RemoteStep.java:65)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:128)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:38)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.util.DefaultTraversal.next(DefaultTraversal.java:186)
> at java_util_Iterator$next.call(Unknown Source)
> at groovysh_evaluate.run(groovysh_evaluate:3)
> at 
> org.codehaus.groovy.vmplugin.v7.IndyInterface.selectMethod(IndyInterface.java:232)
> at org.codehaus.groovy.tools.shell.Interpreter.evaluate(Interpreter.groovy:70)
> at org.codehaus.groovy.tools.shell.Groovysh.execute(Groovysh.groovy:190)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.super$3$execute(GremlinGroovysh.groovy)
> at sun.reflect.GeneratedMethodAccessor16.invoke(Unknown Source)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.execute(GremlinGroovysh.groovy:72)
> at org.codehaus.groovy.tools.shell.Shell.leftShift(Shell.groovy:122)
> at org.codehaus.groovy.tools.shell.ShellRunner.work(ShellRunner.groovy:95)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.super$2$work(InteractiveShellRunner.groovy)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuper0(ScriptBytecodeAdapter.java:152)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.work(InteractiveShellRunner.groovy:124)
> at org.codehaus.groovy.tools.shell.ShellRunner.run(ShellRunner.groovy:59)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.super$2$run(InteractiveShellRunner.groovy)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at

Re: Code Freeze 3.2.3/3.1.5

2016-10-17 Thread Stephen Mallette 
I got delayed on TINKERPOP-1511 - trying to get the release put together
now.

On Sun, Oct 16, 2016 at 8:38 PM, Ted Wilmes  wrote:

> No problem, thanks for the heads up.  I just had been doing some prep and
> dry runs.  I added it to the changelog too and I'll get things finished up
> tomorrow.
>
> --Ted
>
> On Sun, Oct 16, 2016 at 3:36 PM, Stephen Mallette 
> wrote:
>
> > da
> >
> > ted - i just fixed this on tp31:
> >
> > https://issues.apache.org/jira/browse/TINKERPOP-1512
> >
> > sorry - i realized too late that you had done some work on firming up
> that
> > branch for release. hope you don't need to do too much re-work as a
> result
> > of my push.
> >
> > On Sun, Oct 16, 2016 at 11:14 AM, Robert Dale  wrote:
> >
> > > I think TINKERPOP-1511
> > >  should be fixed
> > in
> > > 3.2.3.
> > >
> > > On Fri, Oct 14, 2016 at 10:09 PM, Ted Wilmes 
> wrote:
> > >
> > > > A fresh 3.1.5-SNAPSHOT has been deployed along with the docs.
> > > >
> > > > --Ted
> > > >
> > > >
> > > >
> > > > On Fri, Oct 14, 2016 at 2:10 PM, pieter-gmail <
> pieter.mar...@gmail.com
> > >
> > > > wrote:
> > > >
> > > > > I just upgraded and things are looking good.
> > > > >
> > > > > Cheers
> > > > >
> > > > > Pieter
> > > > >
> > > > >
> > > > > On 14/10/2016 20:50, Stephen Mallette wrote:
> > > > > > It's been fairly quiet on this thread for this release for some
> > > > reason. I
> > > > > > assume that can only mean that this is to be the best release
> ever!
> > > > > >
> > > > > > I just published the latest 3.2.3-SNAPSHOT again to Apache
> > Snapshots
> > > > > > Repository and also republished the docs:
> > > > > >
> > > > > > http://tinkerpop.apache.org/docs/3.2.3-SNAPSHOT/
> > > > > >
> > > > > > and they look pretty good (there was a problem with upgrade doc
> > > > > > formatting). Upgrade docs look really solid for this release.
> > > > Hopefully,
> > > > > > master is fully stable now and we won't need any more changes
> > before
> > > I
> > > > > > build up the release for vote on monday.
> > > > > >
> > > > > >
> > > > > > On Tue, Oct 11, 2016 at 7:20 PM, Ted Wilmes 
> > > wrote:
> > > > > >
> > > > > >> I was getting failures earlier today off of master.  Just did a
> > pull
> > > > and
> > > > > >> things are looking good.
> > > > > >>
> > > > > >> --Ted
> > > > > >>
> > > > > >> On Tue, Oct 11, 2016 at 5:48 PM, Stephen Mallette <
> > > > spmalle...@gmail.com
> > > > > >
> > > > > >> wrote:
> > > > > >>
> > > > > >>> Push a commit to master earlier today to fix that issue we
> talked
> > > > about
> > > > > >>> last week regarding the failing TraversalInterruptionTest.
> Travis
> > > has
> > > > > >> been
> > > > > >>> happy and I can't seem to get it to fail locally. I think it's
> in
> > > > good
> > > > > >>> shape. If you were having problems with that before, please
> give
> > > it a
> > > > > try
> > > > > >>> now. Marko is still planning to some work to fix up
> PeerPressure
> > > test
> > > > > >>> (can't say I've have trouble with that one myself).
> > > > > >>>
> > > > > >>> Also, I published a 3.2.3 -SNAPSHOT earlier today btw to the
> > Apache
> > > > > >>> snapshot repository for testing.
> > > > > >>>
> > > > > >>> On Fri, Oct 7, 2016 at 6:44 PM, Stephen Mallette <
> > > > spmalle...@gmail.com
> > > > > >
> > > > > >>> wrote:
> > > > > >>>
> > > > >  We're supposed to start code freeze tomorrow, but we are a
> > little
> > > > > >> behind.
> > > > >  Still have one PR left to merge and it needs a rebase:
> > > > > 
> > > > >  https://github.com/apache/tinkerpop/pull/448
> > > > > 
> > > > >  So expect that to get merged for 3.2.3 during code freeze
> week,
> > > but
> > > > >  nothing in that PR should preclude providers from testing
> their
> > > > >  implementations.  Other than that, I think everything else of
> > > > > substance
> > > > > >>> is
> > > > >  in.
> > > > > 
> > > > >  I do have one worry about that TraversalInterruption test that
> > has
> > > > > been
> > > > >  failing randomly since the LazyBarrierStrategy stuff went in
> (i
> > > > > think).
> > > > >  Marko also mentioned the PeerPressure test. We'll put some
> elbow
> > > > > grease
> > > > >  into that next week and try to get those figured out and more
> > > > stable.
> > > > > 
> > > > >  As a reminder Ted will be release manager for 3.1.5 and I'll
> be
> > > > doing
> > > > >  3.2.3. As usual, we will use this thread to coordinate during
> > code
> > > > > >> freeze
> > > > >  week. Please bring up relevant issues here.
> > > > > 
> > > > >  Thanks,
> > > > > 
> > > > >  Stephen
> > > > > 
> > > > >
> > > > >
> > > >
> > >
> > >
> > >
> > > --
> > > Robert Dale
> > >
> >
>

[jira] [Closed] (TINKERPOP-1511) Remote client addV, V()

2016-10-17 Thread stephen mallette (JIRA) 

 [ 
https://issues.apache.org/jira/browse/TINKERPOP-1511?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stephen mallette closed TINKERPOP-1511.
---
   Resolution: Fixed
 Assignee: stephen mallette
Fix Version/s: 3.2.3

Opened TINKERPOP-1513 for future thinking.

> Remote client addV, V()
> ---
>
> Key: TINKERPOP-1511
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1511
> Project: TinkerPop
>  Issue Type: Bug
>  Components: server
>Affects Versions: 3.2.2
>Reporter: Robert Dale
>Assignee: stephen mallette
>Priority: Critical
>  Labels: console, groovy, neo4j, python
> Fix For: 3.2.3
>
> Attachments: fail-time.groovy
>
>
> I could reproduce on remote neo4j, but not embedded, nor remote TinkerGraph.  
> server:
> ./bin/gremlin-server.sh conf/gremlin-server-neo4j.yaml
> console:
> ./bin/gremlin.sh
> gremlin> graph = EmptyGraph.instance()
> ==>emptygraph[empty]
> gremlin> g = graph.traversal().withRemote('conf/remote-graph.properties')
> ==>graphtraversalsource[emptygraph[empty], standard]
> gremlin> for (n in 0..1)
> { x = g.addV('block').property('hash', 
> '1234').property('txt','text').id().next(); g.V(x).id().next(); }
> java.util.NoSuchElementException
> Type ':help' or ':h' for help.
> Display stack trace? [yN]y
> java.util.NoSuchElementException
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:158)
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:140)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:118)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:103)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal.nextTraverser(DriverRemoteTraversal.java:95)
> at 
> org.apache.tinkerpop.gremlin.process.remote.traversal.step.map.RemoteStep.processNextStart(RemoteStep.java:65)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:128)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:38)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.util.DefaultTraversal.next(DefaultTraversal.java:186)
> at java_util_Iterator$next.call(Unknown Source)
> at groovysh_evaluate.run(groovysh_evaluate:3)
> at 
> org.codehaus.groovy.vmplugin.v7.IndyInterface.selectMethod(IndyInterface.java:232)
> at org.codehaus.groovy.tools.shell.Interpreter.evaluate(Interpreter.groovy:70)
> at org.codehaus.groovy.tools.shell.Groovysh.execute(Groovysh.groovy:190)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.super$3$execute(GremlinGroovysh.groovy)
> at sun.reflect.GeneratedMethodAccessor16.invoke(Unknown Source)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.execute(GremlinGroovysh.groovy:72)
> at org.codehaus.groovy.tools.shell.Shell.leftShift(Shell.groovy:122)
> at org.codehaus.groovy.tools.shell.ShellRunner.work(ShellRunner.groovy:95)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.super$2$work(InteractiveShellRunner.groovy)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuper0(ScriptBytecodeAdapter.java:152)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.work(InteractiveShellRunner.groovy:124)
> at org.codehaus.groovy.tools.shell.ShellRunner.run(ShellRunner.groovy:59)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.super$2$run(InteractiveShellRunner.groovy)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at 
>

[jira] [Created] (TINKERPOP-1513) Consider transaction boundaries given withRemote()

2016-10-17 Thread stephen mallette (JIRA) 
stephen mallette created TINKERPOP-1513:
---

 Summary: Consider transaction boundaries given withRemote()
 Key: TINKERPOP-1513
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1513
 Project: TinkerPop
  Issue Type: Improvement
  Components: driver, server
Affects Versions: 3.2.2
Reporter: stephen mallette


Not sure what this one means, but given TINKERPOP-1511 we probably need to make 
some improvements in this area.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

[jira] [Commented] (TINKERPOP-1511) Remote client addV, V()

2016-10-17 Thread Robert Dale (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1511?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15582877#comment-15582877
 ] 

Robert Dale commented on TINKERPOP-1511:


Just following up that using toList() is solid on 3.2.3-SNAPSHOT

> Remote client addV, V()
> ---
>
> Key: TINKERPOP-1511
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1511
> Project: TinkerPop
>  Issue Type: Bug
>  Components: server
>Affects Versions: 3.2.2
>Reporter: Robert Dale
>Priority: Critical
>  Labels: console, groovy, neo4j, python
> Attachments: fail-time.groovy
>
>
> I could reproduce on remote neo4j, but not embedded, nor remote TinkerGraph.  
> server:
> ./bin/gremlin-server.sh conf/gremlin-server-neo4j.yaml
> console:
> ./bin/gremlin.sh
> gremlin> graph = EmptyGraph.instance()
> ==>emptygraph[empty]
> gremlin> g = graph.traversal().withRemote('conf/remote-graph.properties')
> ==>graphtraversalsource[emptygraph[empty], standard]
> gremlin> for (n in 0..1)
> { x = g.addV('block').property('hash', 
> '1234').property('txt','text').id().next(); g.V(x).id().next(); }
> java.util.NoSuchElementException
> Type ':help' or ':h' for help.
> Display stack trace? [yN]y
> java.util.NoSuchElementException
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:158)
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:140)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:118)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:103)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal.nextTraverser(DriverRemoteTraversal.java:95)
> at 
> org.apache.tinkerpop.gremlin.process.remote.traversal.step.map.RemoteStep.processNextStart(RemoteStep.java:65)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:128)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:38)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.util.DefaultTraversal.next(DefaultTraversal.java:186)
> at java_util_Iterator$next.call(Unknown Source)
> at groovysh_evaluate.run(groovysh_evaluate:3)
> at 
> org.codehaus.groovy.vmplugin.v7.IndyInterface.selectMethod(IndyInterface.java:232)
> at org.codehaus.groovy.tools.shell.Interpreter.evaluate(Interpreter.groovy:70)
> at org.codehaus.groovy.tools.shell.Groovysh.execute(Groovysh.groovy:190)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.super$3$execute(GremlinGroovysh.groovy)
> at sun.reflect.GeneratedMethodAccessor16.invoke(Unknown Source)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.execute(GremlinGroovysh.groovy:72)
> at org.codehaus.groovy.tools.shell.Shell.leftShift(Shell.groovy:122)
> at org.codehaus.groovy.tools.shell.ShellRunner.work(ShellRunner.groovy:95)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.super$2$work(InteractiveShellRunner.groovy)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuper0(ScriptBytecodeAdapter.java:152)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.work(InteractiveShellRunner.groovy:124)
> at org.codehaus.groovy.tools.shell.ShellRunner.run(ShellRunner.groovy:59)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.super$2$run(InteractiveShellRunner.groovy)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at

[jira] [Commented] (TINKERPOP-1511) Remote client addV, V()

2016-10-17 Thread Robert Dale (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1511?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15582760#comment-15582760
 ] 

Robert Dale commented on TINKERPOP-1511:


Ah, that's explains why I've never seen this. I always iterate out with 
toList() or some such.  Did not know that must be done to finalize a 
transaction!  I updated the script to use toList().first() and it does not fail.

Great find!  Thanks!


> Remote client addV, V()
> ---
>
> Key: TINKERPOP-1511
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1511
> Project: TinkerPop
>  Issue Type: Bug
>  Components: server
>Affects Versions: 3.2.2
>Reporter: Robert Dale
>Priority: Critical
>  Labels: console, groovy, neo4j, python
> Attachments: fail-time.groovy
>
>
> I could reproduce on remote neo4j, but not embedded, nor remote TinkerGraph.  
> server:
> ./bin/gremlin-server.sh conf/gremlin-server-neo4j.yaml
> console:
> ./bin/gremlin.sh
> gremlin> graph = EmptyGraph.instance()
> ==>emptygraph[empty]
> gremlin> g = graph.traversal().withRemote('conf/remote-graph.properties')
> ==>graphtraversalsource[emptygraph[empty], standard]
> gremlin> for (n in 0..1)
> { x = g.addV('block').property('hash', 
> '1234').property('txt','text').id().next(); g.V(x).id().next(); }
> java.util.NoSuchElementException
> Type ':help' or ':h' for help.
> Display stack trace? [yN]y
> java.util.NoSuchElementException
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:158)
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:140)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:118)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:103)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal.nextTraverser(DriverRemoteTraversal.java:95)
> at 
> org.apache.tinkerpop.gremlin.process.remote.traversal.step.map.RemoteStep.processNextStart(RemoteStep.java:65)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:128)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:38)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.util.DefaultTraversal.next(DefaultTraversal.java:186)
> at java_util_Iterator$next.call(Unknown Source)
> at groovysh_evaluate.run(groovysh_evaluate:3)
> at 
> org.codehaus.groovy.vmplugin.v7.IndyInterface.selectMethod(IndyInterface.java:232)
> at org.codehaus.groovy.tools.shell.Interpreter.evaluate(Interpreter.groovy:70)
> at org.codehaus.groovy.tools.shell.Groovysh.execute(Groovysh.groovy:190)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.super$3$execute(GremlinGroovysh.groovy)
> at sun.reflect.GeneratedMethodAccessor16.invoke(Unknown Source)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.execute(GremlinGroovysh.groovy:72)
> at org.codehaus.groovy.tools.shell.Shell.leftShift(Shell.groovy:122)
> at org.codehaus.groovy.tools.shell.ShellRunner.work(ShellRunner.groovy:95)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.super$2$work(InteractiveShellRunner.groovy)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuper0(ScriptBytecodeAdapter.java:152)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.work(InteractiveShellRunner.groovy:124)
> at org.codehaus.groovy.tools.shell.ShellRunner.run(ShellRunner.groovy:59)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.super$2$run(InteractiveShellRunner.groovy)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at 
>

[jira] [Commented] (TINKERPOP-1511) Remote client addV, V()

2016-10-17 Thread stephen mallette (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1511?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15582630#comment-15582630
 ] 

stephen mallette commented on TINKERPOP-1511:
-

I think I have this solved. To prevent breaking things i had to copy/paste a 
bit of code, but at least the issue is resolved. I'll look to improve the 
copy/paste stuff for next release. The basic issues was one we've contended 
with before: the server would send a "final" response message before the commit 
was complete on the server. that would allow the next request which would 
probably execute in a different thread to not see the change.

still testing, but i think i have it fixed. it does of course require that you 
fully iterate out a traversal to ensure that the commit completed. I changed 
the logic a little bit in your fail script:

{code}
for (n in 1..1) {
//def x = 
client.submit("g.addV('block').id()").all().get().get(0).getLong()
//assert client.submit("g.V(x).id().hasNext()", ["x": 
x]).all().get().get(0).getBoolean()
def t1 = g.addV('block').id()
x = t1.next(); 
assert t1.hasNext() == false
LockSupport.parkNanos(nanos); 
assert g.V(x).id().hasNext() == true
  }
{code}

Very thankful for that script by the way. You can see how i make sure there are 
no additional objects in the traversal with {{hasNext()}}. ugly and sucky to 
say the least - that's gotta get better. 

Running integration tests now - hopefully nothing new is broken as a result of 
this change. 

> Remote client addV, V()
> ---
>
> Key: TINKERPOP-1511
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1511
> Project: TinkerPop
>  Issue Type: Bug
>  Components: server
>Affects Versions: 3.2.2
>Reporter: Robert Dale
>Priority: Critical
>  Labels: console, groovy, neo4j, python
> Attachments: fail-time.groovy
>
>
> I could reproduce on remote neo4j, but not embedded, nor remote TinkerGraph.  
> server:
> ./bin/gremlin-server.sh conf/gremlin-server-neo4j.yaml
> console:
> ./bin/gremlin.sh
> gremlin> graph = EmptyGraph.instance()
> ==>emptygraph[empty]
> gremlin> g = graph.traversal().withRemote('conf/remote-graph.properties')
> ==>graphtraversalsource[emptygraph[empty], standard]
> gremlin> for (n in 0..1)
> { x = g.addV('block').property('hash', 
> '1234').property('txt','text').id().next(); g.V(x).id().next(); }
> java.util.NoSuchElementException
> Type ':help' or ':h' for help.
> Display stack trace? [yN]y
> java.util.NoSuchElementException
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:158)
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:140)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:118)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:103)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal.nextTraverser(DriverRemoteTraversal.java:95)
> at 
> org.apache.tinkerpop.gremlin.process.remote.traversal.step.map.RemoteStep.processNextStart(RemoteStep.java:65)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:128)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:38)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.util.DefaultTraversal.next(DefaultTraversal.java:186)
> at java_util_Iterator$next.call(Unknown Source)
> at groovysh_evaluate.run(groovysh_evaluate:3)
> at 
> org.codehaus.groovy.vmplugin.v7.IndyInterface.selectMethod(IndyInterface.java:232)
> at org.codehaus.groovy.tools.shell.Interpreter.evaluate(Interpreter.groovy:70)
> at org.codehaus.groovy.tools.shell.Groovysh.execute(Groovysh.groovy:190)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.super$3$execute(GremlinGroovysh.groovy)
> at sun.reflect.GeneratedMethodAccessor16.invoke(Unknown Source)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.execute(GremlinGroovysh.groovy:72)
> at org.codehaus.groovy.tools.shell.Shell.leftShift(Shell.groovy:122)
> at org.codehaus.groovy.tools.shell.ShellRunner.work(ShellRunner.groovy:95)
> at 
>

[jira] [Commented] (TINKERPOP-1511) Remote client addV, V()

2016-10-17 Thread Robert Dale (JIRA) 

[ 
https://issues.apache.org/jira/browse/TINKERPOP-1511?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15582134#comment-15582134
 ] 

Robert Dale commented on TINKERPOP-1511:


I tried a few things like:

* upgrading to neo4j 2.3.7 and 3.0.3
* swapped gremlin-server jar for 3.2.2 in 3.2.3, but still acted like 3.2.3
* different configurations

None made any difference.  My bet is that the problem is transaction handling 
in gremlin-core. Unfortunately I don't have any time in the near future to dig 
deeper.


> Remote client addV, V()
> ---
>
> Key: TINKERPOP-1511
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1511
> Project: TinkerPop
>  Issue Type: Bug
>  Components: server
>Affects Versions: 3.2.2
>Reporter: Robert Dale
>Priority: Critical
>  Labels: console, groovy, neo4j, python
> Attachments: fail-time.groovy
>
>
> I could reproduce on remote neo4j, but not embedded, nor remote TinkerGraph.  
> server:
> ./bin/gremlin-server.sh conf/gremlin-server-neo4j.yaml
> console:
> ./bin/gremlin.sh
> gremlin> graph = EmptyGraph.instance()
> ==>emptygraph[empty]
> gremlin> g = graph.traversal().withRemote('conf/remote-graph.properties')
> ==>graphtraversalsource[emptygraph[empty], standard]
> gremlin> for (n in 0..1)
> { x = g.addV('block').property('hash', 
> '1234').property('txt','text').id().next(); g.V(x).id().next(); }
> java.util.NoSuchElementException
> Type ':help' or ':h' for help.
> Display stack trace? [yN]y
> java.util.NoSuchElementException
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:158)
> at org.apache.tinkerpop.gremlin.driver.ResultSet$1.next(ResultSet.java:140)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:118)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal$TraverserIterator.next(DriverRemoteTraversal.java:103)
> at 
> org.apache.tinkerpop.gremlin.driver.remote.DriverRemoteTraversal.nextTraverser(DriverRemoteTraversal.java:95)
> at 
> org.apache.tinkerpop.gremlin.process.remote.traversal.step.map.RemoteStep.processNextStart(RemoteStep.java:65)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:128)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.step.util.AbstractStep.next(AbstractStep.java:38)
> at 
> org.apache.tinkerpop.gremlin.process.traversal.util.DefaultTraversal.next(DefaultTraversal.java:186)
> at java_util_Iterator$next.call(Unknown Source)
> at groovysh_evaluate.run(groovysh_evaluate:3)
> at 
> org.codehaus.groovy.vmplugin.v7.IndyInterface.selectMethod(IndyInterface.java:232)
> at org.codehaus.groovy.tools.shell.Interpreter.evaluate(Interpreter.groovy:70)
> at org.codehaus.groovy.tools.shell.Groovysh.execute(Groovysh.groovy:190)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.super$3$execute(GremlinGroovysh.groovy)
> at sun.reflect.GeneratedMethodAccessor16.invoke(Unknown Source)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.apache.tinkerpop.gremlin.console.GremlinGroovysh.execute(GremlinGroovysh.groovy:72)
> at org.codehaus.groovy.tools.shell.Shell.leftShift(Shell.groovy:122)
> at org.codehaus.groovy.tools.shell.ShellRunner.work(ShellRunner.groovy:95)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.super$2$work(InteractiveShellRunner.groovy)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
> at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> at java.lang.reflect.Method.invoke(Method.java:498)
> at org.codehaus.groovy.reflection.CachedMethod.invoke(CachedMethod.java:93)
> at groovy.lang.MetaMethod.doMethodInvoke(MetaMethod.java:325)
> at groovy.lang.MetaClassImpl.invokeMethod(MetaClassImpl.java:1215)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuperN(ScriptBytecodeAdapter.java:132)
> at 
> org.codehaus.groovy.runtime.ScriptBytecodeAdapter.invokeMethodOnSuper0(ScriptBytecodeAdapter.java:152)
> at 
> org.codehaus.groovy.tools.shell.InteractiveShellRunner.work(InteractiveShellRunner.groovy:124)
> at org.codehaus.groovy.tools.shell.ShellRunner.run(ShellRunner.groovy:59)
> at 
>