[FLINK-3798] Clean up RocksDB state backend access modifiers

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Component/s: Connectors / Common
• Assignee: Gyula Fora
• Reporter: Gyula Fora
• Created: 20/Apr/16 19:26
• Updated: 21/Apr/16 10:21
• Resolved: 21/Apr/16 10:02

Description

The RocksDB state backend uses a lot package private methods and fields which makes it very hard to subclass the different parts for added functionality. I think these should be protected instead.

Also the AbstractRocksDBState declares some methods final when there are use-cases when a subclass migh want to call them.

Just to give you an example I am creating a version of the value state which would keep a small cache on heap. For this it would be enough to subclass the RockDBStateBackend and RocksDBVAlue state classes if the above mentioned changes were made. Now I have to use reflection to access package private fields and actually copy classes due to final methods.