Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed MergeV/MergeE steps to work when onCreate is immutable map #2028

Merged
merged 4 commits into from
Apr 21, 2023

Conversation

vkagamlyk
Copy link
Contributor

Current implementation of MergeV/MergeE doesn't work when Merge.onCreate is immutable map.

https://issues.apache.org/jira/browse/TINKERPOP-2926

@codecov-commenter
Copy link

codecov-commenter commented Apr 19, 2023

Codecov Report

Merging #2028 (8349c5d) into 3.6-dev (975c69c) will increase coverage by 5.03%.
The diff coverage is 69.23%.

@@              Coverage Diff              @@
##             3.6-dev    #2028      +/-   ##
=============================================
+ Coverage      64.43%   69.47%   +5.03%     
- Complexity         0     9340    +9340     
=============================================
  Files             25      878     +853     
  Lines           3793    42234   +38441     
  Branches           0     5645    +5645     
=============================================
+ Hits            2444    29341   +26897     
- Misses          1181    10902    +9721     
- Partials         168     1991    +1823     
Impacted Files Coverage Δ
...in/process/traversal/step/map/MergeVertexStep.java 86.95% <57.14%> (ø)
...mlin/process/traversal/step/map/MergeEdgeStep.java 80.34% <83.33%> (ø)

... and 852 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

* Introduced `Writing` and `Deleting` marker interfaces to identify whether a step can perform write or delete or both on Graph.
* Added static map capturing possible Traversal steps that shall be added to traversal for a given operator


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small nit: added extra new line

@kenhuuu
Copy link
Contributor

kenhuuu commented Apr 21, 2023

VOTE +1

@vkagamlyk
Copy link
Contributor Author

VOTE+1

@xiazcy
Copy link
Contributor

xiazcy commented Apr 21, 2023

VOTE +1

@vkagamlyk vkagamlyk merged commit a8f0990 into apache:3.6-dev Apr 21, 2023
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants