Stephen Mallette created TINKERPOP-3246:
-------------------------------------------
Summary: Prevent multiple assignments to key/value on
AddPropertyStep
Key: TINKERPOP-3246
URL: https://issues.apache.org/jira/browse/TINKERPOP-3246
Project: TinkerPop
Issue Type: Improvement
Components: process
Affects Versions: 3.8.1
Reporter: Stephen Mallette
Assignee: Cole Greer
AddPropertyStep uses an internal Parameters object to store the property key
and value. The Parameters class in TinkerPop is designed to support multi-maps
multiple values per key.
1. Initial State: When AddPropertyStep is constructed, it sets T.key and
T.value in the Parameters object.
2. Adding More: If something calls addProperty(T.key, "newKey") or
addProperty(T.value, "newValue") on the step, Parameters simply appends these
new
values to the internal list for those keys.
3. getKey() and getValue() just pop off the first item.
seems unlikely this would happen given GraphTraversal, but i think its worth a
guard against this just to be sure.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
