Better logging for S3DataSegmentPuller getVersion (apache#4657)

* Eventual consistency of S3 means a `404` can be thrown. It helps to know the URI that was attempted.

extensions-core/s3-extensions/src/main/java/io/druid/storage/s3/S3DataSegmentPuller.java

@@ -30,6 +30,7 @@
 import io.druid.java.util.common.IAE;
 import io.druid.java.util.common.IOE;
 import io.druid.java.util.common.MapUtils;
+import io.druid.java.util.common.RE;
 import io.druid.java.util.common.StringUtils;
 import io.druid.java.util.common.UOE;
 import io.druid.java.util.common.logger.Logger;
@@ -300,7 +301,7 @@ public String getVersion(URI uri) throws IOException
         // The recoverable logic is always true for IOException, so we want to only pass IOException if it is recoverable
         throw new IOE(e, "Could not fetch last modified timestamp from URI [%s]", uri);
       } else {
-        throw Throwables.propagate(e);
+        throw new RE(e, "Error fetching last modified timestamp from URI [%s]", uri);
       }
     }
   }