Revert urlretrieve_with_retry() to support "filename" argument

This allows urlretrieve_with_retry() to be called with or without filename. Mock tests can continue to pass filename and remain unchanged, since they do not create a temp file using NamedTemporaryFile. maybe_download() will work properly since it will use the temp file created within urlretrieve()

Author: chadkennedyonline

tensorflow/contrib/learn/python/learn/datasets/base.py

@@ -186,8 +186,8 @@ def _is_retriable(e):
 
 
 @retry(initial_delay=1.0, max_delay=16.0, is_retriable=_is_retriable)
-def urlretrieve_with_retry(url):
-  return urllib.request.urlretrieve(url)
+def urlretrieve_with_retry(url, filename=None):
+  return urllib.request.urlretrieve(url, filename)
 
 
 def maybe_download(filename, work_directory, source_url):