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

support clone graph without config param #1709

Merged
merged 4 commits into from
Dec 27, 2021
Merged

Conversation

javeme
Copy link
Contributor

@javeme javeme commented Dec 26, 2021

Change-Id: I290cf7f51cf92a082a94d60b15638ae699a18515

@javeme javeme force-pushed the clone-graph-without-config branch 2 times, most recently from 1f938b5 to b53e0b4 Compare December 26, 2021 12:49
@codecov
Copy link

codecov bot commented Dec 26, 2021

Codecov Report

Merging #1709 (82cdc9b) into master (b6abf96) will increase coverage by 17.00%.
The diff coverage is 6.34%.

Impacted file tree graph

@@              Coverage Diff              @@
##             master    #1709       +/-   ##
=============================================
+ Coverage     49.53%   66.53%   +17.00%     
- Complexity     5350     7082     +1732     
=============================================
  Files           423      423               
  Lines         35488    35525       +37     
  Branches       4948     4946        -2     
=============================================
+ Hits          17580    23638     +6058     
+ Misses        15975     9501     -6474     
- Partials       1933     2386      +453     
Impacted Files Coverage Δ
...ava/com/baidu/hugegraph/api/profile/GraphsAPI.java 10.76% <0.00%> (+10.76%) ⬆️
...com/baidu/hugegraph/auth/ContextGremlinServer.java 32.25% <0.00%> (+32.25%) ⬆️
...a/com/baidu/hugegraph/auth/HugeGraphAuthProxy.java 58.09% <0.00%> (+58.09%) ⬆️
...e/src/main/java/com/baidu/hugegraph/HugeGraph.java 65.30% <ø> (ø)
...in/java/com/baidu/hugegraph/StandardHugeGraph.java 66.43% <0.00%> (+1.07%) ⬆️
...ph/backend/store/AbstractBackendStoreProvider.java 79.78% <0.00%> (-1.54%) ⬇️
...h/backend/store/raft/RaftBackendStoreProvider.java 0.00% <0.00%> (ø)
...va/com/baidu/hugegraph/dist/HugeGremlinServer.java 52.38% <0.00%> (+52.38%) ⬆️
...ph/backend/store/rocksdb/RocksDBStoreProvider.java 31.57% <0.00%> (-68.43%) ⬇️
...in/java/com/baidu/hugegraph/core/GraphManager.java 39.60% <8.19%> (+39.60%) ⬆️
... and 253 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 b6abf96...82cdc9b. Read the comment docs.

zhoney
zhoney previously approved these changes Dec 27, 2021
Change-Id: I738237035438590b8c9414dd1281f950a3f2b7bd
Change-Id: If26d647ae3b1f7eb1f046a42c02715cf36e31856
Change-Id: I290cf7f51cf92a082a94d60b15638ae699a18515
Change-Id: Id6d9d6207a67ce2587d8d923c7ce9455c14eed9f
@javeme
Copy link
Contributor Author

javeme commented Dec 27, 2021

unit test error:

Error:  Tests run: 390, Failures: 0, Errors: 1, Skipped: 1, Time elapsed: 720.896 s <<< FAILURE! - in com.baidu.hugegraph.unit.UnitTestSuite
Error:  testIntFixedSeConcurrent(com.baidu.hugegraph.unit.util.collection.IntSetTest)  Time elapsed: 14.524 s  <<< ERROR!
java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.AssertionError: expected:<10000> but was:<9999>
	at com.baidu.hugegraph.unit.util.collection.IntSetTest.testIntSetConcurrent(IntSetTest.java:145)
	at com.baidu.hugegraph.unit.util.collection.IntSetTest.testIntFixedSeConcurrent(IntSetTest.java:72)
Caused by: java.util.concurrent.ExecutionException: java.lang.AssertionError: expected:<10000> but was:<9999>
	at com.baidu.hugegraph.unit.util.collection.IntSetTest.testIntSetConcurrent(IntSetTest.java:145)
	at com.baidu.hugegraph.unit.util.collection.IntSetTest.testIntFixedSeConcurrent(IntSetTest.java:72)
Caused by: java.lang.AssertionError: expected:<10000> but was:<9999>
	at com.baidu.hugegraph.unit.util.collection.IntSetTest.lambda$testIntSetConcurrent$6(IntSetTest.java:152)

@javeme javeme merged commit 0580bae into master Dec 27, 2021
@javeme javeme deleted the clone-graph-without-config branch December 27, 2021 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants