This is an automated email from the ASF dual-hosted git repository.

chesnay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git


The following commit(s) were added to refs/heads/master by this push:
     new 4a91354  [FLINK-25064][rest] Remove incorrect @VisibleForTesting
4a91354 is described below

commit 4a91354147c8cea8fc71d8939b8feb21934e7c5d
Author: Chesnay Schepler <[email protected]>
AuthorDate: Thu Nov 25 16:08:57 2021 +0100

    [FLINK-25064][rest] Remove incorrect @VisibleForTesting
---
 .../violations/e5126cae-f3fe-48aa-b6fb-60ae6cc3fcd5                     | 2 --
 .../src/main/java/org/apache/flink/runtime/rest/RestServerEndpoint.java | 1 -
 2 files changed, 3 deletions(-)

diff --git 
a/flink-architecture-tests/violations/e5126cae-f3fe-48aa-b6fb-60ae6cc3fcd5 
b/flink-architecture-tests/violations/e5126cae-f3fe-48aa-b6fb-60ae6cc3fcd5
index a681331..f5e3ff0 100644
--- a/flink-architecture-tests/violations/e5126cae-f3fe-48aa-b6fb-60ae6cc3fcd5
+++ b/flink-architecture-tests/violations/e5126cae-f3fe-48aa-b6fb-60ae6cc3fcd5
@@ -29,8 +29,6 @@ Method 
<org.apache.flink.runtime.io.network.partition.consumer.RemoteInputChanne
 Method 
<org.apache.flink.runtime.io.network.partition.consumer.RemoteInputChannel.getNumberOfRequiredBuffers()>
 calls method 
<org.apache.flink.runtime.io.network.partition.consumer.BufferManager.unsynchronizedGetNumberOfRequiredBuffers()>
 in (RemoteInputChannel.java:341)
 Method 
<org.apache.flink.runtime.io.network.partition.consumer.RemoteInputChannel.isWaitingForFloatingBuffers()>
 calls method 
<org.apache.flink.runtime.io.network.partition.consumer.BufferManager.unsynchronizedIsWaitingForFloatingBuffers()>
 in (RemoteInputChannel.java:351)
 Method 
<org.apache.flink.runtime.operators.coordination.RecreateOnResetOperatorCoordinator$DeferrableCoordinator.closeAsync(long)>
 calls method 
<org.apache.flink.runtime.operators.coordination.RecreateOnResetOperatorCoordinator$QuiesceableContext.quiesce()>
 in (RecreateOnResetOperatorCoordinator.java:319)
-Method 
<org.apache.flink.runtime.rest.FileUploadHandler.channelRead0(org.apache.flink.shaded.netty4.io.netty.channel.ChannelHandlerContext,
 org.apache.flink.shaded.netty4.io.netty.handler.codec.http.HttpObject)> calls 
method 
<org.apache.flink.runtime.rest.RestServerEndpoint.createUploadDir(java.nio.file.Path,
 org.slf4j.Logger, boolean)> in (FileUploadHandler.java:132)
-Method 
<org.apache.flink.runtime.rest.FileUploadHandler.channelRead0(org.apache.flink.shaded.netty4.io.netty.channel.ChannelHandlerContext,
 org.apache.flink.shaded.netty4.io.netty.handler.codec.http.HttpObject)> calls 
method 
<org.apache.flink.runtime.rest.RestServerEndpoint.createUploadDir(java.nio.file.Path,
 org.slf4j.Logger, boolean)> in (FileUploadHandler.java:147)
 Method 
<org.apache.flink.runtime.state.heap.HeapKeyedStateBackend.numKeyValueStateEntries(java.lang.Object)>
 calls method 
<org.apache.flink.runtime.state.heap.StateTable.sizeOfNamespace(java.lang.Object)>
 in (HeapKeyedStateBackend.java:391)
 Method 
<org.apache.flink.runtime.state.heap.StateTable.sizeOfNamespace(java.lang.Object)>
 calls method 
<org.apache.flink.runtime.state.heap.StateMap.sizeOfNamespace(java.lang.Object)>
 in (StateTable.java:361)
 Method 
<org.apache.flink.runtime.taskexecutor.TaskExecutor.registerNewJobAndCreateServices(org.apache.flink.api.common.JobID,
 java.lang.String)> calls method 
<org.apache.flink.runtime.taskexecutor.TaskExecutor$TaskExecutorJobServices.create(org.apache.flink.runtime.execution.librarycache.LibraryCacheManager$ClassLoaderLease,
 java.lang.Runnable)> in (TaskExecutor.java:1121)
diff --git 
a/flink-runtime/src/main/java/org/apache/flink/runtime/rest/RestServerEndpoint.java
 
b/flink-runtime/src/main/java/org/apache/flink/runtime/rest/RestServerEndpoint.java
index cb07a32..cfadcd4 100644
--- 
a/flink-runtime/src/main/java/org/apache/flink/runtime/rest/RestServerEndpoint.java
+++ 
b/flink-runtime/src/main/java/org/apache/flink/runtime/rest/RestServerEndpoint.java
@@ -550,7 +550,6 @@ public abstract class RestServerEndpoint implements 
AutoCloseableAsync {
     }
 
     /** Creates the upload dir if needed. */
-    @VisibleForTesting
     static void createUploadDir(
             final Path uploadDir, final Logger log, final boolean 
initialCreation)
             throws IOException {

Reply via email to