locationtech / geowave

GeoWave provides geospatial and temporal indexing on top of Accumulo, HBase, BigTable, Cassandra, Kudu, Redis, RocksDB, and DynamoDB.
Apache License 2.0
500 stars 190 forks source link

RocksDB is used after close() can result in lockfile contention #1824

Closed rfecher closed 2 years ago

rfecher commented 3 years ago

RocksDBClient is cached unless the RocksDBDataStore is closed. When it is closed its removed from the cache but the client is still potentially usable. If it is in fact used and another datastore is created, there will be lockfile contention. To resolve this issue, we need to make sure RocksDBClients are always re-used from the cache regardless of close()