[jira] [Commented] (TINKERPOP-1975) Introduce with() step modulator

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TINKERPOP-1975?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16500882#comment-16500882 ] 

ASF GitHub Bot commented on TINKERPOP-1975:
-------------------------------------------

GitHub user spmallette opened a pull request:

    https://github.com/apache/tinkerpop/pull/871

    TINKERPOP-1975 Introduced with() step and Configuring interface

    https://issues.apache.org/jira/browse/TINKERPOP-1975
    
    This PR is about `with()` step which introduces a way to provide a way to supply configurations to steps. Especially useful for algorithm type steps like `pageRank()` which had previously relied on `by()` which was insufficient for this purposes and a bit forced to try to prevent additional expansion of the core API.
    
    As a result of `with()` we now have the `Configuring` interface to apply to steps that can accept `with()` options. Moved `Parameterizing` to the package containing the other step interfaces. Changed interface hierarchy with `Parameterizing` and `Mutating` now that `Configuring` is present. Both of those changes make this body of work a breaking change to existing APIs though it should really only affect graph providers and even then the fix is quite simple as described in the upgrade documentation.
    
    Will edit this description when ready for review - once tests complete.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/apache/tinkerpop TINKERPOP-1975

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/tinkerpop/pull/871.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #871
    
----
commit 0dca3ff9d2f78e145a807e28fd58c1a0cf452cb6
Author: Stephen Mallette <sp...@...>
Date:   2018-05-31T16:55:13Z

    TINKERPOP-1975 Introduced with() step and Configuring interface
    
    Moved Parameterizing to the package containing the other step interfaces. Changed interface hierarchy with Parameterizing and Mutating now that Configuring is present. Both of those changes make this body of work a breaking change to existing APIs though it should really only affect graph providers and even then the fix is quite simple as described in the upgrade documentation.

----


> Introduce with() step modulator
> -------------------------------
>
>                 Key: TINKERPOP-1975
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1975
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: process
>    Affects Versions: 3.3.3
>            Reporter: stephen mallette
>            Assignee: stephen mallette
>            Priority: Major
>              Labels: breaking
>             Fix For: 3.4.0
>
>
> The {{with()}} step modulator provides a general way to provide configurations to steps in much the same way {{withStrategies()}} or {{withSideEffects()}} are used to configure traversals.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)