Re: [DISCUSS] Apache TinkerPop 4 Vision

2018-10-30 Thread Jason Plurad
+1 I'd like to review it. Thanks, Stephen.
On Tue, Oct 30, 2018 at 4:03 AM Stephen Mallette 
wrote:

> We've had many discussion on this list about what TP4 will be. We've
> migrated a number of those ideas to the "future" scratchpad[1]. We've even
> introduced a number of concepts to TP3 which will help form the basis for
> TP4, like with() for provider specific instructions and Jorge's neat new
> work on the binary serialization format[2] which looks really promising.
> There have been numerous blog posts that created a lot of the thinking for
> what TP4 should be about as well[3][4].
>
> Parallel to the TP4 design strategy is the position TinkerPop takes in the
> graph community. We want that position clear for users so that they are
> aware of the benefits they gain by considering TinkerPop-enabled systems.
> We want that position clear to graph providers, so that they can feel
> confident that fully leveraging/supporting TinkerPop is in their best
> interest and yields them the greatest flexibility/capability in building
> their offering. Finally, we want this position clear for ourselves so that
> we can use it to help guide TP4 decision making along those lines. Months
> ago, I conveyed my thoughts on our positioning informally on our mailing
> lists[5].
>
> The problem here is that these thoughts and works are spread unevenly
> across a lot of different places. Marko recently set to work with
> collaboration from me to help drag all of these things together into a
> single coherent document. This document explains the history of the
> TinkerPop project in relation to its lore (never previously unified or
> fully explained) and describes the vision for TP4 using that background
> context.
>
> There are no timelines for TP4 at the moment save for perhaps a starting
> date in the middle of 2019 somewhere...hard to say when such development
> would end obviously and I wouldn't venture to guess. The document does not
> yet address compatibility which I think should be rectified - going to
> suggest that to Marko today. I think forward/backward compatibility  seems
> achievable given bytecode i.e. it's just translating TP3 bytecode to TP4
> bytecode (or the reverse). I'd like to keep compatibility forefront in our
> minds to try to make adoption and transition as smooth as possible.
>
> At this point, we would be interested in feedback that could be
> incorporated into the document from the community. The dev list strips
> attachments and I don't really want to make this document widely released
> yet (i.e. put it on the general internet) so if you are interested in
> reviewing a copy, please just reply to this thread and I will share the PDF
> with you. Please continue to discuss feedback here on this thread though.
>
>
> [1] http://tinkerpop.apache.org/docs/current/dev/future/
> [2] https://issues.apache.org/jira/browse/TINKERPOP-1942
> [3] https://www.datastax.com/dev/blog/the-von-gremlin-architecture
> [4]
>
> https://www.datastax.com/dev/blog/a-gremlin-implementation-of-the-gremlin-traversal-machine
> [5] https://groups.google.com/d/msg/gremlin-users/czzHhYsbEmg/htgJGF4iBAAJ
>


Re: [DISCUSS] Review Process

2018-07-10 Thread Jason Plurad
Thanks for starting this conversation, Stephen. Lots of interesting tidbits
here, and perhaps some we can apply to other OSS projects.

> I'm not sure if committers/PMC members have just not had time to do
reviews or have not felt comfortable doing them

Probably a combination of both, especially with the GLVs.

> I personally chase votes in the background to get PRs to merge.and, I
don't want to do that anymore.

Amazing that you did that, but I agree that nagging is not a great path
forward.

> it is perfectly fine to review/VOTE in the following manner (as examples)

It'd be great to have these examples added to the maintainer guidelines.
When I do code reviews, sometimes I feel like one-liner votes are a bit of
a cop out, but having examples like this would lower the mental hurdle to
getting started on reviewing.

> It would also be nice for non-committers to do reviews - i don't know how
to encourage that behavior though.

I agree on this, and it would be particularly important on areas of the
code where we only have one primary committer, like each GLV. If we come to
agreement on a new policy, I'd suggest that if we get the docs written up
and published, then we can mention it on gremlin-users sort of as a heads
up to those interested in getting more involved. Their participation helps
drive out releases, and new releases attract more users.

Regarding the proposal, a single binding +1 from a committer with a 1 week
lazy consensus sounds fine to me. If the contribution is a major feature or
significant change, the expectation is that the committer realizes this and
holds it open for 3 votes before committing.



On Tue, Jul 10, 2018 at 1:46 PM, Stephen Mallette 
wrote:

> Good point, Ted - that wasn't clear and in truth I didn't think that
> through well. I think we could say that that the +1 would come from a
> committer. If the committer and submitter are one in the same then it has
> its single VOTE and technically, the PR just goes through the week long
> cooling period and could be merged after that. In the event the PR
> submitter is not a committer then they would require at least one committer
> to be on board with a +1 and a week long wait.
>
> Ideally, I think we can trust committers enough to do smart things with
> this change in process. I would hope that a committer who submits a PR that
> is especially complex and touches scary code parts or breaks user/provider
> APIs requests an actual review from another committer who might be able to
> spot some problems even if the 1 week cool down passes by. I don't want to
> subvert the good things that reviews do, but I don't want them holding up
> simple code changes either. I'd really like it if we introduced this change
> and we still got multiple +1s on PRs. It would also be nice for
> non-committers to do reviews - i don't know how to encourage that behavior
> though.
>
>
>
>
> On Tue, Jul 10, 2018 at 1:26 PM Ted Wilmes  wrote:
>
> > I fell way off the PR review train, I'll get back on. For clarification,
> is
> > that a +1 on top of the submitter +1? I'm thinking you
> > all just meant the submitter's +1 would be adequate after the lazy
> > consensus period but wanted to be sure. I'd be fine to moving with that.
> My
> > impression is that with the folks involved, if a submitter feels that
> > another set of eyes is really required and lazy consensus is not
> adequate,
> > regardless of the policy, that review will be sought and performed prior
> to
> > merge.
> >
> > --Ted
> >
> > On Tue, Jul 10, 2018 at 11:44 AM Stephen Mallette 
> > wrote:
> >
> > > >   It looks like its disabled for this project.
> > >
> > > I don't think we can use the GitHub integration without getting off our
> > > Apache Mirror (which we've discussed, but not really pulled the trigger
> > on
> > > for no particular reason other than the hassle of changing everything).
> > >
> > > >   Does it have to be in that order?
> > >
> > > I was thinking that the as long as there is a single +1 at any time in
> > that
> > > week (or after that week) then it would be good to merge
> > >
> > > On Tue, Jul 10, 2018 at 12:36 PM Robert Dale 
> wrote:
> > >
> > > > There might be a better alternative to privately nagging ;-)  Github
> > has
> > > a
> > > > feature on the sidebar that can be used to request reviews from
> > > individuals
> > > > or groups. The heading has 'Reviewers' and, when it's active, has a
> > gear
> > > > icon to select people.  Github will then email the reviewers with the
> > > > request.  It looks like its disabled for this project.
> > > >
> > > > I like the idea of adding the option of having a single vote and a
> week
> > > to
> > > > soak.  Does it have to be in that order?  Or can the vote take place
> > any
> > > > time during that week?  Otherwise, you could end up with no one
> voting
> > in
> > > > the first week, get a vote, then waiting another week.
> > > >
> > > > Robert Dale
> > > >
> > > >
> > > > On Tue, Jul 10, 2018 at 7:22 AM Jorge 

Re: [VOTE] TinkerPop 3.2.9 Release

2018-05-11 Thread Jason Plurad
Validated signatures, reviewed the docs, ran manual tests with Gremlin
Server and Python GLV.

VOTE: +1

On Thu, May 10, 2018 at 5:48 PM, Dylan Millikin 
wrote:

> Tests with the php driver all pass. No regressions or anything noticeable.
>
> VOTE +1
>
>
>
> On Thu, May 10, 2018 at 10:48 AM, Daniel Kuppitz  wrote:
>
> > VOTE: +1
> >
> > On Wed, May 9, 2018 at 10:15 AM, Ted Wilmes  wrote:
> >
> > > Docs and validate looked good +1
> > >
> > > Thanks,
> > > Ted
> > >
> > > On Wed, May 9, 2018 at 6:26 AM, Stephen Mallette  >
> > > wrote:
> > >
> > > > VOTE +1 - reviewed published docs a bit and validate-distribution.sh
> is
> > > > good
> > > >
> > > > On Tue, May 8, 2018 at 4:43 PM, Robert Dale 
> wrote:
> > > >
> > > > > Hello,
> > > > >
> > > > > We are happy to announce that TinkerPop 3.2.9 is ready for release.
> > > > >
> > > > > The release artifacts can be found at this location:
> > > > > https://dist.apache.org/repos/dist/dev/tinkerpop/3.2.9/
> > > > >
> > > > > The source distribution is provided by:
> > > > > apache-tinkerpop-3.2.9-src.zip
> > > > >
> > > > > Two binary distributions are provided for user convenience:
> > > > > apache-tinkerpop-gremlin-console-3.2.9-bin.zip
> > > > > apache-tinkerpop-gremlin-server-3.2.9-bin.zip
> > > > >
> > > > > The GPG key used to sign the release artifacts is available at:
> > > > > https://dist.apache.org/repos/dist/dev/tinkerpop/KEYS
> > > > >
> > > > > The online docs can be found here:
> > > > > http://tinkerpop.apache.org/docs/3.2.9/ (user docs)
> > > > > http://tinkerpop.apache.org/docs/3.2.9/upgrade/ (upgrade
> > docs)
> > > > > http://tinkerpop.apache.org/javadocs/3.2.9/core/ (core
> > > javadoc)
> > > > > http://tinkerpop.apache.org/javadocs/3.2.9/full/ (full
> > > javadoc)
> > > > >
> > > > > The tag in Apache Git can be found here:
> > > > >
> > > > > https://git-wip-us.apache.org/repos/asf?p=tinkerpop.git;a=tag;h=
> > > > > 721fd3c32d0a02337f156ef8d99e7ffb9f1cdc98
> > > > >
> > > > > The release notes are available here:
> > > > >
> > > > > https://github.com/apache/tinkerpop/blob/3.2.9/
> > > > > CHANGELOG.asciidoc#release-3-2-9
> > > > >
> > > > > The [VOTE] will be open for the next 72 hours --- closing Friday
> (May
> > > 11
> > > > > 2018) at 17:00 EDT.
> > > > >
> > > > > My vote is +1.
> > > > >
> > > > > Thank you very much,
> > > > > Robert Dale
> > > > >
> > > >
> > >
> >
>


Re: [VOTE] TinkerPop 3.3.2 Release

2018-04-06 Thread Jason Plurad
gt; >> > all main branches; it's just that:
> >> >
> >> > -[ `gpg ${ZIP_FILENAME}.asc 2>&1 | grep -c '^gpg: Good signature from
> >> > "Stephen Mallette <spmalle...@apache.org>"$'` -eq 1 ] || \
> >> > -[ `gpg ${ZIP_FILENAME}.asc 2>&1 | grep -c '^gpg: Good signature from
> >> > "Marko Rodriguez <ok...@apache.org>"$'` -eq 1 ] || \
> >> > +[ `gpg ${ZIP_FILENAME}.asc 2>&1 | grep -c '^gpg: Good signature from
> >> > "Stephen Mallette <spmalle...@apache.org>"'` -eq 1 ] || \
> >> > +[ `gpg ${ZIP_FILENAME}.asc 2>&1 | grep -c '^gpg: Good signature from
> >> > "Marko Rodriguez <ok...@apache.org>"'` -eq 1 ] || \
> >> >
> >> > Cheers,
> >> > Daniel
> >> >
> >> >
> >> > On Wed, Apr 4, 2018 at 1:49 PM, Stephen Mallette <
> spmalle...@gmail.com>
> >> > wrote:
> >> >
> >> > > I haven't changed anything - i wonder why this is suddenly an
> issue. I
> >> > > guess it worked for me locally because i was the person who signed
> it?
> >> > >
> >> > > On Wed, Apr 4, 2018 at 4:47 PM, Daniel Kuppitz <m...@gremlin.guru>
> >> wrote:
> >> > >
> >> > > > gpg: Signature made Tue 03 Apr 2018 12:53:16 PM MST
> >> > > > gpg:using RSA key EA53A99854EAB0E6
> >> > > > gpg: Good signature from "Stephen Mallette <spmalle...@apache.org
> >"
> >> > > > [unknown]
> >> > > > gpg: WARNING: This key is not certified with a trusted signature!
> >> > > > gpg:  There is no indication that the signature belongs to
> >> the
> >> > > > owner.
> >> > > > Primary key fingerprint: 0871 A360 AAB5 FD42 2516  E2FB EA53 A998
> >> 54EA
> >> > > B0E6
> >> > > >
> >> > > > The [unkown] suffix is what breaks the check.
> >> > > >
> >> > > > If you haven't changed anything in the way you're signing the
> >> > artifacts,
> >> > > we
> >> > > > can probably just ignore the suffix.
> >> > > >
> >> > > >
> >> > > >
> >> > > > On Wed, Apr 4, 2018 at 1:39 PM, Daniel Kuppitz <m...@gremlin.guru>
> >> > wrote:
> >> > > >
> >> > > > > The PGP signature check fails for me.
> >> > > > >
> >> > > > > daniel@cube /projects/apache/tinkerpop (tp33) $
> >> > > > > bin/validate-distribution.sh 3.3.2
> >> > > > >
> >> > > > > Validating binary distributions
> >> > > > >
> >> > > > > * downloading Apache TinkerPop Gremlin
> (apache-tinkerpop-gremlin-
> >> > > > console-3.3.2-bin.zip)...
> >> > > > > OK
> >> > > > > * validating signatures and checksums ...
> >> > > > >   * PGP signature ... failed
> >> > > > >
> >> > > > > daniel@cube /projects/apache/tinkerpop (tp33) $ pushd /tmp
> >> > > > > /tmp /projects/apache/tinkerpop
> >> > > > > daniel@cube /tmp $ wget https://dist.apache.org/repos/
> >> > > > > dist/dev/tinkerpop/KEYS
> >> > > > > --2018-04-04 13:36:03--  https://dist.apache.org/repos/
> >> > > > > dist/dev/tinkerpop/KEYS
> >> > > > > Resolving dist.apache.org (dist.apache.org)... 209.188.14.144
> >> > > > > Connecting to dist.apache.org (dist.apache.org)|209.188.14.
> >> > 144|:443...
> >> > > > > connected.
> >> > > > > HTTP request sent, awaiting response... 200 OK
> >> > > > > Length: 13788 (13K) [text/plain]
> >> > > > > Saving to: ‘KEYS’
> >> > > > >
> >> > > > > KEYS
> >> > > > >  100%[==
> >> > > > > ==>]
> >> 13.46K
> >> > > > > --.-KB/sin 0s
> >> > > > >
> >> > > > > 2018-04-04 13:36:03 (98.0 MB/s) - ‘KEYS’ saved [13788/13788]
> >> > > > >
> >> > > > > daniel@cube /tmp $ gpg --import KEYS
> &g

Re: [VOTE] TinkerPop 3.2.8 Release

2018-04-05 Thread Jason Plurad
pluradj

On Thu, Apr 5, 2018 at 4:07 PM Stephen Mallette <spmalle...@gmail.com>
wrote:

> what's you're nuget account name?
>
> On Thu, Apr 5, 2018 at 6:53 PM, Jason Plurad <plur...@gmail.com> wrote:
>
> > I created a nuget account also. I believe I'm already associated with the
> > TinkerPop on npm.
> >
> > On Thu, Apr 5, 2018 at 3:52 PM, Jason Plurad <plur...@gmail.com> wrote:
> >
> > > Validated the binaries, did several manual tests with the Gremlin
> Server
> > +
> > > Gremlin Console, looked over the documentation.
> > >
> > > VOTE: +1
> > >
> > > On Wed, Apr 4, 2018 at 9:22 AM, Ted Wilmes <twil...@gmail.com> wrote:
> > >
> > >> Thanks for catching that Jorge. I'm okay with the hack. I'll setup an
> > >> npms.js and nuget account.
> > >>
> > >> --Ted
> > >>
> > >> On Wed, Apr 4, 2018 at 7:29 AM, Stephen Mallette <
> spmalle...@gmail.com>
> > >> wrote:
> > >>
> > >> > If everyone else is ok with the little hack job to get this first
> > >> > deployment out, I think I am.
> > >> >
> > >> >
> > >> > On Wed, Apr 4, 2018 at 7:55 AM, Jorge Bay Gondra <
> > >> jorgebaygon...@gmail.com
> > >> > >
> > >> > wrote:
> > >> >
> > >> > > Whatever you feel more comfortable, as the only risk is publishing
> > it
> > >> > under
> > >> > > the name gremlin-javascript which is going to be deprecated
> anyways.
> > >> > >
> > >> > > If you are going to manually execute "npm publish" in your
> terminal,
> > >> you
> > >> > > have to only change the package name in package.json.
> > >> > > If you are planning to use maven with the nodejs plugin to deploy,
> > you
> > >> > have
> > >> > > to change the groovy template file"glv/PackageJson.template" in
> > order
> > >> to
> > >> > > change the generated "package.json" file.
> > >> > >
> > >> > > 2018-04-04 12:31 GMT+02:00 Stephen Mallette <spmalle...@gmail.com
> >:
> > >> > >
> > >> > > > ugh - not to be sloppy but do we need to kill the artifacts
> we've
> > >> > already
> > >> > > > built over that? how bad would it be if we simply remembered to
> > make
> > >> > that
> > >> > > > change at time of deployment? would that cause problems?
> > >> > > >
> > >> > > > On Wed, Apr 4, 2018 at 5:44 AM, Jorge Bay Gondra <
> > >> > > jorgebaygon...@gmail.com
> > >> > > > >
> > >> > > > wrote:
> > >> > > >
> > >> > > > > It looks like the JavaScript package still points to
> > >> > > > "gremlin-javascript",
> > >> > > > > instead of "gremlin"
> > >> > > > >
> > >> > > > > https://github.com/apache/tinkerpop/blob/3.2.8/gremlin-
> > >> > > > > javascript/src/main/javascript/gremlin-javascript/
> > package.json#L2
> > >> > > > >
> > >> > > > > We should change that on both 3.2 and 3.3 branches.
> > >> > > > >
> > >> > > > > Also, if you are planning to do the JavaScript and C# GLV
> > >> releases,
> > >> > you
> > >> > > > > should create an account on npmjs.com (and be part of the
> > >> tinkerpop
> > >> > > > team)
> > >> > > > > and nuget.org (and be given access rights).
> > >> > > > >
> > >> > > > > Cheers,
> > >> > > > > Jorge
> > >> > > > >
> > >> > > > > 2018-04-03 18:58 GMT+02:00 Stephen Mallette <
> > spmalle...@gmail.com
> > >> >:
> > >> > > > >
> > >> > > > > > validate-distributions.sh was clean for me. also gave some
> > >> review
> > >> > to
> > >> > > > the
> > >> > > > > > docs and what i looked at seemed good to go:
> > >> > > > > >
> > >> > > > > > VOTE +1
> > >&

Re: [VOTE] TinkerPop 3.2.8 Release

2018-04-05 Thread Jason Plurad
I created a nuget account also. I believe I'm already associated with the
TinkerPop on npm.

On Thu, Apr 5, 2018 at 3:52 PM, Jason Plurad <plur...@gmail.com> wrote:

> Validated the binaries, did several manual tests with the Gremlin Server +
> Gremlin Console, looked over the documentation.
>
> VOTE: +1
>
> On Wed, Apr 4, 2018 at 9:22 AM, Ted Wilmes <twil...@gmail.com> wrote:
>
>> Thanks for catching that Jorge. I'm okay with the hack. I'll setup an
>> npms.js and nuget account.
>>
>> --Ted
>>
>> On Wed, Apr 4, 2018 at 7:29 AM, Stephen Mallette <spmalle...@gmail.com>
>> wrote:
>>
>> > If everyone else is ok with the little hack job to get this first
>> > deployment out, I think I am.
>> >
>> >
>> > On Wed, Apr 4, 2018 at 7:55 AM, Jorge Bay Gondra <
>> jorgebaygon...@gmail.com
>> > >
>> > wrote:
>> >
>> > > Whatever you feel more comfortable, as the only risk is publishing it
>> > under
>> > > the name gremlin-javascript which is going to be deprecated anyways.
>> > >
>> > > If you are going to manually execute "npm publish" in your terminal,
>> you
>> > > have to only change the package name in package.json.
>> > > If you are planning to use maven with the nodejs plugin to deploy, you
>> > have
>> > > to change the groovy template file"glv/PackageJson.template" in order
>> to
>> > > change the generated "package.json" file.
>> > >
>> > > 2018-04-04 12:31 GMT+02:00 Stephen Mallette <spmalle...@gmail.com>:
>> > >
>> > > > ugh - not to be sloppy but do we need to kill the artifacts we've
>> > already
>> > > > built over that? how bad would it be if we simply remembered to make
>> > that
>> > > > change at time of deployment? would that cause problems?
>> > > >
>> > > > On Wed, Apr 4, 2018 at 5:44 AM, Jorge Bay Gondra <
>> > > jorgebaygon...@gmail.com
>> > > > >
>> > > > wrote:
>> > > >
>> > > > > It looks like the JavaScript package still points to
>> > > > "gremlin-javascript",
>> > > > > instead of "gremlin"
>> > > > >
>> > > > > https://github.com/apache/tinkerpop/blob/3.2.8/gremlin-
>> > > > > javascript/src/main/javascript/gremlin-javascript/package.json#L2
>> > > > >
>> > > > > We should change that on both 3.2 and 3.3 branches.
>> > > > >
>> > > > > Also, if you are planning to do the JavaScript and C# GLV
>> releases,
>> > you
>> > > > > should create an account on npmjs.com (and be part of the
>> tinkerpop
>> > > > team)
>> > > > > and nuget.org (and be given access rights).
>> > > > >
>> > > > > Cheers,
>> > > > > Jorge
>> > > > >
>> > > > > 2018-04-03 18:58 GMT+02:00 Stephen Mallette <spmalle...@gmail.com
>> >:
>> > > > >
>> > > > > > validate-distributions.sh was clean for me. also gave some
>> review
>> > to
>> > > > the
>> > > > > > docs and what i looked at seemed good to go:
>> > > > > >
>> > > > > > VOTE +1
>> > > > > >
>> > > > > > On Tue, Apr 3, 2018 at 12:05 PM, Daniel Kuppitz <m...@gremlin.guru
>> >
>> > > > wrote:
>> > > > > >
>> > > > > > > Looks good.
>> > > > > > >
>> > > > > > > *Validating binary distributions*
>> > > > > > >
>> > > > > > > * downloading Apache TinkerPop Gremlin
>> > > > > > > (apache-tinkerpop-gremlin-console-3.2.8-bin.zip)... OK
>> > > > > > > * validating signatures and checksums ...
>> > > > > > >   * PGP signature ... OK
>> > > > > > >   * MD5 checksum ... OK
>> > > > > > >   * SHA1 checksum ... OK
>> > > > > > > * unzipping Apache TinkerPop Gremlin ... OK
>> > > > > > > * validating Apache TinkerPop Gremlin's docs ... OK
>> > > > > > > * validating Apache TinkerPop Gremlin's binaries ... OK
>> > > > > > > * validating Apa

Re: [VOTE] TinkerPop 3.2.8 Release

2018-04-05 Thread Jason Plurad
Validated the binaries, did several manual tests with the Gremlin Server +
Gremlin Console, looked over the documentation.

VOTE: +1

On Wed, Apr 4, 2018 at 9:22 AM, Ted Wilmes  wrote:

> Thanks for catching that Jorge. I'm okay with the hack. I'll setup an
> npms.js and nuget account.
>
> --Ted
>
> On Wed, Apr 4, 2018 at 7:29 AM, Stephen Mallette 
> wrote:
>
> > If everyone else is ok with the little hack job to get this first
> > deployment out, I think I am.
> >
> >
> > On Wed, Apr 4, 2018 at 7:55 AM, Jorge Bay Gondra <
> jorgebaygon...@gmail.com
> > >
> > wrote:
> >
> > > Whatever you feel more comfortable, as the only risk is publishing it
> > under
> > > the name gremlin-javascript which is going to be deprecated anyways.
> > >
> > > If you are going to manually execute "npm publish" in your terminal,
> you
> > > have to only change the package name in package.json.
> > > If you are planning to use maven with the nodejs plugin to deploy, you
> > have
> > > to change the groovy template file"glv/PackageJson.template" in order
> to
> > > change the generated "package.json" file.
> > >
> > > 2018-04-04 12:31 GMT+02:00 Stephen Mallette :
> > >
> > > > ugh - not to be sloppy but do we need to kill the artifacts we've
> > already
> > > > built over that? how bad would it be if we simply remembered to make
> > that
> > > > change at time of deployment? would that cause problems?
> > > >
> > > > On Wed, Apr 4, 2018 at 5:44 AM, Jorge Bay Gondra <
> > > jorgebaygon...@gmail.com
> > > > >
> > > > wrote:
> > > >
> > > > > It looks like the JavaScript package still points to
> > > > "gremlin-javascript",
> > > > > instead of "gremlin"
> > > > >
> > > > > https://github.com/apache/tinkerpop/blob/3.2.8/gremlin-
> > > > > javascript/src/main/javascript/gremlin-javascript/package.json#L2
> > > > >
> > > > > We should change that on both 3.2 and 3.3 branches.
> > > > >
> > > > > Also, if you are planning to do the JavaScript and C# GLV releases,
> > you
> > > > > should create an account on npmjs.com (and be part of the
> tinkerpop
> > > > team)
> > > > > and nuget.org (and be given access rights).
> > > > >
> > > > > Cheers,
> > > > > Jorge
> > > > >
> > > > > 2018-04-03 18:58 GMT+02:00 Stephen Mallette  >:
> > > > >
> > > > > > validate-distributions.sh was clean for me. also gave some review
> > to
> > > > the
> > > > > > docs and what i looked at seemed good to go:
> > > > > >
> > > > > > VOTE +1
> > > > > >
> > > > > > On Tue, Apr 3, 2018 at 12:05 PM, Daniel Kuppitz  >
> > > > wrote:
> > > > > >
> > > > > > > Looks good.
> > > > > > >
> > > > > > > *Validating binary distributions*
> > > > > > >
> > > > > > > * downloading Apache TinkerPop Gremlin
> > > > > > > (apache-tinkerpop-gremlin-console-3.2.8-bin.zip)... OK
> > > > > > > * validating signatures and checksums ...
> > > > > > >   * PGP signature ... OK
> > > > > > >   * MD5 checksum ... OK
> > > > > > >   * SHA1 checksum ... OK
> > > > > > > * unzipping Apache TinkerPop Gremlin ... OK
> > > > > > > * validating Apache TinkerPop Gremlin's docs ... OK
> > > > > > > * validating Apache TinkerPop Gremlin's binaries ... OK
> > > > > > > * validating Apache TinkerPop Gremlin's legal files ...
> > > > > > >   * LICENSE ... OK
> > > > > > >   * NOTICE ... OK
> > > > > > > * validating Apache TinkerPop Gremlin's plugin directory ... OK
> > > > > > > * validating Apache TinkerPop Gremlin's lib directory ... OK
> > > > > > > * testing script evaluation ... OK
> > > > > > >
> > > > > > > * downloading Apache TinkerPop Gremlin
> > > > > > > (apache-tinkerpop-gremlin-server-3.2.8-bin.zip)... OK
> > > > > > > * validating signatures and checksums ...
> > > > > > >   * PGP signature ... OK
> > > > > > >   * MD5 checksum ... OK
> > > > > > >   * SHA1 checksum ... OK
> > > > > > > * unzipping Apache TinkerPop Gremlin ... OK
> > > > > > > * validating Apache TinkerPop Gremlin's docs ... OK
> > > > > > > * validating Apache TinkerPop Gremlin's binaries ... OK
> > > > > > > * validating Apache TinkerPop Gremlin's legal files ...
> > > > > > >   * LICENSE ... OK
> > > > > > >   * NOTICE ... OK
> > > > > > > * validating Apache TinkerPop Gremlin's plugin directory ... OK
> > > > > > > * validating Apache TinkerPop Gremlin's lib directory ... OK
> > > > > > >
> > > > > > > Validating source distribution
> > > > > > >
> > > > > > > * downloading Apache TinkerPop 3.2.8
> (apache-tinkerpop-3.2.8-src.
> > > > > zip)...
> > > > > > > OK
> > > > > > > * validating signatures and checksums ...
> > > > > > >   * PGP signature ... OK
> > > > > > >   * MD5 checksum ... OK
> > > > > > >   * SHA1 checksum ... OK
> > > > > > > * unzipping Apache TinkerPop 3.2.8 ... OK
> > > > > > > * building project ... OK
> > > > > > >
> > > > > > >
> > > > > > > VOTE: +1
> > > > > > >
> > > > > > > Cheers,
> > > > > > > Daniel
> > > > > > >
> > > > > > >
> > > > > > > On Tue, Apr 3, 2018 at 8:27 AM, 

Re: [DISCUSS] Name of 3.4.x

2018-03-01 Thread Jason Plurad
Boyz II Men - ha, I hadn't considered that angle. I assumed the 4 Seasons
was a Vivaldi reference.

How about David Bowie? The Gremlin Who Sold the World
On Thu, Mar 1, 2018 at 8:06 AM Stephen Mallette 
wrote:

> I'm guessing at the reference for "4 Seasons of Gremlin" - are we talking
> Boyz || Men and "4 Seasons of Loneliness"?
>
> >  Yeah, I'd like to see what Gremlyn Manson would look like!
>
> would he look that much different than our "Nine Inch Nails Gremlin"?
>
>
>
> On Tue, Feb 27, 2018 at 10:58 AM, Robert Dale  wrote:
>
> > Yeah, I'd like to see what Gremlyn Manson would look like!
> >
> > Robert Dale
> >
> > On Tue, Feb 27, 2018 at 10:40 AM, Daniel Kuppitz 
> wrote:
> >
> > > Four Rusted Gremlins (reference
> > > ) :)
> > >
> > > On Tue, Feb 27, 2018 at 7:47 AM, Stephen Mallette <
> spmalle...@gmail.com>
> > > wrote:
> > >
> > > > We need a release name and associated Gremlin logo for the 3.4.x
> series
> > > of
> > > > TinkerPop. Recall that names should be related to music and include a
> > > > number. Thus far we've had:
> > > >
> > > > * Gremlin Symphony #40 in G Minor (3.3.x)
> > > > * Nine Inch Gremlins (3.2.x)
> > > > * A 187 On The Undercover Gremlinz (3.1.x)
> > > > * A Gremlin Rāga in 7/16 Time (3.0.x)
> > > >
> > > > Please submit a name with a logo idea - a name where we can't get a
> > logo
> > > > made doesn't really work too well. Thanks.
> > > >
> > >
> >
>


[jira] [Commented] (TINKERPOP-1880) Gremlin.NET Strong name signature could not be verified. (HRESULT: 0x80131045)

2018-02-12 Thread Jason Plurad (JIRA)

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

Jason Plurad commented on TINKERPOP-1880:
-

Sorry I'm late to the party. [~spmallette] I'll have you know that I recently 
obtained an Windows 10 laptop, but I wouldn't want to be the bottleneck in the 
release process. I'm only an occasional Windows developer, and it beats me if I 
have the right software on the machine to do the deploy. If we need a PMC 
member on the account, you can add me. Let's make sure that the release process 
for these artifacts is documented.

> Gremlin.NET Strong name signature could not be verified. (HRESULT: 0x80131045)
> --
>
> Key: TINKERPOP-1880
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1880
> Project: TinkerPop
>  Issue Type: Bug
>  Components: dotnet
>Affects Versions: 3.2.7, 3.3.1
> Environment: Windows 10, IIS Express 10, .Net Framework 4.7.1
>Reporter: Klaus Stephan
>Priority: Major
> Attachments: 31-01-_2018_14-50-15.png
>
>
> I am trying to use the Gremlin.NET nuget package in a web application (.NET 
> Framework 4.7.1).
> The project builds successfully, but when starting it in IIS Express, I get 
> an error
> "Strong name signature could not be verified. (HRESULT: 0x80131045)"
> I tried different versions, but always get the same error.
> Please fix the nuget packages so that they become usable.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


Re: [VOTE] TinkerPop 3.3.1 Release

2017-12-20 Thread Jason Plurad
VOTE: +1
On Tue, Dec 19, 2017 at 11:29 AM Florian Hockmann 
wrote:

> VOTE: +1
>
>
> Am 19.12.2017 um 17:22 schrieb Ted Wilmes:
> > Docs and binaries look good, thanks for heading this up Stephen.
> >
> > VOTE: +1
> >
> > Thanks,
> > Ted
> >
> > On Tue, Dec 19, 2017 at 2:30 AM, pieter gmail 
> > wrote:
> >
> >> Hi,
> >>
> >> Tested Sqlg on 3.3.1.
> >>
> >> Custom, structured and process tests all pass.
> >>
> >> VOTE +1
> >>
> >> Cheers
> >> Pieter
> >>
> >>
> >> On 17/12/2017 18:54, Stephen Mallette wrote:
> >>
> >>> Hello,
> >>>
> >>> We are happy to announce that TinkerPop 3.3.1 is ready for release.
> >>>
> >>> The release artifacts can be found at this location:
> >>>  https://dist.apache.org/repos/dist/dev/tinkerpop/3.3.1/
> >>>
> >>> The source distribution is provided by:
> >>>  apache-tinkerpop-3.3.1-src.zip
> >>>
> >>> Two binary distributions are provided for user convenience:
> >>>  apache-tinkerpop-gremlin-console-3.3.1-bin.zip
> >>>  apache-tinkerpop-gremlin-server-3.3.1-bin.zip
> >>>
> >>> The GPG key used to sign the release artifacts is available at:
> >>>  https://dist.apache.org/repos/dist/dev/tinkerpop/KEYS
> >>>
> >>> The online docs can be found here:
> >>>  http://tinkerpop.apache.org/docs/3.3.1/ (user docs)
> >>>  http://tinkerpop.apache.org/docs/3.3.1/upgrade/ (upgrade
> docs)
> >>>  http://tinkerpop.apache.org/javadocs/3.3.1/core/ (core
> javadoc)
> >>>  http://tinkerpop.apache.org/javadocs/3.3.1/full/ (full
> javadoc)
> >>>
> >>> The tag in Apache Git can be found here:
> >>>
> >>> https://git-wip-us.apache.org/repos/asf?p=tinkerpop.git;a=ta
> >>> g;h=48c762d3e6fd6dda95b806a0eb1126989f7569ee
> >>>
> >>> The release notes are available here:
> >>>
> >>> https://github.com/apache/tinkerpop/blob/3.3.1/CHANGELOG.
> >>> asciidoc#tinkerpop-331-release-date-december-17-2017
> >>>
> >>> The [VOTE] will be open for the next 72 hours --- closing Wednesday,
> >>> December 20, 2017, at 12:00pm EST.
> >>>
> >>> My vote is +1.
> >>>
> >>> Thank you very much,
> >>> Stephen
> >>>
> >>>
>
>


Re: [VOTE] TinkerPop 3.2.7 Release

2017-12-20 Thread Jason Plurad
VOTE: +1

On Tue, Dec 19, 2017 at 11:29 AM, Florian Hockmann 
wrote:

> VOTE: +1
>
>
> Am 19.12.2017 um 17:22 schrieb Ted Wilmes:
> > VOTE: +1
> >
> > Thanks,
> > Ted
> >
> > On Mon, Dec 18, 2017 at 2:15 PM, Daniel Kuppitz  wrote:
> >
> >> /projects/apache/tinkerpop (tp32) $ bin/validate-distribution.sh 3.2.7
> >>
> >> Validating binary distributions
> >>
> >> * downloading Apache TinkerPop Gremlin
> >> (apache-tinkerpop-gremlin-console-3.2.7-bin.zip)... OK
> >> * validating signatures and checksums ...
> >>   * PGP signature ... OK
> >>   * MD5 checksum ... OK
> >>   * SHA1 checksum ... OK
> >> * unzipping Apache TinkerPop Gremlin ... OK
> >> * validating Apache TinkerPop Gremlin's docs ... OK
> >> * validating Apache TinkerPop Gremlin's binaries ... OK
> >> * validating Apache TinkerPop Gremlin's legal files ...
> >>   * LICENSE ... OK
> >>   * NOTICE ... OK
> >> * validating Apache TinkerPop Gremlin's plugin directory ... OK
> >> * validating Apache TinkerPop Gremlin's lib directory ... OK
> >> * testing script evaluation ... OK
> >>
> >> * downloading Apache TinkerPop Gremlin
> >> (apache-tinkerpop-gremlin-server-3.2.7-bin.zip)... OK
> >> * validating signatures and checksums ...
> >>   * PGP signature ... OK
> >>   * MD5 checksum ... OK
> >>   * SHA1 checksum ... OK
> >> * unzipping Apache TinkerPop Gremlin ... OK
> >> * validating Apache TinkerPop Gremlin's docs ... OK
> >> * validating Apache TinkerPop Gremlin's binaries ... OK
> >> * validating Apache TinkerPop Gremlin's legal files ...
> >>   * LICENSE ... OK
> >>   * NOTICE ... OK
> >> * validating Apache TinkerPop Gremlin's plugin directory ... OK
> >> * validating Apache TinkerPop Gremlin's lib directory ... OK
> >>
> >> Validating source distribution
> >>
> >> * downloading Apache TinkerPop 3.2.7 (apache-tinkerpop-3.2.7-src.
> zip)...
> >> OK
> >> * validating signatures and checksums ...
> >>   * PGP signature ... OK
> >>   * MD5 checksum ... OK
> >>   * SHA1 checksum ... OK
> >> * unzipping Apache TinkerPop 3.2.7 ... OK
> >> * building project ... OK
> >>
> >>
> >> VOTE: +1
> >>
> >>
> >> On Sun, Dec 17, 2017 at 6:22 AM, Stephen Mallette  >
> >> wrote:
> >>
> >>> Hello,
> >>>
> >>> We are happy to announce that TinkerPop 3.2.7  is ready for release.
> >>>
> >>> The release artifacts can be found at this location:
> >>> https://dist.apache.org/repos/dist/dev/tinkerpop/3.2.7/
> >>>
> >>> The source distribution is provided by:
> >>> apache-tinkerpop-3.2.7-src.zip
> >>>
> >>> Two binary distributions are provided for user convenience:
> >>> apache-tinkerpop-gremlin-console-3.2.7-bin.zip
> >>> apache-tinkerpop-gremlin-server-3.2.7-bin.zip
> >>>
> >>> The GPG key used to sign the release artifacts is available at:
> >>> https://dist.apache.org/repos/dist/dev/tinkerpop/KEYS
> >>>
> >>> The online docs can be found here:
> >>> http://tinkerpop.apache.org/docs/3.2.7/ (user docs)
> >>> http://tinkerpop.apache.org/docs/3.2.7/upgrade/ (upgrade docs)
> >>> http://tinkerpop.apache.org/javadocs/3.2.7/core/ (core
> javadoc)
> >>> http://tinkerpop.apache.org/javadocs/3.2.7/full/ (full
> javadoc)
> >>>
> >>> The tag in Apache Git can be found here:
> >>>
> >>> https://git-wip-us.apache.org/repos/asf?p=tinkerpop.git;a=tag;h=
> >>> 540bf5c9a52f33a9448899091e11b9ce5be2792b
> >>>
> >>> The release notes are available here:
> >>>
> >>> https://github.com/apache/tinkerpop/blob/tp32/CHANGELOG.
> >>> asciidoc#tinkerpop-327-release-date-december-17-2017
> >>>
> >>> The [VOTE] will be open for the next 72 hours --- closing Wednesday,
> >>> December 20, 2017, at 8:30am EST.
> >>>
> >>> My vote is +1.
> >>>
> >>> Thank you very much,
> >>> Stephen
> >>>
>
>


[jira] [Created] (TINKERPOP-1769) Python graph[empty] string representation is confusing

2017-09-06 Thread Jason Plurad (JIRA)
Jason Plurad created TINKERPOP-1769:
---

 Summary: Python graph[empty] string representation is confusing
 Key: TINKERPOP-1769
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1769
 Project: TinkerPop
  Issue Type: Bug
  Components: language-variant
Affects Versions: 3.2.6
Reporter: Jason Plurad
Priority: Minor


>From this [StackOverflow 
>post|https://stackoverflow.com/questions/46070265/gremlin-python-returning-empty-graph/46082468#46082468],
> the string representation of the [Python 
>Graph|https://github.com/apache/tinkerpop/blob/3.2.6/gremlin-python/src/main/jython/gremlin_python/structure/graph.py#L37]
> object is:

{noformat}graph[empty]{noformat}

This is likely trying to be similar to {{EmptyGraph}} in Java, but this can be 
misleading for new users, especially if they had been using TinkerGraph which 
shows how many vertices and edges are in the graph. Something like one of these 
could be less confusing:

{noformat}
graph[remote]
graph[]
{noformat}




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


Re: Duy's Gremlin Blog Post Series

2017-08-30 Thread Jason Plurad
+1

On Wed, Aug 30, 2017 at 11:00 AM, Stephen Mallette 
wrote:

> i was going to recommend this as well. it's a really good series. a home
> page would be nice, but we also have more flexibility now in how present
> those "publications" now that all of that has been moved to the
> "compendium". however they get on they get on the site - +1
>
> On Wed, Aug 30, 2017 at 10:56 AM, Marko Rodriguez 
> wrote:
>
> > Hello,
> >
> > I think we should add Duy’s Gremlin Blog Post Series to the
> > tinkerpop.apache.org website:
> >
> > http://www.doanduyhai.com/blog/?p=13224 <
> > http://www.doanduyhai.com/blog/?p=13224>
> > http://www.doanduyhai.com/blog/?p=13260
> > http://www.doanduyhai.com/blog/?p=13285
> > http://www.doanduyhai.com/blog/?p=13301
> > http://www.doanduyhai.com/blog/?p=13320
> > http://www.doanduyhai.com/blog/?p=13352
> > http://www.doanduyhai.com/blog/?p=13374
> > http://www.doanduyhai.com/blog/?p=13404
> > http://www.doanduyhai.com/blog/?p=13424 <
> > http://www.doanduyhai.com/blog/?p=13424>
> >
> > If people agree, I can ask Duy to make a single “homepage” for all the
> > posts.
> >
> > Thoughts?,
> > Marko.
> >
> > http://markorodriguez.com
> >
> >
> >
> >
>


Re: [TinkerPop] Apache S2Graph With Gremlin Support

2017-08-25 Thread Jason Plurad
+1 congrats to the Apache S2Graph community
On Fri, Aug 25, 2017 at 7:07 PM Stephen Mallette 
wrote:

> I just saw that Apache S2Graph just released 0.2.0-incubating with
> TinkerPop support:
>
>
> http://s2graph.incubator.apache.org/releases/s2graph-release-0.2.0-incubating.html
>
> Nice to see yet another TinkerPop-enabled Graph.
>
> Stephen
>
> --
> You received this message because you are subscribed to the Google Groups
> "Gremlin-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to gremlin-users+unsubscr...@googlegroups.com.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/gremlin-users/CAA-H43_q4pHa5U6mfesQtdEy3KMJ1Cqb-tSkFSrKsh-PtNYQ3w%40mail.gmail.com
> 
> .
> For more options, visit https://groups.google.com/d/optout.
>


Re: [VOTE] TinkerPop 3.2.6 Release

2017-08-25 Thread Jason Plurad
Ran bin/validate-distribution.sh 3.2.6 OK
Built successfully from source zip
Tested both binary zips
Reviewed the docs on the website and in the binary zips

VOTE: +1

On Fri, Aug 25, 2017 at 11:30 AM, Ted Wilmes  wrote:

> Validate distribution passed, docs look good.
>
> VOTE: +1
>
> --Ted
>
> On Wed, Aug 23, 2017 at 7:51 AM, Stephen Mallette 
> wrote:
>
> > weird - thanks for noticing. i'd be against stopping release over those
> > things - as you said, they are minor.  let's fix those after release.
> >
> > On Wed, Aug 23, 2017 at 7:29 AM, Florian Hockmann <
> f...@florian-hockmann.de>
> > wrote:
> >
> > > I found a few minor issues with the reference documentation, mainly
> > > related to the new Gremlin.Net GLV:
> > > * Custom Serialization includes the following: 'by creating serializers
> > > and deserializers in C (or any other .NET language)'. The 'C'  should
> be
> > > 'C#' here. I don't know what went wrong here as it's correct in the
> > > asciidoc file and it works just fine for other occurrences of 'C#'. [1]
> > > * The formatting and style of the Conclusion looks wrong. It's
> different
> > > than that of the remaining document. [2]
> > > * We should probably replace Gremlin-DotNet everywhere with Gremlin.Net
> > to
> > > stay consistent.
> > >
> > > I don't know if these minor issues should halt the release. Especially
> > the
> > > Gremlin.Net specific issues can also be fixed for the next release in
> my
> > > opinion.
> > >
> > > Apart from that I built and tested Gremlin.Net without any problems.
> So,
> > > my VOTE is +0
> > >
> > > [1] http://tinkerpop.apache.org/docs/3.2.6/reference/#_custom_
> > > serialization_2
> > > [2] http://tinkerpop.apache.org/docs/3.2.6/reference/#conclusion
> > >
> > > -Ursprüngliche Nachricht-
> > > Von: pieter gmail [mailto:pieter.mar...@gmail.com]
> > > Gesendet: Dienstag, 22. August 2017 21:19
> > > An: dev@tinkerpop.apache.org
> > > Betreff: Re: [VOTE] TinkerPop 3.2.6 Release
> > >
> > > Ran Sqlg's own test suite and TinkerPop's structured and process test
> > > suites on Sqlg.
> > > All passes.
> > >
> > > VOTE +1
> > >
> > > Cheers
> > > Pieter
> > >
> > >
> > > On 22/08/2017 20:29, Robert Dale wrote:
> > > > VOTE +1
> > > >
> > > > Robert Dale
> > > >
> > > > On Tue, Aug 22, 2017 at 10:33 AM, Daniel Kuppitz 
> > > wrote:
> > > >
> > > >> *Validating binary distributions*
> > > >>
> > > >> * downloading Apache TinkerPop Gremlin
> > > >> (apache-tinkerpop-gremlin-console-3.2.6-bin.zip)... OK
> > > >> * validating signatures and checksums ...
> > > >>* PGP signature ... OK
> > > >>* MD5 checksum ... OK
> > > >>* SHA1 checksum ... OK
> > > >> * unzipping Apache TinkerPop Gremlin ... OK
> > > >> * validating Apache TinkerPop Gremlin's docs ... OK
> > > >> * validating Apache TinkerPop Gremlin's binaries ... OK
> > > >> * validating Apache TinkerPop Gremlin's legal files ...
> > > >>* LICENSE ... OK
> > > >>* NOTICE ... OK
> > > >> * validating Apache TinkerPop Gremlin's plugin directory ... OK
> > > >> * validating Apache TinkerPop Gremlin's lib directory ... OK
> > > >> * testing script evaluation ... OK
> > > >>
> > > >> * downloading Apache TinkerPop Gremlin
> > > >> (apache-tinkerpop-gremlin-server-3.2.6-bin.zip)... OK
> > > >> * validating signatures and checksums ...
> > > >>* PGP signature ... OK
> > > >>* MD5 checksum ... OK
> > > >>* SHA1 checksum ... OK
> > > >> * unzipping Apache TinkerPop Gremlin ... OK
> > > >> * validating Apache TinkerPop Gremlin's docs ... OK
> > > >> * validating Apache TinkerPop Gremlin's binaries ... OK
> > > >> * validating Apache TinkerPop Gremlin's legal files ...
> > > >>* LICENSE ... OK
> > > >>* NOTICE ... OK
> > > >> * validating Apache TinkerPop Gremlin's plugin directory ... OK
> > > >> * validating Apache TinkerPop Gremlin's lib directory ... OK
> > > >>
> > > >> Validating source distribution
> > > >>
> > > >> * downloading Apache TinkerPop 3.2.6 (apache-tinkerpop-3.2.6-src.
> > > zip)...
> > > >> OK
> > > >> * validating signatures and checksums ...
> > > >>* PGP signature ... OK
> > > >>* MD5 checksum ... OK
> > > >>* SHA1 checksum ... OK
> > > >> * unzipping Apache TinkerPop 3.2.6 ... OK
> > > >> * building project ... OK
> > > >>
> > > >>
> > > >>
> > > >> VOTE: +1
> > > >>
> > > >> Cheers,
> > > >> Daniel
> > > >>
> > > >>
> > > >> On Tue, Aug 22, 2017 at 4:54 AM, Stephen Mallette
> > > >> 
> > > >> wrote:
> > > >>
> > > >>> Hello,
> > > >>>
> > > >>> We are happy to announce that TinkerPop 3.2.6 is ready for release.
> > > >>>
> > > >>> The release artifacts can be found at this location:
> > > >>>  https://dist.apache.org/repos/dist/dev/tinkerpop/3.2.6/
> > > >>>
> > > >>> The source distribution is provided by:
> > > >>>  apache-tinkerpop-3.2.6-src.zip
> > > >>>
> > > >>> Two binary distributions are provided for user convenience:
> > > >>>

[jira] [Updated] (TINKERPOP-1755) No docs for ReferenceElements

2017-08-20 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1755:

Description: 
https://stackoverflow.com/questions/45764199/janusgraph-cluster-always-returns-vertex-without-properties-referencevertex

I'll submit a pull request for this. 

  
was:https://stackoverflow.com/questions/45764199/janusgraph-cluster-always-returns-vertex-without-properties-referencevertex


> No docs for ReferenceElements
> -
>
> Key: TINKERPOP-1755
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1755
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 3.2.5
>Reporter: Jason Plurad
>
> https://stackoverflow.com/questions/45764199/janusgraph-cluster-always-returns-vertex-without-properties-referencevertex
> I'll submit a pull request for this. 



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Created] (TINKERPOP-1755) No docs for ReferenceElements

2017-08-20 Thread Jason Plurad (JIRA)
Jason Plurad created TINKERPOP-1755:
---

 Summary: No docs for ReferenceElements
 Key: TINKERPOP-1755
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1755
 Project: TinkerPop
  Issue Type: Improvement
  Components: documentation
Affects Versions: 3.2.5
Reporter: Jason Plurad


https://stackoverflow.com/questions/45764199/janusgraph-cluster-always-returns-vertex-without-properties-referencevertex



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (TINKERPOP-1505) Transaction Status listener list gets overwritten

2017-07-24 Thread Jason Plurad (JIRA)

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

Jason Plurad commented on TINKERPOP-1505:
-

Thanks for the feedback, Stephen. I guess we'll look into a provider-specific 
solution for now.

> Transaction Status listener list gets overwritten
> -
>
> Key: TINKERPOP-1505
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1505
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process
>Affects Versions: 3.1.4, 3.2.2
>Reporter: Jason Plurad
>
> Transaction status listener is not called when using Gremlin Server
> * Caused by the transaction listener list initialization in 
> [AbstractThreadLocalTransaction|https://github.com/apache/tinkerpop/blob/3.1.4/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/util/AbstractThreadLocalTransaction.java#L54-L59].
> 
>   * There are 2 different threads in action during the Gremlin Server 
> initialization.
> # The Groovy startup script evaluation which adds the listener
> # The server itself handles the transaction commit or rollback. Both 
> threads end up calling on the {{ThreadLocal initialValue()}} which returns 
> {{new ArrayList()}} and effectively wipes out the listener.
> * Fix is to initialize the list once.
> {noformat}
> protected final ThreadLocal<List<Consumer>> 
> transactionListeners = new ThreadLocal<List<Consumer>>() {
> private List<Consumer> list = new ArrayList<>();
> @Override
> protected List<Consumer> initialValue() {
> return list;
> }
> };
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (TINKERPOP-1716) Traversal strategies are not applied with remote in Gremlin Console

2017-07-20 Thread Jason Plurad (JIRA)

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

Jason Plurad commented on TINKERPOP-1716:
-

Thanks for investigating. Using GraphSON serializer with 3.2.x seems like a 
reasonable solution.

> Traversal strategies are not applied with remote in Gremlin Console
> ---
>
> Key: TINKERPOP-1716
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1716
> Project: TinkerPop
>  Issue Type: Bug
>  Components: server
>Affects Versions: 3.2.3, 3.2.5
>Reporter: Jason Plurad
>Assignee: stephen mallette
> Fix For: 3.3.0
>
>
> Originally from a [StackOverflow 
> post|https://stackoverflow.com/questions/44907843/can-strategies-be-used-with-remote-traversal-in-gremlin]:
> This subgraph strategy works (only the `battled` edges are returned) with a 
> remote from Gremlin-Python:
> {noformat}
> from gremlin_python import statics
> from gremlin_python.structure.graph import Graph
> from gremlin_python.process.graph_traversal import __
> from gremlin_python.process.strategies import *
> from gremlin_python.driver.driver_remote_connection import 
> DriverRemoteConnection
> statics.load_statics(globals())
> graph = Graph()
> g = 
> graph.traversal().withRemote(DriverRemoteConnection('ws://localhost:8182/gremlin','g')).withStrategies(SubgraphStrategy(edges=__.hasLabel('battled')))
> g.E().label().groupCount().toList()
> {noformat}
> But the subgraph strategy does not work (all edges are returned) with a 
> remote from the Gremlin Console:
> {noformat}
> graph = EmptyGraph.instance()
> g = 
> graph.traversal().withRemote('conf/remote-graph.properties').withStrategies(SubgraphStrategy.build().edges(__.hasLabel("battled")).create())
> g.E().label().groupCount().toList()
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Created] (TINKERPOP-1716) Traversal strategies are not applied with remote in Gremlin Console

2017-07-09 Thread Jason Plurad (JIRA)
Jason Plurad created TINKERPOP-1716:
---

 Summary: Traversal strategies are not applied with remote in 
Gremlin Console
 Key: TINKERPOP-1716
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1716
 Project: TinkerPop
  Issue Type: Bug
  Components: server
Affects Versions: 3.2.5, 3.2.3
Reporter: Jason Plurad


Originally from a [StackOverflow 
post|https://stackoverflow.com/questions/44907843/can-strategies-be-used-with-remote-traversal-in-gremlin]:

This subgraph strategy works (only the `battled` edges are returned) with a 
remote from Gremlin-Python:

{noformat}
from gremlin_python import statics
from gremlin_python.structure.graph import Graph
from gremlin_python.process.graph_traversal import __
from gremlin_python.process.strategies import *
from gremlin_python.driver.driver_remote_connection import 
DriverRemoteConnection
statics.load_statics(globals())

graph = Graph()
g = 
graph.traversal().withRemote(DriverRemoteConnection('ws://localhost:8182/gremlin','g')).withStrategies(SubgraphStrategy(edges=__.hasLabel('battled')))
g.E().label().groupCount().toList()
{noformat}

But the subgraph strategy does not work (all edges are returned) with a remote 
from the Gremlin Console:

{noformat}
graph = EmptyGraph.instance()
g = 
graph.traversal().withRemote('conf/remote-graph.properties').withStrategies(SubgraphStrategy.build().edges(__.hasLabel("battled")).create())
g.E().label().groupCount().toList()
{noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


Re: broken builds on release branches

2017-06-01 Thread Jason Plurad
I ran into that yesterday. I think it had something to do with pip getting
confused between Python 2 and 3 on my system. I ended up uninstalling
Python 3, which I had installed via Anaconda, and the TP build worked.
On Thu, Jun 1, 2017 at 8:02 AM Robert Dale  wrote:

> Works for me. I built tp32 yesterday and then this morning on
> commit e4bb1010.  Did your python env get updated?  Looks like you are
> using python 3.4.  I have python 2.7.13 and 3.5.3.   They both have 'six'
> installed:
>
> /usr/lib/python3.5/site-packages/six-1.10.0-py3.5.egg-info
> /usr/lib/python3.5/site-packages/six.py
> /usr/lib/python2.7/site-packages/six-1.10.0-py2.7.egg-info
> /usr/lib/python2.7/site-packages/six.py
>
>
>
>
> Robert Dale
>
> On Thu, Jun 1, 2017 at 7:44 AM, Stephen Mallette 
> wrote:
>
> > It's dying for me on the python3 build even when i try to run it by hand.
> > Oddly Travis is happy as of two hours ago  - maybe it's just me:
> >
> > $ env/bin/python setup.py build --build-lib Lib
> > Traceback (most recent call last):
> >   File
> > "/home/smallette/git/apache/incubator-tinkerpop/gremlin-
> >
> python/target/python3/env/lib/python3.4/site-packages/setuptools/dist.py",
> > line 418, in fetch_build_egg
> > cmd = self._egg_fetcher
> > AttributeError: 'Distribution' object has no attribute '_egg_fetcher'
> >
> > During handling of the above exception, another exception occurred:
> >
> > Traceback (most recent call last):
> >   File "setup.py", line 82, in 
> > "Programming Language :: Python :: 3.5",
> >   File "/usr/lib/python3.4/distutils/core.py", line 108, in setup
> > _setup_distribution = dist = klass(attrs)
> >   File
> > "/home/smallette/git/apache/incubator-tinkerpop/gremlin-
> >
> python/target/python3/env/lib/python3.4/site-packages/setuptools/dist.py",
> > line 320, in __init__
> > self.fetch_build_eggs(attrs['setup_requires'])
> >   File
> > "/home/smallette/git/apache/incubator-tinkerpop/gremlin-
> >
> python/target/python3/env/lib/python3.4/site-packages/setuptools/dist.py",
> > line 377, in fetch_build_eggs
> > replace_conflicting=True,
> >   File
> > "/home/smallette/git/apache/incubator-tinkerpop/gremlin-
> > python/target/python3/env/lib/python3.4/site-packages/pkg_
> > resources/__init__.py",
> > line 852, in resolve
> > dist = best[req.key] = env.best_match(req, ws, installer)
> >   File
> > "/home/smallette/git/apache/incubator-tinkerpop/gremlin-
> > python/target/python3/env/lib/python3.4/site-packages/pkg_
> > resources/__init__.py",
> > line 1124, in best_match
> > return self.obtain(req, installer)
> >   File
> > "/home/smallette/git/apache/incubator-tinkerpop/gremlin-
> > python/target/python3/env/lib/python3.4/site-packages/pkg_
> > resources/__init__.py",
> > line 1136, in obtain
> > return installer(requirement)
> >   File
> > "/home/smallette/git/apache/incubator-tinkerpop/gremlin-
> >
> python/target/python3/env/lib/python3.4/site-packages/setuptools/dist.py",
> > line 421, in fetch_build_egg
> > from setuptools.command.easy_install import easy_install
> >   File
> > "/home/smallette/git/apache/incubator-tinkerpop/gremlin-
> > python/target/python3/env/lib/python3.4/site-packages/
> > setuptools/command/easy_install.py",
> > line 49, in 
> > from setuptools.py27compat import rmtree_safe
> >   File
> > "/home/smallette/git/apache/incubator-tinkerpop/gremlin-
> > python/target/python3/env/lib/python3.4/site-packages/
> > setuptools/py27compat.py",
> > line 7, in 
> > import six
> > ImportError: No module named 'six'
> >
> >
> >
> >
> > On Thu, Jun 1, 2017 at 6:52 AM, Stephen Mallette 
> > wrote:
> >
> > > Something has gone bad with the build on tp32 and master for
> > > gremlin-python. I got a clean run before I shut down last night but
> this
> > > morning it's strangely tanking (and i didnt' pull new changes) -
> > something
> > > around the python dependency "six":
> > >
> > > ImportError: No module named six
> > >
> > > Not sure if anyone has any clues, but we're dead in the water on builds
> > > right now. I hope this is one of those things where I send a warning to
> > the
> > > dev list and then it magically gets fixed thirty seconds later.
> > >
> >
>


[DISCUSS] Scala 2.11 and Spark 2.0 support

2017-05-24 Thread Jason Plurad
This question
 on
TinkerPop 3.3 on the Gremlin users list ultimately comes back to Scala 2.11
support
.
I can understand that some features slated for 3.3 might be far off (
TINKERPOP-1564 ), but
I don't think we should hold up on updating core dependencies such as
Spark+Scala.

Azure Cosmos DB (based on TP 3.2.4) recently published

about connecting
it to Spark, and it tells the developer to build TinkerPop master (TP
3.3.0-SNAPSHOT) because Azure HDInsight uses Spark 2.0.

We ran into a similar issue when TInkerPop 3.0 came out with the Hadoop 1
dependency rather than Hadoop 2, and I think a lot of users had already moved
to Hadoop 2
 by
then. Side note, I don't know what the schedule for Hadoop 3 is, but they
already have alpha versions available.

Spark 2.0 came out in July 2016, and it has been integrated on master
since September
2016
.
Spark 2 builds with Scala 2.11 by default. If we can't get the next
TinkerPop release until next year, TinkerPop's Spark+Scala dependencies
will be very far out of date.

Would it make sense to cherry pick Spark 2.0 support into the tp32 branch?


Re: Microsoft's CosmoDB supports Gremlin

2017-05-11 Thread Jason Plurad
I'm +1 on adding CosmoDB.

I think we have a bit of a branding issue though. Most of their docs talk
about the Gremlin graph language, and they generally fail to mention
"Apache TinkerPop", which perhaps makes sense if they're not directly
implementing TinkerPop's Java APIs. I don't think "Apache Gremlin" was
trademarked by the ASF?

Google still isn't helping either since many of the top links refer to
TinkerPop 2 items (which do have the deprecated warning). A couple ideas
based on the top search results:
* redirect http://gremlin.tinkerpop.com to
http://tinkerpop.apache.org/gremlin.html
* update the text description and URL to make Apache TinkerPop more
prominent on https://github.com/tinkerpop/gremlin


On Wed, May 10, 2017 at 1:13 PM, Luca Garulli <l.garu...@gmail.com> wrote:

> Another Multi-Model? Cool, if anybody can try it I'm curious about how ti
> goes.
>
> On 10 May 2017 at 21:42, Stephen Mallette <spmalle...@gmail.com> wrote:
>
> > Another interesting question is if they implemented the Gremlin Server
> > protocol, does that mean that their C#/NodeJS/etc stuff work with a
> > standalone Gremlin Server?
> >
> > On Wed, May 10, 2017 at 2:23 PM, Florian Hockmann <
> f...@florian-hockmann.de>
> > wrote:
> >
> > > I have no idea to be honest. I would probably just create a query
> > > builder or directly a GLV, but the formulation sounds to me more like
> > > they they wrote their own Gremlin Server in C# and embed that in the
> > > library just like it can be done for JanusGraph or Titan in Java. That
> > > would explain at least how they "connect directly to DocumenDB data
> > > partitions". Sounds like a Titan/JanusGraph equivalent written in C#
> for
> > > their document database as the storage backend, but maybe the
> > > formulation is just odd and it's really only a query builder.
> > >
> > > I will probably play a bit with their client and try to find out how it
> > > works. Fortunately they provide a docker image for Cosmos DB:
> > > https://docs.microsoft.com/en-us/azure/documentdb/
> > documentdb-nosql-local-
> > > emulator
> > >
> > > We will now for sure how it works when they publish the source code
> > > which they promise to do at some point in the future:
> > > https://github.com/Azure/azure-documentdb-dotnet/tree/master/sdk
> > >
> > > Am 10.05.2017 um 20:06 schrieb Stephen Mallette:
> > > > I didn't notice that. I assume it's generating Gremlin script
> > underneath?
> > > > Maybeit's just behaving as a query builder? Or do you think it's
> doing
> > > > something more advanced?
> > > >
> > > > On Wed, May 10, 2017 at 1:59 PM, Florian Hockmann <
> > > f...@florian-hockmann.de>
> > > > wrote:
> > > >
> > > >> I think it is especially interesting that they seem to provide a
> > > >> complete implementation of the GraphTraversal class for .NET:
> > > >>
> > > >> https://msdn.microsoft.com/en-us/library/microsoft.azure.
> > > >> graphs.graphtraversal.aspx
> > > >>
> > > >> This looks a bit like a GLV, but it seems that they don't use
> Gremlin
> > > >> Bytecode and instead embed their own Gremlin Server version in the
> > > client:
> > > >>
> > > >> "Azure Cosmos DB also provides a first-party .NET library with
> Gremlin
> > > >> extension methods on top of the Azure Cosmos DB SDKs via NuGet.
> *This
> > > >> library provide an "in-proc" Gremlin server* that can be used to
> > connect
> > > >> directly to DocumenDB data partitions."
> > > >>
> > > >> (From:
> > > >> https://docs.microsoft.com/en-us/azure/cosmos-db/graph-
> > > >> introduction#getting-started)
> > > >>
> > > >> Am 10.05.2017 um 19:02 schrieb Stephen Mallette:
> > > >>> In case anyone is interested here's some more docs to look at:
> > > >>>
> > > >>> https://docs.microsoft.com/en-us/azure/cosmos-db/create-
> graph-dotnet
> > > >>> https://docs.microsoft.com/en-us/azure/cosmos-db/create-
> > > >>> graph-gremlin-console
> > > >>> https://docs.microsoft.com/en-us/azure/cosmos-db/create-graph-java
> > > >>> https://docs.microsoft.com/en-us/azure/cosmos-db/create-
> graph-nodejs
> > > >>>
> > > >>>
> > > >>>
> >

Re: [TinkerPop] New PMC Member: Robert Dale

2017-04-18 Thread Jason Plurad
Congrats!

On Tue, Apr 18, 2017 at 11:48 AM, Robert Dale  wrote:

> Mmm.. cookies.
>
> Robert Dale
>
> On Tue, Apr 18, 2017 at 11:16 AM, Stephen Mallette 
> wrote:
>
>> We would like to announce that Robert Dale is now a member of the Apache
>> TinkerPop PMC. Robert has been a helpful to the TinkerPop community in a
>> variety of ways for some time now as a committer and has demonstrated
>> ongoing interest in the growth of the project which makes him a welcome
>> member to the PMC.
>>
>> As per tradition, I include with this announcement the TinkerPop Guild
>> portrait as a reminder of the olden and golden days of doing graphs:
>>
>> https://upload.wikimedia.org/wikipedia/commons/c/ce/Jan_de_Bray_002.jpg
>>
>> Welcome to the PMC, Robert - we'll have to get the portrait amended to
>> include you. Or you can just be the guy with the big cookie thing.
>>
>> --
>> You received this message because you are subscribed to the Google Groups
>> "Gremlin-users" group.
>> To unsubscribe from this group and stop receiving emails from it, send an
>> email to gremlin-users+unsubscr...@googlegroups.com.
>> To view this discussion on the web visit https://groups.google.com/d/ms
>> gid/gremlin-users/CAA-H438zY86jjnKK1XD241%3DUorZA1tKoacXUbdS
>> Rfq4g1zETYQ%40mail.gmail.com
>> 
>> .
>> For more options, visit https://groups.google.com/d/optout.
>>
>
> --
> You received this message because you are subscribed to the Google Groups
> "Gremlin-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to gremlin-users+unsubscr...@googlegroups.com.
> To view this discussion on the web visit https://groups.google.com/d/ms
> gid/gremlin-users/CABed_4rO8jamj85mNmSHD5FmTB62VsBy8Y4rBdJh5
> Lb0q7L1RQ%40mail.gmail.com
> 
> .
>
> For more options, visit https://groups.google.com/d/optout.
>


[jira] [Resolved] (TINKERPOP-1667) Inconsistent number comparison with Contains

2017-04-13 Thread Jason Plurad (JIRA)

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

Jason Plurad resolved TINKERPOP-1667.
-
Resolution: Duplicate

> Inconsistent number comparison with Contains
> 
>
> Key: TINKERPOP-1667
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1667
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process
>Affects Versions: 3.2.4
>Reporter: Jason Plurad
>Priority: Minor
>
> Steps to recreate:
> {noformat}
> gremlin> conf = new BaseConfiguration()
> ==>org.apache.commons.configuration.BaseConfiguration@7048535f
> gremlin> conf.setProperty("gremlin.tinkergraph.vertexIdManager","LONG")
> ==>null
> gremlin> conf.setProperty("gremlin.tinkergraph.edgeIdManager","LONG")
> ==>null
> gremlin> graph = TinkerGraph.open(conf)
> ==>tinkergraph[vertices:0 edges:0]
> gremlin> graph.io(gryo()).readGraph('data/tinkerpop-modern.kryo')
> ==>null
> gremlin> g = graph.traversal()
> ==>graphtraversalsource[tinkergraph[vertices:6 edges:6], standard]
> gremlin> g.V().hasId(within(1..6)).out().hasId(within(1..6))
> gremlin> g.V().hasId(within(1..6)).out().hasId(between(0,7))
> ==>v[3]
> ==>v[2]
> ==>v[4]
> ==>v[5]
> ==>v[3]
> ==>v[3]
> {noformat}
> `P.within` and `P.without` rely on `Collection.contains()` which ends up 
> doing an Object type comparison during the equality check.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Closed] (TINKERPOP-1667) Inconsistent number comparison with Contains

2017-04-13 Thread Jason Plurad (JIRA)

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

Jason Plurad closed TINKERPOP-1667.
---

> Inconsistent number comparison with Contains
> 
>
> Key: TINKERPOP-1667
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1667
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process
>Affects Versions: 3.2.4
>Reporter: Jason Plurad
>Priority: Minor
>
> Steps to recreate:
> {noformat}
> gremlin> conf = new BaseConfiguration()
> ==>org.apache.commons.configuration.BaseConfiguration@7048535f
> gremlin> conf.setProperty("gremlin.tinkergraph.vertexIdManager","LONG")
> ==>null
> gremlin> conf.setProperty("gremlin.tinkergraph.edgeIdManager","LONG")
> ==>null
> gremlin> graph = TinkerGraph.open(conf)
> ==>tinkergraph[vertices:0 edges:0]
> gremlin> graph.io(gryo()).readGraph('data/tinkerpop-modern.kryo')
> ==>null
> gremlin> g = graph.traversal()
> ==>graphtraversalsource[tinkergraph[vertices:6 edges:6], standard]
> gremlin> g.V().hasId(within(1..6)).out().hasId(within(1..6))
> gremlin> g.V().hasId(within(1..6)).out().hasId(between(0,7))
> ==>v[3]
> ==>v[2]
> ==>v[4]
> ==>v[5]
> ==>v[3]
> ==>v[3]
> {noformat}
> `P.within` and `P.without` rely on `Collection.contains()` which ends up 
> doing an Object type comparison during the equality check.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Updated] (TINKERPOP-1048) Vertex lookups by id are inconsistent

2017-04-13 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1048:

Description: 
{{graph.vertices(id)}}, {{g.V(id)}} and {{g.V().hasId(id)}} should all return 
the same result. However, currently only the latter respects the {{toString()}} 
representation of ids.

{noformat}
gremlin> g.addV().id()
==>12
gremlin> graph.vertices("12")
gremlin> g.V("12")
gremlin> g.V().hasId("12")
==>v[12]
{noformat}

Inconsistent number comparison with Contains:

{noformat}
gremlin> conf = new BaseConfiguration()
==>org.apache.commons.configuration.BaseConfiguration@7048535f
gremlin> conf.setProperty("gremlin.tinkergraph.vertexIdManager","LONG")
==>null
gremlin> conf.setProperty("gremlin.tinkergraph.edgeIdManager","LONG")
==>null
gremlin> graph = TinkerGraph.open(conf)
==>tinkergraph[vertices:0 edges:0]
gremlin> graph.io(gryo()).readGraph('data/tinkerpop-modern.kryo')
==>null
gremlin> g = graph.traversal()
==>graphtraversalsource[tinkergraph[vertices:6 edges:6], standard]
gremlin> g.V().hasId(within(1..6)).out().hasId(within(1..6))
gremlin> g.V().hasId(within(1..6)).out().hasId(between(0,7))
==>v[3]
==>v[2]
==>v[4]
==>v[5]
==>v[3]
==>v[3]
{noformat}

`P.within` and `P.without` rely on `Collection.contains()` which ends up doing 
an Object type comparison during the equality check.

  was:
{{graph.vertices(id)}}, {{g.V(id)}} and {{g.V().hasId(id)}} should all return 
the same result. However, currently only the latter respects the {{toString()}} 
representation of ids.

{noformat}
gremlin> g.addV().id()
==>12
gremlin> graph.vertices("12")
gremlin> g.V("12")
gremlin> g.V().hasId("12")
==>v[12]
{noformat}


> Vertex lookups by id are inconsistent
> -
>
> Key: TINKERPOP-1048
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1048
> Project: TinkerPop
>  Issue Type: Bug
>  Components: tinkergraph
>Affects Versions: 3.1.1-incubating
>Reporter: Daniel Kuppitz
>Assignee: stephen mallette
>
> {{graph.vertices(id)}}, {{g.V(id)}} and {{g.V().hasId(id)}} should all return 
> the same result. However, currently only the latter respects the 
> {{toString()}} representation of ids.
> {noformat}
> gremlin> g.addV().id()
> ==>12
> gremlin> graph.vertices("12")
> gremlin> g.V("12")
> gremlin> g.V().hasId("12")
> ==>v[12]
> {noformat}
> Inconsistent number comparison with Contains:
> {noformat}
> gremlin> conf = new BaseConfiguration()
> ==>org.apache.commons.configuration.BaseConfiguration@7048535f
> gremlin> conf.setProperty("gremlin.tinkergraph.vertexIdManager","LONG")
> ==>null
> gremlin> conf.setProperty("gremlin.tinkergraph.edgeIdManager","LONG")
> ==>null
> gremlin> graph = TinkerGraph.open(conf)
> ==>tinkergraph[vertices:0 edges:0]
> gremlin> graph.io(gryo()).readGraph('data/tinkerpop-modern.kryo')
> ==>null
> gremlin> g = graph.traversal()
> ==>graphtraversalsource[tinkergraph[vertices:6 edges:6], standard]
> gremlin> g.V().hasId(within(1..6)).out().hasId(within(1..6))
> gremlin> g.V().hasId(within(1..6)).out().hasId(between(0,7))
> ==>v[3]
> ==>v[2]
> ==>v[4]
> ==>v[5]
> ==>v[3]
> ==>v[3]
> {noformat}
> `P.within` and `P.without` rely on `Collection.contains()` which ends up 
> doing an Object type comparison during the equality check.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (TINKERPOP-1667) Inconsistent number comparison with Contains

2017-04-13 Thread Jason Plurad (JIRA)

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

Jason Plurad commented on TINKERPOP-1667:
-

I'll move the scenario over to the other ticket. Consistency for this should be 
handled at the same time as the other issue.

> Inconsistent number comparison with Contains
> 
>
> Key: TINKERPOP-1667
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1667
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process
>Affects Versions: 3.2.4
>Reporter: Jason Plurad
>Priority: Minor
>
> Steps to recreate:
> {noformat}
> gremlin> conf = new BaseConfiguration()
> ==>org.apache.commons.configuration.BaseConfiguration@7048535f
> gremlin> conf.setProperty("gremlin.tinkergraph.vertexIdManager","LONG")
> ==>null
> gremlin> conf.setProperty("gremlin.tinkergraph.edgeIdManager","LONG")
> ==>null
> gremlin> graph = TinkerGraph.open(conf)
> ==>tinkergraph[vertices:0 edges:0]
> gremlin> graph.io(gryo()).readGraph('data/tinkerpop-modern.kryo')
> ==>null
> gremlin> g = graph.traversal()
> ==>graphtraversalsource[tinkergraph[vertices:6 edges:6], standard]
> gremlin> g.V().hasId(within(1..6)).out().hasId(within(1..6))
> gremlin> g.V().hasId(within(1..6)).out().hasId(between(0,7))
> ==>v[3]
> ==>v[2]
> ==>v[4]
> ==>v[5]
> ==>v[3]
> ==>v[3]
> {noformat}
> `P.within` and `P.without` rely on `Collection.contains()` which ends up 
> doing an Object type comparison during the equality check.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Created] (TINKERPOP-1667) Inconsistent number comparison with Contains

2017-04-10 Thread Jason Plurad (JIRA)
Jason Plurad created TINKERPOP-1667:
---

 Summary: Inconsistent number comparison with Contains
 Key: TINKERPOP-1667
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1667
 Project: TinkerPop
  Issue Type: Bug
  Components: process
Affects Versions: 3.2.4
Reporter: Jason Plurad
Priority: Minor


Steps to recreate:

{noformat}
gremlin> conf = new BaseConfiguration()
==>org.apache.commons.configuration.BaseConfiguration@7048535f
gremlin> conf.setProperty("gremlin.tinkergraph.vertexIdManager","LONG")
==>null
gremlin> conf.setProperty("gremlin.tinkergraph.edgeIdManager","LONG")
==>null
gremlin> graph = TinkerGraph.open(conf)
==>tinkergraph[vertices:0 edges:0]
gremlin> graph.io(gryo()).readGraph('data/tinkerpop-modern.kryo')
==>null
gremlin> g = graph.traversal()
==>graphtraversalsource[tinkergraph[vertices:6 edges:6], standard]
gremlin> g.V().hasId(within(1..6)).out().hasId(within(1..6))
gremlin> g.V().hasId(within(1..6)).out().hasId(between(0,7))
==>v[3]
==>v[2]
==>v[4]
==>v[5]
==>v[3]
==>v[3]
{noformat}

`P.within` and `P.without` rely on `Collection.contains()` which ends up doing 
an Object type comparison during the equality check.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (TINKERPOP-1648) XyzPropertyChangedEvent contains incomplete data

2017-03-14 Thread Jason Plurad (JIRA)

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

Jason Plurad commented on TINKERPOP-1648:
-

Possible duplicate of https://issues.apache.org/jira/browse/TINKERPOP-1504

> XyzPropertyChangedEvent contains incomplete data
> 
>
> Key: TINKERPOP-1648
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1648
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process
>Affects Versions: 3.1.6
>Reporter: Daniel Kuppitz
>Assignee: stephen mallette
>
> https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/AddPropertyStep.java#L94-L98
> In case of new properties we create events that only contain the new value of 
> the new property, but the actual property key is unknown.
> Haven't set the affected version(s) as it affects all versions.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Updated] (TINKERPOP-1646) Support custom implementor specific graph management

2017-03-06 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1646:

Affects Version/s: 3.2.4

> Support custom implementor specific graph management
> 
>
> Key: TINKERPOP-1646
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1646
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.2.4
>Reporter: David Pitera
>Priority: Minor
>
> Currently, Tinkerpop uses the 
> [GraphManager](https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/GraphManager.java)
>  to handle graph management and store references to graph objects. 
> I propose that we change the graphManager in three ways:
> 1. The graphManager should be an interface, and the existing functionality 
> should be moved into a `BasicGraphManager`. This allows the implementor to 
> design their own graph manager so suit their graph management needs. 
> Furthermore, this will need to allow the implementor to supply a graphManager 
> class in the YAML, so the 
> [Settings](https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/Settings.java)
>  must support a graphmanager, which of course can default to the 
> `BasicGraphManager`.
> 2. We define additional methods on the interface to improve how the 
> graphManager is used in existing Tinkerpop code. For example, 
> [here](https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/util/ServerGremlinExecutor.java#L156)
>  we mutate an object directly outside the scope of a getter or setter on said 
> object. 
> 3. We define additional methods on the interface to allow for custom graph 
> opening implementation:
> ```
> /**
>  * Implementation that allows for custom graph-opening implementations.
>  */
> public Graph openGraph(String graphName, Supplier supplier);
> ```



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Updated] (TINKERPOP-1646) Support custom implementor specific graph management

2017-03-06 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1646:

Issue Type: Improvement  (was: New Feature)

> Support custom implementor specific graph management
> 
>
> Key: TINKERPOP-1646
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1646
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.2.4
>Reporter: David Pitera
>Priority: Minor
>
> Currently, Tinkerpop uses the 
> [GraphManager](https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/GraphManager.java)
>  to handle graph management and store references to graph objects. 
> I propose that we change the graphManager in three ways:
> 1. The graphManager should be an interface, and the existing functionality 
> should be moved into a `BasicGraphManager`. This allows the implementor to 
> design their own graph manager so suit their graph management needs. 
> Furthermore, this will need to allow the implementor to supply a graphManager 
> class in the YAML, so the 
> [Settings](https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/Settings.java)
>  must support a graphmanager, which of course can default to the 
> `BasicGraphManager`.
> 2. We define additional methods on the interface to improve how the 
> graphManager is used in existing Tinkerpop code. For example, 
> [here](https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/util/ServerGremlinExecutor.java#L156)
>  we mutate an object directly outside the scope of a getter or setter on said 
> object. 
> 3. We define additional methods on the interface to allow for custom graph 
> opening implementation:
> ```
> /**
>  * Implementation that allows for custom graph-opening implementations.
>  */
> public Graph openGraph(String graphName, Supplier supplier);
> ```



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Updated] (TINKERPOP-1646) Support custom implementor specific graph management

2017-03-06 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1646:

Priority: Major  (was: Minor)

> Support custom implementor specific graph management
> 
>
> Key: TINKERPOP-1646
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1646
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.2.4
>Reporter: David Pitera
>
> Currently, Tinkerpop uses the 
> [GraphManager](https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/GraphManager.java)
>  to handle graph management and store references to graph objects. 
> I propose that we change the graphManager in three ways:
> 1. The graphManager should be an interface, and the existing functionality 
> should be moved into a `BasicGraphManager`. This allows the implementor to 
> design their own graph manager so suit their graph management needs. 
> Furthermore, this will need to allow the implementor to supply a graphManager 
> class in the YAML, so the 
> [Settings](https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/Settings.java)
>  must support a graphmanager, which of course can default to the 
> `BasicGraphManager`.
> 2. We define additional methods on the interface to improve how the 
> graphManager is used in existing Tinkerpop code. For example, 
> [here](https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/util/ServerGremlinExecutor.java#L156)
>  we mutate an object directly outside the scope of a getter or setter on said 
> object. 
> 3. We define additional methods on the interface to allow for custom graph 
> opening implementation:
> ```
> /**
>  * Implementation that allows for custom graph-opening implementations.
>  */
> public Graph openGraph(String graphName, Supplier supplier);
> ```



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Updated] (TINKERPOP-1646) Support custom implementor specific graph management

2017-03-06 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1646:

Component/s: server

> Support custom implementor specific graph management
> 
>
> Key: TINKERPOP-1646
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1646
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: server
>Affects Versions: 3.2.4
>Reporter: David Pitera
>Priority: Minor
>
> Currently, Tinkerpop uses the 
> [GraphManager](https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/GraphManager.java)
>  to handle graph management and store references to graph objects. 
> I propose that we change the graphManager in three ways:
> 1. The graphManager should be an interface, and the existing functionality 
> should be moved into a `BasicGraphManager`. This allows the implementor to 
> design their own graph manager so suit their graph management needs. 
> Furthermore, this will need to allow the implementor to supply a graphManager 
> class in the YAML, so the 
> [Settings](https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/Settings.java)
>  must support a graphmanager, which of course can default to the 
> `BasicGraphManager`.
> 2. We define additional methods on the interface to improve how the 
> graphManager is used in existing Tinkerpop code. For example, 
> [here](https://github.com/apache/tinkerpop/blob/master/gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/util/ServerGremlinExecutor.java#L156)
>  we mutate an object directly outside the scope of a getter or setter on said 
> object. 
> 3. We define additional methods on the interface to allow for custom graph 
> opening implementation:
> ```
> /**
>  * Implementation that allows for custom graph-opening implementations.
>  */
> public Graph openGraph(String graphName, Supplier supplier);
> ```



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Closed] (TINKERPOP-1362) nest steps in reference doc

2017-03-01 Thread Jason Plurad (JIRA)

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

Jason Plurad closed TINKERPOP-1362.
---
Resolution: Won't Fix

> nest steps in reference doc
> ---
>
> Key: TINKERPOP-1362
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1362
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 3.2.0-incubating, 3.1.2-incubating
>Reporter: Jason Plurad
>Priority: Trivial
>  Labels: easyfix, newbie
>
> A colleague recommended that the reference doc should include links on the 
> left to some of the steps that are hidden underneath other steps. For example:
> * Lambda Steps
> ** Map
> ** FlatMap
> ** Filter
> ** SideEffect
> ** Branch
> * Has Step
> ** Has
> ** HasLabel
> ** HasId
> ** HasKey
> ** HasValue
> ** HasNot



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (TINKERPOP-1362) nest steps in reference doc

2017-03-01 Thread Jason Plurad (JIRA)

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

Jason Plurad commented on TINKERPOP-1362:
-

I see what you're saying about the headers. I think the idea is good, but in 
practice it'll end up making the TOC too fine-grained. It is already very long 
as it is.

> nest steps in reference doc
> ---
>
> Key: TINKERPOP-1362
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1362
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 3.2.0-incubating, 3.1.2-incubating
>Reporter: Jason Plurad
>Priority: Trivial
>  Labels: easyfix, newbie
>
> A colleague recommended that the reference doc should include links on the 
> left to some of the steps that are hidden underneath other steps. For example:
> * Lambda Steps
> ** Map
> ** FlatMap
> ** Filter
> ** SideEffect
> ** Branch
> * Has Step
> ** Has
> ** HasLabel
> ** HasId
> ** HasKey
> ** HasValue
> ** HasNot



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Assigned] (TINKERPOP-1340) docs do not state at what version an API was introduced (or deprecated)

2017-03-01 Thread Jason Plurad (JIRA)

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

Jason Plurad reassigned TINKERPOP-1340:
---

Assignee: stephen mallette

> docs do not state at what version an API was introduced (or deprecated)
> ---
>
> Key: TINKERPOP-1340
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1340
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 3.2.0-incubating, 3.1.2-incubating
>Reporter: Jason Plurad
>Assignee: stephen mallette
>Priority: Trivial
> Fix For: 3.2.5
>
>
> An all-to-common problem for new developers coming to the TinkerPop is 
> figuring out which APIs work against the graph they are using. Since 
> TinkerPop is ahead of many graph implementation out there (Titan/TP 3.0.1, 
> Stardog/TP 3.0.2, Blazegraph/TP 3.1.0 -- kudos to Sqlg and Unipop for TP 
> 3.2.0!), it's really important that developers know which APIs in the current 
> docs are valid. Ideally the developers would go directly to that version of 
> the TP docs, but that doesn't always happen thanks to Google.
> I propose doc updates on each Graph Traversal Step in the [reference 
> docs|http://tinkerpop.apache.org/docs/current/reference/#graph-traversal-steps]
>  and in the javadocs (on 
> [__.java|http://tinkerpop.apache.org/javadocs/current/full/org/apache/tinkerpop/gremlin/process/traversal/dsl/graph/__.html]
>  and on specific step implementations, i.e. 
> [AddEdgeStep.java|http://tinkerpop.apache.org/javadocs/current/full/org/apache/tinkerpop/gremlin/process/traversal/step/map/AddEdgeStep.html]).
> It should state the specific version (3.y.z) the API was introduced or 
> deprecated.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Closed] (TINKERPOP-1340) docs do not state at what version an API was introduced (or deprecated)

2017-03-01 Thread Jason Plurad (JIRA)

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

Jason Plurad closed TINKERPOP-1340.
---
Resolution: Fixed

> docs do not state at what version an API was introduced (or deprecated)
> ---
>
> Key: TINKERPOP-1340
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1340
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 3.2.0-incubating, 3.1.2-incubating
>Reporter: Jason Plurad
>Assignee: stephen mallette
>Priority: Trivial
> Fix For: 3.2.5
>
>
> An all-to-common problem for new developers coming to the TinkerPop is 
> figuring out which APIs work against the graph they are using. Since 
> TinkerPop is ahead of many graph implementation out there (Titan/TP 3.0.1, 
> Stardog/TP 3.0.2, Blazegraph/TP 3.1.0 -- kudos to Sqlg and Unipop for TP 
> 3.2.0!), it's really important that developers know which APIs in the current 
> docs are valid. Ideally the developers would go directly to that version of 
> the TP docs, but that doesn't always happen thanks to Google.
> I propose doc updates on each Graph Traversal Step in the [reference 
> docs|http://tinkerpop.apache.org/docs/current/reference/#graph-traversal-steps]
>  and in the javadocs (on 
> [__.java|http://tinkerpop.apache.org/javadocs/current/full/org/apache/tinkerpop/gremlin/process/traversal/dsl/graph/__.html]
>  and on specific step implementations, i.e. 
> [AddEdgeStep.java|http://tinkerpop.apache.org/javadocs/current/full/org/apache/tinkerpop/gremlin/process/traversal/step/map/AddEdgeStep.html]).
> It should state the specific version (3.y.z) the API was introduced or 
> deprecated.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (TINKERPOP-1340) docs do not state at what version an API was introduced (or deprecated)

2017-03-01 Thread Jason Plurad (JIRA)

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

Jason Plurad commented on TINKERPOP-1340:
-

Yes, this looks good. We'll be able to redirect version questions on the 
mailing list specifically to the GraphTraversal javadoc. Thanks for handling 
this.

> docs do not state at what version an API was introduced (or deprecated)
> ---
>
> Key: TINKERPOP-1340
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1340
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: documentation
>Affects Versions: 3.2.0-incubating, 3.1.2-incubating
>Reporter: Jason Plurad
>Priority: Trivial
> Fix For: 3.2.5
>
>
> An all-to-common problem for new developers coming to the TinkerPop is 
> figuring out which APIs work against the graph they are using. Since 
> TinkerPop is ahead of many graph implementation out there (Titan/TP 3.0.1, 
> Stardog/TP 3.0.2, Blazegraph/TP 3.1.0 -- kudos to Sqlg and Unipop for TP 
> 3.2.0!), it's really important that developers know which APIs in the current 
> docs are valid. Ideally the developers would go directly to that version of 
> the TP docs, but that doesn't always happen thanks to Google.
> I propose doc updates on each Graph Traversal Step in the [reference 
> docs|http://tinkerpop.apache.org/docs/current/reference/#graph-traversal-steps]
>  and in the javadocs (on 
> [__.java|http://tinkerpop.apache.org/javadocs/current/full/org/apache/tinkerpop/gremlin/process/traversal/dsl/graph/__.html]
>  and on specific step implementations, i.e. 
> [AddEdgeStep.java|http://tinkerpop.apache.org/javadocs/current/full/org/apache/tinkerpop/gremlin/process/traversal/step/map/AddEdgeStep.html]).
> It should state the specific version (3.y.z) the API was introduced or 
> deprecated.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[TinkerPop] TinkerPop 3.2.4 Released: Nine Inch Gremlins

2017-02-14 Thread Jason Plurad
Hello,

TinkerPop 3.2.4 has just been released. We fixed a number of bugs for this
release, provided more documentation, and rolled out GremlinScriptEngine to
better support infrastructure reuse across different Gremlin Language
Variants.

The release artifacts can be found at this location:

https://www.apache.org/dyn/closer.lua/tinkerpop/3.2.4/apache-tinkerpop-gremlin-console-3.2.4-bin.zip
https://www.apache.org/dyn/closer.lua/tinkerpop/3.2.4/apache-tinkerpop-gremlin-server-3.2.4-bin.zip

The online docs can be found here:

http://tinkerpop.apache.org/docs/3.2.4/reference/ (user docs)
http://tinkerpop.apache.org/docs/3.2.4/upgrade/#_tinkerpop_3_2_4 (upgrade
docs)
http://tinkerpop.apache.org/javadocs/3.2.4/core/ (core javadoc)
http://tinkerpop.apache.org/javadocs/3.2.4/full/ (full javadoc)
http://tinkerpop.apache.org/docs/3.2.4/recipes/ (recipes for connected
components and duplicate vertex & edge detection) [NEW!]

The release notes are available here:

https://github.com/apache/tinkerpop/blob/3.2.4/CHANGELOG.asciidoc#tinkerpop-324-release-date-february-8-2017

The Central Maven repo has sync'd as well:

https://repo1.maven.org/maven2/org/apache/tinkerpop/tinkerpop/3.2.4/

Python artifacts have also been deployed to pypi:

https://pypi.python.org/pypi/gremlinpython/3.2.4


​


3.2.x code freeze lifted

2017-02-14 Thread Jason Plurad
The code freeze is lifted on the 3.2.x line and the 3.2.5-SNAPSHOT has been
published.

-- Jason


[RESULT][VOTE] TinkerPop 3.2.4 Release

2017-02-14 Thread Jason Plurad
This vote is now closed with a total of 4 +1s, no +0s and no -1s. The
results are:

BINDING VOTES:

+1  (4 -- Daniel Kuppitz, Stephen Mallette, Jason Plurad, Ted Wilmes)
0   (0)
-1  (0)

NON-BINDING VOTES:

+1 (4 -- David Brown, Robert Dale, Paul Jackson, Pieter Martin)
0  (0)
-1 (0)

Thank you very much,
Jason Plurad


Re: [VOTE] TinkerPop 3.1.6 Release

2017-02-08 Thread Jason Plurad
Ran bin/validate-distribution.sh 3.1.6 OK
Built successfully from source zip
Tested both binary zips
Reviewed the docs on the website and in the binary zips

VOTE: +1

On Wed, Feb 8, 2017 at 9:05 AM, Stephen Mallette 
wrote:

> Thanks Ted. Jason is still working on the 3.2.4 release. I think it would
> be good to wait until we get the voted on that before we announce 3.1.6 as
> out the door so that they go out together.
>
> On Wed, Feb 8, 2017 at 12:27 AM, Ted Wilmes  wrote:
>
> > Thanks for taking the time to vote.  If there aren't any objections by
> EOD
> > tomorrow, I'll call it and complete the release.
> >
> > Thanks,
> > Ted
> >
> > On Mon, Feb 6, 2017 at 1:52 PM, Dylan Millikin  >
> > wrote:
> >
> > > I ran the php driver tests against this locally and everything is good.
> > >
> > > VOTE +1 from me!!
> > >
> > > Cheers.
> > >
> > > On Mon, Feb 6, 2017 at 7:48 AM, Stephen Mallette  >
> > > wrote:
> > >
> > > > I can't remember the error I got, but I think it makes sense to test
> > > > validate-distribution.sh from the branch that the release was
> generated
> > > > from. I probably shouldn't have tried to test from master - that
> sorta
> > > > doesn't make sense.
> > > >
> > > > On Sun, Feb 5, 2017 at 7:58 AM, Daniel Kuppitz 
> > wrote:
> > > >
> > > > > >
> > > > > > Note that I couldn't use the validate-distribution.sh on master
> for
> > > > some
> > > > > reason
> > > > > > - it failed
> > > > >
> > > > >
> > > > > Did it fail when it tested gremlin.sh -e..? This test was added
> > > recently
> > > > > and I think that's expected to fail in 3.1.x. All others should
> > succeed
> > > > > though, but in general we should always use the validation script
> > from
> > > > the
> > > > > respective development branches (tp31/, tp32/  and master/).
> > > > >
> > > > > Validation worked for me too, hence ...
> > > > >
> > > > > VOTE: +1
> > > > >
> > > > > Cheers,
> > > > > Daniel
> > > > >
> > > > >
> > > > > On Sat, Feb 4, 2017 at 2:20 AM, Stephen Mallette <
> > spmalle...@gmail.com
> > > >
> > > > > wrote:
> > > > >
> > > > > > Hey Ted - You're a little early as we agreed a while back to let
> > two
> > > > > > weekends pass from code freeze to VOTE day (maybe release docs
> need
> > > to
> > > > be
> > > > > > updated to reflect that). Not a big deal imo as I doubt there are
> > > many
> > > > > > folks still testing on 3.1.6 at this point, so I'm fine to VOTE
> > from
> > > > here
> > > > > > if everyone else is. If anyone feels like we need to extend the
> > > VOTEing
> > > > > > time, just yell.  3.2.4 voting definitely shouldn't start until
> > > Monday.
> > > > > >
> > > > > > In the mean time, as it is right now:
> > > > > >
> > > > > > $ bin/validate-distribution.sh 3.1.6
> > > > > >
> > > > > > Validating binary distributions
> > > > > >
> > > > > > * downloading Apache TinkerPop Gremlin
> > > > > > (apache-tinkerpop-gremlin-console-3.1.6-bin.zip)... OK
> > > > > > * validating signatures and checksums ...
> > > > > >   * PGP signature ... OK
> > > > > >   * MD5 checksum ... OK
> > > > > >   * SHA1 checksum ... OK
> > > > > > * unzipping Apache TinkerPop Gremlin ... OK
> > > > > > * validating Apache TinkerPop Gremlin's docs ... OK
> > > > > > * validating Apache TinkerPop Gremlin's binaries ... OK
> > > > > > * validating Apache TinkerPop Gremlin's legal files ...
> > > > > >   * LICENSE ... OK
> > > > > >   * NOTICE ... OK
> > > > > > * validating Apache TinkerPop Gremlin's plugin directory ... OK
> > > > > > * validating Apache TinkerPop Gremlin's lib directory ... OK
> > > > > > * testing script evaluation ... OK
> > > > > >
> > > > > > * downloading Apache TinkerPop Gremlin
> > > > > > (apache-tinkerpop-gremlin-server-3.1.6-bin.zip)... OK
> > > > > > * validating signatures and checksums ...
> > > > > >   * PGP signature ... OK
> > > > > >   * MD5 checksum ... OK
> > > > > >   * SHA1 checksum ... OK
> > > > > > * unzipping Apache TinkerPop Gremlin ... OK
> > > > > > * validating Apache TinkerPop Gremlin's docs ... OK
> > > > > > * validating Apache TinkerPop Gremlin's binaries ... OK
> > > > > > * validating Apache TinkerPop Gremlin's legal files ...
> > > > > >   * LICENSE ... OK
> > > > > >   * NOTICE ... OK
> > > > > > * validating Apache TinkerPop Gremlin's plugin directory ... OK
> > > > > > * validating Apache TinkerPop Gremlin's lib directory ... OK
> > > > > >
> > > > > > Validating source distribution
> > > > > >
> > > > > > * downloading Apache TinkerPop 3.1.6 (apache-tinkerpop-3.1.6-src.
> > > > zip)...
> > > > > > OK
> > > > > > * validating signatures and checksums ...
> > > > > >   * PGP signature ... OK
> > > > > >   * MD5 checksum ... OK
> > > > > >   * SHA1 checksum ... OK
> > > > > > * unzipping Apache TinkerPop 3.1.6 ... OK
> > > > > > * building project ... OK
> > > > > >
> > > > > > Note that I couldn't use the validate-distribution.sh on master
> for
> 

[VOTE] TinkerPop 3.2.4 Release

2017-02-08 Thread Jason Plurad
Hello,

We are happy to announce that TinkerPop 3.2.4 is ready for release.

The release artifacts can be found at this location:
https://dist.apache.org/repos/dist/dev/tinkerpop/3.2.4/

The source distribution is provided by:
apache-tinkerpop-3.2.4-src.zip

Two binary distributions are provided for user convenience:
apache-tinkerpop-gremlin-console-3.2.4-bin.zip
apache-tinkerpop-gremlin-server-3.2.4-bin.zip

The GPG key used to sign the release artifacts is available at:
https://dist.apache.org/repos/dist/dev/tinkerpop/KEYS

The online docs can be found here:
http://tinkerpop.apache.org/docs/3.2.4/ (user docs)
http://tinkerpop.apache.org/docs/3.2.4/upgrade/ (upgrade docs)
http://tinkerpop.apache.org/javadocs/3.2.4/core/ (core javadoc)
http://tinkerpop.apache.org/javadocs/3.2.4/full/ (full javadoc)

The tag in Apache Git can be found here:

https://git-wip-us.apache.org/repos/asf?p=tinkerpop.git;a=tag;h=refs/tags/3.2.4

The release notes are available here:

https://github.com/apache/tinkerpop/blob/3.2.4/CHANGELOG.asciidoc#release-3-2-4

The [VOTE] will be open for the next 72 hours --- closing Saturday
(February 11, 2017) at 10:00 AM EST.

My vote is +1.

Thank you very much,
Jason Plurad


Re: Code Freeze 3.2.4

2017-02-02 Thread Jason Plurad
I've published the 3.2.4 SNAPSHOT out to Sonatype. Also note, that I've
added my key to the KEYS file, so you those of you that want to validate
the distribution will need to import it again.

https://lists.apache.org/thread.html/909821a85693eaee31b4bf89f9881b
d619ca2ff5db677f79c1bbbec8@%3Cdev.tinkerpop.apache.org%3E


On Thu, Feb 2, 2017 at 9:51 AM, Stephen Mallette <spmalle...@gmail.com>
wrote:

> With the vote from kuppitz 548 is done. I just merged it.
>
> On Thu, Feb 2, 2017 at 7:57 AM, Stephen Mallette <spmalle...@gmail.com>
> wrote:
>
> > We still need one more vote to get https://github.com/apache/
> > tinkerpop/pull/548 - any committers free to handle that so we can get
> > that merged?
> >
> > On Mon, Jan 30, 2017 at 11:08 AM, Jason Plurad <plur...@gmail.com>
> wrote:
> >
> >> We are now under code freeze for 3.2.4.
> >>
> >> There are still 2 outstanding pull requests that should be included for
> >> 3.2.4.
> >> https://github.com/apache/tinkerpop/pull/548
> >> https://github.com/apache/tinkerpop/pull/551
> >>
> >> If there are others currently in the queue that are critical for
> release,
> >> please let us know. As mentioned last week, it has been several months
> >> since we've had a maintenance release and in particular, there is a
> Groovy
> >> bump for a security fix that we need to make available.
> >>
> >> I'll start getting SNAPSHOT artifact and docs published. I'll update
> this
> >> thread again when they are available.
> >>
> >> -- Jason
> >>
> >
> >
>


Code Freeze 3.2.4

2017-01-30 Thread Jason Plurad
We are now under code freeze for 3.2.4.

There are still 2 outstanding pull requests that should be included for
3.2.4.
https://github.com/apache/tinkerpop/pull/548
https://github.com/apache/tinkerpop/pull/551

If there are others currently in the queue that are critical for release,
please let us know. As mentioned last week, it has been several months
since we've had a maintenance release and in particular, there is a Groovy
bump for a security fix that we need to make available.

I'll start getting SNAPSHOT artifact and docs published. I'll update this
thread again when they are available.

-- Jason


[jira] [Updated] (TINKERPOP-1608) TP2-to-TP3 GraphML XSLT

2017-01-20 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1608:

Description: 
There were some changes in the GraphML format between TinkerPop 2 and TinkerPop 
3. [~joshsh] has [started work|https://github.com/apache/tinkerpop/pull/501] on 
an XSLT to transform between the two formats.

I performed an initial test of the transform using this code within the Gremlin 
Console.

{noformat}
import javax.xml.parsers.DocumentBuilderFactory
import javax.xml.transform.TransformerFactory
import javax.xml.transform.dom.DOMSource
import javax.xml.transform.stream.StreamSource
import javax.xml.transform.stream.StreamResult

stylesheet = new File('/tmp/tp-graphml-convert.xslt')
datafile = new File('/tmp/tp2-graphml.xml')
outfile = new File('/tmp/tp3-graphml.xml')

tFactory = TransformerFactory.newInstance()
stylesource = new StreamSource(stylesheet)
transformer = tFactory.newTransformer(stylesource)

source = new StreamSource(datafile)
result = new StreamResult(new FileWriter(outfile))
transformer.transform(source, result)
{noformat}

Example TinkerPop 2 GraphML: 
https://github.com/tinkerpop/gremlin/blob/master/data/graph-example-1.xml

Example TinkerPop 3 GraphML: 
https://github.com/apache/tinkerpop/blob/3.1.5/data/tinkerpop-classic.xml

  was:
There were some changes in the GraphML format between TinkerPop 2 and TinkerPop 
3. [~joshsh] has [started work|https://github.com/apache/tinkerpop/pull/501] on 
an XSLT to transform between the two formats.

I performed an initial test of the transform using this code within the Gremlin 
Console.

{noformat}
import javax.xml.parsers.DocumentBuilderFactory
import javax.xml.transform.TransformerFactory
import javax.xml.transform.dom.DOMSource
import javax.xml.transform.stream.StreamSource
import javax.xml.transform.stream.StreamResult
stylesheet = new File('/tmp/tp-graphml-convert.xslt')
datafile = new File('/tmp/tp2-graphml.xml')
outfile = new File('/tmp/tp3-graphml.xml')
factory = DocumentBuilderFactory.newInstance()
builder = factory.newDocumentBuilder()
document = builder.parse(datafile)
tFactory = TransformerFactory.newInstance()
stylesource = new StreamSource(stylesheet)
transformer = tFactory.newTransformer(stylesource)
source = new DOMSource(document)
result = new StreamResult(new FileWriter(outfile))
transformer.transform(source, result)
{noformat}

There are still some remaining changes required for compatibility.

These keys need to be defined:

{noformat}


{noformat}

A vertex label needs to be inserted on all nodes. Vertex labels are new in TP3. 
Not sure if this is required by the TP3 parser:

{noformat}
vertex
{noformat}

The edge label needs to move from the edge tag to a nested child tag:

{noformat}
created
{noformat}

Example TinkerPop 2 GraphML: 
https://github.com/tinkerpop/gremlin/blob/master/data/graph-example-1.xml

Example TinkerPop 3 GraphML: 
https://github.com/apache/tinkerpop/blob/3.1.5/data/tinkerpop-classic.xml


> TP2-to-TP3 GraphML XSLT
> ---
>
> Key: TINKERPOP-1608
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1608
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: io
>Affects Versions: 3.1.5, 3.2.3
>Reporter: Jason Plurad
>
> There were some changes in the GraphML format between TinkerPop 2 and 
> TinkerPop 3. [~joshsh] has [started 
> work|https://github.com/apache/tinkerpop/pull/501] on an XSLT to transform 
> between the two formats.
> I performed an initial test of the transform using this code within the 
> Gremlin Console.
> {noformat}
> import javax.xml.parsers.DocumentBuilderFactory
> import javax.xml.transform.TransformerFactory
> import javax.xml.transform.dom.DOMSource
> import javax.xml.transform.stream.StreamSource
> import javax.xml.transform.stream.StreamResult
> stylesheet = new File('/tmp/tp-graphml-convert.xslt')
> datafile = new File('/tmp/tp2-graphml.xml')
> outfile = new File('/tmp/tp3-graphml.xml')
> tFactory = TransformerFactory.newInstance()
> stylesource = new StreamSource(stylesheet)
> transformer = tFactory.newTransformer(stylesource)
> source = new StreamSource(datafile)
> result = new StreamResult(new FileWriter(outfile))
> transformer.transform(source, result)
> {noformat}
> Example TinkerPop 2 GraphML: 
> https://github.com/tinkerpop/gremlin/blob/master/data/graph-example-1.xml
> Example TinkerPop 3 GraphML: 
> https://github.com/apache/tinkerpop/blob/3.1.5/data/tinkerpop-classic.xml



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


[jira] [Created] (TINKERPOP-1608) TP2-to-TP3 GraphML XSLT

2017-01-20 Thread Jason Plurad (JIRA)
Jason Plurad created TINKERPOP-1608:
---

 Summary: TP2-to-TP3 GraphML XSLT
 Key: TINKERPOP-1608
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1608
 Project: TinkerPop
  Issue Type: Improvement
  Components: io
Affects Versions: 3.2.3, 3.1.5
Reporter: Jason Plurad






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


JanusGraph project at The Linux Foundation

2017-01-12 Thread Jason Plurad
Today, The Linux Foundation announced
 the
creation of the JanusGraph  project. We're excited
to push forward the open source, collaborative effort on scalable graph
databases that was initiated by the Aurelius team with Titan. JanusGraph
will continue to be a native Apache TinkerPop implementation. The first
effort definitely includes finally upgrading beyond 3.0.1-incubating :)

There are janusgraph-users
 and
janusgraph-dev 
Google Groups created for public mailing list collaboration, but as is the
case with other providers in the TinkerPop ecosystem, I'd expect
cross-traffic to continue with the TinkerPop lists.

If you will be in Austin this weekend for Graph Day Texas
, there is a great lineup of graph-related talks. Ted
Wilmes and I from Apache TinkerPop will be there, and so will others
involved in getting JanusGraph established. After the Graph Day happy hour,
we will gather up for an informal meetup/birds-of-a-feather with anybody
interested.


Have a good one,
Jason


Re: Open graph effort at the Linux Foundation

2016-12-16 Thread Jason Plurad
As a followup to my post on December 8th, I wanted to share that we had a 
kickoff meeting this morning with the Linux Foundation to review the 
charter for our community graph project.

We are forking the Titan code base. The name "Titan" cannot be used, so a 
new name has been chosen.

We will continue and coordinate the efforts already underway for the past 
year by organizations around the world. We will post a plan with more 
specific details of features and future development plans, including 
participation guidelines, in January.

Happy Holidays!

-- Jason

On Friday, December 16, 2016 at 12:39:35 PM UTC-5, Austin Sharp wrote:
>
> This is my question as well. Is this new project starting over from 
> scratch, or will it be something evolutionary based on the Titan codebase?
>
> On Tuesday, December 13, 2016 at 6:14:10 AM UTC-8, Tom Ellis wrote:
>>
>> Does this therefore spell the death of Titan? There will be no fork?
>>
>> On Thursday, 8 December 2016 19:24:33 UTC, Jason Plurad wrote:
>>>
>>> Many folks in the Titan community have continued to reach out wondering 
>>> how to continue development on an Apache-licensed, open source, and 
>>> scalable graph database with pluggable backends. I want to let you know 
>>> that the Linux Foundation is establishing an open community graph project, 
>>> including developers from various backend providers, to fulfill that need. 
>>> The logistics for this new home are being finalized, and it will carry on 
>>> the open source heritage of Titan with open governance. The Apache license 
>>> will be maintained, and the community will operate along the same 
>>> principles of an Apache project. Once naming the new project is complete, 
>>> all are welcome to join, contribute, and drive forward this scalable graph 
>>> solution.
>>>
>>> -- Jason
>>>
>>

Open graph effort at the Linux Foundation

2016-12-08 Thread Jason Plurad
Many folks in the Titan community have continued to reach out wondering how
to continue development on an Apache-licensed, open source, and scalable
graph database with pluggable backends. I want to let you know that the
Linux Foundation is establishing an open community graph project, including
developers from various backend providers, to fulfill that need. The
logistics for this new home are being finalized, and it will carry on the
open source heritage of Titan with open governance. The Apache license will
be maintained, and the community will operate along the same principles of
an Apache project. Once naming the new project is complete, all are welcome
to join, contribute, and drive forward this scalable graph solution.

-- Jason


Re: [TinkerPop] gremlin-x

2016-12-01 Thread Jason Plurad
> + I'll come back to this in the points below, but as a big fat general
question: how will we deal with transactions?

Bullseye. I'd add that assuming that TinkerGraph would be the "one true
TinkerPop graph" for gremlin-x testing, it needs to support transactions
which it doesn't do currently. Or introduce a new reference implementation
graph that does.

On Wed, Nov 30, 2016 at 4:58 PM, Stephen Mallette 
wrote:

> Added back "dev"
>
> I think we've tried a lot of things for TinkerPop 3 and that there are too
> many options for application development as well as an over-exposed API, so
> I like this direction, though a fair bit of complexity lies in making it
> happen. Before getting to that complexity, I don't really think of this as
> a "new idea". To me the description of gremlin-x is really just a GLV for
> Java. I mean - consider that gremlin-python (something we classify as a
> GLV) is basically "everything a python dev needs to build applications -
> nothing more/less" and that's pretty much what we want for java in
> gremlin-x. gremlin-x would basically be "everything a java dev needs to
> build applications - nothing more/less". So I'm not seeing much distinction
> there (nor the need for naming it "gremlin-x" really - seems to be as
> simple as "gremlin-java" to be in line with other GLVs we will have).
>
> Back to the "complexity" - and I think the questions in my mind that
> follow apply with or without this gremlin-x idea (they are general issues
> that just are faced by "remote" and GLVs):
>
> + I'll come back to this in the points below, but as a big fat general
> question: how will we deal with transactions?
> + What will be the nature of Graph.io() - in one sense i'd like to see
> that as something hidden as it is the source of a fair bit of confusion
> (i.e. someone trying to load a multi GB graphson file) but on the other
> hand it's convenient for small graphs - that story could be made better.
> Maybe Graph.io() being a part of "Graph" is hidden and only used by the
> test suite to load data into graph providers implementations for testing
> purposes. Of course, we will need our story in order for loading data in
> different formats if that isn't available (which I guess we need to do
> anyway).
> + Can remoting Gremlin bytecode cover everything that users currently do
> in embedded/local mode? The drawback with a remote traversal is that it is
> bound to a single transaction and you therefore lose some efficiency with
> certain graphs databases as logic with three traversals that might span a
> single transaction, for example, will be executed less efficiently as three
> different transaction refreshing a transaction cache each time. Maybe the
> server protocol needs to be session based? Maybe, going back to the first
> bullet i had, the transaction model needs to change so that its not bound
> to a single thread by default?
> + Related to the last point, is there an easy way to blend client code
> with server code? Meaning, if i have a body of complex logic right now, my
> only option is to submit a script. I think the ScriptEngine stuff should
> really just be used for those situations where you need a lambda. That's a
> nice, simple, easy-to-explain rule, whereas "do it if there is complex
> logic" is a bit more subjective (and potentially troublesome). How can GLVs
> get complex logic on the "server" (let's say the logic is written in java
> and available to gremlin server) so that it could be executed as part of
> some bytecode in any GLV?
> + It would be neat to see what kind of "server" changes we could make to
> optimize for bytecode. Right now we sorta tied bytecode execution into the
> same Gremlin Server execution pipeline as ScriptEngine processing. I think
> we could probably do better on that.
>
> Not sure there are any immediate answers to any of the above, but it's
> what I've been thinking about with respect to TinkerPop 3.3.0 lately.
>
>
>
> On Wed, Nov 30, 2016 at 1:11 PM, pieter-gmail 
> wrote:
>
>> Ok, if Sqlg just implements its own RemoteConnection and no need for an
>> additional server in the architecture then I am a lot less nervous about
>> gremlin-x.
>>
>> As an aside, Sqlg has its own peculiarities and optimization outside the
>> standard TinkerPop interface. Most providers will have such features. If
>> the graph is not exposed directly there will need to be some way to call
>> custom features. No idea how now, but still.
>>
>> E.g. for Sqlg the most used such feature is
>> SqlgGraph.tx().batchMode(batchModeType) but there are others. Index
>> management will be another. Without being able to access these features
>> too many benefits of choosing a particular provider will be lost.
>>
>> Cheers
>> Pieter
>>
>> On 30/11/2016 19:31, Marko Rodriguez wrote:
>> > Hi,
>> >
>> >> I generally recommend the exact opposite. Neo4j shines when embedded.
>> >
>> > And thats fine. There would simply be a 

Re: [DISCUSS] Add HGraphDB to graph system listing

2016-11-11 Thread Jason Plurad
+1 on the HGraphDB index listing.

I think another minor concern is how to handle projects that have gone
dormant, like ts-tinkerpop or scalajs-gremlin-client for example.
Understanding how versions line up between TinkerPop and whatever database
or library they want to use has been a pain point for many users.

My emphasis below:

The project must be *actively* developed/maintained to a current or
previous "y" version of Apache TinkerPop (3.y.z).

-- Jason

On Thu, Nov 10, 2016 at 3:21 PM, Stephen Mallette 
wrote:

> I sorta share that sentiment however, I think we had that discussion back
> when we determined our listing requirements and we figured we couldn't
> figure out how to easily quantify "mature".
>
> On Thu, Nov 10, 2016 at 3:17 PM, Jean-Baptiste Musso 
> wrote:
>
> > Interesting thing. +1 to adding HGraphDB to the list.
> >
> > Minor concern: I'm not sure if we want to manage a curated list of
> > implementations and maybe restrict to more mature projects. I'm just
> > thinking out loud here: I'm not targeting HGraphDB specifically (and I
> wish
> > them good luck - *cheers*). My main concern is that the project is very
> > young, but graph databases need as much publicity as they can.
> >
> > Jean-Baptiste
> >
> > On Thu, Nov 10, 2016 at 8:58 PM, Ted Wilmes  wrote:
> >
> > > Hello everyone,
> > > Robert Yokota notified us today that he has developed an
> > > Apache TinkerPop enabled graph db: https://github.com/rayokota/
> hgraphdb.
> > > Correct me if I'm wrong Robert, but it looks like it's at the TinkerPop
> > > 3.2.3 version.
> > >
> > > I think HGraphDB meets our index listing requirements but I wanted to
> > bring
> > > it up
> > > just in case anyone felt differently.  Unless there are any objections,
> > > I'll assume lazy
> > > consensus in three days (Sunday, November 13, 2016 2:00 CST) and add
> > > HGraphDB
> > > to the graph systems listing.
> > >
> > > Thanks,
> > > Ted
> > >
> >
>


Re: Code Reviews - for committers

2016-11-07 Thread Jason Plurad
For Docker, I was running into the OOME, but then I bumped up the
container/virtualbox memory to 4 GB (default is 2 GB) and it works fine.

-- Jason

On Sun, Nov 6, 2016 at 6:37 PM, Stephen Mallette 
wrote:

> That stinks. Never got OOME with docker. Any other kinds of issues you can
> recall or is it pretty much just that?
>
> On Nov 6, 2016 5:43 PM, "Robert Dale"  wrote:
>
> > One day I'll figure out why docker always fails for me.  It's usually
> > around some java.lang.OutOfMemoryError: unable to create new native
> > thread.  Usually happens in tinkergraph-gremlin. Sometimes it makes it to
> > spark or hadoop, but inevitably fails miserably there.  Tried latest
> master
> > and TINKERPOP-1434 but no joy. Even rebuilt docker containers from
> scratch.
> >
> > Robert Dale
> >
> > On Fri, Oct 28, 2016 at 6:22 PM, Stephen Mallette 
> > wrote:
> >
> > > That's probably fair. Perhaps that's a good rule. At least one person
> > > should use CPU to do the docker build and mention that on the PR as
> part
> > of
> > > their VOTE. I also suppose some PRs won't even need the docker run at
> > all.
> > > We can just use some common sense with that "rule".
> > >
> > > On Fri, Oct 28, 2016 at 5:29 PM, Daniel Kuppitz 
> wrote:
> > >
> > > > Talking about sparing CPU cycles: docker/build.sh -t -i -n takes
> time,
> > > lots
> > > > of time. I used to run it regardless of whether somebody else already
> > did
> > > > it or not. I won't do that anymore and instead rely on idempotent
> > results
> > > > (in the end that's why we originally created the Docker containers).
> > > Hence,
> > > > if anybody already ran the full build, I will only look through the
> > code
> > > > changes and if they look good to me - VOTE: +1.
> > > >
> > > > Cheers,
> > > > Daniel
> > > >
> > > >
> > > > On Fri, Oct 28, 2016 at 9:28 PM, Stephen Mallette <
> > spmalle...@gmail.com>
> > > > wrote:
> > > >
> > > > > We're backing up pretty heavily on pull request reviews/votes.
> > > > >
> > > > > https://github.com/apache/tinkerpop/pulls
> > > > >
> > > > > Can any committers help us along with some reviews/votes?
> > > > >
> > > > > Remember - you don't need to have full knowledge of the code to
> > > > participate
> > > > > in a code review. If the code comes from a core contributor (e.g.
> > marko
> > > > > submitting a gremlin-spark bug fix), you mostly need to focus on
> the
> > > big
> > > > > picture and generalities of the change and then do a "mvn clean
> > > install"
> > > > or
> > > > > better yet "docker.build.sh -t -i -n". If it all works then "VOTE
> > +1".
> > > > >
> > > > > Most pull requests for review are super simple, like this one:
> > > > >
> > > > > https://github.com/apache/tinkerpop/pull/465 (removed deprecated
> > > > classes)
> > > > >
> > > > > All you need to do is check if the classes are gone, if upgrade
> docs
> > > are
> > > > > updated and if the thing builds...done. VOTE +1.
> > > > >
> > > > > Doing reviews is a huge help to the project (even for those who are
> > not
> > > > > committers and don't have binding votes, as it is a great way to
> get
> > > > > visibility in the project and learn more about how it works). It
> > would
> > > be
> > > > > great if folks could make a habit of sparing a few CPU cycles for
> > > > TinkerPop
> > > > > at the end of a work day or while you sleep so that we can continue
> > to
> > > > stay
> > > > > agile in our development.
> > > > >
> > > > > Thanks
> > > > >
> > > >
> > >
> >
>


Re: [TinkerPop] New Committer: Robert Dale

2016-10-27 Thread Jason Plurad
+1!

-- Jason
On Thu, Oct 27, 2016 at 8:17 AM Jean-Baptiste Musso 
wrote:

> Welcome aboard Robert!
>
> On Thursday, 27 October 2016, Stephen Mallette 
> wrote:
>
> The Project Management Committee (PMC) for Apache TinkerPop has asked
> Robert Dale to become a committer and we are pleased to announce that he
> has accepted.
>
> Robert has submitted a number of quality pull requests to TinkerPop
> covering both new features and bug/documentation fixes and has been engaged
> in many different development discussions that have shaped the direction
> TinkerPop has taken. He's also been helpful on the user mailing list,
> providing some nice answers to those who have needed support. All of that
> activity made him a sound choice for committership.
>
> Let this proclamation traverse the entirety of The TinkerPop so that all
> may know to welcome Robert to the TinkerPop Guild!
>
>
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "Gremlin-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to gremlin-users+unsubscr...@googlegroups.com.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/gremlin-users/CAA-H43-DpGHAAxxgb6ib-qXKML-etQnP%2BRP5jfeJm4hRGrt_vg%40mail.gmail.com
> 
> .
> For more options, visit https://groups.google.com/d/optout.
>
>
>
> --
> Jean-Baptiste
>
> --
> You received this message because you are subscribed to the Google Groups
> "Gremlin-users" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to gremlin-users+unsubscr...@googlegroups.com.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/gremlin-users/CAC3-3jirKkyn%3DByYC%3DROdEr2dLJnm-Cb9KC25RpS%2BnByDvn2%2BQ%40mail.gmail.com
> 
> .
> For more options, visit https://groups.google.com/d/optout.
>


Re: [VOTE] TinkerPop 3.2.3 Release

2016-10-20 Thread Jason Plurad
Validate distribution was clean.
Manual tests with Gremlin Console and Gremlin Server were good.
Reviewed the upgrade docs and change logs.

VOTE: +1

On Thu, Oct 20, 2016 at 9:58 AM, Daniel Kuppitz  wrote:

> docker/build.sh -t -i -n -j -d (1x everything) succeeded.
>
> VOTE: +1
>
> On Mon, Oct 17, 2016 at 11:21 PM, Stephen Mallette 
> wrote:
>
> > Hello,
> >
> > We are happy to announce that TinkerPop 3.2.3 is ready for release.
> >
> > The release artifacts can be found at this location:
> > https://dist.apache.org/repos/dist/dev/tinkerpop/3.2.3/
> >
> > The source distribution is provided by:
> > apache-tinkerpop-3.2.3-src.zip
> >
> > Two binary distributions are provided for user convenience:
> > apache-tinkerpop-gremlin-console-3.2.3-bin.zip
> > apache-tinkerpop-gremlin-server-3.2.3-bin.zip
> >
> > The GPG key used to sign the release artifacts is available at:
> > https://dist.apache.org/repos/dist/dev/tinkerpop/KEYS
> >
> > The online docs can be found here:
> > http://tinkerpop.apache.org/docs/3.2.3/ (user docs)
> > http://tinkerpop.apache.org/docs/3.2.3/upgrade/ (upgrade docs)
> > http://tinkerpop.apache.org/javadocs/3.2.3/core/ (core javadoc)
> > http://tinkerpop.apache.org/javadocs/3.2.3/full/ (full javadoc)
> >
> > The tag in Apache Git can be found here:
> >
> > https://git-wip-us.apache.org/repos/asf?p=tinkerpop.git;a=tag;h=
> > 0fdd98d8b657185b766310562926c155427594d6
> >
> > The release notes are available here:
> >
> > https://github.com/apache/tinkerpop/blob/master/
> > CHANGELOG.asciidoc#tinkerpop-323-release-date-october-17-2016
> >
> > The [VOTE] will be open for the next 72 hours --- closing Thursday
> (October
> > 20, 2016) at 5:30pm EST.
> >
> > My vote is +1.
> >
> > Thank you very much,
> > Stephen
> >
>


[jira] [Commented] (TINKERPOP-1493) Groovy project doesn't build on Windows

2016-10-10 Thread Jason Plurad (JIRA)

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

Jason Plurad commented on TINKERPOP-1493:
-

[~pauljackson] Thanks for looking into this. Nice work.

If you're working off master, there were recent changes that moved it into 
integration test (TINKERPOP-1477). You can execute these by running

{noformat}
mvn clean verify -DskipIntegrationTests=false
{noformat}

I tried out your fixes, and they mostly look good. I found that it needed to 
call {{jar.close()}} in {{DependencyGrabber.alterPaths()}} also.

Will you be able to submit a pull request?

> Groovy project doesn't build on Windows
> ---
>
> Key: TINKERPOP-1493
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1493
> Project: TinkerPop
>  Issue Type: Bug
>  Components: groovy
>Affects Versions: 3.2.2
>Reporter: Paul Jackson
>Priority: Minor
>
> Builds on Windows fail for two reasons. First the line to create extTestDir 
> is creating a path consisting of two full paths concatenated together. The 
> second drive letter is seen as an illegal character:
> {code}private static final File extTestDir = new 
> File(System.getProperty("user.dir"), 
> TestHelper.makeTestDataDirectory(DependencyGrabberTest.class));{code}
> Second, when it comes time to delete the directory it is locked. This is 
> because some instances of JarFile are created on it but not closed.



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


Re: [DISCUSS] Test Suite Changes for 3.3.0

2016-09-20 Thread Jason Plurad
Nice discussion thread, Stephen. I've tinkered around minimally with
writing a graph implementation, so hopefully we'll get more feedback from
others. From what I have done, +1 on killing @OptOut test annotations. They
seem out of place on the Graph impl class.

You mentioned "there is at least one method that could be called on
Features that is
typically dynamically driven based on schema" -- which method is that?

-- Jason

On Mon, Sep 19, 2016 at 4:33 PM, Stephen Mallette 
wrote:

> I've spent the middle portion of the day reviewing our test infrastructure
> and related open tickets and have some ideas to make some things better. I
> titled this post for 3.3.0, but, in truth, I'm not sure what must be 3.3.0
> and what might yet be useful and good for 3.2.x. I'm also using this email
> as a way to organize my notes/ideas from the day, so apologies if I'm
> dumping a lot of stuff here to follow.
>
> (1) Of all the things I came up with, I think the biggest breaker is this
> one: have one uber test suite in gremlin-test. In other words, merge
> gremlin-groovy-test to gremlin-test and get rid of that all together. Then.
> stop publishing test artifacts out of hadoop-gremlin (and wherever else we
> might be doing that). We can make groovy and hadoop dependencies optional
> so that if providers aren't using them, they don't have to have them sucked
> in and can just depend on them as needed.
>
> (2) Next biggest breaker - how does everyone feel about killing OptOut and
> OptIn and getting those concepts out of gremlin-core and into features of
> gremlin-test. I've heard at least two Graph providers mention a problem
> where they want to "OptOut" more at the GraphProvider level as opposed to
> the Graph level as their configurations in the GraphProvider do more to
> drive that setting than the Graph does. I don't think we lose anything by
> moving "OptOut" except for the describeGraph() functionality:
>
> http://tinkerpop.apache.org/docs/current/reference/#describe-graph
>
> which I'm not sure is that big a deal to worry about. That was a bit of a
> nice idea that didn't really develop any further than where it is right
> now.
>
> (3) 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).
>
> (4) I think this approach is nice because it spreads into something else
> that I think is important to us: getting maximum value for time out of our
> tests. As we add GLVs and more tests (I think that without integration
> tests right now, we're over 12000 tests), the time it takes to do a basic
> mvn clean install is getting longer and longer. We want that that as short
> as possible while maximizing code coverage. To that end, I'll make several
> points:
>
> + jacoco is now good with java 8 (i think it has been for a while, but i
> hadn't noticed). i worked with it a bit today and we should be able to get
> a good aggregate test coverage report with it (assuming we are ok with
> adding a new "gremlin-coverage" module to maven - stinks, but perhaps isn't
> so different than having added gremlin-benchmarks in some respects). If we
> have that we can find out what combinations of GraphProviders give us the
> best coverage for time and make that our standard testing profile.
> + We can build some fairly exotic GraphProviderSource implementations that
> can help us test all possible configuration options for TinkerGraph or
> cover ranges of settings in Neo4jGraph or randomize the returned
> GraphProviders - these could all be options we execute in docker during
> code freeze week (and perhaps periodically during our dev cycles) to ensure
> we're not breaking anything as a result of running the "maximized"
> configuration of just mvn clean install.
> + If that works, we can eliminate the use or Random in our test suite for a
> standard mvn clean install thus eliminating the chance of some
> non-deterministic behavior. Rather than be "random" we just test all the
> cases.
> + Perhaps we could have different maven profiles that ran different
> GraphProviderSource implementations. I'm thinking that those might be
> triggered from different docker runs to help parallelize the tests and
> allow us to test more permutations more quickly???
>
> (5) Finally, I think we could speed up our 

Re: permission request to use character scene graphics

2016-09-06 Thread Jason Plurad
Ultimately I decided against using the character scene graphic. Blog post
is public here
https://developer.ibm.com/opentech/2016/09/06/summer-monster-graph-databases/
On Thu, Sep 1, 2016 at 8:58 AM Marko Rodriguez <okramma...@gmail.com> wrote:

> Its TinkerPop related so this is “okay” by me. If there are no objections,
> please feel to use the image publicly in 72 hours.
>
> Thanks,
> Marko.
>
> http://markorodriguez <http://markorodriguez/>.com
>
>
> > On Sep 1, 2016, at 9:51 AM, Jason Plurad <plur...@gmail.com> wrote:
> >
> > Hi,
> >
> > I'm drafting a blog post about Apache TinkerPop and several TinkerPop
> > providers, and I'd like to use this cool image in the post. I'll touch on
> > TinkerPop's graduation and other news/releases from this summer.
> >
> >
> https://raw.githubusercontent.com/apache/tinkerpop/3.2.1/docs/static/images/gremlin-dashboard.png
> >
> > Please let me know if it would be ok. Thanks.
> >
> > -- Jason
>
>


[jira] [Reopened] (TINKERPOP-1360) intermittent error in spark-gremlin integration test

2016-07-30 Thread Jason Plurad (JIRA)

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

Jason Plurad reopened TINKERPOP-1360:
-

test case fix made a bad assumption that vertex id is always numeric

> intermittent error in spark-gremlin integration test
> 
>
> Key: TINKERPOP-1360
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1360
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.1.2-incubating
>Reporter: Jason Plurad
>    Assignee: Jason Plurad
>Priority: Minor
> Fix For: 3.1.3, 3.2.1
>
>
> This error intermittently comes up during the Spark integration tests
> {noformat}
> Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 
> sec <<< FAILURE! - in 
> org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
> g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
>   Time elapsed: 0.185 sec  <<< FAILURE!
> java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
> java.lang.Long<6>
>   at org.junit.Assert.fail(Assert.java:88)
>   at org.junit.Assert.failNotEquals(Assert.java:834)
>   at org.junit.Assert.assertEquals(Assert.java:118)
>   at org.junit.Assert.assertEquals(Assert.java:144)
>   at 
> org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest.g_V_hasXageX_propertiesXnameX(PropertiesTest.java:103)
>   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:497)
>   at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>   at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>   at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>   at 
> org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
>   at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
>   at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>   at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
>   at org.junit.rules.RunRules.evaluate(RunRules.java:20)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:53)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:37)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at org.junit.runners.Suite.runChild(Suite.java:128)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:264)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:124)
>   at 
> org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:200)
>   at 
> org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:153)
>   at 
> org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)
> {noformat}
> Reproduce steps:
>

[jira] [Updated] (TINKERPOP-1379) unaccounted excess in TailGlobalStep

2016-07-30 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1379:

Fix Version/s: 3.3.0
   3.2.2
   3.1.4

> unaccounted excess in TailGlobalStep
> 
>
> Key: TINKERPOP-1379
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1379
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process
>Affects Versions: 3.0.2-incubating, 3.1.3, 3.2.1
>Reporter: Jason Plurad
> Fix For: 3.1.4, 3.2.2, 3.3.0
>
>
> https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/TailGlobalStep.java#L71-L74
> This code doesn't account for the excess removed from {{tailBulk}}. This can 
> cause the code to set incorrect bulk values when there are multiple 
> traversers in the tail buffer.
> I observed this behavior intermittently in TitanGraph 
> (https://github.com/thinkaurelius/titan/pull/1312), which doesn't allow user 
> defined ids, so the ordering of the traversers in the TraverserSet is a lot 
> more random than the unit tests using TinkerGraph.
> This issue is reproducible with TinkerGraph (master/3.2.1). Instead of 
> loading from one of the data files, manually create the graph with the ids in 
> inverted order.
> {noformat}
> graph = TinkerGraph.open();
> // graph.io(IoCore.gryo()).readGraph("tinkerpop-modern.kryo");
> final Vertex v1 = graph.addVertex(T.id, 6L, T.label, "person", "name", 
> "marko", "age", 29);
> final Vertex v2 = graph.addVertex(T.id, 5L, T.label, "person", "name", 
> "vadas", "age", 27);
> final Vertex v3 = graph.addVertex(T.id, 4L, T.label, "software", "name", 
> "lop", "lang", "java");
> final Vertex v4 = graph.addVertex(T.id, 3L, T.label, "person", "name", 
> "josh", "age", 32);
> final Vertex v5 = graph.addVertex(T.id, 2L, T.label, "software", "name", 
> "ripple", "lang", "java");
> final Vertex v6 = graph.addVertex(T.id, 1L, T.label, "person", "name", 
> "peter", "age", 35);
> v1.addEdge("knows", v2, "weight", 0.5d);
> v1.addEdge("knows", v4, "weight", 1.0d);
> v1.addEdge("created", v2, "weight", 0.4d);
> v4.addEdge("knows", v5, "weight", 1.0d);
> v4.addEdge("knows", v3, "weight", 0.4d);
> v6.addEdge("knows", v3, "weight", 0.2d);
> if (graph.features().graph().supportsTransactions()) graph.tx().commit();
> final GraphTraversalSource g = graph.traversal();
> String result = 
> g.V().repeat(both()).times(3).tail(7).count().next().toString();
> boolean success = "7".equals(result);
> {noformat}
> The fix is this:
> {noformat}
> if (excess > 0) {
> oldest.setBulk(oldestBulk-excess);
> // Account for the loss of excess in the tail buffer
> this.tailBulk -= excess;
> }
> {noformat}



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


[jira] [Updated] (TINKERPOP-1379) unaccounted excess in TailGlobalStep

2016-07-30 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1379:

Affects Version/s: 3.0.2-incubating

> unaccounted excess in TailGlobalStep
> 
>
> Key: TINKERPOP-1379
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1379
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process
>Affects Versions: 3.0.2-incubating, 3.1.3, 3.2.1
>Reporter: Jason Plurad
> Fix For: 3.1.4, 3.2.2, 3.3.0
>
>
> https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/TailGlobalStep.java#L71-L74
> This code doesn't account for the excess removed from {{tailBulk}}. This can 
> cause the code to set incorrect bulk values when there are multiple 
> traversers in the tail buffer.
> I observed this behavior intermittently in TitanGraph 
> (https://github.com/thinkaurelius/titan/pull/1312), which doesn't allow user 
> defined ids, so the ordering of the traversers in the TraverserSet is a lot 
> more random than the unit tests using TinkerGraph.
> This issue is reproducible with TinkerGraph (master/3.2.1). Instead of 
> loading from one of the data files, manually create the graph with the ids in 
> inverted order.
> {noformat}
> graph = TinkerGraph.open();
> // graph.io(IoCore.gryo()).readGraph("tinkerpop-modern.kryo");
> final Vertex v1 = graph.addVertex(T.id, 6L, T.label, "person", "name", 
> "marko", "age", 29);
> final Vertex v2 = graph.addVertex(T.id, 5L, T.label, "person", "name", 
> "vadas", "age", 27);
> final Vertex v3 = graph.addVertex(T.id, 4L, T.label, "software", "name", 
> "lop", "lang", "java");
> final Vertex v4 = graph.addVertex(T.id, 3L, T.label, "person", "name", 
> "josh", "age", 32);
> final Vertex v5 = graph.addVertex(T.id, 2L, T.label, "software", "name", 
> "ripple", "lang", "java");
> final Vertex v6 = graph.addVertex(T.id, 1L, T.label, "person", "name", 
> "peter", "age", 35);
> v1.addEdge("knows", v2, "weight", 0.5d);
> v1.addEdge("knows", v4, "weight", 1.0d);
> v1.addEdge("created", v2, "weight", 0.4d);
> v4.addEdge("knows", v5, "weight", 1.0d);
> v4.addEdge("knows", v3, "weight", 0.4d);
> v6.addEdge("knows", v3, "weight", 0.2d);
> if (graph.features().graph().supportsTransactions()) graph.tx().commit();
> final GraphTraversalSource g = graph.traversal();
> String result = 
> g.V().repeat(both()).times(3).tail(7).count().next().toString();
> boolean success = "7".equals(result);
> {noformat}
> The fix is this:
> {noformat}
> if (excess > 0) {
> oldest.setBulk(oldestBulk-excess);
> // Account for the loss of excess in the tail buffer
> this.tailBulk -= excess;
> }
> {noformat}



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


Re: [GitHub] tinkerpop pull request #363: TINKERPOP-1379 remove excess bulk in tail buffe...

2016-07-25 Thread Jason Plurad
Thanks for the feedback, Marko. I'll try to come up with another reproduce
scenario. Thanks!

On Fri, Jul 22, 2016 at 8:09 AM, Marko A. Rodriguez <okramma...@gmail.com>
wrote:

> More info -- on vacation from ipad s -- dont make anew graph as you
> have. Use modern or Grateful. this way olap will test too. order on age or
> something like that to get the bulking. name you method like all other test
> methods. in short yoy are doing too much and making your test too
> complicated. you just need to get a bulk where the total sum of the tail is
> greater than what is being tailed.
>
> Sent from my iPad
>
> > On Jul 21, 2016, at 5:36 PM, Marko A. Rodriguez <okramma...@gmail.com>
> wrote:
> >
> > The teat method structure and naming convention is bad. Stephen can help
> set it straight.
> >
> >> On Jul 20, 2016, at 12:40 PM, pluradj <g...@git.apache.org> wrote:
> >>
> >> GitHub user pluradj opened a pull request:
> >>
> >>   https://github.com/apache/tinkerpop/pull/363
> >>
> >>   TINKERPOP-1379 remove excess bulk in tail buffer
> >>
> >>   https://issues.apache.org/jira/browse/TINKERPOP-1379
> >>
> >> You can merge this pull request into a Git repository by running:
> >>
> >>   $ git pull https://github.com/apache/tinkerpop TINKERPOP-1379
> >>
> >> Alternatively you can review and apply these changes as the patch at:
> >>
> >>   https://github.com/apache/tinkerpop/pull/363.patch
> >>
> >> To close this pull request, make a commit to your master/trunk branch
> >> with (at least) the following in the commit message:
> >>
> >>   This closes #363
> >>
> >> 
> >> commit d43db2bb2f3f4988503f0cdedbd1877cd542c502
> >> Author: Jason Plurad <plur...@us.ibm.com>
> >> Date:   2016-07-20T16:31:53Z
> >>
> >>   remove excess bulk in tail buffer
> >>
> >> 
> >>
> >>
> >> ---
> >> If your project is set up for it, you can reply to this email and have
> your
> >> reply appear on GitHub as well. If your project does not have this
> feature
> >> enabled and wishes so, or if the feature is enabled but not working,
> please
> >> contact infrastructure at infrastruct...@apache.org or file a JIRA
> ticket
> >> with INFRA.
> >> ---
>


Re: [DISCUSS] Download Page

2016-07-22 Thread Jason Plurad
+1 looks great.

Link to the documentation, maybe next to the release notes link, would be
useful too.

-- Jason

On Fri, Jul 22, 2016 at 2:07 PM, Stephen Mallette 
wrote:

> I think it might be a good idea to include a "Download Page" on our web
> site. It would let us better list the versions we have out there and would
> get us better in line with Apache Infrastructure - for example:
>
> https://issues.apache.org/jira/browse/TINKERPOP-1255
>
> I'm also seeing that we yet have spots where we dont' link to the mirror
> system. And finally, it would be nice to have a "generic" download page
> where we could point external links to - like the new "documentation"
> feature in StackOverflow (right now it's sorta hardcoded to the mirror
> which isn't so good because we'll have to change that all the time).
>
> I quickly took a swipe at such a page:
>
> http://tinkerpop.apache.org/downloads.html
>
> It is NOT "linked to" by anything at this point so it is "hidden". I just
> wanted to have something that everyone could look at to get the idea of
> what such a page might look like.
>
> Thoughts?
>
> Thanks,
>
> Stephen
>


[jira] [Updated] (TINKERPOP-1379) unaccounted excess in TailGlobalStep

2016-07-20 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1379:

Affects Version/s: 3.1.3

> unaccounted excess in TailGlobalStep
> 
>
> Key: TINKERPOP-1379
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1379
> Project: TinkerPop
>  Issue Type: Bug
>  Components: process
>Affects Versions: 3.1.3, 3.2.1
>Reporter: Jason Plurad
>
> https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/TailGlobalStep.java#L71-L74
> This code doesn't account for the excess removed from {{tailBulk}}. This can 
> cause the code to set incorrect bulk values when there are multiple 
> traversers in the tail buffer.
> I observed this behavior intermittently in TitanGraph 
> (https://github.com/thinkaurelius/titan/pull/1312), which doesn't allow user 
> defined ids, so the ordering of the traversers in the TraverserSet is a lot 
> more random than the unit tests using TinkerGraph.
> This issue is reproducible with TinkerGraph (master/3.2.1). Instead of 
> loading from one of the data files, manually create the graph with the ids in 
> inverted order.
> {noformat}
> graph = TinkerGraph.open();
> // graph.io(IoCore.gryo()).readGraph("tinkerpop-modern.kryo");
> final Vertex v1 = graph.addVertex(T.id, 6L, T.label, "person", "name", 
> "marko", "age", 29);
> final Vertex v2 = graph.addVertex(T.id, 5L, T.label, "person", "name", 
> "vadas", "age", 27);
> final Vertex v3 = graph.addVertex(T.id, 4L, T.label, "software", "name", 
> "lop", "lang", "java");
> final Vertex v4 = graph.addVertex(T.id, 3L, T.label, "person", "name", 
> "josh", "age", 32);
> final Vertex v5 = graph.addVertex(T.id, 2L, T.label, "software", "name", 
> "ripple", "lang", "java");
> final Vertex v6 = graph.addVertex(T.id, 1L, T.label, "person", "name", 
> "peter", "age", 35);
> v1.addEdge("knows", v2, "weight", 0.5d);
> v1.addEdge("knows", v4, "weight", 1.0d);
> v1.addEdge("created", v2, "weight", 0.4d);
> v4.addEdge("knows", v5, "weight", 1.0d);
> v4.addEdge("knows", v3, "weight", 0.4d);
> v6.addEdge("knows", v3, "weight", 0.2d);
> if (graph.features().graph().supportsTransactions()) graph.tx().commit();
> final GraphTraversalSource g = graph.traversal();
> String result = 
> g.V().repeat(both()).times(3).tail(7).count().next().toString();
> boolean success = "7".equals(result);
> {noformat}
> The fix is this:
> {noformat}
> if (excess > 0) {
> oldest.setBulk(oldestBulk-excess);
> // Account for the loss of excess in the tail buffer
> this.tailBulk -= excess;
> }
> {noformat}



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


[jira] [Created] (TINKERPOP-1381) improve test coverage of graphs with random ids

2016-07-20 Thread Jason Plurad (JIRA)
Jason Plurad created TINKERPOP-1381:
---

 Summary: improve test coverage of graphs with random ids
 Key: TINKERPOP-1381
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1381
 Project: TinkerPop
  Issue Type: Test
  Components: test-suite
Affects Versions: 3.2.1
Reporter: Jason Plurad


https://issues.apache.org/jira/browse/TINKERPOP-1379 exposed some issues with 
the test suite where some tests were passing because the input graph has 
user-assigned ids in a specific, repeatable order. Many graph systems do not 
allow user-assigned ids and end up with random ids assigned to elements. This 
difference in behavior can cause the TinkerPop suite to pass cleanly against 
TinkerGraph, but fail intermittently against other graphs (Titan, Sqlg).

https://issues.apache.org/jira/browse/TINKERPOP-1365 is perhaps related to 
this, with respect to finding ways to increase coverage on the graph 
configuration permutations.



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


[jira] [Updated] (TINKERPOP-1379) unaccounted excess in TailGlobalStep

2016-07-19 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1379:

Description: 
https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/TailGlobalStep.java#L71-L74

This code doesn't account for the excess removed from {{tailBulk}}. This can 
cause the code to set incorrect bulk values when there are multiple traversers 
in the tail buffer.

I observed this behavior intermittently in TitanGraph 
(https://github.com/thinkaurelius/titan/pull/1312), which doesn't allow user 
defined ids, so the ordering of the traversers in the TraverserSet is a lot 
more random than the unit tests using TinkerGraph.

This issue is reproducible with TinkerGraph (master/3.2.1). Instead of loading 
from one of the data files, manually create the graph with the ids in inverted 
order.
{noformat}
graph = TinkerGraph.open();

// graph.io(IoCore.gryo()).readGraph("tinkerpop-modern.kryo");
final Vertex v1 = graph.addVertex(T.id, 6L, T.label, "person", "name", "marko", 
"age", 29);
final Vertex v2 = graph.addVertex(T.id, 5L, T.label, "person", "name", "vadas", 
"age", 27);
final Vertex v3 = graph.addVertex(T.id, 4L, T.label, "software", "name", "lop", 
"lang", "java");
final Vertex v4 = graph.addVertex(T.id, 3L, T.label, "person", "name", "josh", 
"age", 32);
final Vertex v5 = graph.addVertex(T.id, 2L, T.label, "software", "name", 
"ripple", "lang", "java");
final Vertex v6 = graph.addVertex(T.id, 1L, T.label, "person", "name", "peter", 
"age", 35);
v1.addEdge("knows", v2, "weight", 0.5d);
v1.addEdge("knows", v4, "weight", 1.0d);
v1.addEdge("created", v2, "weight", 0.4d);
v4.addEdge("knows", v5, "weight", 1.0d);
v4.addEdge("knows", v3, "weight", 0.4d);
v6.addEdge("knows", v3, "weight", 0.2d);
if (graph.features().graph().supportsTransactions()) graph.tx().commit();

final GraphTraversalSource g = graph.traversal();
String result = g.V().repeat(both()).times(3).tail(7).count().next().toString();
boolean success = "7".equals(result);
{noformat}

The fix is this:
{noformat}
if (excess > 0) {
oldest.setBulk(oldestBulk-excess);
// Account for the loss of excess in the tail buffer
this.tailBulk -= excess;
}
{noformat}

  was:
https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/TailGlobalStep.java#L71-L74

This code doesn't account for the excess removed from {{tailBulk}}. This can 
cause the code to set incorrect bulk values when there are multiple traversers 
in the tail buffer.

I observed this behavior intermittently in TitanGraph, which doesn't allow user 
defined ids, so the ordering of the traversers in the TraverserSet is a lot 
more random than the unit tests using TinkerGraph.

This issue is reproducible with TinkerGraph (master/3.2.1). Instead of loading 
from one of the data files, manually create the graph with the ids in inverted 
order.
{noformat}
graph = TinkerGraph.open();

// graph.io(IoCore.gryo()).readGraph("tinkerpop-modern.kryo");
final Vertex v1 = graph.addVertex(T.id, 6L, T.label, "person", "name", "marko", 
"age", 29);
final Vertex v2 = graph.addVertex(T.id, 5L, T.label, "person", "name", "vadas", 
"age", 27);
final Vertex v3 = graph.addVertex(T.id, 4L, T.label, "software", "name", "lop", 
"lang", "java");
final Vertex v4 = graph.addVertex(T.id, 3L, T.label, "person", "name", "josh", 
"age", 32);
final Vertex v5 = graph.addVertex(T.id, 2L, T.label, "software", "name", 
"ripple", "lang", "java");
final Vertex v6 = graph.addVertex(T.id, 1L, T.label, "person", "name", "peter", 
"age", 35);
v1.addEdge("knows", v2, "weight", 0.5d);
v1.addEdge("knows", v4, "weight", 1.0d);
v1.addEdge("created", v2, "weight", 0.4d);
v4.addEdge("knows", v5, "weight", 1.0d);
v4.addEdge("knows", v3, "weight", 0.4d);
v6.addEdge("knows", v3, "weight", 0.2d);
if (graph.features().graph().supportsTransactions()) graph.tx().commit();

final GraphTraversalSource g = graph.traversal();
String result = g.V().repeat(both()).times(3).tail(7).count().next(

[jira] [Created] (TINKERPOP-1379) unaccounted excess in TailGlobalStep

2016-07-19 Thread Jason Plurad (JIRA)
Jason Plurad created TINKERPOP-1379:
---

 Summary: unaccounted excess in TailGlobalStep
 Key: TINKERPOP-1379
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1379
 Project: TinkerPop
  Issue Type: Bug
  Components: process
Affects Versions: 3.2.1
Reporter: Jason Plurad


https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/TailGlobalStep.java#L71-L74

This code doesn't account for the excess removed from {{tailBulk}}. This can 
cause the code to set incorrect bulk values when there are multiple traversers 
in the tail buffer.

I observed this behavior intermittently in TitanGraph, which doesn't allow user 
defined ids, so the ordering of the traversers in the TraverserSet is a lot 
more random than the unit tests using TinkerGraph.

This issue is reproducible with TinkerGraph (master/3.2.1). Instead of loading 
from one of the data files, manually create the graph with the ids in inverted 
order.
{noformat}
graph = TinkerGraph.open();

// graph.io(IoCore.gryo()).readGraph("tinkerpop-modern.kryo");
final Vertex v1 = graph.addVertex(T.id, 6L, T.label, "person", "name", "marko", 
"age", 29);
final Vertex v2 = graph.addVertex(T.id, 5L, T.label, "person", "name", "vadas", 
"age", 27);
final Vertex v3 = graph.addVertex(T.id, 4L, T.label, "software", "name", "lop", 
"lang", "java");
final Vertex v4 = graph.addVertex(T.id, 3L, T.label, "person", "name", "josh", 
"age", 32);
final Vertex v5 = graph.addVertex(T.id, 2L, T.label, "software", "name", 
"ripple", "lang", "java");
final Vertex v6 = graph.addVertex(T.id, 1L, T.label, "person", "name", "peter", 
"age", 35);
v1.addEdge("knows", v2, "weight", 0.5d);
v1.addEdge("knows", v4, "weight", 1.0d);
v1.addEdge("created", v2, "weight", 0.4d);
v4.addEdge("knows", v5, "weight", 1.0d);
v4.addEdge("knows", v3, "weight", 0.4d);
v6.addEdge("knows", v3, "weight", 0.2d);
if (graph.features().graph().supportsTransactions()) graph.tx().commit();

final GraphTraversalSource g = graph.traversal();
String result = g.V().repeat(both()).times(3).tail(7).count().next().toString();
boolean success = "7".equals(result);
{noformat}

The fix is this:
{noformat}
if (excess > 0) {
oldest.setBulk(oldestBulk-excess);
// Account for the loss of excess in the tail buffer
this.tailBulk -= excess;
}
{noformat}



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


[jira] [Commented] (TINKERPOP-1367) Preserve path history for min() and max()

2016-07-12 Thread Jason Plurad (JIRA)

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

Jason Plurad commented on TINKERPOP-1367:
-

Right, from the original post on the mailing list, we were trying to get all 
paths that had a property that matched the max value.

> Preserve path history for min() and max()
> -
>
> Key: TINKERPOP-1367
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1367
> Project: TinkerPop
>  Issue Type: Improvement
>  Components: process
>Affects Versions: 3.2.0-incubating
>Reporter: Jason Plurad
>
> Via https://groups.google.com/d/msg/gremlin-users/qZwsvRjw7L4/YyT-s1foBAAJ
> {noformat}
> gremlin> g.V().outE().as('e').values('weight').path()
> ==>[v[1], e[9][1-created->3], 0.4]
> ==>[v[1], e[7][1-knows->2], 0.5]
> ==>[v[1], e[8][1-knows->4], 1.0]
> ==>[v[4], e[10][4-created->5], 1.0]
> ==>[v[4], e[11][4-created->3], 0.4]
> ==>[v[6], e[12][6-created->3], 0.2]
> gremlin> g.V().outE().as('e').values('weight').max().path()
> ==>[1.0]
> {noformat}
> Currently all reducing barriers are treated the same (min, max, mean, etc.). 
> But they are indeed different when it comes to path computations. Some of 
> them could preserve the path history, others could not.
> For max() and min(), we could preserve the path history. In fact, in this 
> respect, max() and min() would NOT be ReducingBarrierSteps, but in fact be 
> some sort of "barrier" FilterStep.



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


[jira] [Closed] (TINKERPOP-906) Install plugin always fails after first unresolved dependency

2016-07-11 Thread Jason Plurad (JIRA)

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

Jason Plurad closed TINKERPOP-906.
--
   Resolution: Fixed
Fix Version/s: 3.2.1
   3.1.3

https://github.com/apache/tinkerpop/commit/450e3ef80c2595953d722bb7c28dcbfd92ae78f5

> Install plugin always fails after first unresolved dependency
> -
>
> Key: TINKERPOP-906
> URL: https://issues.apache.org/jira/browse/TINKERPOP-906
> Project: TinkerPop
>  Issue Type: Bug
>  Components: groovy
>Affects Versions: 3.0.2-incubating
>Reporter: stephen mallette
>    Assignee: Jason Plurad
> Fix For: 3.1.3, 3.2.1
>
>
> Original issue: The {{DependencyGrabberTest}} currently require ordered 
> execution based on the name of the test.  In some environments this seems to 
> cause the tests to fail (as they may execute out of order).  Generally 
> speaking though it seems it would be best if the tests didn't have 
> dependencies on each other in order to pass.
> Actual issue: It turns out that there is a bug in Groovy Grapes. When there 
> is a failed call to {{Grape.resolve()}}, {{GrapeIvy.resolve()}} hangs onto a 
> reference to the IvyGrabRecord for the failed dependency. When subsequent 
> {{resolve()}} calls are made, those failed records are still there, so it 
> will continue to fail from that point on. This explains why the ordering was 
> important in the TinkerPop test case.
> {noformat}
>  \,,,/
>  (o o)
> -oOOo-(3)-oOOo-
> plugin activated: tinkerpop.server
> plugin activated: tinkerpop.utilities
> plugin activated: tinkerpop.tinkergraph
> gremlin> :install bogus bogus 1.0
> ==>Error grabbing Grapes -- [unresolved dependency: bogus#bogus;1.0: not 
> found]
> gremlin> :install org.apache.tinkerpop hadoop-gremlin 3.0.1-incubating
> ==>Error grabbing Grapes -- [unresolved dependency: bogus#bogus;1.0: not 
> found]
> {noformat}
> When Groovy 2.4.6 ships with the fix for GROOVY-7649, the {{@Ignore}} in the 
> test case can be removed if TinkerPop bumps up to the new Groovy version.



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


[jira] [Resolved] (TINKERPOP-1360) intermittent error in spark-gremlin integration test

2016-07-11 Thread Jason Plurad (JIRA)

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

Jason Plurad resolved TINKERPOP-1360.
-
   Resolution: Fixed
Fix Version/s: 3.2.1
   3.1.3

> intermittent error in spark-gremlin integration test
> 
>
> Key: TINKERPOP-1360
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1360
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.1.2-incubating
>Reporter: Jason Plurad
>    Assignee: Jason Plurad
>Priority: Minor
> Fix For: 3.1.3, 3.2.1
>
>
> This error intermittently comes up during the Spark integration tests
> {noformat}
> Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 
> sec <<< FAILURE! - in 
> org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
> g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
>   Time elapsed: 0.185 sec  <<< FAILURE!
> java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
> java.lang.Long<6>
>   at org.junit.Assert.fail(Assert.java:88)
>   at org.junit.Assert.failNotEquals(Assert.java:834)
>   at org.junit.Assert.assertEquals(Assert.java:118)
>   at org.junit.Assert.assertEquals(Assert.java:144)
>   at 
> org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest.g_V_hasXageX_propertiesXnameX(PropertiesTest.java:103)
>   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:497)
>   at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>   at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>   at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>   at 
> org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
>   at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
>   at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>   at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
>   at org.junit.rules.RunRules.evaluate(RunRules.java:20)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:53)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:37)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at org.junit.runners.Suite.runChild(Suite.java:128)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:264)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:124)
>   at 
> org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:200)
>   at 
> org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:153)
>   at 
> org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)
> {noformat}
> Reproduce steps:
>

[jira] [Closed] (TINKERPOP-1360) intermittent error in spark-gremlin integration test

2016-07-11 Thread Jason Plurad (JIRA)

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

Jason Plurad closed TINKERPOP-1360.
---
Resolution: Fixed

> intermittent error in spark-gremlin integration test
> 
>
> Key: TINKERPOP-1360
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1360
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.1.2-incubating
>Reporter: Jason Plurad
>    Assignee: Jason Plurad
>Priority: Minor
>
> This error intermittently comes up during the Spark integration tests
> {noformat}
> Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 
> sec <<< FAILURE! - in 
> org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
> g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
>   Time elapsed: 0.185 sec  <<< FAILURE!
> java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
> java.lang.Long<6>
>   at org.junit.Assert.fail(Assert.java:88)
>   at org.junit.Assert.failNotEquals(Assert.java:834)
>   at org.junit.Assert.assertEquals(Assert.java:118)
>   at org.junit.Assert.assertEquals(Assert.java:144)
>   at 
> org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest.g_V_hasXageX_propertiesXnameX(PropertiesTest.java:103)
>   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:497)
>   at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>   at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>   at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>   at 
> org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
>   at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
>   at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>   at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
>   at org.junit.rules.RunRules.evaluate(RunRules.java:20)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:53)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:37)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at org.junit.runners.Suite.runChild(Suite.java:128)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:264)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:124)
>   at 
> org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:200)
>   at 
> org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:153)
>   at 
> org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)
> {noformat}
> Reproduce steps:
> {noformat}
> mvn verify -q -DskipIntegrationTests=false -pl spark-gremlin
> {noformat}
> E

Re: Documentation deprecation warning on Github Wikis

2016-07-11 Thread Jason Plurad
+1

Could you also update these? I've fielded lots of questions referring to
these old graph I/O pages.

https://github.com/tinkerpop/blueprints/wiki/GraphSON-Reader-and-Writer-Library

https://github.com/tinkerpop/blueprints/wiki/GraphML-Reader-and-Writer-Library

https://github.com/tinkerpop/blueprints/wiki/GML-Reader-and-Writer-Library

https://github.com/tinkerpop/blueprints/wiki/Batch-Implementation

Thanks!
-- Jason
On Mon, Jul 11, 2016 at 6:36 PM Jean-Baptiste Musso 
wrote:

> Quick follow up - the changes to the TP2 Gremlin documentation are now
> live. Props to Dylan since I couldn't remember where this was first
> mentioned.
>
> See for example
> https://github.com/tinkerpop/gremlin/wiki/Depth-First-vs.-Breadth-First
>
> Editing several pages on GitHub wikis was indeed as simple as cloning
> the wiki repository and doing commits (git clone
> https://github.com/tinkerpop/gremlin.wiki). Anything that hits the
> master branch goes live once pushed to Github.
>
> Jean-Baptiste
>
> On Wed, Jun 8, 2016 at 2:20 PM, Stephen Mallette 
> wrote:
> > Jean-Baptiste, +1 for the wiki changes - and thanks for volunteering.
> I've
> > given you access to all the old repos in github so you should be free to
> > make the changes. Maybe give this thread another day to make sure no
> other
> > comments trickle in before you go to work on the changes. I think we
> might
> > want to go the extra step of adding your header to the READMEs and
> replace
> > our current gremlinbusters image - that way everything is consistent.
> >
> > Dylan, the gremlin-users info was very out of date, so I updated the
> About
> > description:
> >
> > https://groups.google.com/forum/#!aboutgroup/gremlin-users
> >
> > and included a welcome message that points to the latest docs/website:
> >
> > https://groups.google.com/forum/#!forum/gremlin-users
> >
> > Everyone good with those changes?
> >
> >
> >
> > On Tue, Jun 7, 2016 at 4:54 PM, Dylan Bethune-Waddell <
> > dylan.bethune.wadd...@mail.utoronto.ca> wrote:
> >
> >> I believe I mentioned this on the Titan or Tinkerpop mailing list but it
> >> was off-topic at the time, so I will restate here:
> >>
> >>
> >> What about a pinned post on the gremlin-users mailing lists linking to
> the
> >> most recent docs and website as another guidepost for users in line with
> >> this one? Does this edit to all the old wikis etc. make such a thing
> >> redundant? My thinking on this is that it would be similar to the pinned
> >> post titled "On proper issue submission" at the top of the Titan mailing
> >> list - short and sweet, saving Tinker-time solving problems on the list
> >> that are not easily reproducible or particularly
> relevant/straightforward.
> >> It could also prevent some posts based on outdated information that
> beget
> >> more relevant questions posted deep down in the thread, which now has a
> >> misleading title that does not match the core of its content.
> >>
> >> 
> >> From: Jean-Baptiste Musso 
> >> Sent: Tuesday, June 7, 2016 4:05:16 PM
> >> To: dev@tinkerpop.apache.org
> >> Subject: Documentation deprecation warning on Github Wikis
> >>
> >> Dear devs,
> >>
> >> I was thinking recently that we could add a deprecation warning
> >> message on top of all former Github Wiki pages and warn visitors that
> >> a new version of TinkerPop is available. I think this was also brought
> >> up recently on the mailing list so I'm opening a discussion here.
> >>
> >> I feel that some newcomers are still hitting old school TinkerPop 2
> >> material when google'ing for graphs, Gremlin and TinkerPop. Adding a
> >> warning message on top of old Wiki pages pointing them to the freshest
> >> development on the Apache TinkerPop website could certainly help.
> >>
> >> If everyone agrees, I'm volunteering to git clone the Wiki on all
> >> repositories and edit all pages. As Stephen noted on HipChat, it's as
> >> simple as following these steps:
> >> https://help.github.com/articles/adding-and-editing-wiki-pages-locally/
> >> Adding and editing wiki pages locally - User Documentation<
> >> https://help.github.com/articles/adding-and-editing-wiki-pages-locally/
> >
> >> help.github.com
> >> Cloning wikis locally to your computer. Every wiki provides an easy way
> to
> >> clone its contents down to your computer: If you're using GitHub
> Desktop,
> >> click Clone in ...
> >>
> >>
> >>
> >>
> >> Here's an example of such header we could add:
> >>
> >> https://gist.github.com/jbmusso/802cf97ceb20547ba6abf0b4112ac3ee
> >>
> >> Thoughts? Feel free to iterate. The wording could be improved.
> >>
> >> Cheers,
> >>
> >> Jean-Baptiste
> >>
>


[DISCUSS] Community Survey

2016-07-11 Thread Jason Plurad
We're about at the 1 year anniversary of the Apache TinkerPop 3.0.0
release. It might be a good chance to reflect and get some feedback on how
the TinkerPop community is shaping up with a survey. I've never done
anything like this before, so I don't know if email is good enough or if we
should use something like a Google Form or Survey Monkey.

Also it would be great if we can get some more folks in the community to
star the Apache TinkerPop  repo
mirror in GitHub. It's still well behind the old Blueprints/Gremlin repos.

Thoughts?

-- Jason


[jira] [Created] (TINKERPOP-1365) Eliminate Random.nextBoolean() usage in tests

2016-07-10 Thread Jason Plurad (JIRA)
Jason Plurad created TINKERPOP-1365:
---

 Summary: Eliminate Random.nextBoolean() usage in tests
 Key: TINKERPOP-1365
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1365
 Project: TinkerPop
  Issue Type: Improvement
  Components: test-suite
Affects Versions: 3.1.2-incubating, 3.2.0-incubating
Reporter: Jason Plurad


HadoopGraphProvider and SparkHadoopGraphProvider use RANDOM.nextBoolean() in an 
attempt to get coverage over different load scenarios. In practice, this ends 
up causing nondeterministic execution of the test suite, which can mask real 
errors lingering in the code or test code.

See https://issues.apache.org/jira/browse/TINKERPOP-1360




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


[jira] [Commented] (TINKERPOP-1360) intermittent error in spark-gremlin integration test

2016-07-09 Thread Jason Plurad (JIRA)

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

Jason Plurad commented on TINKERPOP-1360:
-

This occurs specifically with the {{GraphSONRecordReader}} because the format 
is lossy. The {{GryoRecordReader}} passes because it stores the {{Long}} type 
information for the vertex property id.

The toggle between the 2 input formats is also the result of a different random 
coin flip 
https://github.com/apache/tinkerpop/blob/tp31/hadoop-gremlin/src/test/java/org/apache/tinkerpop/gremlin/hadoop/HadoopGraphProvider.java#L112


> intermittent error in spark-gremlin integration test
> 
>
> Key: TINKERPOP-1360
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1360
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.1.2-incubating
>Reporter: Jason Plurad
>    Assignee: Jason Plurad
>Priority: Minor
>
> This error intermittently comes up during the Spark integration tests
> {noformat}
> Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 
> sec <<< FAILURE! - in 
> org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
> g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
>   Time elapsed: 0.185 sec  <<< FAILURE!
> java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
> java.lang.Long<6>
>   at org.junit.Assert.fail(Assert.java:88)
>   at org.junit.Assert.failNotEquals(Assert.java:834)
>   at org.junit.Assert.assertEquals(Assert.java:118)
>   at org.junit.Assert.assertEquals(Assert.java:144)
>   at 
> org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest.g_V_hasXageX_propertiesXnameX(PropertiesTest.java:103)
>   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:497)
>   at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>   at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>   at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>   at 
> org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
>   at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
>   at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>   at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
>   at org.junit.rules.RunRules.evaluate(RunRules.java:20)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:53)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:37)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at org.junit.runners.Suite.runChild(Suite.java:128)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:264)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153)
>   at 
> org.apache.maven.surefire.junit4.JUnit4P

[jira] [Commented] (TINKERPOP-1360) intermittent error in spark-gremlin integration test

2016-07-09 Thread Jason Plurad (JIRA)

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

Jason Plurad commented on TINKERPOP-1360:
-

https://github.com/apache/tinkerpop/blob/tp31/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/PropertiesTest.java#L103

The value of the vertex property id is the same, but they are coming back as 
different types, {{Long}} vs {{Integer}}, and comparing {{ 
Long.valueOf(6).equals(Integer.valueOf(6)) }} returns {{false}}. Comparing them 
as primitives would work.


> intermittent error in spark-gremlin integration test
> 
>
> Key: TINKERPOP-1360
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1360
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.1.2-incubating
>Reporter: Jason Plurad
>    Assignee: Jason Plurad
>Priority: Minor
>
> This error intermittently comes up during the Spark integration tests
> {noformat}
> Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 
> sec <<< FAILURE! - in 
> org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
> g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
>   Time elapsed: 0.185 sec  <<< FAILURE!
> java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
> java.lang.Long<6>
>   at org.junit.Assert.fail(Assert.java:88)
>   at org.junit.Assert.failNotEquals(Assert.java:834)
>   at org.junit.Assert.assertEquals(Assert.java:118)
>   at org.junit.Assert.assertEquals(Assert.java:144)
>   at 
> org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest.g_V_hasXageX_propertiesXnameX(PropertiesTest.java:103)
>   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:497)
>   at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>   at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>   at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>   at 
> org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
>   at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
>   at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>   at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
>   at org.junit.rules.RunRules.evaluate(RunRules.java:20)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:53)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:37)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at org.junit.runners.Suite.runChild(Suite.java:128)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:264)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:124)
>   at 
> org.apache.maven.surefir

[jira] [Comment Edited] (TINKERPOP-1360) intermittent error in spark-gremlin integration test

2016-07-09 Thread Jason Plurad (JIRA)

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

Jason Plurad edited comment on TINKERPOP-1360 at 7/10/16 1:47 AM:
--

https://github.com/apache/tinkerpop/blob/tp31/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/PropertiesTest.java#L103

The value of the vertex property id is the same, but they are coming back as 
different types, {{Long}} vs {{Integer}}, and comparing 
{{Long.valueOf(6).equals(Integer.valueOf(6))}} returns {{false}}. Comparing 
them as primitives would work.



was (Author: pluradj):
https://github.com/apache/tinkerpop/blob/tp31/gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/PropertiesTest.java#L103

The value of the vertex property id is the same, but they are coming back as 
different types, {{Long}} vs {{Integer}}, and comparing {{ 
Long.valueOf(6).equals(Integer.valueOf(6)) }} returns {{false}}. Comparing them 
as primitives would work.


> intermittent error in spark-gremlin integration test
> 
>
> Key: TINKERPOP-1360
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1360
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.1.2-incubating
>Reporter: Jason Plurad
>    Assignee: Jason Plurad
>Priority: Minor
>
> This error intermittently comes up during the Spark integration tests
> {noformat}
> Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 
> sec <<< FAILURE! - in 
> org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
> g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
>   Time elapsed: 0.185 sec  <<< FAILURE!
> java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
> java.lang.Long<6>
>   at org.junit.Assert.fail(Assert.java:88)
>   at org.junit.Assert.failNotEquals(Assert.java:834)
>   at org.junit.Assert.assertEquals(Assert.java:118)
>   at org.junit.Assert.assertEquals(Assert.java:144)
>   at 
> org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest.g_V_hasXageX_propertiesXnameX(PropertiesTest.java:103)
>   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:497)
>   at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>   at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>   at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>   at 
> org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
>   at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
>   at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>   at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
>   at org.junit.rules.RunRules.evaluate(RunRules.java:20)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:53)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:37)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at org.junit.runners.Suite.runChild(Suite.java:128)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   

[jira] [Comment Edited] (TINKERPOP-1360) intermittent error in spark-gremlin integration test

2016-07-09 Thread Jason Plurad (JIRA)

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

Jason Plurad edited comment on TINKERPOP-1360 at 7/10/16 1:38 AM:
--

Appears that the broken test in question was introduced by these 2 CTR commits:
* 
https://github.com/apache/tinkerpop/commit/9c8c655dd7bd324d6f1341321717491e60c2e894
* 
https://github.com/apache/tinkerpop/commit/ec106fc5a9f507ae7d5be6b06b92be33bc7bde54

The {{RANDOM.nextBoolean()}} is enough to mask the problem often enough because 
there are scenarios that pass.


was (Author: pluradj):
Appears that the broken test in question was introduced by these 2 CTR commits:
* 
https://github.com/apache/tinkerpop/commit/9c8c655dd7bd324d6f1341321717491e60c2e894
* 
https://github.com/apache/tinkerpop/commit/ec106fc5a9f507ae7d5be6b06b92be33bc7bde54

The {RANDOM.nextBoolean()} is enough to mask the problem often enough because 
there are scenarios that pass.

> intermittent error in spark-gremlin integration test
> 
>
> Key: TINKERPOP-1360
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1360
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.1.2-incubating
>Reporter: Jason Plurad
>    Assignee: Jason Plurad
>Priority: Minor
>
> This error intermittently comes up during the Spark integration tests
> {noformat}
> Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 
> sec <<< FAILURE! - in 
> org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
> g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
>   Time elapsed: 0.185 sec  <<< FAILURE!
> java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
> java.lang.Long<6>
>   at org.junit.Assert.fail(Assert.java:88)
>   at org.junit.Assert.failNotEquals(Assert.java:834)
>   at org.junit.Assert.assertEquals(Assert.java:118)
>   at org.junit.Assert.assertEquals(Assert.java:144)
>   at 
> org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest.g_V_hasXageX_propertiesXnameX(PropertiesTest.java:103)
>   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:497)
>   at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>   at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>   at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>   at 
> org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
>   at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
>   at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>   at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
>   at org.junit.rules.RunRules.evaluate(RunRules.java:20)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:53)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:37)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at org.junit.runners.Suite.runChild(Suite.java:128)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>  

[jira] [Commented] (TINKERPOP-1360) intermittent error in spark-gremlin integration test

2016-07-09 Thread Jason Plurad (JIRA)

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

Jason Plurad commented on TINKERPOP-1360:
-

This in part explains the random failure behavior 
https://github.com/apache/tinkerpop/blob/tp31/spark-gremlin/src/test/java/org/apache/tinkerpop/gremlin/spark/process/computer/SparkHadoopGraphProvider.java#L52-L54

{noformat}
 if (!test.equals(FileSystemStorageCheck.class) && null != loadGraphWith && 
RANDOM.nextBoolean()) {
config.put(Constants.GREMLIN_SPARK_GRAPH_INPUT_RDD, 
ToyGraphInputRDD.class.getCanonicalName());
}
{noformat}

The error occurs when the {{ToyGraphInputRDD}} is used with 
{{GraphSONInputFormat}}, and the graph loaded is {{tinkerpop-modern.json}}.


> intermittent error in spark-gremlin integration test
> 
>
> Key: TINKERPOP-1360
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1360
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.1.2-incubating
>Reporter: Jason Plurad
>Assignee: Jason Plurad
>Priority: Minor
>
> This error intermittently comes up during the Spark integration tests
> {noformat}
> Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 
> sec <<< FAILURE! - in 
> org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
> g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
>   Time elapsed: 0.185 sec  <<< FAILURE!
> java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
> java.lang.Long<6>
>   at org.junit.Assert.fail(Assert.java:88)
>   at org.junit.Assert.failNotEquals(Assert.java:834)
>   at org.junit.Assert.assertEquals(Assert.java:118)
>   at org.junit.Assert.assertEquals(Assert.java:144)
>   at 
> org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest.g_V_hasXageX_propertiesXnameX(PropertiesTest.java:103)
>   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:497)
>   at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>   at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>   at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>   at 
> org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
>   at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
>   at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>   at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
>   at org.junit.rules.RunRules.evaluate(RunRules.java:20)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:53)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:37)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at org.junit.runners.Suite.runChild(Suite.java:128)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:264)
>   at 
> org.apache.maven.surefire.juni

[jira] [Assigned] (TINKERPOP-1360) intermittent error in spark-gremlin integration test

2016-07-08 Thread Jason Plurad (JIRA)

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

Jason Plurad reassigned TINKERPOP-1360:
---

Assignee: Jason Plurad

> intermittent error in spark-gremlin integration test
> 
>
> Key: TINKERPOP-1360
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1360
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.1.2-incubating
>Reporter: Jason Plurad
>    Assignee: Jason Plurad
>Priority: Minor
>
> This error intermittently comes up during the Spark integration tests
> {noformat}
> Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 
> sec <<< FAILURE! - in 
> org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
> g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
>   Time elapsed: 0.185 sec  <<< FAILURE!
> java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
> java.lang.Long<6>
>   at org.junit.Assert.fail(Assert.java:88)
>   at org.junit.Assert.failNotEquals(Assert.java:834)
>   at org.junit.Assert.assertEquals(Assert.java:118)
>   at org.junit.Assert.assertEquals(Assert.java:144)
>   at 
> org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest.g_V_hasXageX_propertiesXnameX(PropertiesTest.java:103)
>   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:497)
>   at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>   at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>   at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>   at 
> org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
>   at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
>   at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>   at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
>   at org.junit.rules.RunRules.evaluate(RunRules.java:20)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:53)
>   at 
> org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:37)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at org.junit.runners.Suite.runChild(Suite.java:128)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:264)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153)
>   at 
> org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:124)
>   at 
> org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:200)
>   at 
> org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:153)
>   at 
> org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)
> {noformat}
> Reproduce steps:
> {noformat}
> mvn verify -q -DskipIntegrationTests=false -pl spark-gremlin
> {n

[jira] [Resolved] (TINKERPOP-1319) several FeatureRequirement annotations are incorrect in gremlin-test

2016-07-08 Thread Jason Plurad (JIRA)

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

Jason Plurad resolved TINKERPOP-1319.
-
Resolution: Fixed

> several FeatureRequirement annotations are incorrect in gremlin-test
> 
>
> Key: TINKERPOP-1319
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1319
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.2.0-incubating, 3.1.2-incubating
>Reporter: Jason Plurad
>    Assignee: Jason Plurad
>Priority: Minor
> Fix For: 3.1.3, 3.2.1
>
>
> Several {{@FeatureRequirement}} annotations are incorrect in these 
> {{gremlin-test}} tests
> * EdgeTest.java
> * FeatureSupportTest.java
> * GraphTest.java
> * PropertyTest.java
> * VertexPropertyTest.java
> * VertexTest.java
> I'll submit a patch for this.



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


[jira] [Closed] (TINKERPOP-1319) several FeatureRequirement annotations are incorrect in gremlin-test

2016-07-08 Thread Jason Plurad (JIRA)

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

Jason Plurad closed TINKERPOP-1319.
---

> several FeatureRequirement annotations are incorrect in gremlin-test
> 
>
> Key: TINKERPOP-1319
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1319
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.2.0-incubating, 3.1.2-incubating
>Reporter: Jason Plurad
>    Assignee: Jason Plurad
>Priority: Minor
> Fix For: 3.1.3, 3.2.1
>
>
> Several {{@FeatureRequirement}} annotations are incorrect in these 
> {{gremlin-test}} tests
> * EdgeTest.java
> * FeatureSupportTest.java
> * GraphTest.java
> * PropertyTest.java
> * VertexPropertyTest.java
> * VertexTest.java
> I'll submit a patch for this.



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


[jira] [Updated] (TINKERPOP-1360) intermittent error in spark-gremlin integration test

2016-07-07 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1360:

Description: 
This error intermittently comes up during the Spark integration tests

{noformat}
Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 sec 
<<< FAILURE! - in 
org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
  Time elapsed: 0.185 sec  <<< FAILURE!
java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
java.lang.Long<6>
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:834)
at org.junit.Assert.assertEquals(Assert.java:118)
at org.junit.Assert.assertEquals(Assert.java:144)
at 
org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest.g_V_hasXageX_propertiesXnameX(PropertiesTest.java:103)
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:497)
at 
org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
at 
org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at 
org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
at 
org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at 
org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
at 
org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
at org.junit.rules.RunRules.evaluate(RunRules.java:20)
at 
org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:53)
at 
org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:37)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at org.junit.runners.Suite.runChild(Suite.java:128)
at 
org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
at 
org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:264)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:124)
at 
org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:200)
at 
org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:153)
at 
org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)
{noformat}

Reproduce steps:
{noformat}
mvn verify -q -DskipIntegrationTests=false -pl spark-gremlin
{noformat}

Environments used:
{noformat}
Apache Maven 3.0.5
Maven home: /usr/share/maven
Java version: 1.8.0_91, vendor: Oracle Corporation
Java home: /usr/lib/jvm/java-8-openjdk-amd64/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "linux", version: "3.13.0-76-generic", arch: "amd64", family: "unix"
-
Apache Maven 3.3.3 (7994120775791599e205a5524ec3e0dfe41d4a06; 
2015-04-22T07:57:37-04:00)
Maven home: /usr/lib/apache-maven-3.3.3
Java version: 1.8.0_71, vendor: Oracle Corporation
Java home: /Library/Java/JavaVirtualMachines/jdk1.8.0_71.jdk/Contents/Home/jre
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "10.11.5", arch: "x86_64", family: "mac"
{noformat}

See

[jira] [Updated] (TINKERPOP-1360) intermittent error in spark-gremlin integration test

2016-07-07 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1360:

Description: 
This error intermittently comes up during the Spark integration tests

{noformat}
Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 sec 
<<< FAILURE! - in 
org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
  Time elapsed: 0.185 sec  <<< FAILURE!
java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
java.lang.Long<6>
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:834)
at org.junit.Assert.assertEquals(Assert.java:118)
at org.junit.Assert.assertEquals(Assert.java:144)
at 
org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest.g_V_hasXageX_propertiesXnameX(PropertiesTest.java:103)
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:497)
at 
org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
at 
org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at 
org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
at 
org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at 
org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
at 
org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
at org.junit.rules.RunRules.evaluate(RunRules.java:20)
at 
org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:53)
at 
org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:37)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at org.junit.runners.Suite.runChild(Suite.java:128)
at 
org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
at 
org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:264)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:124)
at 
org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:200)
at 
org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:153)
at 
org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)
{noformat}

Reproduce steps:
{noformat}
mvn verify -q -DskipIntegrationTests=false -pl spark-gremlin
{noformat}

See also https://github.com/apache/tinkerpop/pull/335

  was:
This error intermittently comes up during the Spark integration tests

{noformat}
Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 sec 
<<< FAILURE! - in 
org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
  Time elapsed: 0.185 sec  <<< FAILURE!
java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
java.lang.Long<6>
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:834)
at org.junit.Assert.assertEquals(Assert.java:1

[jira] [Created] (TINKERPOP-1360) intermittent error

2016-07-07 Thread Jason Plurad (JIRA)
Jason Plurad created TINKERPOP-1360:
---

 Summary: intermittent error 
 Key: TINKERPOP-1360
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1360
 Project: TinkerPop
  Issue Type: Bug
  Components: test-suite
Affects Versions: 3.1.2-incubating
Reporter: Jason Plurad
Priority: Minor


This error intermittently comes up during the Spark integration tests

{noformat}
Tests run: 457, Failures: 1, Errors: 0, Skipped: 150, Time elapsed: 110.068 sec 
<<< FAILURE! - in 
org.apache.tinkerpop.gremlin.spark.process.computer.SparkGraphComputerProcessIntegrateTest
g_V_hasXageX_propertiesXnameX(org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest$Traversals)
  Time elapsed: 0.185 sec  <<< FAILURE!
java.lang.AssertionError: expected: java.lang.Integer<6> but was: 
java.lang.Long<6>
at org.junit.Assert.fail(Assert.java:88)
at org.junit.Assert.failNotEquals(Assert.java:834)
at org.junit.Assert.assertEquals(Assert.java:118)
at org.junit.Assert.assertEquals(Assert.java:144)
at 
org.apache.tinkerpop.gremlin.process.traversal.step.map.PropertiesTest.g_V_hasXageX_propertiesXnameX(PropertiesTest.java:103)
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:497)
at 
org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
at 
org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at 
org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
at 
org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at 
org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
at 
org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
at org.junit.rules.RunRules.evaluate(RunRules.java:20)
at 
org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:53)
at 
org.apache.tinkerpop.gremlin.process.GremlinProcessRunner.runChild(GremlinProcessRunner.java:37)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at org.junit.runners.Suite.runChild(Suite.java:128)
at 
org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
at 
org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:264)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:153)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:124)
at 
org.apache.maven.surefire.booter.ForkedBooter.invokeProviderInSameClassLoader(ForkedBooter.java:200)
at 
org.apache.maven.surefire.booter.ForkedBooter.runSuitesInProcess(ForkedBooter.java:153)
at 
org.apache.maven.surefire.booter.ForkedBooter.main(ForkedBooter.java:103)
{noformat}

Reproduce steps:
{noformat}
mvn verify -q -DskipIntegrationTests=false -pl spark-gremlin
{noformat}




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


[jira] [Updated] (TINKERPOP-1318) java.lang.NoSuchMethodError: org/hamcrest/Matcher.describeMismatch

2016-06-08 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1318:

Fix Version/s: 3.2.1
   3.1.3

> java.lang.NoSuchMethodError: org/hamcrest/Matcher.describeMismatch
> --
>
> Key: TINKERPOP-1318
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1318
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.2.0-incubating, 3.1.2-incubating
>Reporter: Jason Plurad
>    Assignee: Jason Plurad
>Priority: Minor
> Fix For: 3.1.3, 3.2.1
>
>
> I don't recall specifically how to make this fail with {{gremlin-test}}, but 
> I did run into it at one point when writing a graph implementation. This blog 
> describes the issue and workaround. 
> https://tedvinke.wordpress.com/2013/12/17/mixing-junit-hamcrest-and-mockito-explaining-nosuchmethoderror/
> The error trace looks like this:
> {noformat}
> java.lang.NoSuchMethodError: 
> org.hamcrest.Matcher.describeMismatch(Ljava/lang/Object;Lorg/hamcrest/Description;)V
> {noformat}
> There is a dependency conflict created by an older version of {{hamcrest}} 
> coming out of {{mockito-all}}. The fix is to use {{mockito-core}} instead.
> I'll submit a patch for this.



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


[jira] [Updated] (TINKERPOP-1317) IoGraphTest throws error: URI is not hierarchical

2016-06-08 Thread Jason Plurad (JIRA)

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

Jason Plurad updated TINKERPOP-1317:

Fix Version/s: 3.2.1
   3.1.3

> IoGraphTest throws error: URI is not hierarchical
> -
>
> Key: TINKERPOP-1317
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1317
> Project: TinkerPop
>  Issue Type: Bug
>  Components: test-suite
>Affects Versions: 3.2.0-incubating, 3.1.2-incubating
>Reporter: Jason Plurad
>    Assignee: Jason Plurad
>Priority: Minor
> Fix For: 3.1.3, 3.2.1
>
>
> Via https://groups.google.com/d/msg/gremlin-users/aap3pxZtGyU/t-eOC6ZyAAAJ
> These methods will fail for graph providers that are trying to implement the 
> test suite:
> * {{IoGraphTest.shouldReadWriteModernToFileWithHelpers()}}
> * {{IoGraphTest.shouldReadWriteClassicToFileWithHelpers()}}
> Example stack trace:
> {noformat}
> shouldReadWriteModernToFileWithHelpers[gryo](org.apache.tinkerpop.gremlin.structure.io.IoGraphTest)
>   Time elapsed: 0.004 sec  <<< ERROR!
> java.lang.RuntimeException: Unable to computePath for class 
> org.apache.tinkerpop.gremlin.structure.io.IoGraphTest
>   at 
> org.apache.tinkerpop.gremlin.TestHelper.computePath(TestHelper.java:117)
>   at 
> org.apache.tinkerpop.gremlin.TestHelper.getRootOfBuildDirectory(TestHelper.java:105)
>   at 
> org.apache.tinkerpop.gremlin.TestHelper.makeTestDataPath(TestHelper.java:70)
>   at 
> org.apache.tinkerpop.gremlin.TestHelper.generateTempFile(TestHelper.java:127)
>   at 
> org.apache.tinkerpop.gremlin.structure.io.IoGraphTest.shouldReadWriteModernToFileWithHelpers(IoGraphTest.java:164)
>   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:497)
>   at 
> org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
>   at 
> org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
>   at 
> org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
>   at 
> org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
>   at 
> org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
>   at 
> org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
>   at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
>   at org.junit.rules.RunRules.evaluate(RunRules.java:20)
>   at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
>   at 
> org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
>   at 
> org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at org.junit.runners.Suite.runChild(Suite.java:128)
>   at org.junit.runners.Suite.runChild(Suite.java:27)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
>   at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
>   at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
>   at org.junit.runners.Suite.runChild(Suite.java:128)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
>   at 
> org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
>   at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
>   at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
>   at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
>   at org.junit.runners.ParentRunner.access$000

[jira] [Created] (TINKERPOP-1318) java.lang.NoSuchMethodError: org/hamcrest/Matcher.describeMismatch

2016-06-01 Thread Jason Plurad (JIRA)
Jason Plurad created TINKERPOP-1318:
---

 Summary: java.lang.NoSuchMethodError: 
org/hamcrest/Matcher.describeMismatch
 Key: TINKERPOP-1318
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1318
 Project: TinkerPop
  Issue Type: Bug
  Components: test-suite
Affects Versions: 3.1.2-incubating, 3.2.0-incubating
Reporter: Jason Plurad
Assignee: Jason Plurad
Priority: Minor


I don't recall specifically how to make this fail with {{gremlin-test}}, but I 
did run into it at one point when writing a graph implementation. This blog 
describes the issue and workaround. 
https://tedvinke.wordpress.com/2013/12/17/mixing-junit-hamcrest-and-mockito-explaining-nosuchmethoderror/

The error trace looks like this:
{noformat}
java.lang.NoSuchMethodError: 
org.hamcrest.Matcher.describeMismatch(Ljava/lang/Object;Lorg/hamcrest/Description;)V
{noformat}

There is a dependency conflict created by an older version of {{hamcrest}} 
coming out of {{mockito-all}}. The fix is to use {{mockito-core}} instead.

I'll submit a patch for this.



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


[jira] [Created] (TINKERPOP-1317) IoGraphTest throws error: URI is not hierarchical

2016-06-01 Thread Jason Plurad (JIRA)
Jason Plurad created TINKERPOP-1317:
---

 Summary: IoGraphTest throws error: URI is not hierarchical
 Key: TINKERPOP-1317
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1317
 Project: TinkerPop
  Issue Type: Bug
  Components: test-suite
Affects Versions: 3.1.2-incubating, 3.2.0-incubating
Reporter: Jason Plurad
Assignee: Jason Plurad
Priority: Minor


Via https://groups.google.com/d/msg/gremlin-users/aap3pxZtGyU/t-eOC6ZyAAAJ

These methods will fail for graph providers that are trying to implement the 
test suite:
* {{IoGraphTest.shouldReadWriteModernToFileWithHelpers()}}
* {{IoGraphTest.shouldReadWriteClassicToFileWithHelpers()}}

Example stack trace:
{noformat}
shouldReadWriteModernToFileWithHelpers[gryo](org.apache.tinkerpop.gremlin.structure.io.IoGraphTest)
  Time elapsed: 0.004 sec  <<< ERROR!
java.lang.RuntimeException: Unable to computePath for class 
org.apache.tinkerpop.gremlin.structure.io.IoGraphTest
at 
org.apache.tinkerpop.gremlin.TestHelper.computePath(TestHelper.java:117)
at 
org.apache.tinkerpop.gremlin.TestHelper.getRootOfBuildDirectory(TestHelper.java:105)
at 
org.apache.tinkerpop.gremlin.TestHelper.makeTestDataPath(TestHelper.java:70)
at 
org.apache.tinkerpop.gremlin.TestHelper.generateTempFile(TestHelper.java:127)
at 
org.apache.tinkerpop.gremlin.structure.io.IoGraphTest.shouldReadWriteModernToFileWithHelpers(IoGraphTest.java:164)
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:497)
at 
org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50)
at 
org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
at 
org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47)
at 
org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
at 
org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26)
at 
org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27)
at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:55)
at org.junit.rules.RunRules.evaluate(RunRules.java:20)
at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325)
at 
org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78)
at 
org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at org.junit.runners.Suite.runChild(Suite.java:128)
at org.junit.runners.Suite.runChild(Suite.java:27)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at org.junit.runners.Suite.runChild(Suite.java:128)
at 
org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:212)
at 
org.apache.tinkerpop.gremlin.AbstractGremlinSuite.runChild(AbstractGremlinSuite.java:50)
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290)
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71)
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288)
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58)
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268)
at org.junit.runners.ParentRunner.run(ParentRunner.java:363)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.execute(JUnit4Provider.java:252)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.executeTestSet(JUnit4Provider.java:141)
at 
org.apache.maven.surefire.junit4.JUnit4Provider.invoke(JUnit4Provider.java:112)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImp

[jira] [Created] (TINKERPOP-1319) several FeatureRequirement annotations are incorrect in gremlin-test

2016-06-01 Thread Jason Plurad (JIRA)
Jason Plurad created TINKERPOP-1319:
---

 Summary: several FeatureRequirement annotations are incorrect in 
gremlin-test
 Key: TINKERPOP-1319
 URL: https://issues.apache.org/jira/browse/TINKERPOP-1319
 Project: TinkerPop
  Issue Type: Bug
  Components: test-suite
Affects Versions: 3.1.2-incubating, 3.2.0-incubating
Reporter: Jason Plurad
Assignee: Jason Plurad
Priority: Minor


Several {{@FeatureRequirement}} annotations are incorrect in these 
{{gremlin-test}} tests
* EdgeTest.java
* FeatureSupportTest.java
* GraphTest.java
* PropertyTest.java
* VertexPropertyTest.java
* VertexTest.java

I'll submit a patch for this.



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