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

optimize schema cache perf through IntObjectMap(array) #1185

Merged
merged 2 commits into from
Sep 23, 2020

Conversation

javeme
Copy link
Contributor

@javeme javeme commented Sep 21, 2020

  • optimize schema cache perf through IntObjectMap(array)
  • let array_cache_size = schema_cache_size / 8

Change-Id: Id0cd9bda4085429181f96731a01fb2ee200225c9

* optimize schema cache perf through IntObjectMap(array)

Change-Id: Id0cd9bda4085429181f96731a01fb2ee200225c9

* let array_cache_size = schema_cache_size / 8

Change-Id: I08107c6919f8633d0b9a76575f48e605edc12c71
@javeme javeme changed the title optimize schema cache perf through IntObjectMap(array) (#56) optimize schema cache perf through IntObjectMap(array) Sep 21, 2020
Linary
Linary previously approved these changes Sep 22, 2020
@javeme
Copy link
Contributor Author

javeme commented Sep 22, 2020

related #1155

Change-Id: I366a7a067df876393fca1c2df3d48476778b6069
@codecov
Copy link

codecov bot commented Sep 22, 2020

Codecov Report

Merging #1185 into master will decrease coverage by 35.32%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             master    #1185       +/-   ##
=============================================
- Coverage     64.98%   29.65%   -35.33%     
+ Complexity     5652      263     -5389     
=============================================
  Files           361       75      -286     
  Lines         29429     4208    -25221     
  Branches       4158      510     -3648     
=============================================
- Hits          19123     1248    -17875     
+ Misses         8352     2797     -5555     
+ Partials       1954      163     -1791     
Impacted Files Coverage Δ Complexity Δ
...n/java/com/baidu/hugegraph/api/job/GremlinAPI.java 47.82% <0.00%> (-2.90%) 3.00% <0.00%> (+3.00%) ⬇️
...va/com/baidu/hugegraph/structure/HugeFeatures.java
...ugegraph/backend/page/SortByCountIdHolderList.java
...java/com/baidu/hugegraph/type/define/HugeKeys.java
...va/com/baidu/hugegraph/analyzer/JcsegAnalyzer.java
...kend/store/rocksdbsst/RocksDBSstStoreProvider.java
...om/baidu/hugegraph/exception/NoIndexException.java
...du/hugegraph/backend/store/raft/RaftException.java
.../backend/store/cassandra/CassandraSessionPool.java
...ugegraph/backend/store/BackendStoreSystemInfo.java
... and 275 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 e35f9fb...6b52e04. Read the comment docs.

@javeme javeme merged commit 58045ba into master Sep 23, 2020
@javeme javeme deleted the schema-array-cache branch September 23, 2020 07:48
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.

4 participants