Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/650
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/644
I guess `dotnet-dev-1.0.1` could be bumped to `dotnet-dev-1.0.4` (that's
currently the latest).
What is `apt-transport-http` needed for?
---
If your project is set up for it, yo
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/643
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/642
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/641
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/633
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/634
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/640
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/639
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/632
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, or if
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r123745535
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/GroupStep.java
---
@@ -79,6 +79,14 @@ public void
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/628
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, or if
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/628#discussion_r123490358
--- Diff: pom.xml ---
@@ -597,6 +597,7 @@ limitations under the License.
com.codahale.metrics
metrics-graphite
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/628#discussion_r123485417
--- Diff: pom.xml ---
@@ -597,6 +597,7 @@ limitations under the License.
com.codahale.metrics
metrics-graphite
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/628#discussion_r123484932
--- Diff: gremlin-server/pom.xml ---
@@ -63,10 +63,12 @@ limitations under the License.
com.codahale.metrics
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/624
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/635
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, or if
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/638#discussion_r123119092
--- Diff:
gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/GremlinServer.java
---
@@ -108,54 +108,24 @@ public GremlinServer(final
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/637
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/636
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/621
Added a few comments and worked on those points that I haven't commented. I
will push in few minutes after getting a clean build.
---
If your project is set up for it, you can reply to
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r122694055
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/GroupStepV3d0.java
---
@@ -111,6 +111,16 @@ public void
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r122736897
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalRing.java
---
@@ -62,6 +62,10 @@ public void
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r122693554
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/GroupStep.java
---
@@ -79,6 +79,14 @@ public void
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/631
VOTE: +1 (and an extra +1 for finding a super long name that actually
exists :))
---
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 opened a pull request:
https://github.com/apache/tinkerpop/pull/621
TINKERPOP-1682 by-modulator optimization strategy
https://issues.apache.org/jira/browse/TINKERPOP-1682
Implemented `ByModulatorOptimizationStrategy` which replaces certain
standard
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/617
TINKERPOP-1681 Multiple hasId's are or'd into GraphStep
https://issues.apache.org/jira/browse/TINKERPOP-1681
Fixed folding of multiple `hasId()`'s into `GraphStep`.
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/613
Not sure if this should really go into `tp32` as it's kinda breaking
(though it breaks something that was broken :)), but if others agree, then
VOTE: +1
---
If your project is s
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/614
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/612
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/616
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/608
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/610
Slick! It's indeed much easier than I thought.
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 pr
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/607
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/604
LGTM.
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/599
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/598
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/601
Cool.
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
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/602
TINKERPOP-1313 Rename RangeByIsCountStrategy
https://issues.apache.org/jira/browse/TINKERPOP-1313
Renamed `RangeByIsCountStrategy` to `CountStrategy`.
`docker/build.sh -t -i -n
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/601
Very rare indeed. Maybe we should just add a tiny note (and/or an example)
in the repeat steps docs.
```
g.V().repeat(out().sideEffect {println it.loops()}).iterate() // don'
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/601
Lambda steps could do something with `it.loops()` and thus should have the
same effect.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/569
Feel free to merge this PR @pluradj. The @spmallette bot is currently being
rebooted / is on vacation ;).
---
If your project is set up for it, you can reply to this email and have your
reply
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/595#discussion_r110655969
--- Diff: gremlin-core/src/main/resources/tp2-to-tp3-graphml.xslt ---
@@ -0,0 +1,59 @@
+
+
+
+
+
+http://graphml.graphdrawing.org
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/595#discussion_r110650888
--- Diff: gremlin-core/src/main/resources/tp2-to-tp3-graphml.xslt ---
@@ -0,0 +1,59 @@
+
+
+
+
+
+http://graphml.graphdrawing.org
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/595
Would be nice if the transformation would work with standard tools like
`xsltproc`.`
```
daniel@cube /projects/apache/tinkerpop (TINKERPOP-1608) TP2-to-TP3 GraphML
XSLT $ xsltproc
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/593
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/583
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/569
Well done.
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/590
VOTE: +1
And just like the Java compiler I really don't care whether it's `null ==
propertyKeyValues[i+1]` or `propertyKeyValues[i+1] == null`.
---
If your project is set
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/596
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/588
I recently got faster internet, so -1 on this.
Just kidding :). Looks good to me and actually doesn't change anything in
the "normal" workflow.
VOTE: +1
---
If
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/587
`docker/build.sh -t -n -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 project does not
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/589
`implementations-giraph` currently fails for me, but that happens
frequently on my system (and has nothing to do with this PR). `the-traversal`
looks good.
VOTE: +1
---
If your
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/589
I know, that's annoying and I think we mainly did it like this, because the
local directory structure is different from the one that we publish. Might be a
good idea to have a ticket for th
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/589
Hmm, never seen it before, but since it's security related, Docker probably
didn't get enough permissions..? Does the user, that is used to run the Docker
daemon, have root privileges
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/587
`docker/build.sh -t -n -i` failed.
Got an NPE in `SparkGraphComputerGroovyProcessIntegrateTest`.
---
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/578
`docker/build.sh -t -n -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 project does not
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/572
Ok, just tested, the TODO is still valid.
---
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/578
Is it time to rebase this branch? Integration test just failed for me
(`shouldDetachVertexWhenRemoved` in `Neo4j Gremlin`).
---
If your project is set up for it, you can reply to this email and
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/572
Sorry, I currently have integration tests running on another branch and
thus can't test it, but the changes in this PR also fixed this `TODO`:
https://github.com/apache/tinkerpop/blob/TINK
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/574
It's all working as expected 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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/574
I'm going to add a different example. Starting with an empty graph:
```
g = TinkerGraph.open().traversal()
g.addV().property(id, "A").as("a").
addV(
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/573
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/574
```
g.V().as("a").emit().repeat(both().as("b").simplePath().from("a").to("b")).path()
```
...emits all paths, including cyclic paths. Without
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/572
`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 project does not have
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/572#discussion_r106552697
--- Diff:
gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/optimization/PathRetractionStrategyTest.java
---
@@ -191,6
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/572#discussion_r106551449
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/optimization/PathRetractionStrategy.java
---
@@ -72,7
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/571
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/570
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, or if
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/570#discussion_r105754194
--- Diff:
gremlin-groovy/src/main/java/org/apache/tinkerpop/gremlin/groovy/jsr223/GremlinGroovyScriptEngine.java
---
@@ -149,19 +159,64
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/567
This part:
```
if (clazz != null) {
return clazz.get();
}
clazz = CompletableFuture.supplyAsync(() -> loader.parseClass(script,
generateScriptN
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/567
Hmm, I haven't done any benchmarks, but this seem to be valid points. Just
a thought: Would it help to have the class map be like
`ManagedConcurrentValueMap> classMap`?
---
If your pr
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/567
Yep, missing some spaces here and there, but the code in general looks good.
VOTE: +1
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/566
`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 project does not have
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/560
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, or if
Github user dkuppitz commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/560#discussion_r103189226
--- Diff:
gremlin-core/src/test/java/org/apache/tinkerpop/gremlin/process/traversal/strategy/optimization/FilterRankingStrategyTest.java
---
@@ -106,7
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/534
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/564
It's true that this was just "would be cool" PR, that would have helped me
in the short term. The general solution to this problem is proposed in
https://issues.apache.org/jira/b
Github user dkuppitz closed the pull request at:
https://github.com/apache/tinkerpop/pull/564
---
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/564
TINKERPOP-1639 Support simple String operations
https://issues.apache.org/jira/browse/TINKERPOP-1639
Added support for simple String operations. `CountLocalStep`,
`RangeLocalStep` and
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/563
TINKERPOP-1638 count() is optimized away in where()
https://issues.apache.org/jira/browse/TINKERPOP-1638
Fixed a bug in `RangeByIsCountStrategy` that changed the meaning of inner
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/562
`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 project does not have
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/558
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/559
`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 project does not have
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/555
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/548
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/549
`docker/build.sh -t -i -n` succeeded. Manual tests were showing epic
performance results. Best PR in a long time.
VOTE: +1
---
If your project is set up for it, you can reply to this
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/541
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/546
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/546
This also fixes
[TINKERPOP-1597](https://issues.apache.org/jira/browse/TINKERPOP-1597), but the
test I've added can only be included in the 3.3.x line, right?
---
If your project is set u
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/543
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/542
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, or if
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/537
Done. We already use `validate-distribution.sh` to test script evaluations,
so it was easy to another test.
https://github.com/apache/tinkerpop/commit
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/537
Oh, that's a good idea! I will add that.
---
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/537
Same here. I can't think of a way to test `gremlin.sh`. The script requires
that the project was built, but tests can be run without building the project
upfront. Going to merge this PR fo
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/537
I don't know. The fix relies on a change in `gremlin.sh`. Is that testable?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/538
Code looks good. Minor improvements were discussed in Slack.
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
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/539
`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 project does not have
GitHub user dkuppitz opened a pull request:
https://github.com/apache/tinkerpop/pull/537
TINKERPOP-1605 gremlin-console 3.2.3 -e can no longer take paths relative
to current working directory
https://issues.apache.org/jira/browse/TINKERPOP-1605
```
daniel@cube /tmp
Github user dkuppitz commented on the issue:
https://github.com/apache/tinkerpop/pull/535
So do you suggest a) to remove the assertions that check the metrics for
specific steps or b) to find the indexes of all steps that were present before
strategies were applied?
---
If your
301 - 400 of 662 matches
Mail list logo