GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/679
TINKERPOP-1679: Detached side-effects aren't attached when remoted
https://issues.apache.org/jira/browse/TINKERPOP-1679
This was occurring because Gremlin-Python did not have serial
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/677
Yes -- tested again to be safe.
```
[INFO]
[INFO] BUILD SUCCESS
[INFO
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/675
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/678
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 the
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/677
TINKERPOP-1729: Remove deprecated select steps.
https://issues.apache.org/jira/browse/TINKERPOP-1729
There is no point is crying over spilled milk. The deed is done. It had to
be done
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/656
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/662
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/676
Ah solid find. 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 okram closed the pull request at:
https://github.com/apache/tinkerpop/pull/664
---
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
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/664
This was merged. For some reason the issue tracker didn't catch it.
---
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 okram opened a pull request:
https://github.com/apache/tinkerpop/pull/673
TINKERPOP-1709: Add a list of all the steps that support
by()/from()/to()/as()/option()
https://issues.apache.org/jira/browse/TINKERPOP-1709
Listed and referenced (linked to) those steps
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/658
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/672
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 the
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/668
TINKERPOP-1720: Remove deprecated Hadoop code
https://issues.apache.org/jira/browse/TINKERPOP-1720
Removed Constants and VertexComputing method. Created another ticket for
the
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/667
TINKERPOP-1721: Remove deprecated Bindings related code
https://issues.apache.org/jira/browse/TINKERPOP-1721
The deed has been done. Mam, these bindings won't be bothering you any
l
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/665
No, `ConfigurationTraversal` was replaced by `PureTraversal` and the
documentation uses `PureTraversal`.
```
this.traversal = PureTraversal.loadState(configuration
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/666
TINKERPOP-1719: Remove deprecated Traversal related code
https://issues.apache.org/jira/browse/TINKERPOP-1719
Removed deprecated `finalization.LazyBarrierStrategy` as well as some
methods
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/665
TINKERPOP-1170: Remove deprecated ConfigurationTraversal.
https://issues.apache.org/jira/browse/TINKERPOP-1170
Removed `ConfigurationTraversal` which really should have never been used
by
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/664
TINKERPOP-834: Remove deprecated sack() method
https://issues.apache.org/jira/browse/TINKERPOP-834
It has been done.
VOTE +1
You can merge this pull request into a Git repository
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/657
I haven't done any of that for any of the deprecation tickets. Once these
are all merged, I will do a CTR of everything since then.
---
If your project is set up for it, you can reply to
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/661
TINKERPOP-880: Remove deprecated GroupStepV3d0 and GroupSideEffectStepV3d0
https://issues.apache.org/jira/browse/TINKERPOP-880
`groupV3d0()` was created for backward compatibility prior to
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/660
TINKERPOP-1142: Remove deprecated valueIncr, valueDecr, keyIncr, keyDecr.
https://issues.apache.org/jira/browse/TINKERPOP-1142
Removed deprecated `valueIncr`, `valueDecr`, `keyIncr`, and
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/659
TINKERPOP-1291: Remove deprecated mapValues and mapKeys methods
https://issues.apache.org/jira/browse/TINKERPOP-1291
Removed `GraphTraversal.mapValues()` and `GraphTraversal.mapKeys
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/618
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/655
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/645
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 the
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/657
TINKERPOP-832: Remove deprecated addV/E/InE/OutE methods
https://issues.apache.org/jira/browse/TINKERPOP-832
Removed deprecated `addV()`, `addE()`, `addInE()`, and `addOutE()` methods
Github user okram commented on the pull request:
https://github.com/apache/tinkerpop/commit/f68b36c44ee7febfa2b2698889c977b11a3e8f49#commitcomment-22955008
Its also going to need to be registered with Gremlin-Python. However, lets
wait to do that as we are about to push a major
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/621
I just noticed that this strategy is not registered with the GraphSON and
Gryo serializers accordingly. Thus, back to VOTE 0.
---
If your project is set up for it, you can reply to this email and
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/621
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/647
Fixed both of @dkuppitz's recommended tweaks.
---
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 okram commented on the issue:
https://github.com/apache/tinkerpop/pull/646
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/651
Travis is complaining cause of some 'time traversal' test in Gremlin-Groovy
test suite. Not related to this PR.
---
If your project is set up for it, you can reply to this email and
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/651
TINKERPOP-1169: Remove deprecated TraversalScriptFunction and
TraversalScriptHelper
https://issues.apache.org/jira/browse/TINKERPOP-1169
Removed deprecated `TraversalScriptFunction` and
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/650
TINKERPOP-1171: Remove deprecated TraversalSource.Builder
https://issues.apache.org/jira/browse/TINKERPOP-1171
Removed deprecated `TraversalSource.Builder`.
VOTE +1.
You can
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/649
TINKERPOP-1289: Remove deprecated ConnectiveP, AndP, and OrP constructors.
https://issues.apache.org/jira/browse/TINKERPOP-1289
Removed deprecated constructors. (for TinkerPop 3.3.0
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/648
TINKERPOP-1710: Add a note on tree() by-modulation and uniqueness of tree
branches.
https://issues.apache.org/jira/browse/TINKERPOP-1710
There was confusion as to what should happen what
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/647
TINKERPOP-1708: Add a "Note on Scopes" document
https://issues.apache.org/jira/browse/TINKERPOP-1708
Added a `Note on Scopes` section to the `the-traversal.asciidoc`.
VO
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/641
Merged. I think that its not being auto-closed cause I changed the PR title.
---
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 okram closed the pull request at:
https://github.com/apache/tinkerpop/pull/641
---
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
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/641
Done.
---
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
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/643
TINKERPOP-1701: HaltedTraverserStrategy should recurse into collections for
detachment.
https://issues.apache.org/jira/browse/TINKERPOP-1701
`HaltedTraverserStrategy` is used by
Github user okram 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 the
Github user okram 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 the
Github user okram 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 the
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/642
TINKERPOP-1704: XXXTranslators are not being respective of BulkSet and Tree.
https://issues.apache.org/jira/browse/TINKERPOP-1704
`JavaTranslator` would convert `Tree` to `HashMap` and
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/641
TINKERPOP-1703: Make DsegEdgeOtherVertexStep non-final
https://issues.apache.org/jira/browse/TINKERPOP-1703
The ticket really is about making {{EdgeOtherVertexStep}} non-final so
Github user okram closed the pull request at:
https://github.com/apache/tinkerpop/pull/639
---
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
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/632
The problem is real, the fix works given the specific test case.
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 okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r123753801
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/GroupStep.java
---
@@ -79,6 +79,14 @@ public void modulateBy
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/631
VOTE +1. (However, I wouldn't add the between the last line and the
`...` but whateves)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r123507142
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalRing.java
---
@@ -62,6 +62,10 @@ public void
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r123506884
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/GroupStep.java
---
@@ -79,6 +79,14 @@ public void modulateBy
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/639
TINKERPOP-1669: EdgeVertexStep should be designed for extension
https://issues.apache.org/jira/browse/TINKERPOP-1669
`EdgeVertexStep` is no longer final and can be extended by providers
Github user okram 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 the
Github user okram 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 the
Github user okram 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 the
Github user okram 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 the
Github user okram 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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/638
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 the
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r121701740
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/GroupStep.java
---
@@ -79,6 +79,14 @@ public void modulateBy
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r121703249
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/OrderGlobalStep.java
---
@@ -99,6 +99,19 @@ public void
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r121704449
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/TraversalStrategies.java
---
@@ -215,7 +216,8 @@ static void visit(Map
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r121702575
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/TraversalRing.java
---
@@ -62,6 +62,10 @@ public void
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r121703823
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/GroupSideEffectStep.java
---
@@ -81,6 +81,14 @@ public
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r121704275
--- Diff: CHANGELOG.asciidoc ---
@@ -25,7 +25,7 @@
image::https://raw.githubusercontent.com/apache/tinkerpop/master/docs/static/ima
[[release-3-3-0
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/621#discussion_r121702035
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/map/GroupStepV3d0.java
---
@@ -111,6 +111,16 @@ public void
Github user okram 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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/613
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/617
Seems right to me. Just to be certain `x.y.z` are ANDs and before, for
`hasLabel()` and `hasId()` they were being OR'd. Test cases make sense.
VOTE +1.
---
If your project is set u
Github user okram 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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/606
Please see https://github.com/apache/tinkerpop/pull/615 . Here is what I
did:
1. Created a JIRA first.
2. Fixed the problem.
3. Created a test case that fails with the problem, but
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/615
TINKERPOP-1678: until(Predicate) is actually calling emit(Predicate)
https://issues.apache.org/jira/browse/TINKERPOP-1678
Fixed a typo-bug in `GraphTraversal` where `until(Predicate)` was
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/609
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/610
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/606
It must effect semantics as `until` is different from `emit`. ?
---
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 okram commented on the issue:
https://github.com/apache/tinkerpop/pull/608
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 wishes so
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/606
You need splits so the traverser remains "local" to the child branch. I
think your fix for this particular problem is correct, it would be great if you
could also add a test case to `
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/607
VOTE +1. Forgot to vote on my own PR.
---
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 okram opened a pull request:
https://github.com/apache/tinkerpop/pull/607
TINKERPOP-1673: GroovyTranslator produces Gremlin that can't execute on
:remote
https://issues.apache.org/jira/browse/TINKERPOP-1673
`GroovyTranslator` now uses Groovy `[...]
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/606
Whoa!? That is a huge problem. Can you do three things for this please?
1. Can you write a test case in `RepeatTest` that shows it breaking, but
with your fix, it working.
2. Can you
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/594
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 the
Github user okram 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 the
Github user okram 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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/601
Yes, and I decided not to include that given that its rare and the only way
to test would be to look for a `LambdaHolder` which would then make the
strategy not apply to lots of other situations. I
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/601
TINKERPOP-1668: RepeatUnrollStrategy should not execute if there is a
LoopStep used.
https://issues.apache.org/jira/browse/TINKERPOP-1668
If the child traversal of `RepeatStep` has a
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/595
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 the
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/590#discussion_r110167099
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/util/ElementHelper.java
---
@@ -116,6 +116,10 @@ public static void
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/590#discussion_r109411238
--- Diff: CHANGELOG.asciidoc ---
@@ -64,6 +64,7 @@ TinkerPop 3.2.5 (Release Date: NOT OFFICIALLY RELEASED
YET)
* Improved error handling of
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/590
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/587
Fixed the bugs that both @dkuppitz and @twilmes discovered.
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 okram commented on the issue:
https://github.com/apache/tinkerpop/pull/587
Note that `tp32/` doesn't have the issue. Thus, the bug is confined to this
branch (luckily).
---
If your project is set up for it, you can reply to this email and have your
reply appear on G
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/584
Solid work. 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 okram commented on the issue:
https://github.com/apache/tinkerpop/pull/572
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 the
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/574
@dkuppitz --- Do you feel like adding your "playlist" example with text to
`the-traversal.asciidoc` in the `SimplePath Step` section?
---
If your project is set up for it, you can rep
Github user okram commented on the issue:
https://github.com/apache/tinkerpop/pull/574
@dkuppitz -- okay? ... so.
---
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 okram commented on the issue:
https://github.com/apache/tinkerpop/pull/578
```
[INFO]
[INFO] BUILD SUCCESS
[INFO
GitHub user okram opened a pull request:
https://github.com/apache/tinkerpop/pull/578
TINKERPOP-1625: Improvements to has() related steps
https://issues.apache.org/jira/browse/TINKERPOP-1625
Added tests for `hasKey()` and `hasValue()`. Also tweaked up `HasTest`
which had
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/574#discussion_r106495135
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/filter/CyclicPathStep.java
---
@@ -1,46 +0,0
Github user okram commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/574#discussion_r106494779
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/dsl/graph/GraphTraversal.java
---
@@ -1034,11 +1034,10
101 - 200 of 587 matches
Mail list logo