[api] Stop reading a server error message as a format string

[plusplusjiajia]

Purpose

DefaultErrorHandler hands the server's error message to java.util.Formatter as syntax. When the message contains a %, the caller gets a formatting exception instead of the REST exception it waits for, and the
server's message is lost.

message: quota 80% exceeded
before : java.util.MissingFormatArgumentException: Format specifier '% e'
after : BadRequestException: quota 80% exceeded

Fixed:

• NotAuthorizedException, ServiceFailureException, NotImplementedException, ServiceUnavailableException call super(String.format(message, args)) and RESTException formats again. The other four already pass
  super(message, args).
• DefaultErrorHandler case 400 and case 501 pass the message as the format; the other seven branches pass "%s" with it as an argument.
• RESTApi.getFunction passes "Invalid function name: " + name as the format, so a name rejected for containing % raises UnknownFormatConversionException.
• RESTCatalog.listSystemTablesPaged does the same with a caught message, and drops its cause. RESTException has a cause-carrying constructor but no subclass exposes one, so this adds it to BadRequestException.

A sweep of every construction of these exceptions outside tests found no other site whose format argument is built from data.

[JingsongLi]

+1