You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tinkerpop.apache.org by okram <gi...@git.apache.org> on 2017/07/10 20:28:12 UTC

[GitHub] tinkerpop pull request #665: TINKERPOP-1170: Remove deprecated Configuration...

GitHub user okram opened a pull request:

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

    TINKERPOP-1170: Remove deprecated ConfigurationTraversal.

    https://issues.apache.org/jira/browse/TINKERPOP-1170
    
    Removed `ConfigurationTraversal` which really should have never been used by anymore, but still. Deprecated accordingly. Its replacement is `PureTraversal`.
    
    VOTE +1

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

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

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

    https://github.com/apache/tinkerpop/pull/665.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 #665
    
----
commit b0de07fb12685cf8c71a090c10810f18f0a45026
Author: Marko A. Rodriguez <ok...@gmail.com>
Date:   2017-07-10T20:26:49Z

    removed deprecated ConfigurationTraversal. It was replaced by PureTraversal.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop issue #665: TINKERPOP-1170: Remove deprecated ConfigurationTravers...

Posted by dkuppitz <gi...@git.apache.org>.
Github user dkuppitz commented on the issue:

    https://github.com/apache/tinkerpop/pull/665
  
    VOTE: +1


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop issue #665: TINKERPOP-1170: Remove deprecated ConfigurationTravers...

Posted by spmallette <gi...@git.apache.org>.
Github user spmallette commented on the issue:

    https://github.com/apache/tinkerpop/pull/665
  
    VOTE +1


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop pull request #665: TINKERPOP-1170: Remove deprecated Configuration...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop issue #665: TINKERPOP-1170: Remove deprecated ConfigurationTravers...

Posted by okram <gi...@git.apache.org>.
Github user okram commented on the issue:

    https://github.com/apache/tinkerpop/pull/665
  
    No, `ConfigurationTraversal` was replaced by `PureTraversal` and the documentation uses `PureTraversal`.
    
    ```
    this.traversal = PureTraversal.loadState(configuration, VertexProgramStep.ROOT_TRAVERSAL, graph);
    ```
    
    Is there somewhere else in particular you are talking about?



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop issue #665: TINKERPOP-1170: Remove deprecated ConfigurationTravers...

Posted by robertdale <gi...@git.apache.org>.
Github user robertdale commented on the issue:

    https://github.com/apache/tinkerpop/pull/665
  
    I only saw referecences to storeState under http://tinkerpop.apache.org/docs/current/reference/#pagerankvertexprogram  (sorry for the previous incorrect link).  I'm not familiar with any of this, just want to make sure the docs are synced.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] tinkerpop issue #665: TINKERPOP-1170: Remove deprecated ConfigurationTravers...

Posted by robertdale <gi...@git.apache.org>.
Github user robertdale commented on the issue:

    https://github.com/apache/tinkerpop/pull/665
  
    Looks like some documentation may need updated.  Not sure if this is the same thing.  http://tinkerpop.apache.org/docs/current/reference/#program-step



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---