Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/498
Wow, can't wait to try that out. @spmallette I bet this will also solve
your `mkdirs` issues.
If it's really as good as you say and if it doesn't have any downsides, we
should make
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/489
`docker/build.sh -t -i -n` kinda succeeded (it got stuck in Giraph tests,
but I guess it wasn't rebased after this issue had been fixed).
VOTE: +1
---
If your project is set up
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/439#discussion_r88381456
--- Diff: docs/src/upgrade/release-3.3.x.asciidoc ---
@@ -31,3 +31,20 @@ Please see the
link:https://github.com/apache/tinkerpop/blob/3.3.3/CHANGELOG.asc
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/439#discussion_r88381587
--- Diff: gremlin-server/src/main/bin/gremlin-server.conf ---
@@ -0,0 +1,42 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/439#discussion_r88381061
--- Diff: docs/src/reference/gremlin-applications.asciidoc ---
@@ -1482,6 +1513,62 @@ and standard deviation evaluation times, as well as
the 75th, 95th
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/439#discussion_r88381003
--- Diff: docs/src/reference/gremlin-applications.asciidoc ---
@@ -1482,6 +1513,62 @@ and standard deviation evaluation times, as well as
the 75th, 95th
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/439#discussion_r88381583
--- Diff: gremlin-server/src/main/bin/gremlin-server.conf ---
@@ -0,0 +1,42 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/439#discussion_r88381095
--- Diff: docs/src/reference/gremlin-applications.asciidoc ---
@@ -1482,6 +1513,62 @@ and standard deviation evaluation times, as well as
the 75th, 95th
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/496
Okay, cool, then...
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/496
Isn't the `DOCKER_OPTS` environment variable used by default? Or is this
only true for the daemon?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/495
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/494
> the `ImmutablePath` method changes are internal changes
I think that's the point. It would have been an internal change if the
methods were internal, but they're public. We can ass
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/495
Since this PR touches `FilterRankingStrategyTest`s, it would be nice to
have this issue fixed as well (and have some tests added to cover this
scenario):
```
gremlin>
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/495
Yea, looks better now. Not sure if we should really keep all the steps in
`g.V(1).outE('knows').hasLabel('created').more().bla()`. It's like the Java
compiler would keep stuff within
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/495
Alright, looks like I pulled too early, just got some more changes coming
through. I will retest the latest.
---
If your project is set up for it, you can reply to this email and have your
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/495
```
gremlin> g.V(1).outE("knows")
==>e[7][1-knows->2]
==>e[8][1-knows->4]
gremlin> g.V(1).outE("knows").hasLabel("created")
==&g
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/457
Btw. I don't think that this error is related to any of our components. The
`mkdirs` problem has been reported in Docker's issue tracker several times and
whenever they thought they solved
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/457
Hmm, that's very unlikely. Check the output of `docker images`. The `TAG`
column shows the Hadoop version.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/493
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/492
All good now.
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/485
Here's an example that shows how it produces the expected result and still
tries to use `barrier()`'s as often as possible:
```
gremlin> g = TinkerFactory.createModern().traver
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/491
Looks good, tests are not affected by other applications using port 8182.
VOTE: +1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/492
There's still an interference between `GremlinDriverIntegrateTest` and a
locally running DSE Graph instance.
```
---
T E S T
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/485
The only change I would like to do: rename "frozen steps" to "lazy steps"
as they're not completely untouchable for all other strategies, it's really
just about `b
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/487
It only has a private constructor already. Making it final is a good call
though.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/485
Well, what you're saying is "put all the logic into `LazyBarrierStrategy`".
But what about `PathRetractionStrategy`? Same rules apply there - copy the code
there as well? And if we a
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/485
Because it's more complex then that. Example:
```
g.V().as("a").out().store("a").in().sideEffect().groupCount().unfold().sideEffect()...
```
In the fi
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/488
This PR is no longer relevant, as we have a discussion going in
https://github.com/apache/tinkerpop/pull/486 (where `SimplePathStep` is no
longer a `PathProcessor`).
---
If your project is set
Github user dkuppitz closed the pull request at:
https://github.com/apache/tinkerpop/pull/488
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/486
Requirements were not cached in `tp31`. I have a neat solution for
`master/`, but not for `tp32`. In `tp32` we can either reset the requirements
after each strategy has been applied or we can
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/486
I think we have 2 options:
1. reset requirements (a simple `this.requirements = null;`) at the end of
mutating methods (`addStep` and `removeStep`)
2. don't cache requirements before
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/486
I see, that sucks. Looks like there's nothing that resets the cache at any
point. IMO the admin methods `addStep` and `removeStep` should invalidate the
cache.
---
If your project is set up
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/486
Oh, you mean `Traversal.getTraverserRequirements()`. That's useless anyways
(in this case). The strategy has to check every single step.
---
If your project is set up for it, you can reply
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/488
TINKERPOP-1545 IncidentToAdjacentStrategy is buggy (tp32)
https://issues.apache.org/jira/browse/TINKERPOP-1545
Made SimplePathStep implement PathProcessor and IncidentToAdjacentStrategy
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/486
I really don't see what you mean by "it gets cached". `getRequirements()`
as well as `getSelfAndChildRequirements()` look pretty stateless to me.
---
If your project is set up for i
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/486
> Why not use the "marker model" ?
This PR targets `tp31`.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/487
TINKERPOP-1399 NumberHelper needs to go into util and have a private
constructor
https://issues.apache.org/jira/browse/TINKERPOP-1399
Moved `NumberHelper
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/482
`docker/build.sh -t -i -n` succeeded.
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/486
Force-pushed. Filtering steps based on 2 conditions (type and traversal
requirements) was easier than I thought (thanks to
`TraversalHelper.anyStepRecursively()`).
---
If your project is set
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/486
You know what, I will revert the change and not implement `PathProcessor`
in `SimplePathStep`. Here's why: `IncidentToAdjacentStrategy` should only rely
on the `getRequirements()` result
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/486
The `PathProcessor` interface has no function for SimpleStep and doesn't
add any overhead. AFAIK `PathProcessor` is only used by strategies to easily
identify traversals with "sp
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/486
TINKERPOP-1545 IncidentToAdjacentStrategy is buggy
https://issues.apache.org/jira/browse/TINKERPOP-1545
Made `SimplePathStep` implement `PathProcessor` and
`IncidentToAdjacentStrategy
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/482
I'm cool with that change. I still have to run a `docker/build.sh -t -i -n`
though - will do so as soon as I have the resources available (unless someone
else volunteers).
---
If your project
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/483
Please don't merge this PR yet. While testing the changes in
`RangeByIsCountStrategy` I realized that we can add a minor tweak to get
another great optimization. I'll push my local changes
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/457
All good here, `docker/build.sh -t -i -n` succeeded 2 times in a row for me.
VOTE: +1
---
If your project is set up for it, you can reply to this email and have your
reply appear
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/485
TINKERPOP-1529 LazyBarrierStrategy is too aggressive (variant 2)
https://issues.apache.org/jira/browse/TINKERPOP-1529
This fixes a long-term bug that was introduced when
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/484
TINKERPOP-1529 LazyBarrierStrategy is too aggressive
https://issues.apache.org/jira/browse/TINKERPOP-1529
This fixes a long-term bug that was introduced when `LazyBarrierStrategy
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/479
Finally! 3 good builds in row, using `docker/build.sh -t -i -n`
VOTE: +10
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/481
Manual tests look good. `docker/build.sh -t -i -n` worked up to the point
where it always fails for me in `tp32`.
VOTE: +1
---
If your project is set up for it, you can reply
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/481#discussion_r86388694
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/GroupSideEffectStep.java
---
@@ -45,7 +45,7
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/479
Unfortunately I still get the same errors. Looks like they always happen in
`RemoteGraphGroovyTranslatorProcessStandardTest`.
---
If your project is set up for it, you can reply to this email
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/474#discussion_r86029705
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/decoration/SubgraphStrategy.java
---
@@ -105,21 +105,11
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/474
VOTE: +1
A little refactoring would be nice though:
```
for (final Step step : traversal.getSteps()) {
if (step instanceof TraversalParent
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/468
So weird. Build succeeded with `docker/build.sh -t -i -n` after I've
rebuilt my containers and removed the `.glv` file from my `gremlin-python`
directory.
VOTE: +1
---
If your
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/472
VOTE: +1
---
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
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/472#discussion_r85858468
--- Diff:
gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/step/branch/ChooseStepTest.java
---
@@ -38,7 +42,10
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/471
I've done some manual tests.
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/470
Interesting. I've had different results in myy benchmark and it looked more
like just a lot of noise in the measurements, since Bryn's branch was sometimes
a bit faster. There's a crazy
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/470
LGTM, thus VOTE: +1.
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/446
Oh, now I see what you're saying. Yes, this would be the right way to go.
The current `PropertyMapStep` implementation is pretty weird.
```java
protected Map<String, E&g
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/446
To support something like `valueMap(T.label, "name")`? This would be cool,
but would go into another PR.
---
If your project is set up for it, you can reply to this email and have
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/470
I've done some manual tests and all looks good to me. Although the changes
in this PR look scary, it doesn't seem to have a bad performance impact (I
couldn't see a performance impact at all
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/468
`docker/build.sh -t -i -n` fails for me with:
```
Tests run: 490, Failures: 0, Errors: 1, Skipped: 54, Time elapsed: 32.5 sec
<<<
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/446
I've seen lots of code where people actually use `id` as a property name.
```
gremlin> g = TinkerGraph.open().traversal()
==>graphtraversalsource[tinkergraph[vertices:0 e
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/461
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/463
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/462
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/458
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/465
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/467
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/455
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/447
Cool.
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/447
Code looks good, `docker/build.sh -t -i` succeeded.
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/444
The gist was that you could live without the 2 additional files, but your
life would be affected by an everlasting sadness. Hence we concluded that we
keep the symbolic links.
However
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/442
Looks good here too.
VOTE: +1
---
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
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/445#discussion_r81550568
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/optimization/InlineFilterStrategy.java
---
@@ -72,97
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/441
FYI: 3x BUILD SUCCESS in a row.
My vote still holds true.
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/444
This branch was rebased and force pushed. If you've had it checked out
before, you'll likely have a merge conflict in your local version. Try to
checkout a fresh copy:
```
git
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/441
As said in a private chat, integration tests were stuck when I ran them
first. However, the second attempt succeeded. More investigation is apparently
needed, but for this PR:
VOTE: +1
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/444
@okram We could get rid of the additional release steps if we would remove
the 2 extra `gremlin.sh` files. Any reason why you prefer those? If you're just
too lazy to type out the `target` path
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/444
```
daniel@cube /projects/apache/tinkerpop (TINKERPOP-1158) gremlin.sh -v emits
log4j initialization errors $ bin/gremlin.sh -v
gremlin 3.1.5-SNAPSHOT
daniel@cube /projects/apache
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/444#discussion_r81171446
--- Diff: gremlin-console/src/main/bin/gremlin.sh ---
@@ -111,3 +115,5 @@ fi
# Start the JVM, execute the application, and return its exit code
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/444
Sorry, I must have lost some changes when I made the last rebase. Unbound
variable errors are fixed now.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/444
I saw this error yesterday, but fixed it (after rebasing). Are you sure
you've had the latest version for your test?
---
If your project is set up for it, you can reply to this email and have
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/444#discussion_r81114584
--- Diff: bin/gremlin.sh ---
@@ -1,22 +0,0 @@
-#!/bin/bash
--- End diff --
It's no longer a file, it's a symbolic link. GitHub only makes
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/444#discussion_r81114609
--- Diff: gremlin-console/bin/gremlin.sh ---
@@ -0,0 +1 @@
+../target/apache-tinkerpop-gremlin-console-3.1.5-SNAPSHOT-standalone/bin/gremlin.sh
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/441#discussion_r81113714
--- Diff:
gremlin-server/src/test/java/org/apache/tinkerpop/gremlin/server/GremlinDriverIntegrateTest.java
---
@@ -1248,6 +1250,73 @@ public void
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/441#discussion_r81113830
--- Diff: docs/src/upgrade/release-3.1.x-incubating.asciidoc ---
@@ -27,6 +27,23 @@ TinkerPop 3.1.5
*Release Date: NOT OFFICIALLY RELEASED YET
Github user dkuppitz closed the pull request at:
https://github.com/apache/tinkerpop/pull/443
---
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
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/444
TINKERPOP-1158 gremlin.sh -v emits log4j initialization errors
https://issues.apache.org/jira/browse/TINKERPOP-1158
Tweaked `gremlin.sh` so that it can be started from anywhere
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/436
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/429
`docker/build.sh -t -i -n` succeeded.
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/424
VOTE: +1
---
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
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/428
TINKERPOP-927 bin/publish-docs.sh should only upload diffs.
https://issues.apache.org/jira/browse/TINKERPOP-927
VOTE: +1
You can merge this pull request into a Git repository
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/427
TINKERPOP-927 bin/publish-docs.sh should only upload diffs.
https://issues.apache.org/jira/browse/TINKERPOP-927
VOTE: +1
You can merge this pull request into a Git repository
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/411
All good, here's the missing
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/420
This PR has been merged. Thank you for **the the** contribution.
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/417
Regarding the `match()` translation, it's indeed very easy:
```
gremlin> g.V().as("a").outE("created").as("b").
..1> inV().as(&qu
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/399
VOTE: +1
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/406
Right, this one was cherry-picked. But now, after a reset, a merge and a
forced push, the commit hashes are still different.
---
If your project is set up for it, you can reply to this email
401 - 500 of 555 matches
Mail list logo