Marko A. Rodriguez created TINKERPOP-1153: ---------------------------------------------
Summary: Add ByModulating and TimesModulating interfaces. Key: TINKERPOP-1153 URL: https://issues.apache.org/jira/browse/TINKERPOP-1153 Project: TinkerPop Issue Type: Improvement Components: process Affects Versions: 3.1.2-incubating Reporter: Marko A. Rodriguez Fix For: 3.2.0-incubating Currently {{GraphTraversal}} is responsible for the logic for {{by()}}- and {{times()}}-modulators. For instance, if you do: {code} ...by('name') {code} It is {{GraphTraversal}} that will compile that to {{__.values('name').limit(1)}}. This should not be the logic of {{GraphTraversal}}, but instead, logic within the step being modulated. Thus, I propose: {code} public interface ByModulating { public void add(String string) throws UnsupportedOperationException public void add(Traversal traversal) throws UnsupportedOperationException public void add(T t) throws UnsupportedOperationException } {code} Likewise for {{TimesModulating}} ... and any other modulators down the road. -- This message was sent by Atlassian JIRA (v6.3.4#6332)