Skip to content

Fix add schema from follower then list all without it#1325

Merged
Linary merged 3 commits intomasterfrom
fix-cache-miss
Jan 14, 2021
Merged

Fix add schema from follower then list all without it#1325
Linary merged 3 commits intomasterfrom
fix-cache-miss

Conversation

@Linary
Copy link
Copy Markdown
Contributor

@Linary Linary commented Jan 7, 2021

Change-Id: I3fa86e813fba61b4d77493e78317ebb451e58785

Change-Id: I3fa86e813fba61b4d77493e78317ebb451e58785
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 7, 2021

Codecov Report

Merging #1325 (ba72d44) into master (af2590f) will decrease coverage by 0.01%.
The diff coverage is 60.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1325      +/-   ##
============================================
- Coverage     62.33%   62.32%   -0.02%     
+ Complexity     5771     5770       -1     
============================================
  Files           377      377              
  Lines         31535    31550      +15     
  Branches       4418     4421       +3     
============================================
+ Hits          19658    19663       +5     
- Misses         9880     9888       +8     
- Partials       1997     1999       +2     
Impacted Files Coverage Δ Complexity Δ
...ugegraph/backend/store/raft/StoreStateMachine.java 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...ugegraph/backend/store/rocksdb/RocksDBOptions.java 99.25% <ø> (ø) 2.00 <0.00> (ø)
...gegraph/backend/cache/CachedSchemaTransaction.java 89.95% <100.00%> (+0.14%) 34.00 <1.00> (+1.00)
...a/com/baidu/hugegraph/backend/query/Condition.java 78.10% <0.00%> (-0.66%) 32.00% <0.00%> (ø%)
...du/hugegraph/traversal/optimize/TraversalUtil.java 56.65% <0.00%> (-0.62%) 122.00% <0.00%> (ø%)
...n/java/com/baidu/hugegraph/schema/PropertyKey.java 77.09% <0.00%> (-0.60%) 51.00% <0.00%> (ø%)
...in/java/com/baidu/hugegraph/StandardHugeGraph.java 78.98% <0.00%> (-0.39%) 119.00% <0.00%> (+1.00%) ⬇️
...aidu/hugegraph/schema/builder/AbstractBuilder.java 89.28% <0.00%> (-0.37%) 13.00% <0.00%> (-1.00%)
...aidu/hugegraph/backend/tx/AbstractTransaction.java 72.90% <0.00%> (-0.35%) 51.00% <0.00%> (-2.00%)
...om/baidu/hugegraph/task/StandardTaskScheduler.java 77.44% <0.00%> (-0.23%) 85.00% <0.00%> (-2.00%)
... and 12 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 af2590f...ba72d44. Read the comment docs.

javeme
javeme previously approved these changes Jan 7, 2021
this.nameCache.invalidate(prefixedName);
}
// Set the cache all flag of the schema type to false
this.cachedTypes().put(type, false);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

add method to line 164: private final void resetCachedAll(type)

Change-Id: I8995bbe6604a9c6f50b6fcf89eafeff623fc1313
// Set the cache all flag of the schema type to false
this.cachedTypes().put(type, false);
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

move to line 140

Change-Id: Iad9d80cabb3d6a874ac66c52de22cc2682debfca
@Linary Linary merged commit 0dc1535 into master Jan 14, 2021
@Linary Linary deleted the fix-cache-miss branch January 14, 2021 06:08
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.

2 participants