speedb-io / speedb

A RocksDB compliant high performance scalable embedded key-value store
https://www.speedb.io/
Apache License 2.0
898 stars 67 forks source link

The EnableSpeedbXX APIs should be renamed for consistency #761

Open mrambacher opened 10 months ago

mrambacher commented 10 months ago

Currently there are following APIs: DBOptions::EnableSpeedbDB ColumnFamilyOptions::EnableSpeedbCF Options::EnableSpeedb

The first two APIs should be renamed to just "EnableSpeedb". This would mean that the Options class does not have three APIs exposed (since it inherits from the other two classes). It would also make the API consistent with how other APIs in the system are setup (such as OptimizeForSmallDb and OldDefaults).