Github user greghogan commented on a diff in the pull request:
https://github.com/apache/flink/pull/5175#discussion_r158056059
--- 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 --
Should we keep and simply rename the constant?
---
