[ https://issues.apache.org/jira/browse/TINKERPOP-3150?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Andrea C updated TINKERPOP-3150: -------------------------------- Description: Sample step will ignore previous by modulators if multiple are specified, which can be misleading to the user. Instead, similar to https://issues.apache.org/jira/browse/TINKERPOP-3121 multiple by modulators should be prevented with sample. {code:java} gremlin> g.V().sample(10).by(id).elementMap() ==>[id:1,label:person,name:marko,age:29] ==>[id:2,label:person,name:vadas,age:27] ==>[id:3,label:software,name:lop,lang:java] ==>[id:4,label:person,name:josh,age:32] ==>[id:5,label:software,name:ripple,lang:java] ==>[id:6,label:person,name:peter,age:35] gremlin> g.V().sample(10).by('age').elementMap() ==>[id:1,label:person,name:marko,age:29] ==>[id:2,label:person,name:vadas,age:27] ==>[id:4,label:person,name:josh,age:32] ==>[id:6,label:person,name:peter,age:35] gremlin> g.V().sample(10).by('age').by(id).elementMap() ==>[id:1,label:person,name:marko,age:29] ==>[id:2,label:person,name:vadas,age:27] ==>[id:3,label:software,name:lop,lang:java] ==>[id:4,label:person,name:josh,age:32] ==>[id:5,label:software,name:ripple,lang:java] ==>[id:6,label:person,name:peter,age:35]{code} was: Aggregate step will ignore previous by modulators if multiple are specified, which can be misleading to the user. Instead, similar to https://issues.apache.org/jira/browse/TINKERPOP-3121 multiple by modulators should be prevented with aggregate. {code:java} gremlin> g.V().aggregate('x').by('name').cap('x') ==>[marko,vadas,lop,josh,ripple,peter] gremlin> g.V().aggregate('x').by('name').by('age').cap('x') ==>[29,27,32,35] {code} > Prevent sample step from having multiple by modulators > ------------------------------------------------------ > > Key: TINKERPOP-3150 > URL: https://issues.apache.org/jira/browse/TINKERPOP-3150 > Project: TinkerPop > Issue Type: Improvement > Components: process > Affects Versions: 3.7.3 > Reporter: Andrea C > Priority: Minor > > Sample step will ignore previous by modulators if multiple are specified, > which can be misleading to the user. Instead, similar to > https://issues.apache.org/jira/browse/TINKERPOP-3121 multiple by modulators > should be prevented with sample. > {code:java} > gremlin> g.V().sample(10).by(id).elementMap() > ==>[id:1,label:person,name:marko,age:29] > ==>[id:2,label:person,name:vadas,age:27] > ==>[id:3,label:software,name:lop,lang:java] > ==>[id:4,label:person,name:josh,age:32] > ==>[id:5,label:software,name:ripple,lang:java] > ==>[id:6,label:person,name:peter,age:35] > gremlin> g.V().sample(10).by('age').elementMap() > ==>[id:1,label:person,name:marko,age:29] > ==>[id:2,label:person,name:vadas,age:27] > ==>[id:4,label:person,name:josh,age:32] > ==>[id:6,label:person,name:peter,age:35] > gremlin> g.V().sample(10).by('age').by(id).elementMap() > ==>[id:1,label:person,name:marko,age:29] > ==>[id:2,label:person,name:vadas,age:27] > ==>[id:3,label:software,name:lop,lang:java] > ==>[id:4,label:person,name:josh,age:32] > ==>[id:5,label:software,name:ripple,lang:java] > ==>[id:6,label:person,name:peter,age:35]{code} -- This message was sent by Atlassian Jira (v8.20.10#820010)