[HBASE-19333] Consider exposing ExportSnapshot#getSnapshotFiles through POJO class

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Incomplete
• Assignee: Unassigned
• Reporter: Ted Yu
• Created: 22/Nov/17 22:02
• Updated: 27/Nov/17 22:53
• Resolved: 22/Nov/17 22:44

Description

In the thread, http://search-hadoop.com/m/HBase/YGbbUxY9FyU74X?subj=Re+Deleting+and+cleaning+old+snapshots+exported+to+S3 , Timothy mentioned that he used reflection to get to ExportSnapshot#getSnapshotFiles().

  private static List<Pair<SnapshotFileInfo, Long>> getSnapshotFiles(final Configuration conf,
      final FileSystem fs, final Path snapshotDir) throws IOException {

SnapshotFileInfo is protobuf.

We should consider exposing the API by replacing the protobuf class with POJO class.