You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tinkerpop.apache.org by "Stephen Mallette (Jira)" <ji...@apache.org> on 2021/02/25 18:46:00 UTC

[jira] [Commented] (TINKERPOP-2511) Determine if assertion is needed on EarlyLimitStrategyTest

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

Stephen Mallette commented on TINKERPOP-2511:
---------------------------------------------

still can't recall why this assertion was relevant - removed it: https://github.com/apache/tinkerpop/commit/9d035d7c8584e5a80d898d3c67264429c64e506a and resolving as invalid.

> Determine if assertion is needed on EarlyLimitStrategyTest
> ----------------------------------------------------------
>
>                 Key: TINKERPOP-2511
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2511
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: test-suite
>    Affects Versions: 3.5.0
>            Reporter: Stephen Mallette
>            Assignee: Stephen Mallette
>            Priority: Minor
>
> Commented out an assertion which is swapping behavior between applying strategies and not applying any. It may not be possible to have a single assertion here anymore:
> https://github.com/apache/tinkerpop/commit/0e75c645582380ec686f91eeb9b8ba663609175d#diff-eea1b8379c087f4f81360649d09e7b4787c327f1b5477147807567eddf33aa46R109



--
This message was sent by Atlassian Jira
(v8.3.4#803005)