[HBASE-8112] Deprecate HTable#batch(final List<? extends Row>)

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.98.0, 0.99.0
• Hadoop Flags: Reviewed
• Assignee: Jean-Marc Spaggiari
• Reporter: Ted Yu
• Created: 14/Mar/13 22:27
• Updated: 21/Feb/15 23:31
• Resolved: 18/Dec/13 23:03

Description

This was brought up by Amit's inquiry on mailing list, entitled 'Batch returned value and exception handling'

Here is his sample code:

Object[] res = null;
try {
  res = table.batch(batch);
} catch (RetriesExhaustedWithDetailsException retriesExhaustedWithDetailsException) {
  retriesExhaustedWithDetailsException.printStackTrace();
}
if (res == null) {
  System.out.println("No results - returned null.");
}

When RetriesExhaustedWithDetailsException was thrown from batch() call, variable res carried value of null.
Meaning user wouldn't get partial result along with the exception.

We should deprecate

HTable#batch(final List<? extends Row>)

and refer to the following method:

void batch(final List<?extends Row> actions, final Object[] results) throws IOException, InterruptedException;