[
https://issues.apache.org/jira/browse/TINKERPOP-2972?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17763168#comment-17763168
]
ASF GitHub Bot commented on TINKERPOP-2972:
-------------------------------------------
Cole-Greer commented on code in PR #2217:
URL: https://github.com/apache/tinkerpop/pull/2217#discussion_r1320099551
##########
gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/test/features/map/Project.feature:
##########
@@ -106,4 +106,13 @@ Feature: Step - project()
| m[{"a":"d[0].l", "b":"d[29].i"}] |
| m[{"a":"d[1].l", "b":"d[27].i"}] |
| m[{"a":"d[1].l", "b":"d[32].i"}] |
- | m[{"a":"d[0].l", "b":"d[35].i"}] |
\ No newline at end of file
+ | m[{"a":"d[0].l", "b":"d[35].i"}] |
+
+
+ Scenario: g_V_projectXa_aX
Review Comment:
Sorry I missed that this scenario is incompatible with the feature tests.
VOTE +1 (with failing feature test removed)
> ProjectStep will produce an unexpected output when dealing with duplicate keys
> ------------------------------------------------------------------------------
>
> Key: TINKERPOP-2972
> URL: https://issues.apache.org/jira/browse/TINKERPOP-2972
> Project: TinkerPop
> Issue Type: Improvement
> Components: process
> Affects Versions: 3.6.4
> Reporter: Jianzhong Liu
> Priority: Major
>
> g.V().project("a", "b", "b").by("lang").by(T.id).by("age")
> TP:
> 1 \{'b': 29}
> 2 \{'b': 27}
> 3 \{'a': 'java', 'b': '3'}
> 4 \{'b': 32}
> 5 \{'a': 'java', 'b': '5'}
> 6 \{'b': 35}
> TinkerPop erroneously generates an output from the final 'productive'
> by-traversal when a key is used more than once, a situation that should not
> occur. A proposed solution is to throw an exception when a redundant key is
> detected.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
