[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-14 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r805899247 ## File path: flink-runtime/src/test/java/org/apache/flink/runtime/highavailability/AbstractHaJobRunITCase.java ## @@ -0,0 +1,133 @@ +/* + * Licensed to t

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-14 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r805896966 ## File path: flink-runtime/src/main/java/org/apache/flink/runtime/highavailability/FileSystemJobResultStore.java ## @@ -52,7 +52,18 @@ */ public class

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-14 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r805896473 ## File path: flink-runtime/src/main/java/org/apache/flink/runtime/highavailability/FileSystemJobResultStore.java ## @@ -155,18 +166,15 @@ public boolean

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-14 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r805893942 ## File path: flink-runtime/src/test/java/org/apache/flink/runtime/highavailability/AbstractHaJobRunITCase.java ## @@ -0,0 +1,133 @@ +/* + * Licensed to t

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-14 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r805789848 ## File path: flink-test-utils-parent/flink-test-utils-junit/src/main/java/org/apache/flink/util/DockerImageVersions.java ## @@ -45,4 +45,6 @@ public

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-11 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r804553091 ## File path: flink-filesystems/flink-s3-fs-base/src/test/java/org/apache/flink/fs/s3/common/HAJobRunOnMinioS3StoreITCase.java ## @@ -0,0 +1,123 @@ +/* +

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-11 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r804552415 ## File path: flink-runtime/src/test/java/org/apache/flink/runtime/highavailability/AbstractHaJobRunITCase.java ## @@ -0,0 +1,123 @@ +/* + * Licensed to t

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-11 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r804549715 ## File path: flink-filesystems/flink-s3-fs-base/pom.xml ## @@ -235,6 +265,27 @@ under the License. true

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-11 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r804511045 ## File path: flink-runtime/src/test/java/org/apache/flink/runtime/highavailability/AbstractHaJobRunITCase.java ## @@ -0,0 +1,127 @@ +/* + * Licensed to t

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-11 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r804508088 ## File path: flink-filesystems/flink-s3-fs-base/pom.xml ## @@ -221,6 +221,36 @@ under the License.

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-11 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r804480623 ## File path: flink-test-utils-parent/flink-test-utils-junit/src/main/java/org/apache/flink/core/testutils/TestContainerExtension.java ## @@ -0,0 +1,72 @@

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-10 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r803621829 ## File path: flink-filesystems/flink-s3-fs-presto/src/test/java/org/apache/flink/fs/s3presto/MinioTestContainer.java ## @@ -0,0 +1,146 @@ +/* + * License

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-10 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r803564915 ## File path: flink-test-utils-parent/flink-test-utils-junit/src/main/java/org/apache/flink/util/DockerImageVersions.java ## @@ -45,4 +45,6 @@ public

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-10 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r803521572 ## File path: flink-filesystems/flink-s3-fs-presto/src/test/java/org/apache/flink/fs/s3presto/MinioTestContainer.java ## @@ -0,0 +1,146 @@ +/* + * License

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-10 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r803513590 ## File path: flink-filesystems/flink-s3-fs-presto/pom.xml ## @@ -46,6 +46,36 @@ under the License. provided +

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-10 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r803513214 ## File path: flink-filesystems/flink-s3-fs-presto/pom.xml ## @@ -46,6 +46,36 @@ under the License. provided +

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-10 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r803512948 ## File path: flink-core/src/main/java/org/apache/flink/core/fs/FileSystem.java ## @@ -588,8 +588,10 @@ public static URI getDefaultFsUri() { *

[GitHub] [flink] zentol commented on a change in pull request #18692: [FLINK-26015] Fixes object store bug

2022-02-10 Thread GitBox
zentol commented on a change in pull request #18692: URL: https://github.com/apache/flink/pull/18692#discussion_r803512647 ## File path: flink-core/src/main/java/org/apache/flink/core/fs/FileSystem.java ## @@ -684,7 +686,9 @@ public boolean exists(final Path f) throws IOExcepti