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

refactor: optimizing code of project update api #1537

Merged
merged 2 commits into from
Jul 14, 2021
Merged

Conversation

zyxxoo
Copy link
Contributor

@zyxxoo zyxxoo commented Jul 7, 2021

No description provided.

@codecov
Copy link

codecov bot commented Jul 7, 2021

Codecov Report

Merging #1537 (ff215a4) into master (6bff777) will increase coverage by 0.50%.
The diff coverage is 67.85%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1537      +/-   ##
============================================
+ Coverage     59.42%   59.92%   +0.50%     
- Complexity     6187     6236      +49     
============================================
  Files           412      412              
  Lines         33603    33657      +54     
  Branches       4618     4638      +20     
============================================
+ Hits          19967    20169     +202     
+ Misses        11544    11392     -152     
- Partials       2092     2096       +4     
Impacted Files Coverage Δ
.../java/com/baidu/hugegraph/api/auth/ProjectAPI.java 82.05% <67.85%> (-4.91%) ⬇️
.../baidu/hugegraph/backend/query/ConditionQuery.java 85.01% <0.00%> (-0.99%) ⬇️
...du/hugegraph/schema/builder/IndexLabelBuilder.java 87.46% <0.00%> (+0.11%) ⬆️
...c/main/java/com/baidu/hugegraph/task/HugeTask.java 72.00% <0.00%> (+0.30%) ⬆️
...a/com/baidu/hugegraph/backend/query/Condition.java 78.83% <0.00%> (+0.36%) ⬆️
...va/com/baidu/hugegraph/backend/cache/RamCache.java 73.03% <0.00%> (+0.56%) ⬆️
...in/java/com/baidu/hugegraph/schema/IndexLabel.java 80.21% <0.00%> (+0.64%) ⬆️
...du/hugegraph/backend/tx/GraphIndexTransaction.java 81.95% <0.00%> (+0.79%) ⬆️
...du/hugegraph/traversal/optimize/TraversalUtil.java 57.51% <0.00%> (+0.85%) ⬆️
...ava/com/baidu/hugegraph/structure/HugeElement.java 71.68% <0.00%> (+0.91%) ⬆️
... and 8 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 6bff777...ff215a4. Read the comment docs.

@zyxxoo zyxxoo requested a review from javeme July 8, 2021 02:51
Linary
Linary previously approved these changes Jul 12, 2021
@Linary Linary merged commit 379859a into master Jul 14, 2021
@Linary Linary deleted the zy_auth_project branch July 14, 2021 03:59
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

4 participants