nitpick: My personal preference is to have the main line of code have as little nesting as possible...I would probably rewrite this as:

```java
if (doNotUpload) {
  log
  return;
}

boolean success = ...
...
```