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

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

codecov-commenter commented on PR #2019:
URL: https://github.com/apache/tinkerpop/pull/2019#issuecomment-1505928566

   ## 
[Codecov](https://codecov.io/gh/apache/tinkerpop/pull/2019?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#2019](https://codecov.io/gh/apache/tinkerpop/pull/2019?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (fbc381f) into 
[3.5-dev](https://codecov.io/gh/apache/tinkerpop/commit/deba2153bf3dd5e174db8cbd176eaf37b660033e?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (deba215) will **decrease** coverage by `5.18%`.
   > The diff coverage is `n/a`.
   
   ```diff
   @@              Coverage Diff              @@
   ##             3.5-dev    #2019      +/-   ##
   =============================================
   - Coverage      69.33%   64.16%   -5.18%     
   =============================================
     Files            866       25     -841     
     Lines          41219     3750   -37469     
     Branches        5432        0    -5432     
   =============================================
   - Hits           28581     2406   -26175     
   + Misses         10722     1166    -9556     
   + Partials        1916      178    -1738     
   ```
   
   
   [see 841 files with indirect coverage 
changes](https://codecov.io/gh/apache/tinkerpop/pull/2019/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   
   :mega: We’re building smart automated test selection to slash your CI/CD 
build times. [Learn 
more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   




> HasId Step generates incorrect results when given a list of IDs mid-traversal
> -----------------------------------------------------------------------------
>
>                 Key: TINKERPOP-2863
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-2863
>             Project: TinkerPop
>          Issue Type: Bug
>          Components: process
>    Affects Versions: 3.6.2
>            Reporter: Taylor Riggan
>            Priority: Critical
>
> In most situations, hasId() will accept a list of potential IDs to filter on 
> and implicitly use within() filtering semantics to return the correct 
> results.  Examples:
> {code:java}
> g.V().hasId(['1','2'])
> {code}
> returns:
> {code:java}
> [v[1], v[2]]{code}
> or
> {code:java}
> g.E().hasId(['5140','5261']){code}
> returns:
> {code:java}
> [e[5140][1-route->51], e[5261][1-route->398]]{code}
> However, when using the same form of semantics mid-traversal, both of these 
> queries return empty results:
> {code:java}
> g.V().has('code','ATL').outE('route').hasId(['5140','5261'])
> g.V().has('code','ATL').outE('route').inV().hasId(['2','3'])
> {code}
> When using profile() against both queries, the hasId() bytecode gets 
> transformed into
> {code:java}
> HasStep([~id.eq([5140, 5261])]) {code}
> This equates to finding a vertex or edge with an ID that matches the entire 
> list instead of the elements within the list.
> The preceding was tested against Gremlin Server 3.6.2 with the Airroutes 
> dataset.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to