Skip to content

Conversation

coderzc
Copy link
Member

@coderzc coderzc commented Oct 21, 2021

No description provided.

@codecov
Copy link

codecov bot commented Oct 21, 2021

Codecov Report

Merging #126 (9a48362) into master (89fe866) will decrease coverage by 0.01%.
The diff coverage is 64.28%.

Impacted file tree graph

@@ Coverage Diff @@ ## master #126 +/- ## ============================================ - Coverage 88.00% 87.98% -0.02%  Complexity 2854 2854 ============================================ Files 300 300 Lines 10768 10787 +19 Branches 902 904 +2 ============================================ + Hits 9476 9491 +15  - Misses 864 867 +3  - Partials 428 429 +1 
Impacted Files Coverage Δ
...gorithm/path/rings/filter/RingsDetectionValue.java 77.27% <0.00%> (ø)
...egraph/computer/core/graph/value/BooleanValue.java 100.00% <ø> (+3.84%) ⬆️
...gegraph/computer/core/graph/value/DoubleValue.java 86.20% <ø> (+2.87%) ⬆️
...ugegraph/computer/core/graph/value/FloatValue.java 86.20% <ø> (+2.87%) ⬆️
.../hugegraph/computer/core/graph/value/IntValue.java 86.20% <ø> (-0.46%) ⬇️
...hugegraph/computer/core/graph/value/ListValue.java 94.44% <ø> (ø)
...hugegraph/computer/core/graph/value/LongValue.java 86.20% <ø> (+2.87%) ⬆️
...hugegraph/computer/core/graph/value/NullValue.java 85.71% <ø> (ø)
...gegraph/computer/core/graph/value/StringValue.java 91.66% <ø> (+3.66%) ⬆️
...aidu/hugegraph/computer/core/graph/id/BytesId.java 82.27% <ø> (ø)
... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 89fe866...9a48362. Read the comment docs.


@Override
public Object object() {
public Object value() {
Copy link
Contributor

Choose a reason for hiding this comment

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

also return List<T>


@Override
public Object object() {
public Object value() {
Copy link
Contributor

Choose a reason for hiding this comment

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

unify UnsupportedOperationException and NotSupportedException

/**
* @return result string value of this object
*/
default String resultString() {
Copy link
Contributor

Choose a reason for hiding this comment

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

string() is more concise

@javeme javeme merged commit 2442fe6 into master Oct 25, 2021
@javeme javeme deleted the improve-value-interface branch October 25, 2021 04:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants