You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by ch...@apache.org on 2022/06/14 07:55:22 UTC

[flink] 01/02: [hotfix] Cleanup EnumValueSerializerCompatibilityTest

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

commit cc59fab54ac7ae904413c18d41f24abca0f7501d
Author: Chesnay Schepler <ch...@apache.org>
AuthorDate: Wed Dec 1 12:11:27 2021 +0100

    [hotfix] Cleanup EnumValueSerializerCompatibilityTest
    
    - 'in' variable was unused
    - 'out' variable was unnecessary because the reporter by default prints to stderr anyway, and there isn't a real benefit in using stdout
---
 .../api/scala/typeutils/EnumValueSerializerCompatibilityTest.scala | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/flink-scala/src/test/scala/org/apache/flink/api/scala/typeutils/EnumValueSerializerCompatibilityTest.scala b/flink-scala/src/test/scala/org/apache/flink/api/scala/typeutils/EnumValueSerializerCompatibilityTest.scala
index a5dcc619b5a..b077933c5f7 100644
--- a/flink-scala/src/test/scala/org/apache/flink/api/scala/typeutils/EnumValueSerializerCompatibilityTest.scala
+++ b/flink-scala/src/test/scala/org/apache/flink/api/scala/typeutils/EnumValueSerializerCompatibilityTest.scala
@@ -30,7 +30,7 @@ import java.io._
 import java.net.{URL, URLClassLoader}
 
 import scala.reflect.NameTransformer
-import scala.tools.nsc.{GenericRunnerSettings, Global}
+import scala.tools.nsc.{Global, Settings}
 import scala.tools.nsc.reporters.ConsoleReporter
 
 class EnumValueSerializerCompatibilityTest extends TestLogger with JUnitSuiteLike {
@@ -177,10 +177,7 @@ object EnumValueSerializerCompatibilityTest {
   }
 
   def compileScalaFile(file: File): Unit = {
-    val in = new BufferedReader(new StringReader(""))
-    val out = new PrintWriter(new BufferedWriter(new OutputStreamWriter(System.out)))
-
-    val settings = new GenericRunnerSettings(out.println _)
+    val settings = new Settings()
 
     // use the java classpath so that scala libraries are available to the compiler
     settings.usejavacp.value = true