dah - I had a feeling you would say that. On Wed, Jul 27, 2016 at 7:53 AM, Marko A. Rodriguez <[email protected]> wrote:
> ah. please dont do this. having that extensible may not be a good idea. > please ask why this is needed. i dont like our internals extendable. > usually a sign of the requester doing it wrong. > > Sent from my iPad > > > On Jul 27, 2016, at 6:49 AM, [email protected] wrote: > > > > Merge remote-tracking branch 'origin/tp31' > > > > > > Project: http://git-wip-us.apache.org/repos/asf/tinkerpop/repo > > Commit: http://git-wip-us.apache.org/repos/asf/tinkerpop/commit/3135c30f > > Tree: http://git-wip-us.apache.org/repos/asf/tinkerpop/tree/3135c30f > > Diff: http://git-wip-us.apache.org/repos/asf/tinkerpop/diff/3135c30f > > > > Branch: refs/heads/master > > Commit: 3135c30fbef0f04964f8bef56cbdace925f2ab01 > > Parents: 2f2b373 4b9212e > > Author: Stephen Mallette <[email protected]> > > Authored: Wed Jul 27 06:38:52 2016 -0400 > > Committer: Stephen Mallette <[email protected]> > > Committed: Wed Jul 27 06:38:52 2016 -0400 > > > > ---------------------------------------------------------------------- > > .../process/traversal/step/util/ExpandableStepIterator.java | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > ---------------------------------------------------------------------- > > > > > > > http://git-wip-us.apache.org/repos/asf/tinkerpop/blob/3135c30f/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/util/ExpandableStepIterator.java > > ---------------------------------------------------------------------- > > diff --cc > gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/util/ExpandableStepIterator.java > > index 30558e8,f5bd239..dc61ad2 > > --- > a/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/util/ExpandableStepIterator.java > > +++ > b/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/util/ExpandableStepIterator.java > > @@@ -28,12 -29,13 +28,12 @@@ import java.util.Iterator > > /** > > * @author Marko A. Rodriguez (http://markorodriguez.com) > > */ > > - public final class ExpandableStepIterator<S> implements > Iterator<Traverser.Admin<S>>, Serializable { > > + public class ExpandableStepIterator<S> implements > Iterator<Traverser.Admin<S>>, Serializable { > > > > private final TraverserSet<S> traverserSet = new TraverserSet<>(); > > - private final MultiIterator<Traverser.Admin<S>> traverserIterators > = new MultiIterator<>(); > > - private final Step<S,?> hostStep; > > + private final Step<S, ?> hostStep; > > > > - public ExpandableStepIterator(final Step<S,?> hostStep) { > > + public ExpandableStepIterator(final Step<S, ?> hostStep) { > > this.hostStep = hostStep; > > } > > > > >
