Address some of Reynold's minor comments
Preserve attemptId behavior and deprecate it:

- Introduce new `attemptNumber` and `taskAttemptId` methods to avoid
  ambuiguity.
- Change `attemptNumber` to return Int instead of Long, since it was
  being treated as an Int elsewhere.
- Add more Javadocs.
- Add Mima excludes for new methods.