Github user NicoK commented on a diff in the pull request:
https://github.com/apache/flink/pull/5175#discussion_r159209751
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/blob/BlobUtils.java ---
@@ -150,8 +150,7 @@ static File initLocalStorageDirectory(String basePath)
throws IOException {
File storageDir;
// NOTE: although we will be using UUIDs, there may be
collisions
- final int MAX_ATTEMPTS = 10;
- for(int attempt = 0; attempt < MAX_ATTEMPTS; attempt++) {
+ for (int attempt = 0; attempt < 10; attempt++) {
--- End diff --
ok, sure - I admit, I was tempted to change it and went for it - let me
change that back
---