[HADOOP-9646] Inconsistent exception specifications in FileUtils#chmod

• Type: Bug
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Target Version/s: 2.1.0-beta
• Hadoop Flags: Incompatible change
• Assignee: Colin McCabe
• Reporter: Colin McCabe
• Created: 14/Jun/13 18:20
• Updated: 10/Mar/15 04:17

Description

There are two FileUtils#chmod methods:

public static int chmod(String filename, String perm
                          ) throws IOException, InterruptedException;
public static int chmod(String filename, String perm, boolean recursive)
                            throws IOException;

The first one just calls the second one with recursive = false, but despite that it is declared as throwing InterruptedException, something the second one doesn't declare.

The new Java7 chmod API, which we will transition to once JDK6 support is dropped, does not throw InterruptedException

See http://docs.oracle.com/javase/7/docs/api/java/nio/file/Files.html#setOwner(java.nio.file.Path, java.nio.file.attribute.UserPrincipal)

So we should make these consistent by removing the InterruptedException

Issue Links

• is duplicated by: HADOOP-9647 Hadoop streaming needs minor update for exception signature change