gchq / Gaffer

A large-scale entity and relation database supporting aggregation of properties
Apache License 2.0
1.75k stars 354 forks source link

Gh-3223: Change GafferPopEdge id representation #3230

Closed cn337131 closed 3 weeks ago

cn337131 commented 1 month ago

Able to now query edges with the id in format [1, knows, 2] in addition to [1, 2]. If the label is included this is used in the GetElements View.

Label also added to the returned edge id.

Related issue

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 82.14286% with 5 lines in your changes missing coverage. Please review.

Project coverage is 67.02%. Comparing base (cb343aa) to head (f7ced05). Report is 1 commits behind head on develop.

Files Patch % Lines
...a/uk/gov/gchq/gaffer/tinkerpop/GafferPopGraph.java 81.48% 2 Missing and 3 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #3230 +/- ## ========================================== Coverage 67.01% 67.02% - Complexity 2558 2559 +1 ========================================== Files 914 914 Lines 29394 29411 +17 Branches 3276 3278 +2 ========================================== + Hits 19699 19713 +14 - Misses 8252 8254 +2 - Partials 1443 1444 +1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

sonarcloud[bot] commented 3 weeks ago

Quality Gate Passed Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud