incubator-graphar icon indicating copy to clipboard operation
incubator-graphar copied to clipboard

feat(java,info): support multi-property in yaml

Open yangxk1 opened this issue 3 months ago • 2 comments

Reason for this PR

close #778

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

yangxk1 avatar Sep 29 '25 07:09 yangxk1

Codecov Report

:x: Patch coverage is 59.42029% with 28 lines in your changes missing coverage. Please review. :white_check_mark: Project coverage is 76.60%. Comparing base (6f112c6) to head (b808b9b). :warning: Report is 11 commits behind head on main.

Files with missing lines Patch % Lines
...rc/main/java/org/apache/graphar/info/EdgeInfo.java 0.00% 3 Missing and 3 partials :warning:
...c/main/java/org/apache/graphar/info/GraphInfo.java 0.00% 3 Missing and 3 partials :warning:
...java/org/apache/graphar/info/type/Cardinality.java 62.50% 3 Missing and 3 partials :warning:
.../apache/graphar/info/saver/BaseGraphInfoSaver.java 28.57% 3 Missing and 2 partials :warning:
.../java/org/apache/graphar/info/yaml/VertexYaml.java 66.66% 1 Missing and 1 partial :warning:
...rc/main/java/org/apache/graphar/info/Property.java 90.00% 0 Missing and 1 partial :warning:
.../main/java/org/apache/graphar/info/VertexInfo.java 66.66% 0 Missing and 1 partial :warning:
...ava/org/apache/graphar/info/yaml/PropertyYaml.java 90.90% 0 Missing and 1 partial :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##               main     #780    +/-   ##
==========================================
  Coverage     76.59%   76.60%            
- Complexity      560      585    +25     
==========================================
  Files            83       84     +1     
  Lines          8679     8788   +109     
  Branches       1006     1018    +12     
==========================================
+ Hits           6648     6732    +84     
- Misses         1813     1826    +13     
- Partials        218      230    +12     
Flag Coverage Δ
java-info 82.20% <59.42%> (-0.54%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov-commenter avatar Sep 29 '25 07:09 codecov-commenter

Hey @Thespica, please help me review.

yangxk1 avatar Oct 09 '25 06:10 yangxk1