[TINKERPOP-2283] GraphStep's ids null exception

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.4.2
• Fix Version/s: 3.3.9, 3.4.4, 3.5.0
• Component/s: process
• Assignee: Stephen Mallette
• Reporter: Stark Arya
• Created: 15/Aug/19 00:53
• Updated: 03/Sep/19 15:34
• Resolved: 03/Sep/19 15:34

Description

Variable ids maybe null in GraphStep.java, s simple sample list here:

gremlin> g.V().hasId([]).profile()
==>Traversal Metrics
Step Count Traversers Time (ms) % Dur
=============================================================================================================
TinkerGraphStep(vertex,null) 0.037 100.00
>TOTAL - - 0.037 - 

So, GraphStep.java has many error when dealing with ids，just like：

@Override
public int hashCode() {
int result = super.hashCode() ^ this.returnClass.hashCode();
for (final Object id : this.ids) {
result ^= id.hashCode();
}
return result;
}

@Override
public String toString() {
if (this.hasContainers.isEmpty())
return super.toString();
else
return 0 == this.ids.length ?
StringFactory.stepString(this, this.returnClass.getSimpleName().toLowerCase(), this.hasContainers) :
StringFactory.stepString(this, this.returnClass.getSimpleName().toLowerCase(), Arrays.toString(this.ids), this.hasContainers);
}

I thinks this is bug not just appear from 3.4.2