Uploaded image for project: 'TinkerPop'
  1. TinkerPop
  2. TINKERPOP-1153

Add ByModulating and TimesModulating interfaces.

    Details

    • 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
    • Labels:
      None

      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.

        Attachments

          Activity

            People

            • Assignee:
              okram Marko A. Rodriguez
              Reporter:
              okram Marko A. Rodriguez
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: