Ken Hu created TINKERPOP-3216:
---------------------------------
Summary: Throw error for by modulators that have undefined behavior
Key: TINKERPOP-3216
URL: https://issues.apache.org/jira/browse/TINKERPOP-3216
Project: TinkerPop
Issue Type: Improvement
Components: process
Affects Versions: 4.0.0, 3.8.0
Reporter: Ken Hu
Going through the current list of steps that are ByModulating, some of them
shouldn't allow multiple by()s.
||Step Name||Behavior||Correctness||
|DedupGlobal|Single only already|yes|
|PathFilter|Requires multiple|yes|
|SampleGlobal|Single only already|yes|
|WherePredicate|*Allows multiple -> should only allow one*|no|
|Format|Requires multiple|yes|
|GroupCount|Single only already|yes|
|GroupStep|Double only already|yes|
|Math|Requires multiple|yes|
|OrderGlobal|*Allows same modulator more than once -> only allow unique
modulators*|no|
|OrderLocal|*Allows same modulator more than once* {*}-> only allow unique
modulators{*}{*}{*}|no|
|Path|Requires multiple|yes|
|Project|Requires multiple|yes|
|PropertyMap|Single only already|yes|
|SelectOne|*Uses last modulator ->* {*}should only allow one{*}{*}{*}|no|
|Select|Requires multiple|yes|
|TraversalSelect|*Uses last modulator* *->* {*}should only allow
one{*}{*}{*}{*}{*}|no|
|Tree|Requires multiple|yes|
|Aggregate|Single only already|yes|
|GroupCountSideEffect|Single only already|yes|
|GroupSideEffect|Double only already|yes|
|SackValue|Single only already|yes|
|TreeSideEffect|Requires multiple|yes|
These should get changed in 5.x so that exceptions are thrown in these cases.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
