search

vesoft-inc / nebula-java

Client API and data importer of Nebula Graph in Java
Apache License 2.0
164 stars 121 forks source link

replace the protocol with header protocol #505

Closed Nicole00 closed 1 year ago

Nicole00 commented 1 year ago

What type of PR is this?

What problem(s) does this PR solve?

Issue(s) number:

https://github.com/vesoft-inc/nebula/issues/5191

Description:

when server transfer a large dataframe, client will hang. Using header protocol can throw the exception but not hang.

How do you solve it?

Special notes for your reviewer, ex. impact of this fix, design document, etc:

codecov-commenter commented 1 year ago

Codecov Report

Base: 63.47% // Head: 63.48% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (fd8fa64) compared to base (6f5e897). Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #505 +/- ## ========================================= Coverage 63.47% 63.48% Complexity 788 788 ========================================= Files 71 71 Lines 3762 3763 +1 Branches 538 538 ========================================= + Hits 2388 2389 +1 - Misses 1001 1003 +2 + Partials 373 371 -2 ``` | [Impacted Files](https://codecov.io/gh/vesoft-inc/nebula-java/pull/505?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vesoft-inc) | Coverage Δ | | |---|---|---| | [.../vesoft/nebula/client/meta/AbstractMetaClient.java](https://codecov.io/gh/vesoft-inc/nebula-java/pull/505?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vesoft-inc#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvY29tL3Zlc29mdC9uZWJ1bGEvY2xpZW50L21ldGEvQWJzdHJhY3RNZXRhQ2xpZW50LmphdmE=) | `47.61% <ø> (ø)` | | | [...vesoft/nebula/client/graph/net/SyncConnection.java](https://codecov.io/gh/vesoft-inc/nebula-java/pull/505?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vesoft-inc#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvY29tL3Zlc29mdC9uZWJ1bGEvY2xpZW50L2dyYXBoL25ldC9TeW5jQ29ubmVjdGlvbi5qYXZh) | `54.09% <100.00%> (ø)` | | | [...java/com/vesoft/nebula/client/meta/MetaClient.java](https://codecov.io/gh/vesoft-inc/nebula-java/pull/505?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vesoft-inc#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvY29tL3Zlc29mdC9uZWJ1bGEvY2xpZW50L21ldGEvTWV0YUNsaWVudC5qYXZh) | `52.60% <100.00%> (+0.22%)` | :arrow_up: | | [.../nebula/client/storage/GraphStorageConnection.java](https://codecov.io/gh/vesoft-inc/nebula-java/pull/505?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vesoft-inc#diff-Y2xpZW50L3NyYy9tYWluL2phdmEvY29tL3Zlc29mdC9uZWJ1bGEvY2xpZW50L3N0b3JhZ2UvR3JhcGhTdG9yYWdlQ29ubmVjdGlvbi5qYXZh) | `87.50% <100.00%> (ø)` | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vesoft-inc). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=vesoft-inc)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.