[
https://issues.apache.org/jira/browse/TINKERPOP-3225?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18072125#comment-18072125
]
ASF GitHub Bot commented on TINKERPOP-3225:
-------------------------------------------
xiazcy commented on code in PR #3372:
URL: https://github.com/apache/tinkerpop/pull/3372#discussion_r3054683039
##########
gremlin-test/src/main/resources/org/apache/tinkerpop/gremlin/test/features/map/Conjoin.feature:
##########
@@ -153,3 +153,29 @@ Feature: Step - conjoin()
Then the result should be unordered
| result |
| 3;three |
+
+ @GraphComputerVerificationInjectionNotSupported
+ Scenario: g_injectXnull_a_null_bX_fold_conjoinXplusX
Review Comment:
```suggestion
Scenario: g_injectXnull_a_null_bX_conjoinXplusX
```
> conjoin has incorrect null handling
> -----------------------------------
>
> Key: TINKERPOP-3225
> URL: https://issues.apache.org/jira/browse/TINKERPOP-3225
> Project: TinkerPop
> Issue Type: Bug
> Components: process
> Affects Versions: 3.8.0, 3.7.5
> Reporter: Ken Hu
> Priority: Major
>
> The conjoin() step doesn't behave as stated in the semantics documentation.
> Namely, the "Null values are ignored" isn't being followed.
> {code:java}
> gremlin> g.inject([null]).conjoin("-")
> ==>null
> gremlin> g.inject([null, null]).conjoin("-")
> ==>null {code}
> show that the nulls are being passed through similar to concat() but that
> isn't correct according to the semantics.
> Instead it should be the following:
> {code:java}
> gremlin> g.inject([]).conjoin("+")
> ==>
> gremlin> g.inject([null]).conjoin("+")
> ==>
> gremlin> g.inject([null, null]).conjoin("+")
> ==>{code}
> where similar to empty array, the result is an empty string.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
