Skip to content

AKI-646 switch default DB to the rocksDB#1107

Merged
AionJayT merged 2 commits intomasterfrom
AKI-646
Jan 28, 2020
Merged

AKI-646 switch default DB to the rocksDB#1107
AionJayT merged 2 commits intomasterfrom
AKI-646

Conversation

@AionJayT
Copy link
Copy Markdown
Collaborator

Description

Fixes Issue # .

Type of change

  • Bug fix.
  • New feature.
  • Enhancement.
  • Unit test.
  • Breaking change (a fix or feature that causes existing functionality to not work as expected).
  • Requires documentation update.

Testing

@AionJayT AionJayT added the enhancement New feature or request label Jan 21, 2020
@AionJayT AionJayT added this to the 1.3 milestone Jan 21, 2020
@AionJayT AionJayT self-assigned this Jan 21, 2020
<vendor>rocksdb</vendor>
<!--Boolean value. Enable/disable database compression to trade storage space for execution time.-->
<enable_db_compression>false</enable_db_compression>
<enable_db_compression>true</enable_db_compression>
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.

For LevelDB the compression made performance fairly bad. Do we have experimental data to show that it's better to enable compression with RocksDB?

Copy link
Copy Markdown
Collaborator Author

@AionJayT AionJayT Jan 23, 2020

Choose a reason for hiding this comment

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

I can do the benchmark. But base on my understanding, Enable the data compression will increase CPU usage but significantly reduce the data size been wrote into the disk, especially the compacting happens. Also I doubt the leveldb library might use the old version of the native levelDB, So the compression performance is not good. for the benchmark I did before. You can look at the AKI-550.
Also,you can look at the reference link for the details of the rocksdb data compression
https://github.com/facebook/rocksdb/wiki/Compression

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

@AionJayT AionJayT merged commit e9405e2 into master Jan 28, 2020
@AionJayT AionJayT deleted the AKI-646 branch January 28, 2020 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants