[TINKERPOP-1153] Add ByModulating and TimesModulating interfaces.

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Implemented
• Affects Version/s: 3.1.2-incubating
• Fix Version/s: 3.2.0-incubating
• Component/s: process
• Assignee: Marko A. Rodriguez
• Reporter: Marko A. Rodriguez
• Created: 15/Feb/16 13:22
• Updated: 19/Feb/16 16:27
• Resolved: 19/Feb/16 16:27

Description

Currently GraphTraversal is responsible for the logic for by()- and times()-modulators. For instance, if you do:

...by('name')

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:

public interface ByModulating {

  public void add(String string) throws UnsupportedOperationException
  public void add(Traversal traversal) throws UnsupportedOperationException
  public void add(T t) throws UnsupportedOperationException

}

Likewise for TimesModulating ... and any other modulators down the road.