[HBASE-7995] Export$Exporter could be replaced with IdentityTableMapper

• Type: Sub-task
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.98.0, 0.95.1
• Component/s: mapreduce
• Hadoop Flags: Reviewed
• Assignee: Nick Dimiduk
• Reporter: Nick Dimiduk
• Created: 04/Mar/13 21:35
• Updated: 23/Sep/13 19:08
• Resolved: 02/Apr/13 17:48

Description

The Mapper implementation in Export looks identical to IdentityTableMapper, except for the (poor) exception handling. I'd like to consolidate on IdentityTableMapper, but I'm looking for some historical perspective before doing so (cc stack)

Export$Exporter's map method:

    public void map(ImmutableBytesWritable row, Result value, Context context)
    throws IOException {
      try {
        context.write(row, value);
      } catch (InterruptedException e) {
        e.printStackTrace();
      }
    }

IdentityTableMapper's map method:

  public void map(ImmutableBytesWritable key, Result value, Context context)
  throws IOException, InterruptedException {
    context.write(key, value);
  }