add conf entry jdbc.storage_engine to change storage engine by wenzhenghu · Pull Request #555 · apache/hugegraph

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Merged

zhoney merged 7 commits intoapache:masterfrom

wenzhenghu:myrocks

Jun 5, 2019

Merged

Conversation

@wenzhenghu

Copy link

Contributor

default innodb, could change to rocksdb, but not myisam, as hugegraph will create large index that myisam could not support ("Specified key was too long; max key length is 1000 bytes")

@CLAassistant

Copy link

CLAassistant commented

Jun 5, 2019

edited

Loading

CLA assistant check
All committers have signed the CLA.

@codecov

Copy link

codecov bot commented

Jun 5, 2019

edited

Loading

Codecov Report

Merging #555 into master will increase coverage by 0.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #555      +/-   ##
============================================
+ Coverage     70.19%   70.21%   +0.01%     
- Complexity     3418     3419       +1     
============================================
  Files           216      216              
  Lines         16461    16465       +4     
  Branches       2351     2351              
============================================
+ Hits          11555    11561       +6     
- Misses         3650     3651       +1     
+ Partials       1256     1253       -3
Impacted Files Coverage Δ Complexity Δ
...raph/backend/store/postgresql/PostgresqlTable.java 96.72% <ø> (ø) 17 <0> (ø) ⬇️
...u/hugegraph/backend/store/mysql/MysqlSessions.java 70.9% <100%> (+0.78%) 18 <0> (+1) ⬆️
...du/hugegraph/backend/store/mysql/MysqlOptions.java 95.65% <100%> (+0.41%) 3 <0> (ø) ⬇️
...aidu/hugegraph/backend/store/mysql/MysqlTable.java 76.84% <100%> (+0.07%) 70 <1> (ø) ⬇️
...ain/java/com/baidu/hugegraph/task/TaskManager.java 60.86% <0%> (-4.35%) 10% <0%> (-1%)
...va/com/baidu/hugegraph/backend/cache/RamCache.java 72.54% <0%> (+0.39%) 45% <0%> (+1%) ⬆️
...du/hugegraph/backend/tx/GraphIndexTransaction.java 80% <0%> (+0.47%) 146% <0%> (ø) ⬇️

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 f3a3c35...4cb62c2. Read the comment docs.

@zhoney zhoney merged commit 033c387 into apache:master

Jun 5, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Linary Linary Linary approved these changes

@javeme javeme javeme approved these changes

@zhoney zhoney zhoney approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@wenzhenghu @CLAassistant @Linary @javeme @zhoney