[hotfix] Rename TestJvmProcess#getJvmArgs to `TestJvmProcess#getMai…

…nMethodArgs`
tongcheng-elong · Nov 22, 2023 · 7f2818b · 7f2818b
1 parent 905d599
commit 7f2818b
Show file tree

Hide file tree

Showing 8 changed files with 12 additions and 12 deletions.
diff --git a/.../test/java/org/apache/flink/runtime/entrypoint/ClusterUncaughtExceptionHandlerITCase.java b/.../test/java/org/apache/flink/runtime/entrypoint/ClusterUncaughtExceptionHandlerITCase.java
@@ -130,7 +130,7 @@ public String getName() {
  }
 
  @Override
- public String[] getJvmArgs() {
+ public String[] getMainMethodArgs() {
  return new String[0];
  }
 

diff --git a/flink-runtime/src/test/java/org/apache/flink/runtime/io/disk/FileChannelManagerImplTest.java b/flink-runtime/src/test/java/org/apache/flink/runtime/io/disk/FileChannelManagerImplTest.java
@@ -185,7 +185,7 @@ public String getName() {
  }
 
  @Override
- public String[] getJvmArgs() {
+ public String[] getMainMethodArgs() {
  return new String[] {Boolean.toString(callerHasHook), tmpDirectories, signalFilePath};
  }
 

diff --git a/flink-runtime/src/test/java/org/apache/flink/runtime/testutils/DispatcherProcess.java b/flink-runtime/src/test/java/org/apache/flink/runtime/testutils/DispatcherProcess.java
@@ -78,7 +78,7 @@ public String getName() {
  }
 
  @Override
- public String[] getJvmArgs() {
+ public String[] getMainMethodArgs() {
  return jvmArgs;
  }
 

diff --git a/flink-runtime/src/test/java/org/apache/flink/runtime/testutils/TestJvmProcess.java b/flink-runtime/src/test/java/org/apache/flink/runtime/testutils/TestJvmProcess.java
@@ -101,12 +101,12 @@ public void run() {
  *
  * <p>These can be parsed by the main method of the entry point class.
  */
- public abstract String[] getJvmArgs();
+ public abstract String[] getMainMethodArgs();
 
  /**
  * Returns the name of the class to run.
  *
- * <p>Arguments to the main method can be specified via {@link #getJvmArgs()}.
+ * <p>Arguments to the main method can be specified via {@link #getMainMethodArgs()}.
  */
  public abstract String getEntryPointClassName();
 
@@ -151,10 +151,10 @@ public void startProcess() throws IOException {
 
  cmd = ArrayUtils.add(cmd, getEntryPointClassName());
 
- String[] jvmArgs = getJvmArgs();
+ String[] mainMethodArgs = getMainMethodArgs();
 
- if (jvmArgs != null && jvmArgs.length > 0) {
- cmd = ArrayUtils.addAll(cmd, jvmArgs);
+ if (mainMethodArgs != null && mainMethodArgs.length > 0) {
+ cmd = ArrayUtils.addAll(cmd, mainMethodArgs);
  }
 
  synchronized (createDestroyLock) {

diff --git a/...ime/src/test/java/org/apache/flink/runtime/testutils/TestingClusterEntrypointProcess.java b/...ime/src/test/java/org/apache/flink/runtime/testutils/TestingClusterEntrypointProcess.java
@@ -55,7 +55,7 @@ public String getName() {
  }
 
  @Override
- public String[] getJvmArgs() {
+ public String[] getMainMethodArgs() {
  return new String[] {markerFile.getAbsolutePath()};
  }
 

diff --git a/flink-runtime/src/test/java/org/apache/flink/runtime/util/BlockingShutdownTest.java b/flink-runtime/src/test/java/org/apache/flink/runtime/util/BlockingShutdownTest.java
@@ -164,7 +164,7 @@ public String getName() {
  }
 
  @Override
- public String[] getJvmArgs() {
+ public String[] getMainMethodArgs() {
  return new String[] {
  tempFilePath, String.valueOf(installSignalHandler), String.valueOf(selfKillDelay)
  };

diff --git a/flink-runtime/src/test/java/org/apache/flink/runtime/util/FlinkSecurityManagerITCase.java b/flink-runtime/src/test/java/org/apache/flink/runtime/util/FlinkSecurityManagerITCase.java
@@ -82,7 +82,7 @@ public String getName() {
  }
 
  @Override
- public String[] getJvmArgs() {
+ public String[] getMainMethodArgs() {
  return new String[0];
  }
 

diff --git a/flink-runtime/src/test/java/org/apache/flink/runtime/util/JvmExitOnFatalErrorTest.java b/flink-runtime/src/test/java/org/apache/flink/runtime/util/JvmExitOnFatalErrorTest.java
@@ -131,7 +131,7 @@ public String getName() {
  }
 
  @Override
- public String[] getJvmArgs() {
+ public String[] getMainMethodArgs() {
  return new String[] {temporaryFolder.getAbsolutePath()};
  }