[DRILL-4275] Refactor e/pstore interfaces and their factories to provide a unified mechanism to access stores

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Component/s: Execution - Flow
• Assignee: Hanifi Gunes
• Reporter: Hanifi Gunes
• Created: 15/Jan/16 17:32
• Updated: 19/Feb/16 05:34
• Resolved: 19/Feb/16 05:32

Description

We rely on E/PStore interfaces to persist data. Even though E/PStore stands for Ephemeral and Persistent stores respectively, the current design for EStore does not extend the interface/functionality of PStore at all, which hints abstraction for EStore is redundant. This issue proposes a new unified Store interface replacing the old E/PStore that exposes an additional method that report persistence level as follows:

Store interface

interface Store<V> {
  StoreMode getMode();
  V get(String key);
  ...
}

enum StoreMode {
  EPHEMERAL,
  PERSISTENT,
  ...
}

The new design brings in less redundancy, more centralized code, ease to reason and maintain.

Issue Links

• is depended upon by: DRILL-4186 queue enhancements [umbrella]
• is depended upon by: DRILL-4386 Unify and move serialization logic to common module or package
• links to: GitHub